chore: various code clean (#3591)

* chore: remove media3 dependancy from common

* chore: remove unused

* chore: fix linter change

---------

Co-authored-by: Olivier Bouillet <olivier@OrdinateOlivier.lan>

android/src/main/java/androidx/media3/exoplayer/ima/ImaAdsLoader.java

@@ -11,8 +11,6 @@ import androidx.media3.exoplayer.ExoPlayer;
 import androidx.media3.exoplayer.source.ads.AdsLoader;
 import androidx.media3.exoplayer.source.ads.AdsMediaSource;
 
-import com.facebook.react.uimanager.ThemedReactContext;
-
 import java.io.IOException;
 
 public class ImaAdsLoader implements AdsLoader {

android/src/main/java/com/brentvatne/common/react/VideoEventEmitter.java

@@ -14,8 +14,6 @@ import com.facebook.react.bridge.WritableArray;
 import com.facebook.react.bridge.WritableMap;
 import com.facebook.react.uimanager.UIManagerHelper;
 import com.facebook.react.uimanager.common.ViewUtil;
-import com.facebook.react.uimanager.events.RCTEventEmitter;
-import com.google.ads.interactivemedia.v3.api.AdError;
 
 import java.io.PrintWriter;
 import java.io.StringWriter;
@@ -449,14 +447,14 @@ public class VideoEventEmitter {
         receiveEvent(EVENT_ON_RECEIVE_AD_EVENT, map);
     }
 
-    public void receiveAdErrorEvent(AdError error) {
+    public void receiveAdErrorEvent(String message, String code, String type) {
         WritableMap map = Arguments.createMap();
         map.putString("event", "ERROR");
 
         WritableMap dataMap = Arguments.createMap();
-        dataMap.putString("message", error.getMessage());
+        dataMap.putString("message", message);
-        dataMap.putString("code", String.valueOf(error.getErrorCode()));
+        dataMap.putString("code", code);
-        dataMap.putString("type", String.valueOf(error.getErrorType()));
+        dataMap.putString("type", type);
         map.putMap("data", dataMap);
 
         receiveEvent(EVENT_ON_RECEIVE_AD_EVENT, map);

android/src/main/java/com/brentvatne/common/toolbox/ReactBridgeUtils.kt

@@ -12,67 +12,47 @@ import java.util.HashMap
 
 object ReactBridgeUtils {
     @JvmStatic
-    fun safeGetString(map: ReadableMap?, key: String?, fallback: String?): String? {
+    fun safeGetString(map: ReadableMap?, key: String?, fallback: String?): String? =
-        return if (map != null && map.hasKey(key!!) && !map.isNull(key)) map.getString(key) else fallback
+        if (map != null && map.hasKey(key!!) && !map.isNull(key)) map.getString(key) else fallback
-    }
 
     @JvmStatic
-    fun safeGetString(map: ReadableMap?, key: String?): String? {
+    fun safeGetString(map: ReadableMap?, key: String?): String? = safeGetString(map, key, null)
-        return safeGetString(map, key, null)
-    }
 
     @JvmStatic
-    fun safeGetDynamic(map: ReadableMap?, key: String?, fallback: Dynamic?): Dynamic? {
+    fun safeGetDynamic(map: ReadableMap?, key: String?, fallback: Dynamic?): Dynamic? =
-        return if (map != null && map.hasKey(key!!) && !map.isNull(key)) map.getDynamic(key) else fallback
+        if (map != null && map.hasKey(key!!) && !map.isNull(key)) map.getDynamic(key) else fallback
-    }
 
     @JvmStatic
-    fun safeGetDynamic(map: ReadableMap?, key: String?): Dynamic? {
+    fun safeGetDynamic(map: ReadableMap?, key: String?): Dynamic? = safeGetDynamic(map, key, null)
-        return safeGetDynamic(map, key, null)
-    }
 
     @JvmStatic
-    fun safeGetBool(map: ReadableMap?, key: String?, fallback: Boolean): Boolean {
+    fun safeGetBool(map: ReadableMap?, key: String?, fallback: Boolean): Boolean =
-        return if (map != null && map.hasKey(key!!) && !map.isNull(key)) map.getBoolean(key) else fallback
+        if (map != null && map.hasKey(key!!) && !map.isNull(key)) map.getBoolean(key) else fallback
-    }
 
     @JvmStatic
-    fun safeGetMap(map: ReadableMap?, key: String?): ReadableMap? {
+    fun safeGetMap(map: ReadableMap?, key: String?): ReadableMap? = if (map != null && map.hasKey(key!!) && !map.isNull(key)) map.getMap(key) else null
-        return if (map != null && map.hasKey(key!!) && !map.isNull(key)) map.getMap(key) else null
-    }
 
     @JvmStatic
-    fun safeGetArray(map: ReadableMap?, key: String?): ReadableArray? {
+    fun safeGetArray(map: ReadableMap?, key: String?): ReadableArray? = if (map != null && map.hasKey(key!!) && !map.isNull(key)) map.getArray(key) else null
-        return if (map != null && map.hasKey(key!!) && !map.isNull(key)) map.getArray(key) else null
-    }
 
     @JvmStatic
-    fun safeGetInt(map: ReadableMap?, key: String?, fallback: Int): Int {
+    fun safeGetInt(map: ReadableMap?, key: String?, fallback: Int): Int =
-        return if (map != null && map.hasKey(key!!) && !map.isNull(key)) map.getInt(key) else fallback
+        if (map != null && map.hasKey(key!!) && !map.isNull(key)) map.getInt(key) else fallback
-    }
 
     @JvmStatic
-    fun safeGetInt(map: ReadableMap?, key: String?): Int {
+    fun safeGetInt(map: ReadableMap?, key: String?): Int = safeGetInt(map, key, 0)
-        return safeGetInt(map, key, 0)
-    }
 
     @JvmStatic
-    fun safeGetDouble(map: ReadableMap?, key: String?, fallback: Double): Double {
+    fun safeGetDouble(map: ReadableMap?, key: String?, fallback: Double): Double =
-        return if (map != null && map.hasKey(key!!) && !map.isNull(key)) map.getDouble(key) else fallback
+        if (map != null && map.hasKey(key!!) && !map.isNull(key)) map.getDouble(key) else fallback
-    }
 
     @JvmStatic
-    fun safeGetDouble(map: ReadableMap?, key: String?): Double {
+    fun safeGetDouble(map: ReadableMap?, key: String?): Double = safeGetDouble(map, key, 0.0)
-        return safeGetDouble(map, key, 0.0)
-    }
 
-    @JvmStatic fun safeGetFloat(map: ReadableMap?, key: String?, fallback: Float): Float {
+    @JvmStatic fun safeGetFloat(map: ReadableMap?, key: String?, fallback: Float): Float =
-        return if (map != null && map.hasKey(key!!) && !map.isNull(key)) map.getDouble(key).toFloat() else fallback
+        if (map != null && map.hasKey(key!!) && !map.isNull(key)) map.getDouble(key).toFloat() else fallback
-    }
 
-    @JvmStatic fun safeGetFloat(map: ReadableMap?, key: String?): Float {
+    @JvmStatic fun safeGetFloat(map: ReadableMap?, key: String?): Float = safeGetFloat(map, key, 0.0f)
-        return safeGetFloat(map, key, 0.0f)
-    }
 
     /**
      * toStringMap converts a [ReadableMap] into a HashMap.

android/src/main/java/com/brentvatne/exoplayer/ReactExoplayerView.java

@@ -108,6 +108,7 @@ import com.facebook.react.bridge.ReadableArray;
 import com.facebook.react.bridge.ReadableMap;
 import com.facebook.react.bridge.UiThreadUtil;
 import com.facebook.react.uimanager.ThemedReactContext;
+import com.google.ads.interactivemedia.v3.api.AdError;
 import com.google.ads.interactivemedia.v3.api.AdEvent;
 import com.google.ads.interactivemedia.v3.api.AdErrorEvent;
 import com.google.common.collect.ImmutableList;
@@ -2124,6 +2125,7 @@ public class ReactExoplayerView extends FrameLayout implements
 
     @Override
     public void onAdError(AdErrorEvent adErrorEvent) {
-        eventEmitter.receiveAdErrorEvent(adErrorEvent.getError());
+        AdError error = adErrorEvent.getError();
+        eventEmitter.receiveAdErrorEvent(error.getMessage(), String.valueOf(error.getErrorCode()), String.valueOf(error.getErrorType()));
     }
 }