fix logging

Author: Deepscorn

Components/BlendAnimatorBasedScreen.cs

@@ -2,7 +2,6 @@
 // http://www.apache.org/licenses/LICENSE-2.0
 using UnityEngine;
 using System.Collections;
-using Assets.Sources.Util.Log;
 
 namespace Assets.Sources.Scripts.Basic
 {

Core/NativePlatform/Platforms/NativeEditor.cs

@@ -4,9 +4,9 @@
 
 using System;
 using System.Collections.Generic;
+using Assets.Plugins.DeepLabs.Core.Utils.Log;
 using Assets.Sources.Logic.Data;
 using Assets.Sources.Util.Data;
-using Assets.Sources.Util.Log;
 using Assets.Sources.Util.UI;
 using UnityEngine;
 
@@ -25,7 +25,7 @@ public void BuyInAppPurchase(string productId, string userId, string orderId, st
         {
             var purchase = new PurchaseInfo(userId, orderId, "0", string.Format("productId : {0}", productId), "this is a test from Unity player");
             purchases.Add(purchase);
-            LogProxy.Log("Bought " + purchase.ToString());
+            Lc.D("Bought " + purchase);
 
             GameObject.Find(callBackObj).SendMessage(callbackMethod, null);
         }

Core/Utils/Log/Lc.cs

@@ -0,0 +1,39 @@
+using JetBrains.Annotations;
+using UnityEditor;
+using UnityEngine;
+
+namespace Assets.Plugins.DeepLabs.Core.Utils.Log
+{
+    public class Lc
+    {
+        public static void D([NotNull] object message)
+        {
+            Debug.Log(message.ToString());
+        }
+
+        // check "development build" in Unity build settings and
+        // that method call will result in app crash
+        public static void Assertion([NotNull] object message)
+        {
+            Debug.LogAssertion(message.ToString());
+            // TODO log to crashlytics in release
+            if (Debug.isDebugBuild)
+            {
+                // TODO breakpoint
+                Application.Quit();
+            }
+            else if (!EditorApplication.isPlaying)
+            {
+                // TODO consider just throwing with unhandled exception leads to crash set in Unity
+                //Crashlytics.RecordCustomException(string name, string reason, StackTrace stackTrace);
+                //Crashlytics.RecordCustomException(string name, string reason, string stackTraceString);
+            }
+            if (EditorApplication.isPlaying)
+            {
+                Debug.Break();
+            }
+        }
+
+    }
+
+}

Core/Utils/Log/LogProxy.cs

@@ -2,8 +2,8 @@
 // http://www.apache.org/licenses/LICENSE-2.0
 using System;
 using System.Net;
+using Assets.Plugins.DeepLabs.Core.Utils.Log;
 using Assets.Sources.NativePlatform;
-using Assets.Sources.Util.Log;
 using Assets.Sources.Util.Network.Requests;
 using RestSharp;
 
@@ -77,7 +77,7 @@ public void AddRequest<TResponseType>(RequestBase<TResponseType> request, Action
             {
                 if (request.Error != null)
                 {
-                    LogProxy.Log("Request error (custom error-handling logic is set): " + request.Error);
+                    Lc.D("Request error (custom error-handling logic is set): " + request.Error);
                 }
                 onFinish(response, response.Error);
             });

Core/Utils/Log/UnityLogImpl.cs

@@ -3,7 +3,7 @@
 using System;
 using System.Collections.Generic;
 using System.Linq;
-using Assets.Sources.Util.Log;
+using Assets.Plugins.DeepLabs.Core.Utils.Log;
 using RestSharp;
 
 namespace Assets.Sources.Util.Network.Requests
@@ -130,7 +130,7 @@ private IRestRequest BuildRequest()
             var requestPath = BuildRequestPath();
             var requestMethodType = RequestMethodType();
 
-            LogProxy.Log(requestMethodType + ": " + requestPath);
+            Lc.D(requestMethodType + ": " + requestPath);
 
             var request = new RestRequest(requestPath, requestMethodType);
             request.RequestFormat = DataFormat.Json;
@@ -145,7 +145,7 @@ private IRestRequest BuildRequest()
                 var body = request.Parameters.FirstOrDefault(p => p.Type == ParameterType.RequestBody);
                 if (body != null)
                 {
-                    LogProxy.Log("Request body: " + body.Value);
+                    Lc.D("Request body: " + body.Value);
                 }
             }
 
@@ -166,11 +166,11 @@ private void OnResponse(IRestResponse<TResponseType> response)
                     errorMessage = response.Content;
                 }
                 Error = new RequestError(errorMessage, statusCode, response.ResponseStatus);
-                LogProxy.Log(Error.ToString());
+                Lc.D(Error.ToString());
             }
             else
             {
-                LogProxy.Log("Response data: " + response.Content);
+                Lc.D("Response data: " + response.Content);
             }
         }
 

Core/Utils/Network/RequestManager.cs

@@ -3,8 +3,8 @@
 using System;
 using System.Collections.Generic;
 using System.Threading;
+using Assets.Plugins.DeepLabs.Core.Utils.Log;
 using Assets.Sources.Util.Concurrent;
-using Assets.Sources.Util.Log;
 
 namespace Assets.Sources.Util.Statistics
 {
@@ -47,7 +47,7 @@ private void ExceptionRun()
             }
             catch (Exception e)
             {
-                LogProxy.LogError(e.ToString());
+                Lc.Assertion(e);
             }
         }