fix: remove all stop words by iterating through array backwards (fixes …

…#3)
kbrsh · Jul 19, 2017 · 388ccaa · 388ccaa
1 parent 8507e5f
commit 388ccaa
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 4 deletions.
diff --git a/dist/wade.js b/dist/wade.js
@@ -101,8 +101,9 @@
 
     var removeStopWords = function(str) {
       var words = getWords(str);
+      var i = words.length;
 
-      for(var i = 0; i < words.length; i++) {
+      while((i--) !== 0) {
         if(stopWords.indexOf(words[i]) !== -1) {
           words.splice(i, 1);
         }
@@ -215,6 +216,10 @@
       }
     }
 
+    Wade.config = {
+      stopWords: stopWords
+    };
+
     Wade.version = "0.3.1";
 
     return Wade;

diff --git a/dist/wade.min.js b/dist/wade.min.js
diff --git a/src/index.js b/src/index.js
@@ -91,8 +91,9 @@ var removePunctuation = function(str) {
 
 var removeStopWords = function(str) {
   var words = getWords(str);
+  var i = words.length;
 
-  for(var i = 0; i < words.length; i++) {
+  while((i--) !== 0) {
     if(stopWords.indexOf(words[i]) !== -1) {
       words.splice(i, 1);
     }

diff --git a/test/search/process.js b/test/search/process.js
@@ -6,6 +6,6 @@ describe("Processing", function() {
     const data = Wade(["ALL UPPERCASE!!", "This. is wade"]).data;
 
     expect(data[0]).to.equal('uppercase');
-    expect(data[1]).to.equal('is wade');
+    expect(data[1]).to.equal('wade');
   });
 });