Save one query when inserting a new post

[ozh]

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.

[ozh]

fix wp_insert_attachment() as well

[ozh]

sorry, coding standards... tabs, not spaces

[DrewAPicture]

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

[ozh]

More coding standards: braces on all if

[ozh]

(why 3.0 ?)

[johnbillion]

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

[wonderboymusic]

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