Merge pull request #516 from jorgegaticav/master

improvement of logging, replaced print method for developer.log

+
 import 'package:get/instance_manager.dart';
 import 'package:get/instance_manager.dart';
+import 'dart:developer' as developer;
 
 
 typedef LogWriterCallback = void Function(String text, {bool isError});
 typedef LogWriterCallback = void Function(String text, {bool isError});
 
 
 void defaultLogWriterCallback(String value, {bool isError = false}) {
 void defaultLogWriterCallback(String value, {bool isError = false}) {
-  if (isError || GetConfig.isLogEnable) print(value);
+  if (isError || GetConfig.isLogEnable) developer.log(value, name: 'GETX');
 }
 }

+import 'package:flutter/cupertino.dart';
 import 'package:get/src/core/log.dart';
 import 'package:get/src/core/log.dart';
 import 'package:get/src/navigation/root/smart_management.dart';
 import 'package:get/src/navigation/root/smart_management.dart';
 import 'package:get/src/state_manager/rx/rx_interface.dart';
 import 'package:get/src/state_manager/rx/rx_interface.dart';
@@ -127,7 +128,7 @@ class GetInstance {
 
 
     if (i is DisposableInterface) {
     if (i is DisposableInterface) {
       i.onStart();
       i.onStart();
-      GetConfig.log('[GETX] $key has been initialized');
+      GetConfig.log('$key has been initialized');
     }
     }
   }
   }
 
 
