diff --git a/android/src/main/java/io/ably/flutter/plugin/AblyEventStreamHandler.java b/android/src/main/java/io/ably/flutter/plugin/AblyEventStreamHandler.java index 9a6056970..4db333552 100644 --- a/android/src/main/java/io/ably/flutter/plugin/AblyEventStreamHandler.java +++ b/android/src/main/java/io/ably/flutter/plugin/AblyEventStreamHandler.java @@ -3,6 +3,7 @@ import android.content.Context; import android.os.Handler; import android.os.Looper; +import android.util.Log; import java.util.Map; @@ -26,6 +27,7 @@ */ public class AblyEventStreamHandler implements EventChannel.StreamHandler { + private static final String TAG = AblyEventStreamHandler.class.getName(); /** * Creating an ablyLibrary instance. * As ablyLibrary is a singleton, @@ -197,7 +199,7 @@ public void onListen(Object object, EventChannel.EventSink uiThreadEventSink) { @Override public void onCancel(Object object) { if (object == null) { - System.out.println("Cannot process null input on cancel"); + Log.w(TAG, "onCancel cannot decode null"); return; } final AblyFlutterMessage> ablyMessage = getMessage(object); diff --git a/android/src/main/java/io/ably/flutter/plugin/AblyFlutterPlugin.java b/android/src/main/java/io/ably/flutter/plugin/AblyFlutterPlugin.java index b832576d1..7b6852377 100644 --- a/android/src/main/java/io/ably/flutter/plugin/AblyFlutterPlugin.java +++ b/android/src/main/java/io/ably/flutter/plugin/AblyFlutterPlugin.java @@ -76,9 +76,7 @@ private void setupChannels(BinaryMessenger messenger, Context applicationContext } @Override - public void onDetachedFromEngine(@NonNull FlutterPluginBinding binding) { - System.out.println("Ably Plugin onDetachedFromEngine"); - } + public void onDetachedFromEngine(@NonNull FlutterPluginBinding binding) {} private static MethodCodec createCodec(CipherParamsStorage cipherParamsStorage) { return new StandardMethodCodec(new AblyMessageCodec(cipherParamsStorage)); diff --git a/android/src/main/java/io/ably/flutter/plugin/AblyMessageCodec.java b/android/src/main/java/io/ably/flutter/plugin/AblyMessageCodec.java index a7b176f26..96fb5c72d 100644 --- a/android/src/main/java/io/ably/flutter/plugin/AblyMessageCodec.java +++ b/android/src/main/java/io/ably/flutter/plugin/AblyMessageCodec.java @@ -1,5 +1,7 @@ package io.ably.flutter.plugin; +import android.util.Log; + import androidx.annotation.Nullable; import com.google.firebase.messaging.RemoteMessage; @@ -58,7 +60,7 @@ interface CodecDecoder { } private static class CodecPair { - + private static final String TAG = CodecPair.class.getName(); final CodecEncoder encoder; final CodecDecoder decoder; @@ -69,7 +71,7 @@ private static class CodecPair { Map encode(final Object value) { if (this.encoder == null) { - System.out.println("Codec encoder not defined"); + Log.w(TAG, "Encoder is null"); return null; } return this.encoder.encode((T) value); @@ -77,7 +79,7 @@ Map encode(final Object value) { T decode(Map jsonMap) { if (this.decoder == null) { - System.out.println("Codec decoder not defined"); + Log.w(TAG, "Decoder is null"); return null; } return this.decoder.decode(jsonMap); diff --git a/android/src/main/java/io/ably/flutter/plugin/AblyMethodCallHandler.java b/android/src/main/java/io/ably/flutter/plugin/AblyMethodCallHandler.java index b12924835..dfc8545bb 100644 --- a/android/src/main/java/io/ably/flutter/plugin/AblyMethodCallHandler.java +++ b/android/src/main/java/io/ably/flutter/plugin/AblyMethodCallHandler.java @@ -151,7 +151,6 @@ private void handleAblyException(@NonNull MethodChannel.Result result, @NonNull @Override public void onMethodCall(@NonNull MethodCall call, @NonNull MethodChannel.Result rawResult) { final MethodChannel.Result result = new MethodResultWrapper(rawResult); - System.out.println("Ably Plugin handle: " + call.method); final BiConsumer handler = _map.get(call.method); if (null == handler) { // We don't have a handler for a method with this name so tell the caller. @@ -177,7 +176,6 @@ public void onError(ErrorInfo reason) { } private void register(@NonNull MethodCall call, @NonNull MethodChannel.Result result) { - System.out.println("Registering library instance to clean up any existing instances"); hotRestartCallback.on(); _ably.dispose(); result.success(null); @@ -205,10 +203,7 @@ public void success(@Nullable Object result) { @Override public void error(String errorCode, @Nullable String errorMessage, @Nullable Object errorDetails) { - System.out.println(errorDetails); - if (errorMessage != null) { - result.error("40000", String.format("Error from authCallback: %s", errorMessage), errorDetails); - } + result.error("40000", String.format("Error from authCallback: %s", errorMessage), errorDetails); latch.countDown(); }