#3983 closed defect (bug) (fixed)
Blogroll link adding should use clean_url()
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 2.0.10 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Administration | Keywords: | has-patch commit |
Focuses: | Cc: |
Description
Image Address under Blogroll > Add Link now appends "http://" to the front of all URL addresses. This thwarts the use of relative links and is a change in behavior from previous versions. There's no good reason for this behavioral change and it should be corrected.
Attachments (3)
Change History (16)
#3
@
18 years ago
- Keywords blogroll image address address links removed
- Owner changed from anonymous to rob1n
- Status changed from new to assigned
For starters, I don't see *why* you would be using relative URI's in your blogroll, but that aside, I agree that this "default behavior" should be changed.
Also, I think blogrolls should use clean_url() not the plain regex that is on line 614 of admin-functions.php. I'll fix this.
#4
@
18 years ago
- Summary changed from Blogroll Image Address auto inserting "http://" with relative links to Blogroll link adding should use clean_url()
#5
@
18 years ago
- Keywords has-patch added
Patch changes edit_link() (which is called by add_link()) to use clean_url(), and also use clean_url() on the RSS link (if it's not empty).
#9
@
18 years ago
Maybe clean_url only REQUEST_URI, which should begin with a slash, and not clean_url the edit-users.php link.
#10
@
18 years ago
Code in question is in user_row() in admin-functions.php BTW. We'll need to check other places too.
#11
@
18 years ago
- Milestone changed from 2.2 to 2.0.11
- Resolution fixed deleted
- Status changed from closed to reopened
- Version 2.1.2 deleted
Reopen for 2.0.
Likely relates to #4001.