Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#34488 closed enhancement (fixed)

REST API: Save some preg_match iterations

Reported by: tobiasbg's profile TobiasBg Owned by: wonderboymusic's profile wonderboymusic
Milestone: 4.4 Priority: normal
Severity: normal Version: 4.4
Component: REST API Keywords: has-patch needs-testing
Focuses: Cc:


In the WP_REST_Server, a preg_match call and an if-check can be moved one level of for-loops up.

It doesn't depend on the $handler variable nor is it modified in the inner for-loop, so that this would save some preg_match/REGEXP iterations.

Also see

Attachments (1)

34488.diff (967 bytes) - added by TobiasBg 8 years ago.

Download all attachments as: .zip

Change History (3)

8 years ago

#1 @johnbillion
8 years ago

  • Keywords needs-testing added

#2 @wonderboymusic
8 years ago

  • Owner set to wonderboymusic
  • Resolution set to fixed
  • Status changed from new to closed

In 35514:

REST API: in WP_REST_Server::dispatch(), move preg_match() out of it's current loop, which doesn't affect the context passed to it.

Props TobiasBg.
Fixes #34488.

Note: See TracTickets for help on using tickets.