[wp-trac] [WordPress Trac] #36618: Move WP_Upgrader and WP_Upgrader_Skin subclasses into separate files

WordPress Trac noreply at wordpress.org
Sun May 8 11:12:40 UTC 2016


#36618: Move WP_Upgrader and WP_Upgrader_Skin subclasses into separate files
-------------------------------+-----------------------
 Reporter:  ocean90            |       Owner:  ocean90
     Type:  enhancement        |      Status:  assigned
 Priority:  normal             |   Milestone:  4.6
Component:  Upgrade/Install    |     Version:
 Severity:  normal             |  Resolution:
 Keywords:  2nd-opinion early  |     Focuses:
-------------------------------+-----------------------
Description changed by ocean90:

Old description:

> Related: #34432, #36335
>
> [source:/trunk/src/wp-admin/includes/class-wp-upgrader-skins.php]
> includes `WP_Upgrader_Skin`, `Plugin_Upgrader_Skin`,
> `Bulk_Upgrader_Skin`, `Plugin_Installer_Skin`, `Theme_Installer_Skin`,
> `Theme_Upgrader_Skin`, `Language_Pack_Upgrader_Skin`, and
> `Automatic_Upgrader_Skin`.
>
> [source:/trunk/src/wp-admin/includes/class-wp-upgrader.php] includes
> `WP_Upgrader`, `Plugin_Upgrader`, `Theme_Upgrader`,
> `Language_Pack_Upgrader`, and `Core_Upgrader`.
> The same file includes also `File_Upload_Upgrader` and
> `WP_Automatic_Updater` which are not a subclass of `WP_Upgrader` and the
> file includes `class-wp-upgrader-skins.php`.
>
> Sadly, the naming of the classes isn't quite good because they don't have
> a unique prefix.
>
> Based on the current naming schema we'd have the following new and old
> files:
>
> {{{
> wp-admin/includes/class-wp-upgrader.php // Includes WP_Upgrader and
> requires files below except fo class-wp-upgrader-skins.php.
> wp-admin/includes/class-plugin-upgrader.php
> wp-admin/includes/class-theme-upgrader.php
> wp-admin/includes/class-language-pack-upgrader.php
> wp-admin/includes/class-core-upgrader.php
> wp-admin/includes/class-file-upload-upgrader.php
> wp-admin/includes/class-wp-automatic-updater.php
> wp-admin/includes/class-wp-upgrader-skins.php // Only for back-compat,
> requires files below.
> wp-admin/includes/class-wp-upgrader-skin.php
> wp-admin/includes/class-plugin-upgrader-skin.php
> wp-admin/includes/class-bulk-upgrader-skin.php
> wp-admin/includes/class-plugin-installer-skin.php
> wp-admin/includes/class-theme-installer-skin.php
> wp-admin/includes/class-theme-upgrader-skin.php
> wp-admin/includes/class-language-pack-upgrader-skin.php
> wp-admin/includes/class-automatic-upgrader-skin.php
> }}}
>
> Thoughts?

New description:

 Related: #34432, #36335

 [source:/trunk/src/wp-admin/includes/class-wp-upgrader-skins.php] includes
 `WP_Upgrader_Skin`, `Plugin_Upgrader_Skin`, `Bulk_Upgrader_Skin`,
 `Plugin_Installer_Skin`, `Theme_Installer_Skin`, `Theme_Upgrader_Skin`,
 `Language_Pack_Upgrader_Skin`, and `Automatic_Upgrader_Skin`.

 [source:/trunk/src/wp-admin/includes/class-wp-upgrader.php] includes
 `WP_Upgrader`, `Plugin_Upgrader`, `Theme_Upgrader`,
 `Language_Pack_Upgrader`, and `Core_Upgrader`.
 The same file includes also `File_Upload_Upgrader` and
 `WP_Automatic_Updater` which are not a subclass of `WP_Upgrader` and the
 file includes `class-wp-upgrader-skins.php`.

 Sadly, the naming of the classes isn't quite good because they don't have
 a unique prefix.

 Based on the current naming schema we'd have the following new and old
 files:

 {{{
 wp-admin/includes/class-wp-upgrader.php // Includes WP_Upgrader and
 requires files below except fo class-wp-upgrader-skins.php.
 wp-admin/includes/class-plugin-upgrader.php
 wp-admin/includes/class-theme-upgrader.php
 wp-admin/includes/class-language-pack-upgrader.php
 wp-admin/includes/class-core-upgrader.php
 wp-admin/includes/class-file-upload-upgrader.php
 wp-admin/includes/class-wp-automatic-updater.php
 wp-admin/includes/class-wp-upgrader-skins.php // Only for back-compat,
 requires files below.
 wp-admin/includes/class-automatic-upgrader-skin.php
 wp-admin/includes/class-wp-upgrader-skin.php
 wp-admin/includes/class-bulk-upgrader-skin.php
 wp-admin/includes/class-bulk-plugin-upgrader-skin.php
 wp-admin/includes/class-bulk-theme-upgrader-skin.php
 wp-admin/includes/class-language-pack-upgrader-skin.php
 wp-admin/includes/class-plugin-upgrader-skin.php
 wp-admin/includes/class-theme-upgrader-skin.php
 wp-admin/includes/class-plugin-installer-skin.php
 wp-admin/includes/class-theme-installer-skin.php
 }}}

 Thoughts?

--

--
Ticket URL: <https://core.trac.wordpress.org/ticket/36618#comment:9>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list