Make WordPress Core

Opened 19 years ago

Closed 19 years ago

Last modified 9 years ago

#2271 closed defect (bug) (fixed)

fileupload_url not set when fileupload_realpath changed

Reported by: skeltoac's profile skeltoac Owned by: ryan's profile ryan
Milestone: Priority: high
Severity: major Version: 2.0
Component: Administration Keywords: bg|has-patch
Focuses: Cc:

Description

Not a problem if fileupload_url is empty, because wp_upload_dir() computes the url from the realpath. However, upgraders will be unable to upload if fileupload_url is set.

This patch sets fileupload_url to a computed value when fileupload_realpath is saved.

Attachments (2)

upload-url.diff (754 bytes) - added by skeltoac 19 years ago.
2271.diff (2.5 KB) - added by davidhouse 19 years ago.
Just drop fileupload_url

Download all attachments as: .zip

Change History (8)

@skeltoac
19 years ago

#1 @ryan
19 years ago

We should drop the fileupload_url option since we don't provide a way to change it. It is derived from fileupload_realpath.

@davidhouse
19 years ago

Just drop fileupload_url

#2 @davidhouse
19 years ago

As per Ryan's request. This will break people that have their uploads folder outside of their wordpress root (was allowed in 1.5), but that's getting too picky.

#3 @ryan
19 years ago

I have a slightly different patch that also changes the name of the upload path option so as to avoid interference from the old option. It also puts uploads in wp-content and turns off date dirs if safe_mode is on. safe_mode can screw up mkdir().

#4 @ryan
19 years ago

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

(In [3437]) More upload path tweaks. fixes #2271

#5 @(none)
18 years ago

  • Milestone 2.0.1 deleted

Milestone 2.0.1 deleted

This ticket was mentioned in Slack in #forums by radices. View the logs.


9 years ago

Note: See TracTickets for help on using tickets.