Remove unused local vars in delete_plugins()
, delete_theme()
, WP_Date_Query->validate_date_values()
, global_terms()
, and WP_Text_Diff_Renderer_Table->_changed()
.
This will clear out the "Unused Code" label in the next Scrutinizer report. See #30799. git-svn-id: https://develop.svn.wordpress.org/trunk@31217 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
parent
2ed007c023
commit
5deee71273
@ -805,9 +805,6 @@ function delete_plugins( $plugins, $deprecated = '' ) {
|
||||
|
||||
$plugins_dir = trailingslashit( $plugins_dir );
|
||||
|
||||
$translations_dir = $wp_filesystem->wp_lang_dir();
|
||||
$translations_dir = trailingslashit( $translations_dir );
|
||||
|
||||
$plugin_translations = wp_get_installed_translations( 'plugins' );
|
||||
|
||||
$errors = array();
|
||||
|
@ -69,9 +69,6 @@ function delete_theme($stylesheet, $redirect = '') {
|
||||
return new WP_Error( 'could_not_remove_theme', sprintf( __( 'Could not fully remove the theme %s.' ), $stylesheet ) );
|
||||
}
|
||||
|
||||
$translations_dir = $wp_filesystem->wp_lang_dir();
|
||||
$translations_dir = trailingslashit( $translations_dir );
|
||||
|
||||
$theme_translations = wp_get_installed_translations( 'themes' );
|
||||
|
||||
// Remove language files, silently.
|
||||
|
@ -391,7 +391,6 @@ class WP_Date_Query {
|
||||
}
|
||||
|
||||
// Throw a notice for each failing value.
|
||||
$is_between = true;
|
||||
foreach ( (array) $date_query[ $key ] as $_value ) {
|
||||
$is_between = $_value >= $check['min'] && $_value <= $check['max'];
|
||||
|
||||
|
@ -1805,11 +1805,13 @@ function global_terms( $term_id, $deprecated = '' ) {
|
||||
}
|
||||
} elseif ( $global_id != $term_id ) {
|
||||
$local_id = $wpdb->get_row( $wpdb->prepare( "SELECT term_id FROM $wpdb->terms WHERE term_id = %d", $global_id ) );
|
||||
if ( null != $local_id )
|
||||
$local_id = global_terms( $local_id );
|
||||
if ( 10 < $global_terms_recurse )
|
||||
if ( null != $local_id ) {
|
||||
global_terms( $local_id );
|
||||
if ( 10 < $global_terms_recurse ) {
|
||||
$global_id = $term_id;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
if ( $global_id != $term_id ) {
|
||||
if ( get_option( 'default_category' ) == $term_id )
|
||||
|
@ -273,7 +273,7 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer {
|
||||
$diff = $renderer->render( $text_diff );
|
||||
|
||||
// If they're too different, don't include any <ins> or <dels>
|
||||
if ( $diff_count = preg_match_all( '!(<ins>.*?</ins>|<del>.*?</del>)!', $diff, $diff_matches ) ) {
|
||||
if ( preg_match_all( '!(<ins>.*?</ins>|<del>.*?</del>)!', $diff, $diff_matches ) ) {
|
||||
// length of all text between <ins> or <del>
|
||||
$stripped_matches = strlen(strip_tags( join(' ', $diff_matches[0]) ));
|
||||
// since we count lengith of text between <ins> or <del> (instead of picking just one),
|
||||
|
Loading…
Reference in New Issue
Block a user