Browse Source

Don't use white spaces as separator in RSS "must not contain" rules

This RegExp uses both white spaces and | as separator.
Use only the latter and drop the RegExp as it is no longer required.
adaptive-webui-19844
Gabriele 10 years ago
parent
commit
c6dbe8ea60
  1. 2
      src/rss/automatedrssdownloader.cpp
  2. 2
      src/rss/rssdownloadrule.cpp

2
src/rss/automatedrssdownloader.cpp

@ -605,7 +605,7 @@ void AutomatedRssDownloader::updateMustNotLineValidity() @@ -605,7 +605,7 @@ void AutomatedRssDownloader::updateMustNotLineValidity()
if (ui->checkRegex->isChecked())
tokens << text;
else
tokens << text.split(QRegExp("[\\s|]"));
tokens << text.split("|");
foreach (const QString &token, tokens) {
QRegExp reg(token, Qt::CaseInsensitive, ui->checkRegex->isChecked() ? QRegExp::RegExp : QRegExp::Wildcard);
if (!reg.isValid()) {

2
src/rss/rssdownloadrule.cpp

@ -133,7 +133,7 @@ void RssDownloadRule::setMustNotContain(const QString &tokens) @@ -133,7 +133,7 @@ void RssDownloadRule::setMustNotContain(const QString &tokens)
if (m_useRegex)
m_mustNotContain = QStringList() << tokens;
else
m_mustNotContain = tokens.split(QRegExp("[\\s|]"));
m_mustNotContain = tokens.split("|");
}
RssDownloadRulePtr RssDownloadRule::fromVariantHash(const QVariantHash &rule_hash)

Loading…
Cancel
Save