fix(eject): guard against empty key before access (#7289)

bmuenzenmeyer · Timer · commit 4c0d8b64aa45 · 2019-07-02T20:44:03.000-04:00
closes #7288
diff --git a/packages/react-scripts/scripts/eject.js b/packages/react-scripts/scripts/eject.js
@@ -187,7 +187,10 @@ inquirer
     }
     Object.keys(ownPackage.dependencies).forEach(key => {
       // For some reason optionalDependencies end up in dependencies after install
-      if (ownPackage.optionalDependencies[key]) {
+      if (
+        ownPackage.optionalDependencies &&
+        ownPackage.optionalDependencies[key]
+      ) {
         return;
       }
       console.log(`  Adding ${cyan(key)} to dependencies`);

Original file line number	Diff line number	Diff line change
`@@ -187,7 +187,10 @@ inquirer`
`187`	`187`	`}`
`188`	`188`	`Object.keys(ownPackage.dependencies).forEach(key => {`
`189`	`189`	`// For some reason optionalDependencies end up in dependencies after install`
`190`		`- if (ownPackage.optionalDependencies[key]) {`
	`190`	`+ if (`
	`191`	`+ ownPackage.optionalDependencies &&`
	`192`	`+ ownPackage.optionalDependencies[key]`
	`193`	`+ ) {`
`191`	`194`	`return;`
`192`	`195`	`}`
`193`	`196`	console.log(` Adding ${cyan(key)} to dependencies`);