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

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

CommentFileSizeAuthor
#4 mollom-policy_nofollow-1839788-4.patch788 bytesistryker
#1 mollom.policy-nofollow.1.patch790 bytessun

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Title: No Follow link » Privacy policy link should use rel="nofollow"
Component: Code » User interface
Status: Active » Needs review
StatusFileSize
new mollom.policy-nofollow.1.patch790 bytes

Yeah, that's a good idea.

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

istryker’s picture

istryker commented
Status: Needs review » Reviewed & tested by the community

Looks good

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
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.

istryker’s picture

istryker commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new mollom-policy_nofollow-1839788-4.patch788 bytes

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

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
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.

Status: Fixed » Closed (fixed)

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

  • Commit 62aaab5 on 7.x-2.x, 8.x-2.x, fbajs, actions by sun: 
    - #1839788 by sun, iStryker: Privacy policy link should use rel='...

  • Commit 62aaab5 on 7.x-2.x, 8.x-2.x, fbajs, actions by sun: 
    - #1839788 by sun, iStryker: Privacy policy link should use rel='...