Make WordPress Core

Opened 18 years ago

Closed 18 years ago

Last modified 18 years ago

#3983 closed defect (bug) (fixed)

Blogroll link adding should use clean_url()

Reported by: nateomedia's profile nateomedia Owned by: rob1n's profile rob1n
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)

3983.diff (920 bytes) - added by rob1n 18 years ago.
3983.2.diff (885 bytes) - added by rob1n 18 years ago.
Take out empty check.
3983-branch2.0.diff (895 bytes) - added by rob1n 18 years ago.
For the 2.0 branch.

Download all attachments as: .zip

Change History (16)

#1 @foolswisdom
18 years ago

  • Milestone changed from 2.1.3 to 2.2

#2 @foolswisdom
18 years ago

Likely relates to #4001.

#3 @rob1n
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 @rob1n
18 years ago

  • Summary changed from Blogroll Image Address auto inserting "http://" with relative links to Blogroll link adding should use clean_url()

@rob1n
18 years ago

#5 @rob1n
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).

@rob1n
18 years ago

Take out empty check.

#6 @rob1n
18 years ago

  • Keywords commit added

#7 @ryan
18 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [5068]) Use clean_url instead of inline preg. Props rob1n. fixes #3983 for trunk

#8 @ryan
18 years ago

(In [5069]) Use clean_url instead of inline preg. Props rob1n. fixes #3983 for 2.1

#9 @ryan
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 @ryan
18 years ago

Code in question is in user_row() in admin-functions.php BTW. We'll need to check other places too.

@rob1n
18 years ago

For the 2.0 branch.

#11 @rob1n
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.

#12 @ryan
18 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [5070]) Use clean_url instead of inline preg. Props rob1n. fixes #3983 for 2.0

#13 @rob1n
18 years ago

  • Milestone changed from 2.0.11 to 2.0.10
Note: See TracTickets for help on using tickets.