[wp-trac] [WordPress Trac] #14963: Plugin update shouldn't delete package from local filesystem
WordPress Trac
wp-trac at lists.automattic.com
Sun Sep 26 03:38:15 UTC 2010
#14963: Plugin update shouldn't delete package from local filesystem
-------------------------+--------------------------------------------------
Reporter: joelhardi | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Plugins | Version: 3.1
Severity: normal | Keywords: has-patch dev-feedback
-------------------------+--------------------------------------------------
While working on #14915 I discovered a small separate issue.
Currently, when upgrading a plugin, the package (zip) file is extracted to
the filesystem and then deleted, no matter what the source of the file is.
* Remotely fetched packages, after they're extracted, are deleted from
their temporary location (which is fine, that's garbage collection).
* Local files, however, are also deleted from wherever they happen to be.
I don't think that it's right to silently unlink from the user's local
filesystem like this. If I've got a file sitting on my local system
(especially outside !WordPress' file tree) I don't expect !WordPress to
delete it (without asking).
I doubt this issue has come up as I can't imagine it affects many people
(who has or would manually download a plugin package and then hack the
!WordPress upgrader to install it, instead of just unzipping into the
plugins folder?). Nevertheless I think it's undesirable behavior, and the
attached patch is a trivial enhancement.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/14963>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list