#17206 closed defect (bug) (fixed)
Deprecating wp_clone() causes some BC Issues
Reported by: | technosailor | Owned by: | technosailor |
---|---|---|---|
Milestone: | 3.2 | Priority: | normal |
Severity: | normal | Version: | |
Component: | General | Keywords: | has-patch |
Focuses: | Cc: |
Description
Notably, with the W3 Total Cache plugin. Normally, I'd say we shouldn't worry too much about plugin compatibility, but considering the widespread use of this particular one, we should consider ensuring compatibility.
I've not paid too close attention to the process of deprecation and keeping code BC. Happy to write this patch with some guidance/feedback though.
Errors from logs below for context:
[Thu Apr 21 12:53:11 2011] [error] [client 24.35.200.175] PHP Fatal error: Call to undefined function wp_clone() in /nas/wp/www/sites/aaronbrazell/wp-content/plugins/w3-total-cache/lib/W3/ObjectCache.php on line 223
Attachments (1)
Change History (10)
#3
@
14 years ago
We could move it back to load.php and leave as deprecated. I doubt we want to suddenly start loading it later as it was probably early for a reason.
#4
@
14 years ago
I'm okay with this. Not really sure what the point of deprecating it was anyway, besides dropping PHP4 compat.
#5
@
14 years ago
- Milestone changed from Awaiting Review to 3.2
It was loaded early for just this reason. I moved it to deprecated.php after auditing a bunch of object caches and not seeing any uses of wp_clone(). I obviously forgot this one. Let's put it back in load.php and leave as deprecated, as suggested by duck_.
Related: r17613, #16813