In some case we don't want to accept all of the words that the Yahoo Terms API returns. The logic of blacklisting is already implemented in the code. The request:
- create user interface for specifiying these words.
Possible places:
- general blacklist at admin/settings/yahoo_terms
- per content-type blacklists (see form_alter)

CommentFileSizeAuthor
#1 html_escapes_blacklist.patch2.72 KBalex_b

Comments

alex_b’s picture

Version: 5.x-1.x-dev » 4.7.x-1.x-dev
StatusFileSize
new2.72 KB

Hi Aron,

I think for the beginning a textarea on admin/yahoo_terms with one blacklisted term per line should be fine.

I just eliminated two undesired terms (quot and nbsp) with a hard coded black list, I attach the patch. It is actually only lacking the UI for editing the blacklist - we could grab it from leech's (or was it url_profile??) blacklisted urls.

I move this issue to 4.7.x, because the attached patch is for that version. This feature should be implemented for both yahoo_terms versions though.

designerbrent’s picture

Status: Active » Needs review
Shiny’s picture

Assigned: Unassigned » Shiny
Status: Needs review » Patch (to be ported)

will be ported to the D6 version

Shiny’s picture

Version: 4.7.x-1.x-dev » 6.x-6.x-dev
thedavidmeister’s picture

yes please!

this module seems freaking magical for someone who is administrating a Street Press site, where the writers aren't necessarily the same people uploading content.

the first thing i noticed, after doing a few test tags, is that yahoo can return rather undesired tags, "times new roman", "quot", "nbsp" to name a few.

i'm happy to hardcode my blacklisted tags for now, but a UI would give me peace of mind that i wouldn't forget to save these before I next update the module.