Merge pull request #1809 from keithclark/bug-1804

Fix duplication of parts when renaming them

Author: pazdera

src/app/elements/kc-workspace-frame/kc-parts-controls.ts

@@ -205,7 +205,7 @@ export class KCPartsControls extends LitElement {
         return this.renderRoot.querySelector(`#part-${id}`) as HTMLElement;
     }
     addEntry(model : IStackEntry) : IPartsControlsEntry {
-        const item = model;
+        let item = model;
         const entry : IPartsControlsEntry = {
             _onDidChangeName: new EventEmitter<string>(),
             get onDidChangeName() { return this._onDidChangeName.event },
@@ -214,6 +214,8 @@ export class KCPartsControls extends LitElement {
                 const index = this.parts.indexOf(item);
                 this.parts.splice(index, 1, model);
                 this.parts = [...this.parts];
+                item = model;
+                item.entry = entry;
             },
             dispose: () => {
                 item.inlineDisplay.onDispose();

src/app/lib/editor/toolbox.ts

@@ -78,8 +78,8 @@ export class Toolbox extends Plugin {
         this.update();
     }
     updateEntry(oldEntry : IAPIDefinition, entry : IAPIDefinition) {
-        const index = this.entries.indexOf(oldEntry);
-        this.entries.splice(index, 1, entry);
+        const index = this._entries.indexOf(oldEntry);
+        this._entries.splice(index, 1, entry);
         this.update();
     }
     setWhitelist(whitelist : IToolboxWhitelist|null) {