Allow the opposite : 'nofollow' for all except the listed sites.

CommentFileSizeAuthor
#4 nofollow_admin.patch4.58 KBrszrama

Comments

Abilnet’s picture

Component: Code » User interface

What about choosing from settings, if the default is to follow OR not to follow... and then a possibility to list the sites the default rule is not affecting.

My 2 cents...

greggles’s picture

Component: User interface » Code

There are lots of features that would be valuable. I don't currently have time to provide them.

I think there should be two lists - one to remove nofollow and one to add nofollow. That way you could control both lists separately.

mediafrenzy’s picture

This absolutely makes sense, and is a feature I'd totally support being added! In fact I'm certain I need it for my next project.

rszrama’s picture

Assigned: Unassigned » rszrama
Status: Active » Needs review
StatusFileSize
new4.58 KB

We've grabbed this module and implemented the changes requested for Ubercart.org. This patch adds the appropriate code to add an administrator settings form (hook_menu, hook_perm, and a form builder function) that has a textarea for entering hosts and an option to make the list of hosts a blacklist or a whitelist. A blacklist will add the rel attribute to links going to any listed hosts while a whitelist will add it for all hosts except those listed.

This code is live and appears to be working just fine at Ubercart.

Enjoy!

greggles’s picture

looks great!

do you want commit access to the module and you can commit it yourself?

rszrama’s picture

Sure, I'd be happy to. : )

greggles’s picture

great - I added you to the committers.

Thanks!

rszrama’s picture

Status: Needs review » Fixed

Committed. Rolling it out as a new release.

Anonymous’s picture

Status: Fixed » Closed (fixed)
gurukripa’s picture

Status: Closed (fixed) » Postponed (maintainer needs more info)

i tried this module for no follow.

how does this work?

i chose whitelist..meaning..i want all links to have no follow except 1-2 i choose.

how do i know that this is actually happening ?

i went to the page source of a recent link someone had put up..and it didnt show this tag...so how do i know..pls help. thanks

gurukripa’s picture

sorry i tried it with 5.x-1.1 not dev....will this work..my question still stands...how do i know its working.

greggles’s picture

Status: Postponed (maintainer needs more info) » Closed (fixed)

You'll know it's working because rel=nofollow will be in the html.

Also, don't open up an old issue for new support. Please open a new issue.

All I can say is that I don't know how rszrama's code works, so I hope they'll be able to answer your questions.

Thanks.