[wp-trac] [WordPress Trac] #20759: Implement set_url_scheme() where appropriate

WordPress Trac wp-trac at lists.automattic.com
Fri Sep 14 23:35:26 UTC 2012


#20759: Implement set_url_scheme() where appropriate
-------------------------------------+-----------------------
 Reporter:  johnbillion              |       Owner:  ryan
     Type:  enhancement              |      Status:  reopened
 Priority:  normal                   |   Milestone:  3.5
Component:  General                  |     Version:  3.4
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |
-------------------------------------+-----------------------
Changes (by SergeyBiryukov):

 * type:  defect (bug) => enhancement


Comment:

 The ticket itself is still an enhancement.

 Replying to [comment:15 mindctrl]:
 > It strips out the https and instead of linking https://domain.com it
 only links domain.com (no http* - only relative).

 I could not reproduce stripping off the protocol, it's `'http'` on my
 install. Before [21734], however, it used to be `'https'` in this case.

 In [21734], scheme calculation was removed from `get_site_url()`, but not
 from `get_home_url()`, which is used for "Visit Site" link in the toolbar.

 [attachment:20759.3.diff] brings consistency and fixes the issue with
 "Visit Site" link in my testing.

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


More information about the wp-trac mailing list