blacklist deletion bug fix

eero-lehtinen · May 8, 2020 · d778d83 · d778d83
1 parent 9bdfa4a
commit d778d83
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/js/options.js b/js/options.js
@@ -375,7 +375,7 @@ function addSite(toList, select, input, callback) {
 function removeSite(type, button) {
     $("#" + type + "SiteItems").empty();
     var list = type === "bl" ? blocksetDatas[currentPageId].blacklist : blocksetDatas[currentPageId].whitelist;
-    list.splice(button.parent().attr("id").charAt(6), 1);
+    list.splice(parseInt(button.parent().attr("id").substring(6)), 1);
     displaySites(list, type);
     saveCurrentBlockset();
 }

diff --git a/manifest.json b/manifest.json
@@ -3,7 +3,7 @@
   "name": "Dawdle block",
   "short_name": "Dawdle block",
   "description": "Put time limits on unproductive sites.",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "options_ui": {
     "page": "options.html",
     "chrome_style": true,

diff --git a/patchNotes.txt b/patchNotes.txt
@@ -1,6 +1,9 @@
+Version 1.3.1
+- Fixed bug that caused the wrong blacklist rules to be removed when the amount of rules exceeded 10
+
 Version 1.3.0
-- removed unintended regex matching
-- added intentional regex matching
+- Removed unintended regex matching (eg. characters "?", "*", "+", and "=" didn't work correctly)
+- Added intentional regex matching
 - Added ability to choose if unfocused tabs deplete time
 - Small tweaks