Unified Diff: background.js
Issue 4515985834901504:
Issue 2664 - Remove ignoreHeaders argument from parseFilter(s) and move the logic to the UI (Closed)
Patch Set: Exclude filter list header errors using .forEach()
Created June 8, 2015, 6:36 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
lib/filterValidation.js » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: background.js |
| =================================================================== |
| --- a/background.js |
| +++ b/background.js |
| @@ -370,9 +370,9 @@ |
| case "add-filters": |
| var result = parseFilters(msg.text); |
| - if (result.error) |
| + if (result.errors.length > 0) |
| { |
| - sendResponse({status: "invalid", error: result.error}); |
| + sendResponse({status: "invalid", error: result.errors.join("\n")}); |
| break; |
| } |
« no previous file with comments
|
« no previous file
|
lib/filterValidation.js » ('j')
|
no next file with comments »
