fix(editor): Single/MultiSelect Editors misbehave w/Desc Sort, fixes #46

ghiscoding · Apr 11, 2018 · 71916dc · 71916dc
1 parent 4961cab
commit 71916dc
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/aurelia-slickgrid/src/aurelia-slickgrid/editors/multipleSelectEditor.ts b/aurelia-slickgrid/src/aurelia-slickgrid/editors/multipleSelectEditor.ts
@@ -130,7 +130,7 @@ export class MultipleSelectEditor implements Editor {
     this.defaultValue = item[this.columnDef.field].map((i: any) => i.toString());
 
     this.$editorElm.find('option').each((i: number, $e: any) => {
-      if (this.defaultValue.indexOf($e.value) !== -1) {
+      if (this.defaultValue === $e.value) {
         $e.selected = true;
       } else {
         $e.selected = false;

diff --git a/aurelia-slickgrid/src/aurelia-slickgrid/editors/singleSelectEditor.ts b/aurelia-slickgrid/src/aurelia-slickgrid/editors/singleSelectEditor.ts
@@ -122,7 +122,7 @@ export class SingleSelectEditor implements Editor {
     this.defaultValue = item[this.columnDef.field].toString();
 
     this.$editorElm.find('option').each((i: number, $e: any) => {
-      if (this.defaultValue.indexOf($e.value) !== -1) {
+      if (this.defaultValue === $e.value) {
         $e.selected = true;
       } else {
         $e.selected = false;