Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi there,
I've activated anonymous voting. In the table votingapi_vote I've found at least 40 voting entries with nearly the same ip address: 66.249.71.xxx. I've looked it up at http://www.arin.net/whois/ and found out that this was the Google bot. We should add rel="nofollow" to the voting link or something like this.
Thanks, Bjoern
Comment | File | Size | Author |
---|---|---|---|
#11 | plus1_theming_improvement.patch | 9.79 KB | voxpelli |
Comments
Comment #1
Chill35 CreditAttribution: Chill35 commentedThat is a great idea. I will add the attribute now. Brb.
Comment #2
Chill35 CreditAttribution: Chill35 commentedActually this will not work.
From Wikipedia
So what you have to do is edit the robots.txt file that comes with Drupal.
Add these lines:
Disallow: /plus1/vote/
Disallow: /?q=plus1/vote/
Report back on this, please. If that works, I will add it to the module's documentation.
Comment #3
Chill35 CreditAttribution: Chill35 commentedThis might be a long term check of course. Please try it and report back in 2-3 weeks. That would be awesome!
Not many people use anonymous voting.
Comment #4
timmillwoodAny progress on this, I have been having the same problem on http://blog.to.it now that I have enabled anonymous voting.
Comment #5
Chill35 CreditAttribution: Chill35 commentedtimmillwood, add the 2 lines to robots.txt. It really should fix the problem. And please report back if it does. I do not have a live site that uses plus 1. I cannot test this.
Comment #6
timmillwoodI have added those to the robot.txt but guess it will take a while to filter through.
Will that stop the listings on http://www.google.co.uk/search?q=site%3Ablog.to.it ?
Comment #7
Chill35 CreditAttribution: Chill35 commentedYes.
Comment #8
voxpelli CreditAttribution: voxpelli commentedRobots.txt and nofollow isn't the right solutions - the right solution is to not violate the HTTP-specification of the GET-method and submit votes through POST-request instead. A GET is not allowed to have any impact on the state of the data and letting a GET have that not only makes search engine robots trigger but can also have weird results with caches and similar.
Comment #9
asak CreditAttribution: asak commentedAny progress on this issue...?
Comment #10
voxpelli CreditAttribution: voxpelli commentedI have coded in solutions to this in the themes for the sites I'm using Plus1 on - should be fairly easy to make a patch out of it when I get time.
If any one else has a solution - please don't let that stop you from presenting it though.
Comment #11
voxpelli CreditAttribution: voxpelli commentedHere's a patch that changes the theme function as well as make it more customizable.
The change to the theme layer is that the link is now a button instead and the form POST:s the votes to the backend - which means that no search engines will index it and no cache will stop it from being processed.
This patch is dependent on #696500: Variable misused used to store state.
Comment #12
carlop CreditAttribution: carlop commentedvoxpelli this patch didn't work for me. I have this error
How i can fix this?
Comment #13
cashwilliams CreditAttribution: cashwilliams commentedI 2nd #12 as I get the same error trying to apply the patch.Opps, you have to apply the other patch first, as it says in the description.
Comment #14
cashwilliams CreditAttribution: cashwilliams commentedI've applied the patch
, but it breaks the current version of google chrome. The submit button is shown but not clickable.(had a caching issue)Everything is working and tested
Comment #15
cashwilliams CreditAttribution: cashwilliams commentedComment #16
NancyDruCoding standards: indentation, "elseif".
Comment #17
NancyDruThis also seems to fix the problem in #581032: Not working on IE8.
Comment #18
voxpelli CreditAttribution: voxpelli commentedDo we need to reroll a patch for #16 or can we fix it once we commit?
Comment #19
NancyDruIt is fixed in my copy. Just waiting on commit access.
Comment #20
NancyDruComment #21
NancyDruCommitted to 6.x-2.x-dev.