WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#34488 closed enhancement (fixed)

REST API: Save some preg_match iterations

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

Description

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 ​https://wordpress.slack.com/archives/core-restapi/p1446136013000935.

Attachments (1)

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

Download all attachments as: .zip

Change History (3)

@TobiasBg
11 years ago

#1 @johnbillion
11 years ago

  • Keywords needs-testing added

#2 @wonderboymusic
11 years ago

  • Owner set to wonderboymusic
  • Resolution → fixed
  • Status new → 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.