Fix issue with overriding component usedPropTypes if the component already existed in the map

jseminck · jseminck · commit f85ff448b8f1 · 2017-05-22T07:48:56.000+03:00
diff --git a/lib/util/Components.js b/lib/util/Components.js
@@ -99,9 +99,12 @@ Components.prototype.list = function() {
       component = this.get(node);
     }
     if (component) {
-      usedPropTypes[this._getId(component.node)] = (this._list[i].usedPropTypes || []).filter(function(propType) {
+      var newUsedProps = (this._list[i].usedPropTypes || []).filter(function(propType) {
         return !propType.node || propType.node.kind !== 'init';
       });
+
+      var componentId = this._getId(component.node);
+      usedPropTypes[componentId] = (usedPropTypes[componentId] || []).concat(newUsedProps);
     }
   }
   // Assign used props in not confident components to the parent component

Original file line number	Diff line number	Diff line change
`@@ -99,9 +99,12 @@ Components.prototype.list = function() {`
`99`	`99`	`component = this.get(node);`
`100`	`100`	`}`
`101`	`101`	`if (component) {`
`102`		`- usedPropTypes[this._getId(component.node)] = (this._list[i].usedPropTypes \|\| []).filter(function(propType) {`
	`102`	`+ var newUsedProps = (this._list[i].usedPropTypes \|\| []).filter(function(propType) {`
`103`	`103`	`return !propType.node \|\| propType.node.kind !== 'init';`
`104`	`104`	`});`
	`105`	`+`
	`106`	`+ var componentId = this._getId(component.node);`
	`107`	`+ usedPropTypes[componentId] = (usedPropTypes[componentId] \|\| []).concat(newUsedProps);`
`105`	`108`	`}`
`106`	`109`	`}`
`107`	`110`	`// Assign used props in not confident components to the parent component`