Merge pull request #1527 from OneSignal/fix/privacyConsentLogError

shepherd-l · web-flow · commit a04f55f0fb8e · 2022-02-04T16:45:03.000-05:00
Remove setRequiresUserConesnt call when there is no value in the manifest
diff --git a/OneSignalSDK/onesignal/src/main/java/com/onesignal/OneSignal.java b/OneSignalSDK/onesignal/src/main/java/com/onesignal/OneSignal.java
@@ -883,7 +883,8 @@ private static void setupPrivacyConsent(Context context) {
 
          // Read the current privacy consent setting from AndroidManifest.xml
          String requireSetting = bundle.getString("com.onesignal.PrivacyConsent");
-         setRequiresUserPrivacyConsent("ENABLE".equalsIgnoreCase(requireSetting));
+         if (requireSetting != null)
+            setRequiresUserPrivacyConsent("ENABLE".equalsIgnoreCase(requireSetting));
       } catch (Throwable t) {
          t.printStackTrace();
       }

Original file line number	Diff line number	Diff line change
`@@ -883,7 +883,8 @@ private static void setupPrivacyConsent(Context context) {`
`883`	`883`
`884`	`884`	`// Read the current privacy consent setting from AndroidManifest.xml`
`885`	`885`	`String requireSetting = bundle.getString("com.onesignal.PrivacyConsent");`
`886`		`- setRequiresUserPrivacyConsent("ENABLE".equalsIgnoreCase(requireSetting));`
	`886`	`+ if (requireSetting != null)`
	`887`	`+ setRequiresUserPrivacyConsent("ENABLE".equalsIgnoreCase(requireSetting));`
`887`	`888`	`} catch (Throwable t) {`
`888`	`889`	`t.printStackTrace();`
`889`	`890`	`}`