[drupal-devel] [feature] Implement google's rel="nofollow"

Anonymous drupal-devel at drupal.org
Wed Jan 19 16:52:20 UTC 2005


 Project:      Drupal
 Version:      cvs
 Component:    filter.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  walkah
 Reported by:  walkah
 Updated by:   Anonymous
 Status:       patch

IMO this should not be part of core. Why not add this to either the
weblink module, or spam.module?

Bèr

Anonymous



Previous comments:
------------------------------------------------------------------------

January 19, 2005 - 16:01 : walkah

Attachment: http://drupal.org/files/issues/filter-no-follow.patch (1.74 KB)

google just announced a new attribute rel="nofollow" in an attempt to
combat comment spam. the attached patch adds the option to add
rel="nofollow" as part of HTML Filter

for more info check:
http://www.google.com/googleblog/2005/01/preventing-comment-spam.html

------------------------------------------------------------------------

January 19, 2005 - 16:35 : Steven

Google dictates and the people follow ;). Personally I don't think
implementing it will stop comment spam much as there will always be
blogs and guestbooks that will not have this feature. Experience has
shown that making a medium less effective for spam simply causes
spammers to spam harder. It's only when the medium becomes 100%
ineffective, that they stop.

Still, it's a tiny patch and a popular thing to have. Notes:

- The default value for the variable is 0 in one place, FALSE in the
other. While it won't alter functionality, it's still iffy.

- Bad code style ;) (space in front of parameter)

- The title for the option could be clearer. Something like "Link
spamming discouragement" ? The option's description already tells you
about the rel="nofollow" attribute and what it is for.



------------------------------------------------------------------------

January 19, 2005 - 16:50 : walkah

Attachment: http://drupal.org/files/issues/filter-nofollow2.patch (2.27 KB)

OK, while Dries committed the first patch, he also asked for a bit more
info in the explanation.

Here's another patch that should address all the nitpicks ;)

-- 
View: http://drupal.org/node/15847
Edit: http://drupal.org/project/comments/add/15847





More information about the drupal-devel mailing list