Opened 10 years ago
Closed 10 years ago
#37681 closed defect (bug) (fixed)
Remove `function_exists( 'ini_get' )` from `_load_image_to_edit_path()`
| Reported by: | dd32 | Owned by: | wonderboymusic |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.7 |
| Component: | General | Version: | |
| Severity: | trivial | Keywords: | has-patch commit |
| Cc: | Focuses: |
Description
Introduced with [20806] is a check for function_exists( 'ini_get' ) we don't check the ability to call ini_get() anywhere else in core, so removing this from the check increases the consistency in core.
Attachments (1)
Change History (3)
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
If anyone is curious as to when this code is hit..
It appears to be hit if you're editing an image which was uploaded pre-2.7 and/or if the file metadata has gone missing. In that case, the image editors will treat the image URL as the source image through the filesystem wrappers, bypassing
wp_remote_get(), which is why thefopenandallow_url_fopenare there.