WordPress.org

Make WordPress Core

Opened 2 years ago

Last modified 8 months ago

#27736 new enhancement

Save one query when inserting a new post

Reported by: ozh Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 3.0
Component: Posts, Post Types Keywords: has-patch needs-refresh
Focuses: administration, performance Cc:

Description

The workflow when inserting a new post is the following :

1) wp_insert_post() sets a few vars (post type, post status, ...), updates $wpdb->posts, then calls wp_set_post_categories()

2) first thing wp_set_post_categories() wants to know is $post_type and $post_status, for which it performs an SQL query, instead of getting this info from caller function

Same things goes for wp_insert_attachment()

Patch passes extra optional parameters $post_type and $post_status to wp_set_post_categories(), to save one query when adding a post, or a whole bunch when importing a batch.

Attachments (4)

27736.post.php.patch (1.5 KB) - added by ozh 2 years ago.
27736.post.php.2.patch (1.8 KB) - added by ozh 2 years ago.
fix wp_insert_attachment() as well
27736.post.php.3.patch (1.8 KB) - added by ozh 2 years ago.
sorry, coding standards... tabs, not spaces
27736.post.php.4.patch (2.1 KB) - added by ozh 2 years ago.
More coding standards: braces on all if

Download all attachments as: .zip

Change History (11)

@ozh
2 years ago

@ozh
2 years ago

fix wp_insert_attachment() as well

#1 @ozh
2 years ago

  • Focuses administration performance added
  • Keywords has-patch added

@ozh
2 years ago

sorry, coding standards... tabs, not spaces

#2 @DrewAPicture
2 years ago

This seems pretty straightforward. We're also now always using braces on all if statements.

@ozh
2 years ago

More coding standards: braces on all if

#3 @johnbillion
2 years ago

  • Severity changed from normal to minor
  • Version changed from trunk to 3.0

#4 @ozh
2 years ago

(why 3.0 ?)

#5 @johnbillion
2 years ago

The issue was introduced in version 3.0 ([13184] and [14883]).

#6 @wonderboymusic
21 months ago

  • Keywords needs-refresh added

The patch does not apply and seems to ignore all of the changes made in 4.0 (extract() removal, et al)

#7 @chriscct7
8 months ago

  • Severity changed from minor to normal
Note: See TracTickets for help on using tickets.