[wpmu-trac] [WordPress MU Trac] #1054: Dashboard failing during "Right Now" dashboard widget

[WordPress MU Trac]

#1054: Dashboard failing during "Right Now" dashboard widget
----------------------------+-----------------------------------------------
 Reporter:  jasonbenesch    |        Owner:  somebody
     Type:  defect          |       Status:  new     
 Priority:  high            |    Milestone:  WPMU 2.0
Component:  Administration  |      Version:  2.8.1   
 Severity:  critical        |   Resolution:          
 Keywords:                  |  
----------------------------+-----------------------------------------------
Changes (by jasonbenesch):

  * priority:  normal => high
  * severity:  normal => critical

Comment:

 There were several more severe problems with the themes than just the
 dashboard.  get_themes() was issuing errors causing the themes.php file to
 fail.

 I believe the error is being caused by wp_kses.
 I have uncommented the line mentioned in the previous ticket have
 commented out several lines in other files:

 on wp-includes/theme.php line 211 - 213 and 222 and 225 respectively
 below:

 //$name = $theme = wp_kses( trim( $theme_name[1] ), $themes_allowed_tags
 );
 //$theme_uri = clean_url( trim( $theme_uri[1] ) );
 //$description = wptexturize( wp_kses( trim( $description[1] ),
 $themes_allowed_tags ) );

 $name = $theme = trim( $theme_name[1] );
 $theme_uri = trim( $theme_uri[1] );
 $description = trim( $description[1] );



 //$author = wp_kses( trim( $author_name[1] ), $themes_allowed_tags );
 $author = trim( $author_name[1] );


 //$author = sprintf( '<a href="%1$s" title="%2$s">%3$s</a>', $author_uri,
 __( 'Visit author homepage' ), wp_kses( trim( $author_name[1] ),
 $themes_allowed_tags ) );
                         $author = sprintf( '<a href="%1$s"
 title="%2$s">%3$s</a>', $author_uri, __( 'Visit author homepage' ), trim(
 $author_name[1] ) );




 Also on file wp-includes/kses.php I reverted the function wp_kses_split
 back to the 2.7.1 version:

 function wp_kses_split($string, $allowed_html, $allowed_protocols) {
    return preg_replace('%((<!--.*?(-->|$))|(<[^>]*(>|$)|>))%e',
         "wp_kses_split2('\\1', \$allowed_html, ".'$allowed_protocols)',
 $string);
 }


 That seems to have fixed the problem for the time being.

-- 
Ticket URL: <http://trac.mu.wordpress.org/ticket/1054#comment:1>
WordPress MU Trac <http://mu.wordpress.org/>
WordPress Multiuser