fix(ModelManager): Attach a view ID to new views

floryst · floryst · commit 4f6d1e7e2a0d · 2018-08-20T12:58:21.000-04:00
diff --git a/src/core/ModelManager.js b/src/core/ModelManager.js
@@ -47,6 +47,7 @@ export default class ModelManager {
     this.activeViewIndex = 0;
     this.collapseState = {};
     this.listeners = [];
+    this.nextViewId = 0;
   }
 
   get data() {
@@ -139,7 +140,7 @@ export default class ModelManager {
     const index = viewList.length;
     const name = `${this.localizedData.getView(viewName)} ${index}`;
 
-    viewList.push({ name });
+    viewList.push({ name, id: this.nextViewId++ });
     this.data = assignObjKey(this.data, viewName, viewList);
     this.activateView(viewName, index);
   }

Original file line number	Diff line number	Diff line change
`@@ -47,6 +47,7 @@ export default class ModelManager {`
`47`	`47`	`this.activeViewIndex = 0;`
`48`	`48`	`this.collapseState = {};`
`49`	`49`	`this.listeners = [];`
	`50`	`+ this.nextViewId = 0;`
`50`	`51`	`}`
`51`	`52`
`52`	`53`	`get data() {`
`@@ -139,7 +140,7 @@ export default class ModelManager {`
`139`	`140`	`const index = viewList.length;`
`140`	`141`	const name = `${this.localizedData.getView(viewName)} ${index}`;
`141`	`142`
`142`		`- viewList.push({ name });`
	`143`	`+ viewList.push({ name, id: this.nextViewId++ });`
`143`	`144`	`this.data = assignObjKey(this.data, viewName, viewList);`
`144`	`145`	`this.activateView(viewName, index);`
`145`	`146`	`}`