Fixed bug with blacklist and whitelist.

This commit is contained in:
Mario Basic
2015-06-19 13:03:54 +02:00
parent fb6e8bee75
commit c98fbf93e9
5 changed files with 32 additions and 8 deletions

View File

@@ -959,6 +959,12 @@ function contains(line, list) {
var lines = list.split('\n');
for (var i = 0; i < lines.length; i++) {
// If by any chance one line in the list is empty or contains a blank space, ignore it
// It would probably be better to use regex here to detect blank space, but since
// this is not likely to be triggered anyway, there is no need for that yet.
if (lines[i] === '' || lines[i] === ' ') continue;
if (line.indexOf(lines[i]) > -1) {
return true;
}

View File

@@ -586,6 +586,12 @@ function contains(line, list) {
var lines = list.split('\n');
for (var i = 0; i < lines.length; i++) {
// If by any chance one line in the list is empty or contains a blank space, ignore it
// It would probably be better to use regex here to detect blank space, but since
// this is not likely to be triggered anyway, there is no need for that yet.
if (lines[i] === '' || lines[i] === ' ') continue;
if (line.indexOf(lines[i]) > -1) {
return true;
}

View File

@@ -117,20 +117,23 @@ var Options = React.createClass({
var theme = React.findDOMNode(this.refs.theme).value.trim();
var loggingType = React.findDOMNode(this.refs.loggingType).value.trim();
var loggingStyle = React.findDOMNode(this.refs.loggingStyle).value.trim();
// Trimming blacklist and whitelist removes blank lines and spaces.
var blacklist = that.state.blacklist.trim();
var whitelist = that.state.whitelist.trim();
// Sync options with google storage.
chrome.storage.sync.set({
theme: theme,
blacklist: that.state.blacklist,
whitelist: that.state.whitelist,
blacklist: blacklist,
whitelist: whitelist,
loggingType: loggingType,
loggingStyle: loggingStyle
}, function () {
// Set state to be newly entered values.
that.setState({
theme: theme,
blacklist: that.state.blacklist,
whitelist: that.state.whitelist,
blacklist: blacklist,
whitelist: whitelist,
loggingType: loggingType,
loggingStyle: loggingStyle,
displayAlert: true