fixes #2325
This commit is contained in:
@@ -11,7 +11,11 @@ class DmailFilter < ActiveRecord::Base
|
|||||||
end
|
end
|
||||||
|
|
||||||
def filtered?(dmail)
|
def filtered?(dmail)
|
||||||
dmail.from.level <= User::Levels::MODERATOR && (dmail.body =~ regexp || dmail.subject =~ regexp)
|
dmail.from.level <= User::Levels::MODERATOR && has_filter? && (dmail.body =~ regexp || dmail.subject =~ regexp)
|
||||||
|
end
|
||||||
|
|
||||||
|
def has_filter?
|
||||||
|
!words.strip.empty?
|
||||||
end
|
end
|
||||||
|
|
||||||
def regexp
|
def regexp
|
||||||
|
|||||||
@@ -31,6 +31,16 @@ class DmailTest < ActiveSupport::TestCase
|
|||||||
@dmail.save
|
@dmail.save
|
||||||
assert(@dmail.is_deleted)
|
assert(@dmail.is_deleted)
|
||||||
end
|
end
|
||||||
|
|
||||||
|
context "that is empty" do
|
||||||
|
setup do
|
||||||
|
@recipient.dmail_filter.update_attributes(:words => " ")
|
||||||
|
end
|
||||||
|
|
||||||
|
should "not filter everything" do
|
||||||
|
assert(!@recipient.dmail_filter.filtered?(@dmail))
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
context "from a banned user" do
|
context "from a banned user" do
|
||||||
|
|||||||
Reference in New Issue
Block a user