#21720 closed enhancement (fixed)
Hide upload_path and upload_url_path from the UI for new installs
Reported by: | nacin | Owned by: | nacin |
---|---|---|---|
Milestone: | 3.5 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Media | Keywords: | has-patch |
Focuses: | Cc: |
Description
Changing the upload_path and upload_url_path options are pretty dangerous. They have the potential to break quite a bit of things, not to mention they have the potential of simply not working.
In #19235, I was able to simplify wp_upload_dir() so the UPLOADS constant would again be 100% reliable. If you wanted to specify the exact location of your uploads dir, this is how you should do it, the same way you can move wp-content and other directories.
There's really no good reason to show upload_path or upload_url_path in the UI unless the values are other than the default (wp-content/uploads or ''
for upload_path, and ''
for upload_url_path). With that, most of a section disappears, leaving just year/month folder organization.
Attachments (2)
Change History (12)
#2
@
12 years ago
- Owner set to nacin
- Resolution set to fixed
- Status changed from new to closed
In [21852]:
#7
follow-up:
↓ 8
@
12 years ago
There were a few changes on settings pages, i think you need to audit all the settings "help tabs" to hide (or remove) unnecessary text.
I took a stab at display-only - now shows you the real value of
wp_upload_dir()
'path' and 'url' - so the screen would be a way to debug when your code is changing those values.