Bug fix for #379 where status report can be undefined on notifyApplicationReady

nevir · nevir · commit 8c432e4595d6 · 2016-06-23T10:52:35.000-07:00
I introduced a crash on subsequent launches after an update is applied, where `getNewStatusReport()` returns nothing.  This fixes it!
diff --git a/CodePush.js b/CodePush.js
@@ -173,7 +173,7 @@ const notifyApplicationReady = (() => {
 async function notifyApplicationReadyInternal() {
   await NativeCodePush.notifyApplicationReady();
   const statusReport = await NativeCodePush.getNewStatusReport();
-  tryReportStatus(statusReport); // Don't wait for this to complete.
+  statusReport && tryReportStatus(statusReport); // Don't wait for this to complete.
 
   return statusReport;
 }

Original file line number	Diff line number	Diff line change
`@@ -173,7 +173,7 @@ const notifyApplicationReady = (() => {`
`173`	`173`	`async function notifyApplicationReadyInternal() {`
`174`	`174`	`await NativeCodePush.notifyApplicationReady();`
`175`	`175`	`const statusReport = await NativeCodePush.getNewStatusReport();`
`176`		`- tryReportStatus(statusReport); // Don't wait for this to complete.`
	`176`	`+ statusReport && tryReportStatus(statusReport); // Don't wait for this to complete.`
`177`	`177`
`178`	`178`	`return statusReport;`
`179`	`179`	`}`