Opened 4 years ago

Last modified 4 years ago

#9679 new enhancement

Move Plugin/Theme deletion to new WP_Upgrader class

Reported by: DD32 Owned by: DD32
Priority: normal Milestone: Future Release
Component: Upgrade/Install Version: 2.8
Severity: normal Keywords: needs-patch
Cc:

Description

Expansion off #7875, The plugin/Theme deletions should be also handled by the Upgrader class.

Change History (6)

comment:1   DD324 years ago

  • Type changed from defect (bug) to enhancement

Plugin Deletions do a check upfront to verify that deleting the files of one plugin is interfeering some other plugin (same files used for multiple plugins). also plugin deletion is checking for the plugin uninstall api.

for themes i have no idea.

comment:3   DD324 years ago

Correct.. Which has absolutely no affect upon this ticket other than to mimic current backend UI.

I do not know wether or not the Upgrader Class is there to mimic backend UI or not. Wether it is part of the (then plugin and theme) API or not.

So my concrete question regarding to this ticket is:

1.) Does the upgrader class needs to handle the plugin API, especially the install and uninstall part of it?

comment:5   DD324 years ago

  • Owner set to DD32

The upgrader class is mearly back-backend, It has a UI class for the UI part of the deal.

Dont worry, I'm not going to remove functionality, Only add or improve the process. (Remember I've written this Upgrader class to remove the old code which i seemed to be the main maintainer of, In the sense that I probably understood it the best, I wasn't happy with the old code.)

I see no reason why the uninstall process cant be handled as well, It'll be left as a separate function, but tied in with the upgrader class. The upgrader class is designed to deal with all the connections and various FS methods which WordPress needs to deal with plugins/themes/custom-objects.

  • Keywords needs-patch added
Note: See TracTickets for help on using tickets.