[wp-trac] [WordPress Trac] #27882: Address some issues found running Scrutinizer

WordPress Trac noreply at wordpress.org
Tue May 6 15:18:43 UTC 2014


#27882: Address some issues found running Scrutinizer
----------------------------+-----------------------
 Reporter:  wonderboymusic  |       Owner:
     Type:  defect (bug)    |      Status:  reopened
 Priority:  normal          |   Milestone:  4.0
Component:  General         |     Version:
 Severity:  normal          |  Resolution:
 Keywords:  needs-patch     |     Focuses:
----------------------------+-----------------------

Comment (by wonderboymusic):

 In [changeset:"28292"]:
 {{{
 #!CommitTicketReference repository="" revision="28292"
 In `ajax-actions.php`, remove dead code:

 * In `wp_ajax_add_tag()`, `$post_type` is set and never used.
 * In `wp_ajax_hidden_columns()`, `$hidden` is set twice, but only checks
 for the existence of `$_POST['hidden']` the first time. The two lines can
 be combined and work together.
 * In `wp_ajax_inline_save()`, `$mode` is set and never used.
 * In `wp_ajax_find_posts()`, `$searchand = $search = '';` is leftover
 cruft, neither variable is used. `$wpdb` does not need to be imported, it
 is never used.
 * In `wp_ajax_wp_fullscreen_save_post()`, `$post_type` is set and never
 used.
 * In `wp_ajax_save_attachment_order()`, `$post` is set and never used.
 * In `wp_ajax_send_attachment_to_editor()`, `$title` is set and never
 used.

 See #27882.
 }}}

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


More information about the wp-trac mailing list