Make WordPress Core


Ignore:
Timestamp:
05/19/2014 05:03:44 AM (11 years ago)
Author:
wonderboymusic
Message:

Fix some hackificator odds and ends in wp-admin:

  • wp-activate.php and wp-admin/themes.php don't need the closing PHP tag
  • Switch single quotes for HTML attribute values to double in a few places
  • Convert include_once file.php syntax to include_once( 'file.php' )
  • Add access modifiers to methods/members in: _WP_List_Table_Compat, Walker_Nav_Menu_Edit, Walker_Nav_Menu_Checklist, WP_Screen, Walker_Category_Checklist
  • edit_user() doesn't need to import the $wpdb global
  • wp_list_widgets() doesn't need to import the $sidebars_widgets global
  • switch/endswitch syntax is not supported in Hack
  • A <ul> in wp-admin/users.php is unclosed

See #27881.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-admin/update.php

    r28126 r28500  
    1313require_once( dirname( __FILE__ ) . '/admin.php' );
    1414
    15 include_once ABSPATH . 'wp-admin/includes/class-wp-upgrader.php';
     15include_once( ABSPATH . 'wp-admin/includes/class-wp-upgrader.php' );
    1616
    1717if ( isset($_GET['action']) ) {
     
    9696            wp_die( __( 'You do not have sufficient permissions to install plugins on this site.' ) );
    9797
    98         include_once ABSPATH . 'wp-admin/includes/plugin-install.php'; //for plugins_api..
     98        include_once( ABSPATH . 'wp-admin/includes/plugin-install.php' ); //for plugins_api..
    9999
    100100        check_admin_referer('install-plugin_' . $plugin);
     
    201201            wp_die( __( 'You do not have sufficient permissions to install themes on this site.' ) );
    202202
    203         include_once ABSPATH . 'wp-admin/includes/theme-install.php'; //for themes_api..
     203        include_once( ABSPATH . 'wp-admin/includes/theme-install.php' ); //for themes_api..
    204204
    205205        check_admin_referer( 'install-theme_' . $theme );
Note: See TracChangeset for help on using the changeset viewer.