Closed (fixed)
Project:
Plus 1
Version:
6.x-2.6
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
17 Oct 2008 at 17:40 UTC
Updated:
27 Jan 2011 at 17:10 UTC
Jump to comment: Most recent file
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 commentedThat is a great idea. I will add the attribute now. Brb.
Comment #2
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 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 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 commentedYes.
Comment #8
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 commentedAny progress on this issue...?
Comment #10
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 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 commentedvoxpelli this patch didn't work for me. I have this error
How i can fix this?
Comment #13
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 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 commentedComment #16
nancydruCoding standards: indentation, "elseif".
Comment #17
nancydruThis also seems to fix the problem in #581032: Not working on IE8.
Comment #18
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.