Profanivore not catching an expletive

RESOLVED FIXED

Status

()

bugzilla.mozilla.org
Extensions: Other
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Robert Kaiser, Assigned: glob)

Tracking

Production
x86_64
Linux

Details

(Reporter)

Description

6 years ago
https://bugzilla.mozilla.org/show_bug.cgi?id=755593#c33

The first expletive there wasn't caught by Profanivore.

Unfortunately, this is a somewhat high-profile bug, so quite a few people will read this. We should get rid of the expletive and also try to find out why Profanivore missed it.
(Assignee)

Comment 1

6 years ago
i've filed bug 792008 to get the comment removed from the database.

/me looks at provanivore
Assignee: nobody → glob
Status: NEW → ASSIGNED
(Assignee)

Comment 2

6 years ago
profanivore was case-sensitive (lowercase only); i've committed an 'ignore-case' fix.

Committing to: bzr+ssh://bjones%40mozilla.com@bzr.mozilla.org/bmo/4.0/
modified extensions/Profanivore/Extension.pm
Committed revision 8332.

Committing to: bzr+ssh://bjones%40mozilla.com@bzr.mozilla.org/bmo/4.2/
modified extensions/Profanivore/Extension.pm
Committed revision 8351.
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Thanks, glob :-) And thanks, Kairo, for filing the bug.

Gerv
Let's not forget to upstream extension fixes... :-)

Gerv
You need to log in before you can comment on or make changes to this bug.