[wp-trac] [WordPress Trac] #27048: Export: Allow multiple post types to be selected

WordPress Trac noreply at wordpress.org
Tue Jan 26 08:31:12 UTC 2016


#27048: Export: Allow multiple post types to be selected
-------------------------------------------------+-------------------------
 Reporter:  hlashbrooke                          |       Owner:
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  Future
Component:  Export                               |  Release
 Severity:  normal                               |     Version:  3.8
 Keywords:  has-patch has-screenshots needs-     |  Resolution:
  unit-tests                                     |     Focuses:
                                                 |  administration
-------------------------------------------------+-------------------------
Changes (by netweb):

 * keywords:  has-patch has-screenshots => has-patch has-screenshots needs-
     unit-tests


Comment:

 @hlashbrooke I'm adding back the `needs-unit-tests` keyword, there aren't
 actually and existing tests, hence one of the reasons of importance for
 them. (They are only included in the Export API patch)

 My biggest concern in your latest patch is the switch from '''raw MySQL'''
 queries to `WP_Query`.

 An example of this was r36068 that shipped with WordPress 4.4.1 which
 reverted a change to `WP-Query` from '''raw MySQL''' queries in WordPress
 4.4 for non-public post types, as such testing post types defined with
 `exclude_from_search` and custom post statuses will be required to be
 tested here.

 My knowledge of units testing has come a long way since my last comment in
 this ticket 2 years ago so I'll try and find some time to see if I can get
 some tests up and running.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/27048#comment:34>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list