Since the backend converts to lowercase, filtering blacklist entries does not returns results if the entered text is not lowercase.

CommentFileSizeAuthor
mollom-HEAD.blacklist-filter-lowercase.0.patch1.45 KBsun

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Reviewed & tested by the community

I actually tested this manually. :)

dries’s picture

dries commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed to CVS HEAD. Let's backport to D6.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Patch (to be ported) » Fixed

Thanks for reporting, reviewing, and testing! Committed the identical patch to D6. (That's why we intentionally use a wrong indentation in the D6 .js file. :)

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit edd08d3 on master, fai6, 8.x-2.x, fbajs, actions by Dries: 
    - Patch #939720 by sun: make blacklist text filter lowercase by default...

  • Commit edd08d3 on master, fai6, 8.x-2.x, fbajs, actions by Dries: 
    - Patch #939720 by sun: make blacklist text filter lowercase by default...