return a sorted copy from array.sort

Current behavior is to sort the original source array Fixes #249
kellyselden · Feb 26, 2017 · fc9ce65 · fc9ce65
1 parent c9ad14b
commit fc9ce65
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/addon/array/sort.js b/addon/array/sort.js
@@ -8,7 +8,7 @@ export default function(array, sortDefinition) {
   let computedCallback;
 
   if (sortDefinition === undefined) {
-    computedCallback = array => array.sort();
+    computedCallback = array => array.slice().sort();
   } else {
     computedCallback = function(array, sortDefinition) {
       let sortCallback;
@@ -38,7 +38,7 @@ export default function(array, sortDefinition) {
         };
       }
 
-      return array.sort(sortCallback);
+      return array.slice().sort(sortCallback);
     };
   }