refactor: rename logger classes in Android and iOS

- Rename FlutterOptimizelyLogger to OptimizelyFlutterLogger in Android
- Rename FlutterOptimizelyLogger to OptimizelyFlutterLogger in iOS

Author: muzahidul-opti

android/src/main/java/com/optimizely/optimizely_flutter_sdk/OptimizelyFlutterClient.java

@@ -192,7 +192,7 @@ protected void initializeOptimizely(@NonNull ArgumentsParser argumentsParser, @N
         Boolean useCustomLogger = argumentsParser.getCustomLogger();
         Logger customLogger = null;
         if (useCustomLogger != null && useCustomLogger) {
-            customLogger = new FlutterOptimizelyLogger("OptimizelySDK");
+            customLogger = new OptimizelyFlutterLogger("OptimizelySDK");
             optimizelyManagerBuilder.withLogger(customLogger);
         }
 

android/src/main/java/com/optimizely/optimizely_flutter_sdk/FlutterOptimizelyLogger.java renamed to android/src/main/java/com/optimizely/optimizely_flutter_sdk/OptimizelyFlutterLogger.java

@@ -23,12 +23,12 @@
 import java.util.HashMap;
 import java.util.Map;
 
-public class FlutterOptimizelyLogger implements Logger { 
+public class OptimizelyFlutterLogger implements Logger { 
     static final String LOGGER_CHANNEL = "optimizely_flutter_sdk_logger";
     private static MethodChannel loggerChannel;
     private final String tag;
 
-    public FlutterOptimizelyLogger(String name) {
+    public OptimizelyFlutterLogger(String name) {
         tag = name;
     }
 

android/src/main/java/com/optimizely/optimizely_flutter_sdk/OptimizelyFlutterSdkPlugin.java

@@ -157,21 +157,18 @@ public void onMethodCall(@NonNull MethodCall call, @NonNull Result result) {
   public void onAttachedToEngine(@NonNull FlutterPluginBinding binding) {
     channel = new MethodChannel(binding.getBinaryMessenger(), "optimizely_flutter_sdk");
     channel.setMethodCallHandler(this);
-
-    // Logger channel for custom logging
-    loggerChannel = new MethodChannel(binding.getBinaryMessenger(), FlutterOptimizelyLogger.LOGGER_CHANNEL);
-
-    // Set the logger channel in your logger class
-      FlutterOptimizelyLogger.setChannel(loggerChannel);
+    
+    loggerChannel = new MethodChannel(binding.getBinaryMessenger(), OptimizelyFlutterLogger.LOGGER_CHANNEL);
+    OptimizelyFlutterLogger.setChannel(loggerChannel);
 
     context = binding.getApplicationContext();
   }
 
   @Override
   public void onDetachedFromEngine(@NonNull FlutterPluginBinding binding) {
     channel.setMethodCallHandler(null);
-    loggerChannel.setMethodCallHandler(null); // Clean up logger channel
-    FlutterOptimizelyLogger.setChannel(null);
+    loggerChannel.setMethodCallHandler(null); 
+    OptimizelyFlutterLogger.setChannel(null);
   }
 
   @Override

example/lib/custom_logger.dart

@@ -1,11 +1,14 @@
 import 'package:optimizely_flutter_sdk/optimizely_flutter_sdk.dart';
+import 'package:flutter/foundation.dart';
 
 class CustomLogger implements OptimizelyLogger {
   @override
   OptimizelyLogLevel logLevel = OptimizelyLogLevel.debug;
 
   @override
   void log(OptimizelyLogLevel level, String message) {
-    print('[Flutter LOGGER] ${level.name}: $message');
+    if (kDebugMode) {
+      print('[Flutter LOGGER] ${level.name}: $message');
+    }
   }
 }

example/lib/main.dart

@@ -2,6 +2,14 @@ import 'package:flutter/material.dart';
 import 'dart:async';
 import 'dart:math';
 import 'package:optimizely_flutter_sdk/optimizely_flutter_sdk.dart';
+import 'package:optimizely_flutter_sdk_example/custom_logger.dart';
+
+/**
+ * Logger TODO:
+ * Check thread safety
+ * Android logger fix
+ * Add unit test
+ */
 
 void main() {
   runApp(const MyApp());
@@ -28,16 +36,20 @@ class _MyAppState extends State<MyApp> {
       OptimizelyDecideOption.includeReasons,
       OptimizelyDecideOption.excludeVariables
     };
+    final customLogger = CustomLogger();
+
     var flutterSDK = OptimizelyFlutterSdk("X9mZd2WDywaUL9hZXyh9A",
         datafilePeriodicDownloadInterval: 10 * 60,
         eventOptions: const EventOptions(
             batchSize: 1, timeInterval: 60, maxQueueSize: 10000),
         defaultLogLevel: OptimizelyLogLevel.debug,
-        defaultDecideOptions: defaultOptions);
+        defaultDecideOptions: defaultOptions,
+        logger: customLogger,
+        );
     var response = await flutterSDK.initializeClient();
 
     setState(() {
-      uiResponse = "Optimizely Client initialized: ${response.success} ";
+      uiResponse = "[Optimizely] Client initialized: ${response.success} ";
     });
 
     var rng = Random();

ios/Classes/FlutterOptimizelyLogger.swift renamed to ios/Classes/OptimizelyFlutterLogger.swift

@@ -1,7 +1,7 @@
 import Flutter
 import Optimizely
 
-public class FlutterOptimizelyLogger: NSObject, OPTLogger {
+public class OptimizelyFlutterLogger: NSObject, OPTLogger {
     public static var logLevel: OptimizelyLogLevel = .info
 
     private static let loggerChannel = FlutterMethodChannel(

ios/Classes/SwiftOptimizelyFlutterSdkPlugin.swift

@@ -168,7 +168,7 @@ public class SwiftOptimizelyFlutterSdkPlugin: NSObject, FlutterPlugin {
         // Check if custom logger is requested
         var logger: OPTLogger?
         if let useCustomLogger = parameters[RequestParameterKey.customLogger] as? Bool, useCustomLogger {
-            logger = FlutterOptimizelyLogger()
+            logger = OptimizelyFlutterLogger()
         }
 
         // Creating new instance

lib/src/optimizely_client_wrapper.dart

@@ -27,8 +27,6 @@ import 'package:optimizely_flutter_sdk/src/data_objects/get_vuid_response.dart';
 import 'package:optimizely_flutter_sdk/src/data_objects/optimizely_config_response.dart';
 import 'package:optimizely_flutter_sdk/src/utils/constants.dart';
 import 'package:optimizely_flutter_sdk/src/utils/utils.dart';
-import 'package:optimizely_flutter_sdk/src/logger/flutter_logger.dart';
-import 'package:optimizely_flutter_sdk/src/logger/logger_bridge.dart';
 
 enum ListenerType { activate, track, decision, logEvent, projectConfigUpdate }