#2271 closed defect (bug) (fixed)
fileupload_url not set when fileupload_realpath changed
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | high | Milestone: | |
| Component: | Administration | Version: | 2.0 |
| Severity: | major | Keywords: | bg|has-patch |
| 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)
Change History (7)
comment:2
davidhouse
— 7 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.
comment:3
ryan
— 7 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().
We should drop the fileupload_url option since we don't provide a way to change it. It is derived from fileupload_realpath.