[wp-trac] [WordPress Trac] #43761: CS: Fix violations for wp-admin/install-helper.php

WordPress Trac noreply at wordpress.org
Sat Nov 19 15:40:26 UTC 2022


#43761: CS: Fix violations for wp-admin/install-helper.php
------------------------------+-------------------------------
 Reporter:  jipmoors          |       Owner:  SergeyBiryukov
     Type:  defect (bug)      |      Status:  closed
 Priority:  normal            |   Milestone:  6.2
Component:  Upgrade/Install   |     Version:
 Severity:  normal            |  Resolution:  fixed
 Keywords:  has-patch commit  |     Focuses:  coding-standards
------------------------------+-------------------------------
Changes (by SergeyBiryukov):

 * status:  reviewing => closed
 * resolution:   => fixed


Comment:

 In [changeset:"54858" 54858]:
 {{{
 #!CommitTicketReference repository="" revision="54858"
 Coding Standards: Fix WPCS issues in `wp-admin/install-helper.php`.

 This commit adds inline comments instructing PHPCS to ignore some lines
 for database queries. An explanation is provided with each instruction.

 This resolves a few WPCS warnings along the lines of:
 {{{
 Use placeholders and $wpdb->prepare(); found interpolated variable
 $table_name at "DESC $table_name"
 }}}

 Follow-up to [236], [265], [5778].

 Props jipmoors, costdev, jrf, SergeyBiryukov.
 Fixes #43761.
 }}}

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


More information about the wp-trac mailing list