WordPress.org

Make WordPress Core

Opened 9 years ago

Closed 7 years ago

#5623 closed enhancement (invalid)

Reduce logic structure with better regex in wp-settings.php

Reported by: sambauers Owned by:
Milestone: Priority: lowest
Severity: trivial Version: 2.5
Component: Optimization Keywords: regex has-patch needs-testing needs-unittests
Focuses: Cc:

Description

The attached patch reduces some logic in wp-settings by using slightly more efficient regex in one of the cases.

Low priority as this logic is only called during installation.

Attachments (1)

one_less_regex.diff (866 bytes) - added by sambauers 9 years ago.

Download all attachments as: .zip

Change History (5)

#1 @hansengel
9 years ago

  • Keywords has-patch needs-testing added

#2 @lloydbudd
9 years ago

  • Milestone changed from 2.5 to 2.6

#3 @jacobsantos
8 years ago

  • Keywords needs-unittests added

Needs unit tests.

#4 @Denis-de-Bernardy
7 years ago

  • Milestone 2.9 deleted
  • Resolution set to invalid
  • Status changed from new to closed

semantics between the two regex are different

one will catch wp-admin. the other will only catch wp-admin/.

Note: See TracTickets for help on using tickets.