Make WordPress Core

Opened 8 years ago

Closed 6 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)

@sambauers8 years ago

comment:1 @hansengel8 years ago

  • Keywords has-patch needs-testing added

comment:2 @lloydbudd8 years ago

  • Milestone changed from 2.5 to 2.6

comment:3 @jacobsantos7 years ago

  • Keywords needs-unittests added

Needs unit tests.

comment:4 @Denis-de-Bernardy6 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.