fix(MdTable): provide value mutation (#1789)

assign value instead of mutate value to avoid unexpected behavior fix #1786
vuematerial · Jun 7, 2018 · ae38547 · ae38547
1 parent 21d7e67
commit ae38547
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/components/MdTable/MdTable.vue b/src/components/MdTable/MdTable.vue
@@ -322,7 +322,7 @@
         if (this.MdTable.selectedItems.includes(item)) {
           this.MdTable.selectedItems = this.MdTable.selectedItems.filter(target => target !== item)
         } else {
-          this.MdTable.selectedItems.push(item)
+          this.MdTable.selectedItems = this.MdTable.selectedItems.concat([item])
         }
       },
       sortTable () {

diff --git a/src/components/MdTable/MdTableRow.vue b/src/components/MdTable/MdTableRow.vue
@@ -95,7 +95,7 @@
       },
       addSelection () {
         if (!this.isMultipleSelected) {
-          this.MdTable.selectedItems.push(this.mdItem)
+          this.MdTable.selectedItems = this.MdTable.selectedItems.concat([this.mdItem])
         }
       },
       removeSelection () {
@@ -124,7 +124,7 @@
           return false
         }
 
-        this.MdTable.selectable.push(this.mdItem)
+        this.MdTable.selectable = this.MdTable.selectable.concat([this.mdItem])
       },
       removeSelectableItem (target = this.mdItem) {
         if (this.mdSelectable === 'multiple') {