Make WordPress Core

Opened 11 years ago

Closed 10 years ago

Last modified 10 years ago

#23091 closed enhancement (wontfix)

Need to remove deprecated get_bloginfo('url') from _fix_attachment_links() function

Reported by: hexalys's profile hexalys Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.5
Component: Administration Keywords: has-patch
Focuses: Cc:

Description

get_bloginfo('url'); is deprecated.
Should be home_url();

Attachments (2)

23091.diff (520 bytes) - added by MikeHansenMe 11 years ago.
23091.2.diff (1009 bytes) - added by SergeyBiryukov 11 years ago.

Download all attachments as: .zip

Change History (9)

#1 @alexvorn2
11 years ago

edited

Last edited 11 years ago by alexvorn2 (previous) (diff)

@MikeHansenMe
11 years ago

#2 @MikeHansenMe
11 years ago

  • Cc mdhansen@… added
  • Keywords has-patch needs-testing added; needs-patch removed

#3 @alexvorn2
11 years ago

  • Keywords needs-testing removed

#4 @SergeyBiryukov
11 years ago

  • Component changed from Warnings/Notices to Administration
  • Keywords close added
  • Type changed from defect (bug) to enhancement

23091.2.diff also renames $site_url to $home_url, to prevent confusion.

However, get_bloginfo( 'url' ) is not deprecated, only home and siteurl parameters are:
http://core.trac.wordpress.org/browser/tags/3.5.1/wp-includes/general-template.php#L407

url is still a valid parameter. We use get_bloginfo( 'url' ) in some other places as well.

#5 @MikeHansenMe
11 years ago

This is confusing in the codex as it states 'home' is deprecated but 'url' is on the same line. Then states consider using home_url() instead. I think the patch is still an improvement as get_bloginfo( 'url' ) returns home_url() anyway. Seems removing a step is good and could be done other places in core as well.

#6 @c3mdigital
10 years ago

  • Keywords close removed
  • Resolution set to wontfix
  • Status changed from new to closed
if ( get_bloginfo( 'url' ) == home_url() )
      $ticket = 'wontfix';

get_bloginfo('url') is not deprecated and returns home_url(). No reason to change internally.

#7 @SergeyBiryukov
10 years ago

  • Milestone Awaiting Review deleted
Note: See TracTickets for help on using tickets.