Download & Extend
Mollom » Issues

Privacy policy link should use rel="nofollow"

Posted by iStryker on November 13, 2012 at 8:42pm
Project:Mollom
Version:6.x-2.x-dev
Component:User interface
Category:task
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

I believe this should be a rel="nofollow" link

By submitting this form, you accept the Mollom privacy policy.

Login or register to post comments

Comments

#1

Posted by sun on January 22, 2013 at 4:49pm
Title:No Follow link» Privacy policy link should use rel="nofollow"
Component:Code» User interface
Status:active» needs review

Yeah, that's a good idea.

This will break existing user interface string translations, but well...

AttachmentSizeStatusTest resultOperations
mollom.policy-nofollow.1.patch790 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 4,883 pass(es).View details

#2

Posted by iStryker on January 22, 2013 at 4:59pm
Status:needs review» reviewed & tested by the community

Looks good

#3

Posted by sun on January 22, 2013 at 5:02pm
Version:7.x-2.x-dev» 6.x-2.x-dev
Status:reviewed & tested by the community» patch (to be ported)

Thanks for reporting, reviewing, and testing! Committed to 7.x-2.x.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

This change will need to be backported to 6.x-2.x.

#4

Posted by iStryker on January 22, 2013 at 5:46pm
Status:patch (to be ported)» needs review

patch for 6.x-2.x-dev attached.

AttachmentSizeStatusTest resultOperations
mollom-policy_nofollow-1839788-4.patch788 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 3,855 pass(es).View details

#5

Posted by sun on February 19, 2013 at 10:23pm
Status:needs review» fixed

Thanks for reporting, reviewing, and testing! Committed and pushed to 6.x-2.x.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

#6

Posted by System Message on March 5, 2013 at 10:30pm
Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

nobody click here