Changeset 28500 for trunk/src/wp-admin/includes/screen.php

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:

• trunk/src/wp-admin/includes/screen.php (modified) (5 diffs)

trunk/src/wp-admin/includes/screen.php

@@ -531 +531 @@
      * @since 3.3.0
      */
-    function set_current_screen() {
+    public function set_current_screen() {
         global $current_screen, $taxnow, $typenow;
         $current_screen = $this;
@@ -582 +582 @@
      * @param string $help Help text.
      */
-    static function add_old_compat_help( $screen, $help ) {
+    public static function add_old_compat_help( $screen, $help ) {
         self::$_old_compat_help[ $screen->id ] = $help;
     }
@@ -594 +594 @@
      * @param string $parent_file The parent file of the screen. Typically the $parent_file global.
      */
-    function set_parentage( $parent_file ) {
+    public function set_parentage( $parent_file ) {
         $this->parent_file = $parent_file;
         list( $this->parent_base ) = explode( '?', $parent_file );
@@ -1090 +1090 @@
      * @since 3.3.0
      */
-    function render_screen_layout() {
+    public function render_screen_layout() {
         if ( ! $this->get_option('layout_columns') )
             return;
@@ -1119 +1119 @@
      * @since 3.3.0
      */
-    function render_per_page_options() {
+    public function render_per_page_options() {
         if ( ! $this->get_option( 'per_page' ) )
             return;