fix(select): Selected options now updated when options change

src/modules/select/components/multi-select.ts

@@ -145,11 +145,16 @@ export class SuiMultiSelect<T, U> extends SuiSelectBase<T, U> implements ICustom
     }
 
     protected optionsUpdateHook():void {
+        if (!this._writtenOptions && this.selectedOptions.length > 0) {
+            // We need to check the options still exist.
+            this.writeValue(this.selectedOptions.map(o => this.valueGetter(o)));
+        }
+
         if (this._writtenOptions && this.searchService.options.length > 0) {
             // If there were values written by ngModel before the options had been loaded, this runs to fix it.
             this.selectedOptions = this._writtenOptions
                 // non-null assertion added here because Typescript doesn't recognise the non-null filter.
-                .map(v => this.searchService.options.find(o => v === this.valueGetter(o))!)
+                .map(v => this.findOption(this.searchService.options, v)!)
                 .filter(v => v != undefined);
 
             if (this.selectedOptions.length === this._writtenOptions.length) {
@@ -206,6 +211,8 @@ export class SuiMultiSelect<T, U> extends SuiSelectBase<T, U> implements ICustom
             if (values.length === 0) {
                 this.selectedOptions = [];
             }
+        } else {
+            this.selectedOptions = [];
         }
     }
 

src/modules/select/components/select.ts

@@ -64,6 +64,11 @@ export class SuiSelect<T, U> extends SuiSelectBase<T, U> implements ICustomValue
     }
 
     protected optionsUpdateHook():void {
+        if (!this._writtenOption && this.selectedOption) {
+            // We need to check the option still exists.
+            this.writeValue(this.valueGetter(this.selectedOption));
+        }
+
         if (this._writtenOption && this.searchService.options.length > 0) {
             // If there was an value written by ngModel before the options had been loaded, this runs to fix it.
             this.selectedOption = this.findOption(this.searchService.options, this._writtenOption);