Make WordPress Core

Opened 8 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:


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 8 years ago.

Download all attachments as: .zip

Change History (5)

#1 @hansengel
8 years ago

  • Keywords has-patch needs-testing added

#2 @lloydbudd
8 years ago

  • Milestone changed from 2.5 to 2.6

#3 @jacobsantos
7 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.