Fix no patterns being set for triggers (#2891)

Mudlet · Jul 31, 2019 · 0913755 · 0913755
1 parent 910bf8f
commit 0913755
Showing 1 changed file with 10 additions and 8 deletions.
diff --git a/src/TTrigger.cpp b/src/TTrigger.cpp
@@ -173,14 +173,16 @@ bool TTrigger::setRegexCodeList(QStringList regexList, QList<int> propertyList,
         qDebug() << "[CRITICAL ERROR (plz report):] Trigger name=" << mName << " aborting reason: propertyList.size() != regexList.size()";
     }
 
-    if (!addingTrigger && ((propertyList.empty()) && (!isFolder()) && (!mColorTrigger))) {
-        setError(QStringLiteral("<b><font color='blue'>%1</font></b>")
-                .arg(tr("Error: This trigger has no patterns defined, yet. Add some to activate it.")));
-        mOK_init = false;
-        return false;
-    } else if (addingTrigger) {
-        mOK_init = true;
-        return true;
+    if ((propertyList.empty()) && (!isFolder()) && (!mColorTrigger)) {
+        if (!addingTrigger) {
+            setError(QStringLiteral("<b><font color='blue'>%1</font></b>")
+                     .arg(tr("Error: This trigger has no patterns defined, yet. Add some to activate it.")));
+            mOK_init = false;
+            return false;
+        } else {
+            mOK_init = true;
+            return true;
+        }
     }
 
     bool state = true;