![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#2271 closed defect (bug) (fixed)
fileupload_url not set when fileupload_realpath changed
| Reported by: | skeltoac | Owned by: | ryan |
|---|---|---|---|
| Priority: | high | Milestone: | |
| Component: | Administration | Version: | 2.0 |
| Severity: | major | Keywords: | bg|has-patch |
| Cc: | Focuses: |
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)
Change History (8)
#2
@
21 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
@
21 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().
This ticket was mentioned in Slack in #forums by radices. View the logs.
11 years ago
Note:
See TracTickets
for help on using tickets.
We should drop the fileupload_url option since we don't provide a way to change it. It is derived from fileupload_realpath.