clean code

Author: ABTastyAdel

example/lib/widgets/configuration.dart

@@ -406,8 +406,7 @@ class _ConfigurationState extends State<Configuration> with ShowDialog {
       GestureBinding.instance.pointerRouter
           .addGlobalRoute(_emotionAIGlobalPointerRoute);
     } catch (e) {
-      // Todo later add flagship logger
-      print(e);
+      Flagship.logger(Level.ERROR, e.toString());
     }
 
     Flagship.sharedInstance()

lib/visitor.dart

@@ -40,9 +40,7 @@ enum Instance {
   NEW_INSTANCE
 }
 
-const Duration FSSessionVisitor = Duration(
-    seconds:
-        20); // redo later Duration(seconds: 1 * 60 * 30); // example 30 min
+const Duration FSSessionVisitor = Duration(seconds: 1 * 60 * 30); // 30 min
 
 class Visitor with EmotionAiDelegate {
   /// VisitorId

lib/visitor/strategy/default_strategy.dart

@@ -42,7 +42,7 @@ class DefaultStrategy implements IVisitor {
     Modification modification,
     bool isDuplicated,
   ) async {
-    // Getconfig et le callback
+    // Get config and callback
     final config = Flagship.sharedInstance().getConfiguration();
     final onExposed = config?.onVisitorExposed;
 
@@ -69,7 +69,7 @@ class DefaultStrategy implements IVisitor {
         exposedFlag.alreadyActivatedCampaign = true;
         onExposed(exposedVisitor, exposedFlag);
       }
-      print('Skip the activate: Flag already activated.');
+      Flagship.logger(Level.INFO, " The camapign's flag already activated ");
       return;
     }
 
@@ -88,14 +88,14 @@ class DefaultStrategy implements IVisitor {
       visitorJson,
     );
 
-    // Log interne pour troubleshooting
+    // Send troubleshooting
     DataUsageTracking.sharedInstance().processTroubleShootingHits(
       CriticalPoints.VISITOR_SEND_ACTIVATE.name,
       visitor,
       activateHit,
     );
 
-    // Envoi du hit
+    // Send Activate hit
     try {
       final response = await visitor.trackingManager?.sendActivate(activateHit);
       final status = response?.statusCode ?? -1;

lib/visitor/strategy/not_ready_strategy.dart

@@ -17,11 +17,6 @@ class NotReadyStrategy extends DefaultStrategy {
     return null;
   }
 
-  //@override
-  // Future<void> activateModification(String key) async {
-  // Flagship.logger(Level.ERROR, ACTIVATE_NOT_READY);
-  //}
-
   @override
   Future<void> activateFlag(Modification pFlag,
       {bool isDuplicated = false}) async {

lib/visitor/visitor_delegate.dart

@@ -9,7 +9,6 @@ import 'package:flagship/visitor/strategy/no_consent_strategy.dart';
 import 'package:flagship/visitor/strategy/not_ready_strategy.dart';
 import 'package:flagship/visitor/strategy/panic_strategy.dart';
 import 'package:flagship/flagship.dart';
-import 'package:flutter/material.dart';
 import '../visitor.dart';
 
 class VisitorDelegate implements IVisitor {
@@ -141,7 +140,7 @@ class VisitorDelegate implements IVisitor {
     getStrategy().onAppScreenChange(screenName);
   }
 
-  /// Returns `true` if this (campId, varGrpId) pair is considered “deduplicated”.
+  /// Returns `true` if flag is already activated during visitor session
   bool _isDeduplicatedFlag(String campId, String varGrpId) {
     final DateTime now = DateTime.now();
     final Duration elapsed = now.difference(visitor.sessionDuration);