Khalid - this looks like a great module!

Reading from the google web accelerator help for webmasters, it appears that using the post method (e.g. making it a button instead of a link) would reduce the impact of crawlers. Also, using rel="nofollow" might help. In the end there will always be some crawlers that will follow content, but the impact of nice ones can be reduced.

http://webaccelerator.google.com/webmasterhelp.html#prefetch3

Comments

moshe weitzman’s picture

Title: reduce impact of crawlers by using post method and/or url parameter instead of link and/or rel=nofollow » reduce impact of crawlers by using confirmation page

i think the most consistent and bestest way is to add a confirm page. this gives users a point to reconsider their actions. maybe even restate the site policies on the confirm page.

kbahey’s picture

Priority: Minor » Normal

I don't want the "flag this" link to be a button (if I can help it).

Would the confirm_form() system.module be the best way to do this?

This way the link stays as a link, and the button is displayed in the confirm dialog only.

greggles’s picture

That sounds reasonable to me to have the button on a confirmation page.

How about nofollow on the link? It's mostly famous for SEO reasons, but also has the intended impact of telling crawlers not to follow the link (for those that respect it).

moshe weitzman’s picture

the flag link can remain a link. the button is for the confirm form ... the no_follow attribute is a good idea, but not a requirement. drupal is not so caring about this in general.

greggles’s picture

Status: Active » Needs review
StatusFileSize
new947 bytes

well, here's the nofollow part of it, at least. this was against 4.7 but I imagine it would work against HEAD as well.

greggles’s picture

StatusFileSize
new1.31 KB

actually, that should be on all three of those links, I think.

kbahey’s picture

Status: Needs review » Reviewed & tested by the community

Moshe and Greg,

You now have CVS access to this module.

Feel free to commit whatever patches that make sense.

j9’s picture

Would there be any way to create a patch file or a new release w/ this fix? This code addition really helps a site do community moderation w/out bots flagging all nodes.

Or, perhaps, would anyone know the diff command to use to create the patch file from within the module directory?

Thanks - this is a really nice submission.

kbahey’s picture

I rather have a proper solution for this, a confirmation button.

In other news, I am in discussion with a client who wants a form for the flagging, where they can enter more info (why it was flagged, comments, ...etc.)

If this materializes, then this issue will be fixed by implication.

neclimdul’s picture

StatusFileSize
new4.65 KB

Ok, this adds the confirm form and rel='nofollow' to the 5 branch and also has the nofollow hints as optional through a setting as suggested by kbahey.

kbahey’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 5.x and HEAD.

Thanks man.

neclimdul’s picture

Version: master » 4.7.x-1.x-dev
Status: Fixed » Reviewed & tested by the community

And because I'm such a nice guy, here's a port to 4.7

neclimdul’s picture

StatusFileSize
new4.25 KB

patch...

kbahey’s picture

Status: Reviewed & tested by the community » Fixed

Committed.

Anonymous’s picture

Status: Fixed » Closed (fixed)