Fix the deepCopy() method to extend an empty array instead of object.
authorMike Taylor <mike@indexdata.com>
Tue, 12 May 2015 10:15:16 +0000 (11:15 +0100)
committerMike Taylor <mike@indexdata.com>
Tue, 12 May 2015 10:15:16 +0000 (11:15 +0100)
Remove debugging output from deepCopy().
Remove _list() private accessor method.

src/mkws-filter.js

index 6df47aa..23cb54b 100644 (file)
@@ -126,8 +126,7 @@ function filterSet(team) {
 
   that.deepCopy = function() {
     var fs = filterSet(m_team);
-    document.write("setting list to " + mkws.$.toJSON(m_list) + "<br/>");
-    fs._setList($.extend({}, m_list));
+    fs._setList($.extend([], m_list));
     return fs;
   };
 
@@ -136,11 +135,6 @@ function filterSet(team) {
     m_list = list;
   };
 
-  /// Shouldn't be used at all
-  that._list = function() {
-    return m_list;
-  };
-
   return that;
 }