[wp-trac] [WordPress Trac] #48082: Replace `dirname(__FILE__)` with `__DIR__` (Performance improvement, massive patch)

WordPress Trac noreply at wordpress.org
Fri Sep 20 13:48:12 UTC 2019


#48082: Replace `dirname(__FILE__)` with `__DIR__` (Performance improvement,
massive patch)
-------------------------+--------------------------------------------
 Reporter:  ayeshrajans  |       Owner:  (none)
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  Future Release
Component:  General      |     Version:
 Severity:  minor        |  Resolution:
 Keywords:  has-patch    |     Focuses:  performance, coding-standards
-------------------------+--------------------------------------------

Comment (by joostdevalk):

 Looking through this patch I'm really liking where you're going with this.
 Some minor feedback: I see a lot of `include` statements where I think we
 should be using `require_once`. For instance includes on `admin-
 header.php` don't make sense: without that the admin doesn't make any
 sense. Or an `include` on `wp-load.php` in `xmlrpc.php`, if that fails,
 there's no WP, a require would be much better...

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/48082#comment:6>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list