WordPress.org

Make WordPress Core

Changes between Version 2 and Version 18 of Ticket #24210


Ignore:
Timestamp:
04/29/2013 01:24:55 AM (5 years ago)
Author:
SergeyBiryukov
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #24210

    • Property Milestone changed from Awaiting Review to 3.6
  • Ticket #24210 – Description

    v2 v18  
    11These all look like valid, but minor, issues:
     2{{{
     3--------------------------------
     4File       : wp-includes/class-json.php:495
     5Reason     : UnknownFunction
     6Snippet    : class_name($var)
     7Line       : : new Services_JSON_Error(class_name($var).
     8
     9--------------------------------
     10File       : wp-signup.php:493
     11Reason     : RequiredAfterOptionalParam
     12Snippet    : $user_name = ''
     13Line       : function confirm_blog_signup($domain, $path, $blog_title, $user_name = '', $user_email = '', $meta) {
     14
     15--------------------------------
     16File       : wp-signup.php:493
     17Reason     : RequiredAfterOptionalParam
     18Snippet    : $user_email = ''
     19Line       : function confirm_blog_signup($domain, $path, $blog_title, $user_name = '', $user_email = '', $meta) {
     20
     21--------------------------------
     22File       : wp-includes/widgets.php:76
     23Reason     : RequiredAfterOptionalParam
     24Snippet    : $id_base = false
     25Line       : function WP_Widget( $id_base = false, $name, $widget_options = array(), $control_options = array() ) {
     26
     27--------------------------------
     28File       : wp-includes/widgets.php:93
     29Reason     : RequiredAfterOptionalParam
     30Snippet    : $id_base = false
     31Line       : function __construct( $id_base = false, $name, $widget_options = array(), $control_options = array() ) {
     32
     33--------------------------------
     34File       : wp-includes/post.php:4789
     35Reason     : RequiredAfterOptionalParam
     36Snippet    : $deprecated = ''
     37Line       : function _future_post_hook( $deprecated = '', $post ) {
     38
     39--------------------------------
     40File       : wp-admin/includes/class-wp-terms-list-table.php:173
     41Reason     : RequiredAfterOptionalParam
     42Snippet    : $start = 0
     43Line       : function _rows( $taxonomy, $terms, &$children, $start = 0, $per_page = 20, &$count, $parent = 0, $level = 0 ) {
     44
     45--------------------------------
     46File       : wp-admin/includes/class-wp-terms-list-table.php:173
     47Reason     : RequiredAfterOptionalParam
     48Snippet    : $per_page = 20
     49Line       : function _rows( $taxonomy, $terms, &$children, $start = 0, $per_page = 20, &$count, $parent = 0, $level = 0 ) {
     50
     51--------------------------------
     52File       : wp-admin/includes/class-wp-terms-list-table.php:159
     53Reason     : TooManyArgument
     54Snippet    : $this->single_row($term, 0, $taxonomy)
     55Line       : $out .= $this->single_row( $term, 0, $taxonomy );
     56
     57--------------------------------
     58File       : wp-admin/includes/class-wp-terms-list-table.php:202
     59Reason     : TooManyArgument
     60Snippet    : $this->single_row($my_parent, $level - $num_parents, $taxonomy)
     61Line       : $output .=  "\t" . $this->single_row( $my_parent, $level - $num_parents, $taxonomy );
     62
     63--------------------------------
     64File       : wp-admin/includes/class-wp-terms-list-table.php:208
     65Reason     : TooManyArgument
     66Snippet    : $this->single_row($term, $level, $taxonomy)
     67Line       : $output .= "\t" . $this->single_row( $term, $level, $taxonomy );
     68
     69--------------------------------
     70File       : wp-includes/SimplePie/Locator.php:94
     71Reason     : RequiredAfterOptionalParam
     72Snippet    : $type = SIMPLEPIE_LOCATOR_ALL
     73Line       : public function find($type = SIMPLEPIE_LOCATOR_ALL, &$working)
     74
     75--------------------------------
     76File       : wp-includes/class-wp-walker.php:118
     77Reason     : RequiredAfterOptionalParam
     78Snippet    : $depth = 0
     79Line       : function display_element( $element, &$children_elements, $max_depth, $depth=0, $args, &$output ) {
     80
     81--------------------------------
     82File       : wp-includes/comment-template.php:1298
     83Reason     : RequiredAfterOptionalParam
     84Snippet    : $depth = 0
     85Line       : function display_element( $element, &$children_elements, $max_depth, $depth=0, $args, &$output ) {
     86
     87--------------------------------
     88File       : wp-includes/deprecated.php:802
     89Reason     : RequiredAfterOptionalParam
     90Snippet    : $echo = false
     91Line       : function get_author_link($echo = false, $author_id, $author_nicename = '') {
     92
     93--------------------------------
     94File       : wp-includes/deprecated.php:1709
     95Reason     : TooManyArgument
     96Snippet    : get_the_content($more_link_text, $stripteaser, $more_file)
     97Line       : $content = get_the_content($more_link_text, $stripteaser, $more_file);
     98
     99--------------------------------
     100File       : wp-includes/ID3/module.tag.id3v2.php:433
     101Reason     : TooFewArgument
     102Snippet    : substr($footer[5])
     103Line       : $id3_flags = ord(substr($footer{5}));
     104
     105--------------------------------
     106File       : wp-includes/ID3/module.tag.id3v2.php:1586
     107Reason     : StatementHasNoEffect
     108Snippet    : $frame_ownerid == '';
     109Line       : $frame_ownerid == '';
     110
     111--------------------------------
     112File       : wp-includes/rewrite.php:92
     113Reason     : TooManyArgument
     114Snippet    : remove_action($hook, $hook, 10, 1)
     115Line       : remove_action($hook, $hook, 10, 1);
     116
     117--------------------------------
     118File       : wp-admin/includes/user.php:350
     119Reason     : TooManyArgument
     120Snippet    : delete_user_setting('default_password_nag', $user_ID)
     121Line       : delete_user_setting('default_password_nag', $user_ID);
     122
     123--------------------------------
     124File       : wp-includes/ID3/module.audio.mp3.php:37
     125Reason     : TooManyArgument
     126Snippet    : $this->getOnlyMPEGaudioInfoBruteForce($this->getid3->fp, $info)
     127Line       : $this->getOnlyMPEGaudioInfoBruteForce($this->getid3->fp, $info);
     128
     129--------------------------------
     130File       : wp-includes/SimplePie/Misc.php:127
     131Reason     : TooManyArgument
     132Snippet    : SimplePie_Misc::entities_decode(end($attribs[$j]), 'UTF-8')
     133Line       : $return[$i]['attribs'][strtolower($attribs[$j][1])]['data'] = SimplePie_Misc::entities_decode(end($attribs[$j]),
     134'UTF-8');
     135}}}
     136
     137Resolved:
    2138{{{
    3139--------------------------------
     
    20156
    21157--------------------------------
    22 File       : wp-includes/class-json.php:495
    23 Reason     : UnknownFunction
    24 Snippet    : class_name($var)
    25 Line       : : new Services_JSON_Error(class_name($var).
    26 
    27 --------------------------------
    28 File       : wp-signup.php:493
    29 Reason     : RequiredAfterOptionalParam
    30 Snippet    : $user_name = ''
    31 Line       : function confirm_blog_signup($domain, $path, $blog_title, $user_name = '', $user_email = '', $meta) {
    32 
    33 --------------------------------
    34 File       : wp-signup.php:493
    35 Reason     : RequiredAfterOptionalParam
    36 Snippet    : $user_email = ''
    37 Line       : function confirm_blog_signup($domain, $path, $blog_title, $user_name = '', $user_email = '', $meta) {
    38 
    39 --------------------------------
    40 File       : wp-includes/widgets.php:76
    41 Reason     : RequiredAfterOptionalParam
    42 Snippet    : $id_base = false
    43 Line       : function WP_Widget( $id_base = false, $name, $widget_options = array(), $control_options = array() ) {
    44 
    45 --------------------------------
    46 File       : wp-includes/widgets.php:93
    47 Reason     : RequiredAfterOptionalParam
    48 Snippet    : $id_base = false
    49 Line       : function __construct( $id_base = false, $name, $widget_options = array(), $control_options = array() ) {
    50 
    51 --------------------------------
    52 File       : wp-includes/post.php:4789
    53 Reason     : RequiredAfterOptionalParam
    54 Snippet    : $deprecated = ''
    55 Line       : function _future_post_hook( $deprecated = '', $post ) {
    56 
    57 --------------------------------
    58 File       : wp-admin/includes/class-wp-terms-list-table.php:173
    59 Reason     : RequiredAfterOptionalParam
    60 Snippet    : $start = 0
    61 Line       : function _rows( $taxonomy, $terms, &$children, $start = 0, $per_page = 20, &$count, $parent = 0, $level = 0 ) {
    62 
    63 --------------------------------
    64 File       : wp-admin/includes/class-wp-terms-list-table.php:173
    65 Reason     : RequiredAfterOptionalParam
    66 Snippet    : $per_page = 20
    67 Line       : function _rows( $taxonomy, $terms, &$children, $start = 0, $per_page = 20, &$count, $parent = 0, $level = 0 ) {
    68 
    69 --------------------------------
    70158File       : wp-admin/includes/class-wp-terms-list-table.php:159
    71 Reason     : TooManyArgument
     159Reason     : UseVoidReturn
    72160Snippet    : $this->single_row($term, 0, $taxonomy)
    73161Line       : $out .= $this->single_row( $term, 0, $taxonomy );
     
    75163--------------------------------
    76164File       : wp-admin/includes/class-wp-terms-list-table.php:202
    77 Reason     : TooManyArgument
     165Reason     : UseVoidReturn
    78166Snippet    : $this->single_row($my_parent, $level - $num_parents, $taxonomy)
    79167Line       : $output .=  "\t" . $this->single_row( $my_parent, $level - $num_parents, $taxonomy );
     
    81169--------------------------------
    82170File       : wp-admin/includes/class-wp-terms-list-table.php:208
    83 Reason     : TooManyArgument
     171Reason     : UseVoidReturn
    84172Snippet    : $this->single_row($term, $level, $taxonomy)
    85173Line       : $output .= "\t" . $this->single_row( $term, $level, $taxonomy );
    86174
    87175--------------------------------
    88 File       : wp-admin/includes/class-wp-terms-list-table.php:159
    89 Reason     : UseVoidReturn
    90 Snippet    : $this->single_row($term, 0, $taxonomy)
    91 Line       : $out .= $this->single_row( $term, 0, $taxonomy );
    92 
    93 --------------------------------
    94 File       : wp-admin/includes/class-wp-terms-list-table.php:202
    95 Reason     : UseVoidReturn
    96 Snippet    : $this->single_row($my_parent, $level - $num_parents, $taxonomy)
    97 Line       : $output .=  "\t" . $this->single_row( $my_parent, $level - $num_parents, $taxonomy );
    98 
    99 --------------------------------
    100 File       : wp-admin/includes/class-wp-terms-list-table.php:208
    101 Reason     : UseVoidReturn
    102 Snippet    : $this->single_row($term, $level, $taxonomy)
    103 Line       : $output .= "\t" . $this->single_row( $term, $level, $taxonomy );
    104 
    105 --------------------------------
    106176File       : wp-admin/includes/class-wp-terms-list-table.php:228
    107177Reason     : UseVoidReturn
     
    110180
    111181--------------------------------
    112 File       : wp-includes/SimplePie/Locator.php:94
    113 Reason     : RequiredAfterOptionalParam
    114 Snippet    : $type = SIMPLEPIE_LOCATOR_ALL
    115 Line       : public function find($type = SIMPLEPIE_LOCATOR_ALL, &$working)
    116 
    117 --------------------------------
    118 File       : wp-includes/class-wp-walker.php:118
    119 Reason     : RequiredAfterOptionalParam
    120 Snippet    : $depth = 0
    121 Line       : function display_element( $element, &$children_elements, $max_depth, $depth=0, $args, &$output ) {
    122 
    123 --------------------------------
    124 File       : wp-includes/comment-template.php:1298
    125 Reason     : RequiredAfterOptionalParam
    126 Snippet    : $depth = 0
    127 Line       : function display_element( $element, &$children_elements, $max_depth, $depth=0, $args, &$output ) {
    128 
    129 --------------------------------
    130 File       : wp-includes/deprecated.php:802
    131 Reason     : RequiredAfterOptionalParam
    132 Snippet    : $echo = false
    133 Line       : function get_author_link($echo = false, $author_id, $author_nicename = '') {
    134 
    135 --------------------------------
    136 File       : wp-includes/deprecated.php:1709
    137 Reason     : TooManyArgument
    138 Snippet    : get_the_content($more_link_text, $stripteaser, $more_file)
    139 Line       : $content = get_the_content($more_link_text, $stripteaser, $more_file);
    140 
    141 --------------------------------
    142 File       : wp-includes/ID3/module.tag.id3v2.php:433
    143 Reason     : TooFewArgument
    144 Snippet    : substr($footer[5])
    145 Line       : $id3_flags = ord(substr($footer{5}));
    146 
    147 --------------------------------
    148 File       : wp-includes/ID3/module.tag.id3v2.php:1586
    149 Reason     : StatementHasNoEffect
    150 Snippet    : $frame_ownerid == '';
    151 Line       : $frame_ownerid == '';
    152 
    153 --------------------------------
    154 File       : wp-includes/rewrite.php:92
    155 Reason     : TooManyArgument
    156 Snippet    : remove_action($hook, $hook, 10, 1)
    157 Line       : remove_action($hook, $hook, 10, 1);
    158 
    159 --------------------------------
    160182File       : wp-includes/wp-db.php:648
    161183Reason     : TooManyArgument
     
    180202Snippet    : esc_html(maybe_unserialize($option->option_value), 'single')
    181203Line       : $option->option_value = esc_html( maybe_unserialize( $option->option_value ), 'single' );
    182 
    183 --------------------------------
    184 File       : wp-admin/includes/user.php:350
    185 Reason     : TooManyArgument
    186 Snippet    : delete_user_setting('default_password_nag', $user_ID)
    187 Line       : delete_user_setting('default_password_nag', $user_ID);
    188 
    189 --------------------------------
    190 File       : wp-includes/ID3/module.audio.mp3.php:37
    191 Reason     : TooManyArgument
    192 Snippet    : $this->getOnlyMPEGaudioInfoBruteForce($this->getid3->fp, $info)
    193 Line       : $this->getOnlyMPEGaudioInfoBruteForce($this->getid3->fp, $info);
    194 
    195 --------------------------------
    196 File       : wp-includes/SimplePie/Misc.php:127
    197 Reason     : TooManyArgument
    198 Snippet    : SimplePie_Misc::entities_decode(end($attribs[$j]), 'UTF-8')
    199 Line       : $return[$i]['attribs'][strtolower($attribs[$j][1])]['data'] = SimplePie_Misc::entities_decode(end($attribs[$j]),
    200 'UTF-8');
    201204
    202205--------------------------------