[wp-trac] [WordPress Trac] #34988: Screen Options on taxonomy edit screen

WordPress Trac noreply at wordpress.org
Wed Dec 16 08:50:27 UTC 2015


#34988: Screen Options on taxonomy edit screen
-------------------------------------------------+-------------------------
 Reporter:  milindmore22                         |       Owner:
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  4.5
Component:  Taxonomy                             |     Version:  4.4
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch dev-feedback needs-        |     Focuses:
  testing                                        |  administration
-------------------------------------------------+-------------------------

Comment (by swissspidy):

 From my previous comment:

 > I'll update the patch to move the `$term = get_term( $term_id,
 $taxonomy, OBJECT, 'edit' );` check up. There's no need for a separate
 taxonomy check.

 The whole `term.php` can be simplified even further:

 * Passing a taxonomy is not really necessary because there are no shared
 terms anymore.
 * Passing the post type can be optional, because we can always default to
 the first `object_type` associated with the taxonomy if no post type was
 passed.

 Because of that, [35875] is kinda made obsolete here, because we need to
 check for the term before loading the admin header. Pinging @afercia for
 feedback :)

 Note that in the previous patches, the `<title>` was broken. My new patch
 [attachment:34988.5.patch]  fixes this by correctly setting `$title`.

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


More information about the wp-trac mailing list