[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