@@ -172,7 +173,7 @@ class GetInstance {
       if (!_factory.containsKey(key))
       if (!_factory.containsKey(key))
         throw " $S not found. You need call put<$S>($S()) before";
         throw " $S not found. You need call put<$S>($S()) before";
 
 
-      GetConfig.log('[GETX] $S instance was created at that time');
+      GetConfig.log('$S instance was created at that time');
       S _value = put<S>(_factory[key].builder() as S);
       S _value = put<S>(_factory[key].builder() as S);
 
 
       initDependencies<S>(name: tag);
       initDependencies<S>(name: tag);
@@ -216,15 +217,14 @@ class GetInstance {
 
 
     return queue.add<bool>(() async {
     return queue.add<bool>(() async {
       if (!_singl.containsKey(newKey)) {
       if (!_singl.containsKey(newKey)) {
-        GetConfig.log('[GETX] Instance $newKey already been removed.',
-            isError: true);
+        GetConfig.log('Instance $newKey already been removed.', isError: true);
         return false;
         return false;
       }
       }
 
 
       FcBuilder builder = _singl[newKey] as FcBuilder;
       FcBuilder builder = _singl[newKey] as FcBuilder;
       if (builder.permanent && !force) {
       if (builder.permanent && !force) {
         GetConfig.log(
         GetConfig.log(
-            '[GETX] [$newKey] has been marked as permanent, SmartManagement is not authorized to delete it.',
+            '[$newKey] has been marked as permanent, SmartManagement is not authorized to delete it.',
             isError: true);
             isError: true);
         return false;
         return false;
       }
       }
@@ -235,14 +235,14 @@ class GetInstance {
       }
       }
       if (i is DisposableInterface) {
       if (i is DisposableInterface) {
         await i.onClose();
         await i.onClose();
-        GetConfig.log('[GETX] onClose of $newKey called');
+        GetConfig.log('onClose of $newKey called');
       }
       }
 
 
       _singl.removeWhere((oldKey, value) => (oldKey == newKey));
       _singl.removeWhere((oldKey, value) => (oldKey == newKey));
       if (_singl.containsKey(newKey)) {
       if (_singl.containsKey(newKey)) {
-        GetConfig.log('[GETX] error on remove object $newKey', isError: true);
+        GetConfig.log('error on remove object $newKey', isError: true);
       } else {
       } else {
-        GetConfig.log('[GETX] $newKey deleted from memory');
+        GetConfig.log('$newKey deleted from memory');
       }
       }
       // _routesKey?.remove(key);
       // _routesKey?.remove(key);
       return true;
       return true;

@@ -72,13 +72,13 @@ class GetObserver extends NavigatorObserver {
     String routeName = name(route);
     String routeName = name(route);
 
 
     if (isSnackbar) {
     if (isSnackbar) {
-      GetConfig.log("[GETX] OPEN SNACKBAR $routeName");
+      GetConfig.log("OPEN SNACKBAR $routeName");
     } else if (isBottomSheet) {
     } else if (isBottomSheet) {
-      GetConfig.log("[GETX] OPEN $routeName");
+      GetConfig.log("OPEN $routeName");
     } else if (isDialog) {
     } else if (isDialog) {
-      GetConfig.log("[GETX] OPEN $routeName");
+      GetConfig.log("OPEN $routeName");
     } else if (isGetPageRoute) {
     } else if (isGetPageRoute) {
-      GetConfig.log("[GETX] GOING TO ROUTE $routeName");
+      GetConfig.log("GOING TO ROUTE $routeName");
     }
     }
     GetConfig.currentRoute = routeName;
     GetConfig.currentRoute = routeName;
 
 
@@ -107,13 +107,13 @@ class GetObserver extends NavigatorObserver {
     String routeName = name(route);
     String routeName = name(route);
 
 
     if (isSnackbar) {
     if (isSnackbar) {
-      GetConfig.log("[GETX] CLOSE SNACKBAR $routeName");
+      GetConfig.log("CLOSE SNACKBAR $routeName");
     } else if (isBottomSheet) {
     } else if (isBottomSheet) {
-      GetConfig.log("[GETX] CLOSE $routeName");
+      GetConfig.log("CLOSE $routeName");
     } else if (isDialog) {
     } else if (isDialog) {
-      GetConfig.log("[GETX] CLOSE $routeName");
+      GetConfig.log("CLOSE $routeName");
     } else if (isGetPageRoute) {
     } else if (isGetPageRoute) {
-      GetConfig.log("[GETX] CLOSE TO ROUTE $routeName");
+      GetConfig.log("CLOSE TO ROUTE $routeName");
     }
     }
     GetConfig.currentRoute = routeName;
     GetConfig.currentRoute = routeName;
 
 
@@ -136,8 +136,8 @@ class GetObserver extends NavigatorObserver {
   void didReplace({Route newRoute, Route oldRoute}) {
   void didReplace({Route newRoute, Route oldRoute}) {
     super.didReplace(newRoute: newRoute, oldRoute: oldRoute);
     super.didReplace(newRoute: newRoute, oldRoute: oldRoute);
 
 
-    GetConfig.log("[GETX] REPLACE ROUTE ${oldRoute?.settings?.name}");
-    GetConfig.log("[GETX] NEW ROUTE ${newRoute?.settings?.name}");
+    GetConfig.log("REPLACE ROUTE ${oldRoute?.settings?.name}");
+    GetConfig.log("NEW ROUTE ${newRoute?.settings?.name}");
 
 
     GetConfig.currentRoute = name(newRoute);
     GetConfig.currentRoute = name(newRoute);
 
 
@@ -158,7 +158,7 @@ class GetObserver extends NavigatorObserver {
   @override
   @override
   void didRemove(Route route, Route previousRoute) {
   void didRemove(Route route, Route previousRoute) {
     super.didRemove(route, previousRoute);
     super.didRemove(route, previousRoute);
-    GetConfig.log("[GETX] REMOVING ROUTE ${route?.settings?.name}");
+    GetConfig.log("REMOVING ROUTE ${route?.settings?.name}");
 
 
     _routeSend.update((value) {
     _routeSend.update((value) {
       value.route = previousRoute;
       value.route = previousRoute;

@@ -100,7 +100,7 @@ class Worker {
   final String type;
   final String type;
 
 
   void _message() {
   void _message() {
-    GetConfig.log('[Getx] Worker $type disposed');
+    GetConfig.log('Worker $type disposed');
   }
   }
 
 
   void dispose() {
   void dispose() {