[wp-trac] [WordPress Trac] #16953: Allow symlinked plugins
WordPress Trac
noreply at wordpress.org
Sat Sep 21 16:00:55 UTC 2013
#16953: Allow symlinked plugins
-------------------------------------------------+-------------------------
Reporter: scribu | Owner: rmccue
Type: enhancement | Status: assigned
Priority: normal | Milestone: Future
Component: Plugins | Release
Severity: normal | Version:
Keywords: has-patch dev-feedback needs- | Resolution:
testing |
-------------------------------------------------+-------------------------
Changes (by rmccue):
* keywords: has-patch dev-feedback => has-patch dev-feedback needs-testing
* owner: => rmccue
Comment:
Attached patch creates a realpath cache of sorts to match real directories
up with their original version. Also introduces a `wp_normalize_path()`
function, since there's a lot of duplicate code in core doing exactly that
(I only converted `plugin_basename` while I was there, but there's plenty
more places ripe for patching that in a separate ticket).
Tested locally (Windows system with junction points) and it works. I'd
like to have a few more people test it though to make sure.
The only concern I have here is performance, which is why it only stores
differing paths (plugins are loaded once, but `plugin_basename` is called
many times). Calling `realpath` shouldn't be a concern, since it's called
on the filename itself which should push it into the stat cache, and is
just repeating the `realpath` call that happens in `include` anyway.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/16953#comment:74>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list