This is a Taxonomy Blacklist port to D6.x. After testing on two dev sites, I'm deploying this on a production box. Additional testing would be appreciated.

I've attached a tarball and a patch against Drupal 5.x

Comments

olivo.marco’s picture

Status: Active » Needs review
askibinski’s picture

Works fine so far for me. I especially like the metaphone() check.
There is a small typo in the .info file (drupal 5.x should be 6.x).
Also the menu item appears under 'content' in the admin, I think it should appear under settings.

Some thoughts on how to improve this module:
* blacklisted terms which are used are loggen in watchdog
* ability to send an email to an email-adres or role with notification of used words in context
* (and/or hook into the rules module for this)

cj-a-min’s picture

Works great bobthecow.
I agree the with askibinski, the menu should appear somewhere else.

An addition to the above improvements:

Having the .install script create it's own db table (i.e.blacklist_terms) , rather than using drupal core variable table, would be the next step to a version = "6.x-2.0".

So it would be something like this:

CREATE TABLE IF NOT EXISTS `blacklist_terms` (
  `name` varchar(128) NOT NULL DEFAULT '',
  `value` longtext NOT NULL,
  PRIMARY KEY (`name`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8;
INSERT INTO `blacklist_terms` (`name`, `value`) VALUES
('taxonomy_blacklist_active', 'i:1;'),
('taxonomy_blacklist_badwords', 's:0:"";'),
('taxonomy_blacklist_use_metaphone', 'i:0;');

The above code just needs to be modified to fit into Drupal schema method, so it can be installed with Drupal admin, otherwise you can simply import it via mysql.

The next step is to modify the .module to query the new table, instead of the variable table.

Anyways this is isn't by any means the solution; just a quick sketch.

GL

cj-a-min’s picture

Issue tags: +tags, +blacklist
StatusFileSize
new9.16 KB

Ok, I fixed the following - Super fast fixes:

* Info says version = "6.x-1.0".
* The menu now appears in admin/settings

I didn't make a patch, because the fixings are minor and non-life changing.

kenorb’s picture

Status: Needs review » Patch (to be ported)
kenorb’s picture

Issue summary: View changes
Status: Patch (to be ported) » Closed (outdated)

Closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.