Changeset 47198 for trunk/src/wp-admin/widgets.php

Timestamp:

02/06/2020 06:31:22 AM (5 years ago)

Author:

SergeyBiryukov

Message:

Code Modernization: Replace dirname( __FILE__ ) calls with __DIR__ magic constant.

This avoids the performance overhead of the function call every time dirname( __FILE__ ) was used instead of __DIR__.

This commit also includes:

• Removing unnecessary parentheses from include/require statements. These are language constructs, not function calls.
• Replacing include statements for several files with require_once, for consistency:
  • wp-admin/admin-header.php
  • wp-admin/admin-footer.php
  • wp-includes/version.php

Props ayeshrajans, desrosj, valentinbora, jrf, joostdevalk, netweb.
Fixes #48082.

File:

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

trunk/src/wp-admin/widgets.php

@@ -8 +8 @@
 
 /** WordPress Administration Bootstrap */
-require_once( dirname( __FILE__ ) . '/admin.php' );
+require_once __DIR__ . '/admin.php';
 
 /** WordPress Administration Widgets API */
-require_once( ABSPATH . 'wp-admin/includes/widgets.php' );
+require_once ABSPATH . 'wp-admin/includes/widgets.php';
 
 if ( ! current_user_can( 'edit_theme_options' ) ) {
@@ -289 +289 @@
     $key   = isset( $_GET['key'] ) ? (int) $_GET['key'] : 0;
 
-    require_once( ABSPATH . 'wp-admin/admin-header.php' ); ?>
+    require_once ABSPATH . 'wp-admin/admin-header.php'; ?>
     <div class="wrap">
     <h1><?php echo esc_html( $title ); ?></h1>
@@ -367 +367 @@
     </div>
     <?php
-    require_once( ABSPATH . 'wp-admin/admin-footer.php' );
+    require_once ABSPATH . 'wp-admin/admin-footer.php';
     exit;
 }
@@ -380 +380 @@
 );
 
-require_once( ABSPATH . 'wp-admin/admin-header.php' );
+require_once ABSPATH . 'wp-admin/admin-header.php';
 ?>
 
@@ -576 +576 @@
  */
 do_action( 'sidebar_admin_page' );
-require_once( ABSPATH . 'wp-admin/admin-footer.php' );
+require_once ABSPATH . 'wp-admin/admin-footer.php';