Unused variable $num_ratings in WP_Theme_Install_List_Table

[Corphi]

WP_Theme_Install_List_Table::install_theme_info() doesn’t use its local variable $num_ratings. It seems to be superseded by wp_star_rating().

[SergeyBiryukov]

Introduced in [26380].

Well, the whole file is now unused, not sure if it still makes sense to patch it.

[Corphi]

Well, if the file is unused, shouldn’t it be removed?

[michalzuber]

It's not included

wordpress-svn$ grep -r class-wp-theme-install-list-table src
src/wp-admin/includes/ajax-actions.php:2580:    /** This filter is documented in wp-admin/includes/class-
wp-theme-install-list-table.php */

Tried other variatons, maybe required with some combination with a variable.

wordpress-svn$ grep -r class-wp-theme-install src
No result
wordpress-svn$ grep -r class-wp-theme src
src/wp-settings.php:119:require( ABSPATH . WPINC . '/class-wp-theme.php' );
wordpress-svn$ grep -r theme-install-list src
No result
wordpress-svn$ grep -r install-list src
No result

The class is used

wordpress-svn$ grep -r WP_Theme_Install_List_Table src
src/wp-admin/includes/class-wp-theme-install-list-table.php:10:class WP_Theme_Install_List_Table extends WP_Themes_List_Table
src/wp-admin/includes/list-table.php:32:   'WP_Theme_Install_List_Table' => array( 'themes', 'theme-install' ),
src/wp-admin/includes/theme-install.php:158:  $wp_list_table = _get_list_table('WP_Theme_Install_List_Table');
src/wp-admin/includes/theme-install.php:173:  $wp_list_table = _get_list_table('WP_Theme_Install_List_Table');
src/wp-admin/includes/theme-install.php:199:  $wp_list_table = _get_list_table('WP_Theme_Install_List_Table');

[michalzuber]

svn rm file and imported filter documentation

[michalzuber]

Related #28855

[michalzuber]

Function in following files

wordpress-svn$ grep -r display_theme src
src/wp-admin/includes/theme-install.php:154:function display_theme( $theme ) {
src/wp-admin/includes/theme-install.php:169:function display_themes() {
src/wp-admin/includes/theme-install.php:179:// add_action('install_themes_search', 'display_themes');
src/wp-admin/includes/theme-install.php:180:// add_action('install_themes_featured', 'display_themes');
src/wp-admin/includes/theme-install.php:181:// add_action('install_themes_new', 'display_themes');
src/wp-admin/includes/theme-install.php:182:// add_action('install_themes_updated', 'display_themes');

wordpress-svn$ grep -r install_theme_information src
src/wp-admin/includes/theme-install.php:189:function install_theme_information() {
src/wp-admin/includes/theme-install.php:205:add_action('install_themes_pre_theme-information',
 'install_theme_information');

Hook in the same file only

wordpress-svn$ grep -r install_themes_pre_theme src
src/wp-admin/includes/theme-install.php:170:add_action('install_themes_pre_theme-information',
'install_theme_information');

[michalzuber]

Only install_themes_upload() is used from wp-admin/includes/theme-install.php, everything else is unused.

[michalzuber]

Only install_themes_upload() is used in wp-admin/theme-install.php

[ocean90]

We can't remove the file for back compat reasons. Sounds like wontfix.

[michalzuber]

Replace inline doc

[michalzuber]

Thanks for the info ocean90 ;-)
28749.2.diff might be commited, replace with proper inline doc.

[DrewAPicture]

Hi michalzuber,

The filter doc currently lives in the file it was introduced for -- the second-usage in ajax-actions is merely a duplicate. It doesn't matter if the file is currently loaded or not, if it's still available in core, it's still the correct place to document that hook.

-1 for 28749.2.diff​

On track with the topic of this ticket, I can see applying num_ratings.patch​ as long as it doesn't create an issue elsewhere.

[ocean90]

In 29173:

Remove unused variable in WP_Theme_Install_List_Table::install_theme_info(). Unused since [26380].

props Corphi.
fixes #28749.