Amend logging

Author: ReallySmallSoftware

README.md

@@ -286,6 +286,9 @@ In order to ensure the browser implementation works, it will be necessary to con
 ```
 
 # History
+## 0.0.6
+- Improve logging
+
 ## 0.0.5
 - Change logo to be drawable rather than mipmap for Android
 - Remove leaked (and now deleted) client id

package.json

@@ -1,6 +1,6 @@
 {
   "name": "cordova-plugin-firebaseui-auth",
-  "version": "0.0.5",
+  "version": "0.0.6",
   "description": "Adds support for Firebase UI authentication.",
   "repository": {
     "type": "git",

plugin.xml

@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<plugin xmlns="http://apache.org/cordova/ns/plugins/1.0" xmlns:android="http://schemas.android.com/apk/res/android" id="cordova-plugin-firebaseui-auth" version="0.0.5">
+<plugin xmlns="http://apache.org/cordova/ns/plugins/1.0" xmlns:android="http://schemas.android.com/apk/res/android" id="cordova-plugin-firebaseui-auth" version="0.0.6">
   <name>Firebase UI Authentication</name>
   <description>Adds support for Firebase Authentication to your Cordova/PhoneGap apps.</description>
   <license>Apache 2.0</license>

src/android/uk/co/reallysmall/cordova/plugin/firebase/ui/auth/FirebaseUIAuthPlugin.java

@@ -57,6 +57,7 @@ public boolean execute(String action, JSONArray args, CallbackContext callbackCo
         } else if ("getToken".equals(action)) {
           return getToken(callbackContext);
         } else {
+          Log.w(TAG, "Unknown action : " + action);
           return false;
         }
     }
@@ -76,7 +77,7 @@ private boolean initialise(JSONArray args, final CallbackContext callbackContext
                 anonymous = true;
             }
         } catch (JSONException ex) {
-            Log.d(TAG, "initialise: error getting options: " + ex.getMessage());
+            Log.e(TAG, "initialise: error getting options: " + ex.getMessage());
         }
 
         this.callbackContext = callbackContext;
@@ -138,7 +139,6 @@ private void createProviderList() throws JSONException {
                     providers.add(new AuthUI.IdpConfig.Builder(AuthUI.TWITTER_PROVIDER).build());
                 }
             }
-
         }
     }
 
@@ -200,6 +200,7 @@ private AuthUI.SignInIntentBuilder buildCustomInstance() {
             }
             instance = instance.setIsSmartLockEnabled(smartLockEnabled, smartLockHints);
         } catch (JSONException ex) {
+          Log.e(TAG, "Error in buildCustomInstance ", ex);
         }
 
         return instance;
@@ -248,7 +249,7 @@ public void onComplete(@NonNull Task<Void> task) {
             }).addOnFailureListener(new OnFailureListener() {
                 @Override
                 public void onFailure(@NonNull Exception e) {
-                    Log.d(TAG, "deleteUser: failure", e);
+                    Log.w(TAG, "deleteUser: failure", e);
                     raiseErrorEvent("deleteuserfailure", 1, "This operation requires you to have recently authenticated. Please log out and back in and try again.");
                 }
             });
@@ -285,6 +286,7 @@ private void raiseErrorEvent(String event, int code, String message) {
             data.put("code", code);
             data.put("message", message);
         } catch (JSONException e) {
+          Log.e(TAG, "Error in raiseErrorEvent ", e);
         }
         raiseEvent(callbackContext, event, data);
     }
@@ -303,7 +305,7 @@ private void raiseEventForUser(FirebaseUser user) {
                 resultData.put("photoUrl", user.getPhotoUrl().toString());
             }
         } catch (JSONException e) {
-            Log.d(TAG, e.getMessage());
+          Log.e(TAG, "Error in raiseEventForUser ", e);
         }
 
         raiseEvent(callbackContext, "signinsuccess", resultData);
@@ -342,7 +344,7 @@ private void raiseEvent(CallbackContext callbackContext, String type, Object dat
                 event.put("data", data);
             }
         } catch (JSONException e) {
-            e.printStackTrace();
+          Log.e(TAG, "Error in raiseEvent ", e);
         }
 
         PluginResult result = new PluginResult(PluginResult.Status.OK, event);
@@ -361,6 +363,7 @@ public void onComplete(@NonNull Task<AuthResult> task) {
                 data.put("code", err.getClass().getSimpleName());
                 data.put("message", err.getMessage());
             } catch (JSONException e) {
+              Log.e(TAG, "Error in onComplete ", e);
             }
             raiseEvent(callbackContext, "signinfailure", data);
         }