Opened 6 years ago
Closed 6 years ago
#50085 closed enhancement (maybelater)
Use `__DIR__` magic constant instead of `dirname(__FILE__)`
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Priority: | normal | |
| Severity: | normal | Version: | |
| Component: | General | Keywords: | has-patch |
| Focuses: | performance | Cc: |
Description
Now with WordPress requiring PHP 5.6 the minimum, we can replace dirname(__FILE__) calls with the PHP 5.3-introuced __DIR__ constant.
This ticket does not intend to change such calls in third party libraries, even on the ones that we are the de-facto maintainers due to the package age.
Attachments (1)
Change History (3)
#2
@
6 years ago
- Keywords close removed
- Milestone Awaiting Review deleted
- Resolution set to maybelater
- Status changed from new to closed
- Version trunk deleted
Going to close this one out in favor of submitting the changes upstream to the Requests library.
If in the future updating wp-tests-config-sample.php becomes possible without the backwards compatibility problems, this can be reopened or a new ticket can be created.
Note: See
TracTickets for help on using
tickets.
Thanks for the patch!
class-requests.phpis a part of the Requests external library, any changes should be submitted upstream.The change in
wp-tests-config-sample.phpwas previously reverted in [47201] to avoid breaking unit tests created with WP-CLIscaffoldcommand, see comment:15:ticket:48082 and #49377 for details.