Skip to content

Commit 11c8f36

Browse files
committed
Fixed: FS#758 - disabling mail filter rules doesn't work
1 parent 1cbeab3 commit 11c8f36

File tree

1 file changed

+26
-22
lines changed

1 file changed

+26
-22
lines changed

interface/web/mail/mail_user_filter_edit.php

Lines changed: 26 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,9 @@ function getRule() {
114114
$content = '';
115115
$content .= '### BEGIN FILTER_ID:'.$this->id."\n";
116116

117-
if($this->dataRecord["action"] == 'move') {
117+
if($this->dataRecord["active"] == 'y') {
118+
119+
if($this->dataRecord["action"] == 'move') {
118120

119121
$content .= "
120122
`test -e ".'$DEFAULT/.'.$this->dataRecord["target"]."`
@@ -125,33 +127,35 @@ function getRule() {
125127
`echo INBOX.".$this->dataRecord["target"]." >> ".'$DEFAULT'."/courierimapsubscribed`
126128
}
127129
";
128-
}
130+
}
129131

130-
$content .= "if (/^".$this->dataRecord["source"].":";
132+
$content .= "if (/^".$this->dataRecord["source"].":";
131133

132-
$searchterm = preg_quote($this->dataRecord["searchterm"]);
134+
$searchterm = preg_quote($this->dataRecord["searchterm"]);
133135

134-
if($this->dataRecord["op"] == 'contains') {
135-
$content .= ".*".$searchterm."/:h)\n";
136-
} elseif ($this->dataRecord["op"] == 'is') {
137-
$content .= $searchterm."$/:h)\n";
138-
} elseif ($this->dataRecord["op"] == 'begins') {
139-
$content .= $searchterm."/:h)\n";
140-
} elseif ($this->dataRecord["op"] == 'ends') {
141-
$content .= ".*".$searchterm."$/:h)\n";
142-
}
136+
if($this->dataRecord["op"] == 'contains') {
137+
$content .= ".*".$searchterm."/:h)\n";
138+
} elseif ($this->dataRecord["op"] == 'is') {
139+
$content .= $searchterm."$/:h)\n";
140+
} elseif ($this->dataRecord["op"] == 'begins') {
141+
$content .= $searchterm."/:h)\n";
142+
} elseif ($this->dataRecord["op"] == 'ends') {
143+
$content .= ".*".$searchterm."$/:h)\n";
144+
}
143145

144-
$content .= "{\n";
145-
$content .= "exception {\n";
146+
$content .= "{\n";
147+
$content .= "exception {\n";
146148

147-
if($this->dataRecord["action"] == 'move') {
148-
$content .= 'to $DEFAULT/.'.$this->dataRecord["target"]."/\n";
149-
} else {
150-
$content .= "to /dev/null\n";
151-
}
149+
if($this->dataRecord["action"] == 'move') {
150+
$content .= 'to $DEFAULT/.'.$this->dataRecord["target"]."/\n";
151+
} else {
152+
$content .= "to /dev/null\n";
153+
}
152154

153-
$content .= "}\n";
154-
$content .= "}\n";
155+
$content .= "}\n";
156+
$content .= "}\n";
157+
158+
}
155159

156160
$content .= '### END FILTER_ID:'.$this->id."\n";
157161

0 commit comments

Comments
 (0)