Merge pull request #1341 from OneSignal/fix/get_manifest_meta_bundle

jkasten2 · web-flow · commit 04824e3deed5 · 2021-05-24T14:34:46.000-07:00
Fix getManifestMetaBundle
diff --git a/OneSignalSDK/onesignal/src/main/java/com/onesignal/OSUtils.java b/OneSignalSDK/onesignal/src/main/java/com/onesignal/OSUtils.java
@@ -403,10 +403,10 @@ String getCarrierName() {
    }
 
    static Bundle getManifestMetaBundle(Context context) {
-
-      ApplicationInfo ai = null;
+      ApplicationInfo ai;
       try {
          ai = context.getPackageManager().getApplicationInfo(context.getPackageName(), PackageManager.GET_META_DATA);
+         return ai.metaData;
       } catch (PackageManager.NameNotFoundException e) {
          Log(OneSignal.LOG_LEVEL.ERROR, "Manifest application info not found", e);
       }

Original file line number	Diff line number	Diff line change
`@@ -403,10 +403,10 @@ String getCarrierName() {`
`403`	`403`	`}`
`404`	`404`
`405`	`405`	`static Bundle getManifestMetaBundle(Context context) {`
`406`		`-`
`407`		`- ApplicationInfo ai = null;`
	`406`	`+ ApplicationInfo ai;`
`408`	`407`	`try {`
`409`	`408`	`ai = context.getPackageManager().getApplicationInfo(context.getPackageName(), PackageManager.GET_META_DATA);`
	`409`	`+ return ai.metaData;`
`410`	`410`	`} catch (PackageManager.NameNotFoundException e) {`
`411`	`411`	`Log(OneSignal.LOG_LEVEL.ERROR, "Manifest application info not found", e);`
`412`	`412`	`}`