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

Timestamp:

02/06/2020 06:31:22 AM (7 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/comment.php (modified) (5 diffs)

trunk/src/wp-admin/comment.php

@@ -8 +8 @@
 
 /** Load WordPress Bootstrap */
-require_once( dirname( __FILE__ ) . '/admin.php' );
+require_once __DIR__ . '/admin.php';
 
 $parent_file  = 'edit-comments.php';
@@ -59 +59 @@
 
                 wp_enqueue_script( 'comment' );
-                require_once( ABSPATH . 'wp-admin/admin-header.php' );
+                require_once ABSPATH . 'wp-admin/admin-header.php';
 
                 $comment_id = absint( $_GET['c'] );
@@ -78 +78 @@
                 $comment = get_comment_to_edit( $comment_id );
 
-                include( ABSPATH . 'wp-admin/edit-form-comment.php' );
+                require ABSPATH . 'wp-admin/edit-form-comment.php';
 
                 break;
@@ -107 +107 @@
                 }
 
-                require_once( ABSPATH . 'wp-admin/admin-header.php' );
+                require_once ABSPATH . 'wp-admin/admin-header.php';
 
                 $formaction    = $action . 'comment';
@@ -358 +358 @@
 } // End switch.
 
-include( ABSPATH . 'wp-admin/admin-footer.php' );
+require_once ABSPATH . 'wp-admin/admin-footer.php';