[wp-trac] [WordPress Trac] #11360: Don't nofollow links within the site

WordPress Trac wp-trac at lists.automattic.com
Mon Jan 11 13:51:03 UTC 2010


#11360: Don't nofollow links within the site
--------------------------+-------------------------------------------------
 Reporter:  caesarsgrunt  |       Owner:                         
     Type:  defect (bug)  |      Status:  new                    
 Priority:  normal        |   Milestone:  3.0                    
Component:  Comments      |     Version:  2.9                    
 Severity:  normal        |    Keywords:  has-patch needs-testing
--------------------------+-------------------------------------------------

Comment(by nacin):

 Replying to [comment:8 caesarsgrunt]:
 > Replying to [comment:6 hakre]:
 > > Some feedback: How to reflect mixed usage of HTTP and HTTPS in that
 patch? Any Idea?
 > Not sure what you mean... does http/https make any difference in the
 patch?

 The get_option() call there will only return the http:// link, not
 https://. There's a new function home_url() that could be used, but that
 still wouldn't prevent an http:// link from not being picked up on a
 forced SSL site and vice versa.

 At the very least, using home_url() makes sense to make sure the right
 protocol is searched for. Checking both protocols wouldn't be a bad idea
 though.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/11360#comment:9>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list