Remove getVibrate / getSoundEnabled apis and logics

* Remove getVibrate and enableVibrate apis
* Remove getSoundEnabled and enableSound apis
* Remove logics using them

Author: Jeasmine

OneSignalSDK/onesignal/src/main/java/com/onesignal/GenerateNotification.java

@@ -240,11 +240,11 @@ private static void setAlertnessOptions(JSONObject fcmJson, NotificationCompat.B
          } catch (Throwable t) {
             notificationDefaults |= Notification.DEFAULT_LIGHTS;
          } // Can throw if an old android support lib is used or parse error.
-      }
-      else
+      } else {
          notificationDefaults |= Notification.DEFAULT_LIGHTS;
+      }
 
-      if (OneSignal.getVibrate() && fcmJson.optInt("vib", 1) == 1) {
+      if (fcmJson.optInt("vib", 1) == 1) {
          if (fcmJson.has("vib_pt")) {
             long[] vibrationPattern = OSUtils.parseVibrationPattern(fcmJson);
             if (vibrationPattern != null)
@@ -931,9 +931,7 @@ private static int getDrawableId(String name) {
 
    private static boolean isSoundEnabled(JSONObject fcmJson) {
       String sound = fcmJson.optString("sound", null);
-      if ("null".equals(sound) || "nil".equals(sound))
-         return false;
-      return OneSignal.getSoundEnabled();
+      return !"null".equals(sound) && !"nil".equals(sound);
    }
 
    // Android 5.0 accent color to use, only works when AndroidManifest.xml is targetSdkVersion >= 21

OneSignalSDK/onesignal/src/main/java/com/onesignal/OneSignal.java

@@ -2241,60 +2241,6 @@ static boolean isUserPrivacyConsentRequired() {
    }
    // End Remote params getters
 
-   // If true(default) - Device will always vibrate unless the device is in silent mode.
-   // If false - Device will only vibrate when the device is set on it's vibrate only mode.
-   /**
-    * By default OneSignal always vibrates the device when a notification is displayed unless the
-    * device is in a total silent mode.
-    * <br/><br/>
-    * <i>You can link this action to a UI button to give your user a vibration option for your notifications.</i>
-    * @param enable Passing {@code false} means that the device will only vibrate lightly when the device is in it's vibrate only mode.
-    */
-   public static void enableVibrate(boolean enable) {
-      if (appContext == null) {
-         logger.error("enableVibrate called before initWithContext!");
-         return;
-      }
-
-      OneSignalPrefs.saveBool(
-              OneSignalPrefs.PREFS_ONESIGNAL,
-              OneSignalPrefs.PREFS_GT_VIBRATE_ENABLED,
-              enable);
-   }
-
-   static boolean getVibrate() {
-      return OneSignalPrefs.getBool(
-              OneSignalPrefs.PREFS_ONESIGNAL,
-              OneSignalPrefs.PREFS_GT_VIBRATE_ENABLED,
-              true);
-   }
-
-   // If true(default) - Sound plays when receiving notification. Vibrates when device is on vibrate only mode.
-   // If false - Only vibrates unless EnableVibrate(false) was set.
-   /**
-    * By default OneSignal plays the system's default notification sound when the
-    * device's notification system volume is turned on.
-    * <br/><br/>
-    * <i>You can link this action to a UI button to give your user a different sound option for your notifications.</i>
-    * @param enable Passing {@code false} means that the device will only vibrate unless the device is set to a total silent mode.
-    */
-   public static void enableSound(boolean enable) {
-      if (appContext == null) {
-         logger.error("enableSound called before initWithContext!");
-         return;
-      }
-
-      OneSignalPrefs.saveBool(OneSignalPrefs.PREFS_ONESIGNAL,
-              OneSignalPrefs.PREFS_GT_SOUND_ENABLED,enable);
-   }
-
-   static boolean getSoundEnabled() {
-      return OneSignalPrefs.getBool(
-              OneSignalPrefs.PREFS_ONESIGNAL,
-              OneSignalPrefs.PREFS_GT_SOUND_ENABLED,
-              true);
-   }
-
    static void setLastSessionTime(long time) {
       OneSignalPrefs.saveLong(
               OneSignalPrefs.PREFS_ONESIGNAL,

OneSignalSDK/unittest/src/test/java/com/test/onesignal/MainOneSignalClassRunner.java

@@ -2659,8 +2659,6 @@ public void testMethodCalls_withSetAppIdCalledBeforeMethodCalls() throws Excepti
       OneSignal.deleteTag("key");
       OneSignal.deleteTags("[\"key1\", \"key2\"]");
       OneSignal.disablePush(false);
-      OneSignal.enableVibrate(false);
-      OneSignal.enableSound(false);
       OneSignal.promptLocation();
       OneSignal.postNotification("{}", new OneSignal.PostNotificationResponseHandler() {
          @Override
@@ -2691,8 +2689,6 @@ public void testMethodCalls_withInitWithContextCalledBeforeMethodCalls() throws
       OneSignal.deleteTag("key");
       OneSignal.deleteTags("[\"key1\", \"key2\"]");
       OneSignal.disablePush(false);
-      OneSignal.enableVibrate(false);
-      OneSignal.enableSound(false);
       OneSignal.promptLocation();
       OneSignal.postNotification("{}", new OneSignal.PostNotificationResponseHandler() {
          @Override
@@ -2721,8 +2717,6 @@ public void testMethodCalls_withInitWithContextAndSetAppId() throws Exception {
       OneSignal.deleteTag("key");
       OneSignal.deleteTags("[\"key1\", \"key2\"]");
       OneSignal.disablePush(false);
-      OneSignal.enableVibrate(false);
-      OneSignal.enableSound(false);
       OneSignal.promptLocation();
       OneSignal.postNotification("{}", new OneSignal.PostNotificationResponseHandler() {
          @Override
@@ -2751,8 +2745,6 @@ public void testMethodCalls_withSetAppIdAndInitWithContext() throws Exception {
       OneSignal.deleteTag("key");
       OneSignal.deleteTags("[\"key1\", \"key2\"]");
       OneSignal.disablePush(false);
-      OneSignal.enableVibrate(false);
-      OneSignal.enableSound(false);
       OneSignal.promptLocation();
       OneSignal.postNotification("{}", new OneSignal.PostNotificationResponseHandler() {
          @Override