Add checkDefault method for addressing #33

Author: geraintwhite

android/src/main/AndroidManifest.xml

@@ -2,4 +2,5 @@
 <manifest xmlns:android="http://schemas.android.com/apk/res/android" package="io.wazo.callkeep">
 
     <uses-permission android:name="android.permission.CALL_PHONE" />
+    <uses-permission android:name="android.permission.READ_PHONE_STATE" />
 </manifest>

android/src/main/java/io/wazo/callkeep/RNCallKeepModule.java

@@ -176,6 +176,21 @@ public void checkPhoneAccountPermission(Promise promise) {
         promise.resolve(hasPhoneAccount());
     }
 
+    @ReactMethod
+    public void checkDefaultPhoneAccount(Promise promise) {
+        if (!isConnectionServiceAvailable() || !hasPhoneAccount()) {
+            promise.resolve(true);
+            return;
+        }
+
+        if (!Build.MANUFACTURER.equalsIgnoreCase("Samsung")) {
+            promise.resolve(true);
+            return;
+        }
+
+        promise.resolve(telecomManager.getDefaultOutgoingPhoneAccount("tel") != null);
+    }
+
     @ReactMethod
     public void hasPhoneAccount(Promise promise) {
         promise.resolve(hasPhoneAccount());
@@ -217,6 +232,15 @@ public void openPhoneAccounts() {
             return;
         }
 
+        openPhoneAccountSettings();
+    }
+
+    @ReactMethod
+    public void openPhoneAccountSettings() {
+        if (!isConnectionServiceAvailable()) {
+            return;
+        }
+
         Intent intent = new Intent(TelecomManager.ACTION_CHANGE_PHONE_ACCOUNTS);
         intent.setFlags(Intent.FLAG_ACTIVITY_NEW_TASK | Intent.FLAG_ACTIVITY_MULTIPLE_TASK);
         this.getAppContext().startActivity(intent);

index.js

@@ -37,6 +37,14 @@ class RNCallKeep {
     return this._setupIOS(options.ios);
   };
 
+  checkDefault = async (options) => {
+    if (!isIOS) {
+      return this._checkDefault(options);
+    }
+
+    return;
+  };
+
   displayIncomingCall = (uuid, handle, localizedCallerName, handleType = 'number', hasVideo = false) => {
     if (!isIOS) {
       RNCallKeepModule.displayIncomingCall(handle, localizedCallerName);
@@ -121,33 +129,44 @@ class RNCallKeep {
 
   _setupAndroid = async (options) => {
     const hasAccount = await RNCallKeepModule.checkPhoneAccountPermission();
+    const shouldOpenAccounts = await this._alert(options, hasAccount);
+
+    if (shouldOpenAccounts) {
+      RNCallKeepModule.openPhoneAccounts();
+    }
+  };
+
+  _checkDefault = async (options) => {
+    const hasDefault = await RNCallKeepModule.checkDefaultPhoneAccount();
+    const shouldOpenAccounts = await this._alert(options, hasDefault);
 
-    return new Promise((resolve, reject) => {
-      if (hasAccount) {
-        return resolve();
-      }
-
-      Alert.alert(
-        options.alertTitle,
-        options.alertDescription,
-        [
-          {
-            text: options.cancelButton,
-            onPress: reject,
-            style: 'cancel',
-          },
-          { text: options.okButton,
-            onPress: () => {
-              RNCallKeepModule.openPhoneAccounts();
-              resolve();
-            }
-          },
-        ],
-        { cancelable: true },
-      );
-    });
+    if (shouldOpenAccounts) {
+      RNCallKeepModule.openPhoneAccountSettings();
+    }
   };
 
+  _alert = async (options, condition) => new Promise((resolve, reject) => {
+    if (condition) {
+      return resolve(false);
+    }
+
+    Alert.alert(
+      options.alertTitle,
+      options.alertDescription,
+      [
+        {
+          text: options.cancelButton,
+          onPress: reject,
+          style: 'cancel',
+        },
+        { text: options.okButton,
+          onPress: () => resolve(true)
+        },
+      ],
+      { cancelable: true },
+    );
+  });
+
   backToForeground() {
     if (isIOS) {
       return;