WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#8705 closed defect (bug) (fixed)

New uploaded media relative links are broken upon site move

Reported by: DD32 Owned by:
Milestone: 2.8 Priority: normal
Severity: normal Version: 2.7
Component: General Keywords: has-patch needs-testing
Focuses: Cc:

Description

Seems I (or who ever wrote the final patch) forgot to cover a If branch case for when $file was relative to the uploads dir.

See attached patch.

Attached patch also includes a small notice-level warning correction for a post meta function.

Attachments (2)

8705.diff (873 bytes) - added by DD32 7 years ago.
8705-2.diff (1.0 KB) - added by azaozz 7 years ago.

Download all attachments as: .zip

Change History (6)

@DD327 years ago

comment:1 @DD327 years ago

  • Keywords needs-testing added

Hm, I have a feeling this patch might not work well for old attachments after the WordPress install has moved to a new server, Eg, It appears to be creating links such as:

http://dd32.id.au/wordpress/wp-content/uploads//home/theonly/public_html/dd32.id.au/wordpress/wp-content/uploads/2007/07/uaimage.png

@azaozz7 years ago

comment:2 @azaozz7 years ago

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

(In [10253]) Fix uploaded media relative links when site has been moved, props DD32, fixes #8705 for 2.7

comment:3 @azaozz7 years ago

  • Milestone changed from 2.7.1 to 2.8
  • Resolution fixed deleted
  • Status changed from closed to reopened

Reopen for trunk

comment:4 @azaozz7 years ago

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

(In [10254]) Fix uploaded media relative links when site has been moved, props DD32, fixes #8705 for trunk

Note: See TracTickets for help on using tickets.