Ticket #17997: misc.diff

File misc.diff, 18.4 KB (added by Latz, 23 months ago)

  • \misc.php

    old new  
    1414 * @return unknown 
    1515 */ 
    1616function got_mod_rewrite() { 
    17         $got_rewrite = apache_mod_loaded('mod_rewrite', true); 
    18         return apply_filters('got_rewrite', $got_rewrite); 
     17        $got_rewrite = apache_mod_loaded( 'mod_rewrite', true ); 
     18        return apply_filters( 'got_rewrite', $got_rewrite ); 
    1919} 
    2020 
    2121/** 
    … …  
    3434                return $result; 
    3535        } 
    3636 
    37         if ( $markerdata = explode( "\n", implode( '', file( $filename ) ) )); 
     37        if ( $markerdata = explode( "\n", implode( '', file( $filename ) ) ) ); 
    3838        { 
    3939                $state = false; 
    4040                foreach ( $markerdata as $markerline ) { 
    41                         if (strpos($markerline, '# END ' . $marker) !== false) 
     41                        if ( strpos( $markerline, '# END ' . $marker ) !== false ) 
    4242                                $state = false; 
    4343                        if ( $state ) 
    4444                                $result[] = $markerline; 
    45                         if (strpos($markerline, '# BEGIN ' . $marker) !== false) 
     45                        if ( strpos( $markerline, '# BEGIN ' . $marker ) !== false ) 
    4646                                $state = true; 
    4747                } 
    4848        } 
    … …  
    7979                if ( $markerdata ) { 
    8080                        $state = true; 
    8181                        foreach ( $markerdata as $n => $markerline ) { 
    82                                 if (strpos($markerline, '# BEGIN ' . $marker) !== false) 
     82                                if ( strpos( $markerline, '# BEGIN ' . $marker ) !== false ) 
    8383                                        $state = false; 
    8484                                if ( $state ) { 
    8585                                        if ( $n + 1 < count( $markerdata ) ) 
    … …  
    8787                                        else 
    8888                                                fwrite( $f, "{$markerline}" ); 
    8989                                } 
    90                                 if (strpos($markerline, '# END ' . $marker) !== false) { 
     90                                if ( strpos( $markerline, '# END ' . $marker ) !== false ) { 
    9191                                        fwrite( $f, "# BEGIN {$marker}\n" ); 
    92                                         if ( is_array( $insertion )) 
     92                                        if ( is_array( $insertion ) ) 
    9393                                                foreach ( $insertion as $insertline ) 
    9494                                                        fwrite( $f, "{$insertline}\n" ); 
    9595                                        fwrite( $f, "# END {$marker}\n" ); 
    … …  
    9898                                } 
    9999                        } 
    100100                } 
    101                 if (!$foundit) { 
     101                if ( !$foundit ) { 
    102102                        fwrite( $f, "\n# BEGIN {$marker}\n" ); 
    103103                        foreach ( $insertion as $insertline ) 
    104104                                fwrite( $f, "{$insertline}\n" ); 
    … …  
    126126        global $wp_rewrite; 
    127127 
    128128        $home_path = get_home_path(); 
    129         $htaccess_file = $home_path.'.htaccess'; 
     129        $htaccess_file = $home_path . '.htaccess'; 
    130130 
    131131        // If the file doesn't already exist check for write access to the directory and whether we have some rules. 
    132132        // else check for write access to the file. 
    133         if ((!file_exists($htaccess_file) && is_writable($home_path) && $wp_rewrite->using_mod_rewrite_permalinks()) || is_writable($htaccess_file)) { 
     133        if ( ( !file_exists( $htaccess_file ) && is_writable( $home_path ) && $wp_rewrite->using_mod_rewrite_permalinks() ) || is_writable( $htaccess_file ) ) { 
    134134                if ( got_mod_rewrite() ) { 
    135135                        $rules = explode( "\n", $wp_rewrite->mod_rewrite_rules() ); 
    136136                        return insert_with_markers( $htaccess_file, 'WordPress', $rules ); 
    … …  
    158158        $web_config_file = $home_path . 'web.config'; 
    159159 
    160160        // Using win_is_writable() instead of is_writable() because of a bug in Windows PHP 
    161         if ( iis7_supports_permalinks() && ( ( ! file_exists($web_config_file) && win_is_writable($home_path) && $wp_rewrite->using_mod_rewrite_permalinks() ) || win_is_writable($web_config_file) ) ) { 
    162                 $rule = $wp_rewrite->iis7_url_rewrite_rules(false, '', ''); 
    163                 if ( ! empty($rule) ) { 
    164                         return iis7_add_rewrite_rule($web_config_file, $rule); 
     161        if ( iis7_supports_permalinks() && ( ( ! file_exists( $web_config_file ) && win_is_writable( $home_path ) && $wp_rewrite->using_mod_rewrite_permalinks() ) || win_is_writable( $web_config_file ) ) ) { 
     162                $rule = $wp_rewrite->iis7_url_rewrite_rules( false, '', '' ); 
     163                if ( ! empty( $rule ) ) { 
     164                        return iis7_add_rewrite_rule( $web_config_file, $rule ); 
    165165                } else { 
    166                         return iis7_delete_rewrite_rule($web_config_file); 
     166                        return iis7_delete_rewrite_rule( $web_config_file ); 
    167167                } 
    168168        } 
    169169        return false; 
    … …  
    177177 * @param unknown_type $file 
    178178 */ 
    179179function update_recently_edited( $file ) { 
    180         $oldfiles = (array ) get_option( 'recently_edited' ); 
     180        $oldfiles = ( array ) get_option( 'recently_edited' ); 
    181181        if ( $oldfiles ) { 
    182182                $oldfiles = array_reverse( $oldfiles ); 
    183183                $oldfiles[] = $file; 
    … …  
    202202function update_home_siteurl( $old_value, $value ) { 
    203203        global $wp_rewrite; 
    204204 
    205         if ( defined( "WP_INSTALLING" ) ) 
     205        if ( defined( 'WP_INSTALLING' ) ) 
    206206                return; 
    207207 
    208208        // If home changed, write rewrite rules to new location. 
    … …  
    221221 * @return string 
    222222 */ 
    223223function url_shorten( $url ) { 
    224         $short_url = str_replace( 'http://', '', stripslashes( $url )); 
     224        $short_url = str_replace( 'http://', '', stripslashes( $url ) ); 
    225225        $short_url = str_replace( 'www.', '', $short_url ); 
    226         if ('/' == substr( $short_url, -1 )) 
     226        if ('/' == substr( $short_url, -1 ) ) 
    227227                $short_url = substr( $short_url, 0, -1 ); 
    228228        if ( strlen( $short_url ) > 35 ) 
    229                 $short_url = substr( $short_url, 0, 32 ).'...'; 
     229                $short_url = substr( $short_url, 0, 32 ) . '...'; 
    230230        return $short_url; 
    231231} 
    232232 
    … …  
    242242 * @param array $vars An array of globals to reset. 
    243243 */ 
    244244function wp_reset_vars( $vars ) { 
    245         for ( $i=0; $i<count( $vars ); $i += 1 ) { 
     245        for ( $i = 0; $i < count( $vars ); $i += 1 ) { 
    246246                $var = $vars[$i]; 
    247247                global $$var; 
    248248 
    249                 if ( empty( $_POST[$var] ) ) { 
    250                         if ( empty( $_GET[$var] ) ) 
     249                if ( empty( $_POST[ $var ] ) ) { 
     250                        if ( empty( $_GET[ $var ] ) ) 
    251251                                $$var = ''; 
    252252                        else 
    253                                 $$var = $_GET[$var]; 
     253                                $$var = $_GET[ $var ]; 
    254254                } else { 
    255                         $$var = $_POST[$var]; 
     255                        $$var = $_POST[ $var ]; 
    256256                } 
    257257        } 
    258258} 
    … …  
    264264 * 
    265265 * @param unknown_type $message 
    266266 */ 
    267 function show_message($message) { 
    268         if ( is_wp_error($message) ){ 
     267function show_message( $message ) { 
     268        if ( is_wp_error( $message) ){ 
    269269                if ( $message->get_error_data() ) 
    270270                        $message = $message->get_error_message() . ': ' . $message->get_error_data(); 
    271271                else 
    … …  
    284284 * @param unknown_type $message 
    285285 */ 
    286286function wp_doc_link_parse( $content ) { 
    287         if ( !is_string( $content ) || empty( $content ) ) 
     287        if ( ! is_string( $content ) || empty( $content ) ) 
    288288                return array(); 
    289289 
    290         if ( !function_exists('token_get_all') ) 
     290        if ( ! function_exists( 'token_get_all' ) ) 
    291291                return array(); 
    292292 
    293293        $tokens = token_get_all( $content ); 
    294294        $functions = array(); 
    295295        $ignore_functions = array(); 
    296296        for ( $t = 0, $count = count( $tokens ); $t < $count; $t++ ) { 
    297                 if ( !is_array( $tokens[$t] ) ) continue; 
    298                 if ( T_STRING == $tokens[$t][0] && ( '(' == $tokens[ $t + 1 ] || '(' == $tokens[ $t + 2 ] ) ) { 
     297                if ( ! is_array( $tokens[ $t ] ) ) continue; 
     298                if ( T_STRING == $tokens[ $t ][ 0 ] && ( '(' == $tokens[ $t + 1 ] || '(' == $tokens[ $t + 2 ] ) ) { 
    299299                        // If it's a function or class defined locally, there's not going to be any docs available 
    300                         if ( ( isset( $tokens[ $t - 2 ][1] ) && in_array( $tokens[ $t - 2 ][1], array( 'function', 'class' ) ) ) || ( isset( $tokens[ $t - 2 ][0] ) && T_OBJECT_OPERATOR == $tokens[ $t - 1 ][0] ) ) { 
    301                                 $ignore_functions[] = $tokens[$t][1]; 
     300                        if ( ( isset( $tokens[ $t - 2 ][1] ) && in_array( $tokens[ $t - 2 ][ 1 ], array( 'function', 'class' ) ) ) || ( isset( $tokens[ $t - 2 ][ 0 ] ) && T_OBJECT_OPERATOR == $tokens[ $t - 1 ][ 0 ] ) ) { 
     301                                $ignore_functions[] = $tokens[$t][ 1 ]; 
    302302                        } 
    303303                        // Add this to our stack of unique references 
    304                         $functions[] = $tokens[$t][1]; 
     304                        $functions[] = $tokens[ $t ][ 1 ]; 
    305305                } 
    306306        } 
    307307 
    … …  
    327327**/ 
    328328function set_screen_options() { 
    329329 
    330         if ( isset($_POST['wp_screen_options']) && is_array($_POST['wp_screen_options']) ) { 
     330        if ( isset( $_POST[ 'wp_screen_options' ] ) && is_array( $_POST[ 'wp_screen_options' ] ) ) { 
    331331                check_admin_referer( 'screen-options-nonce', 'screenoptionnonce' ); 
    332332 
    333333                if ( !$user = wp_get_current_user() ) 
    334334                        return; 
    335                 $option = $_POST['wp_screen_options']['option']; 
    336                 $value = $_POST['wp_screen_options']['value']; 
    337  
    338                 if ( !preg_match( '/^[a-z_-]+$/', $option ) ) 
     335                $option = $_POST[ 'wp_screen_options' ][ 'option' ]; 
     336                $value = $_POST[ 'wp_screen_options' ][ 'value' ]; 
     337 
     338                if ( ! preg_match( '/^[a-z_-]+$/', $option ) ) 
    339339                        return; 
    340340 
    341                 $option = str_replace('-', '_', $option); 
     341                $option = str_replace( '-', '_', $option ); 
    342342 
    343343                $map_option = $option; 
    344                 $type = str_replace('edit_', '', $map_option); 
    345                 $type = str_replace('_per_page', '', $type); 
    346                 if ( in_array($type, get_post_types()) ) 
     344                $type = str_replace( 'edit_', '', $map_option ); 
     345                $type = str_replace( '_per_page', '', $type ); 
     346                if ( in_array( $type, get_post_types() ) ) 
    347347                        $map_option = 'edit_per_page'; 
    348                 if ( in_array( $type, get_taxonomies()) ) 
     348                if ( in_array( $type, get_taxonomies() ) ) 
    349349                        $map_option = 'edit_tags_per_page'; 
    350350 
    351351 
    … …  
    363363                        case 'plugins_network_per_page': 
    364364                        case 'themes_network_per_page': 
    365365                        case 'site_themes_network_per_page': 
    366                                 $value = (int) $value; 
     366                                $value = ( int ) $value; 
    367367                                if ( $value < 1 || $value > 999 ) 
    368368                                        return; 
    369369                                break; 
    370370                        default: 
    371                                 $value = apply_filters('set-screen-option', false, $option, $value); 
     371                                $value = apply_filters( 'set-screen-option', false, $option, $value ); 
    372372                                if ( false === $value ) 
    373373                                        return; 
    374374                                break; 
    375375                } 
    376376 
    377                 update_user_meta($user->ID, $option, $value); 
    378                 wp_redirect( remove_query_arg( array('pagenum', 'apage', 'paged'), wp_get_referer() ) ); 
     377                update_user_meta( $user->ID, $option, $value ); 
     378                wp_redirect( remove_query_arg( array( 'pagenum', 'apage', 'paged' ), wp_get_referer() ) ); 
    379379                exit; 
    380380        } 
    381381} 
    … …  
    388388 * @param unknown_type $message 
    389389 */ 
    390390function wp_menu_unfold() { 
    391         if ( isset($_GET['unfoldmenu']) ) { 
     391        if ( isset( $_GET[ 'unfoldmenu' ] ) ) { 
    392392                delete_user_setting('mfold'); 
    393                 wp_redirect( remove_query_arg( 'unfoldmenu', stripslashes($_SERVER['REQUEST_URI']) ) ); 
     393                wp_redirect( remove_query_arg( 'unfoldmenu', stripslashes( $_SERVER[ 'REQUEST_URI' ] ) ) ); 
    394394                exit; 
    395395        } 
    396396} 
    … …  
    403403 * @return bool 
    404404 * @param string $filename The file path to the configuration file 
    405405 */ 
    406 function iis7_rewrite_rule_exists($filename) { 
    407         if ( ! file_exists($filename) ) 
    408                 return false; 
    409         if ( ! class_exists('DOMDocument') ) 
     406function iis7_rewrite_rule_exists( $filename ) { 
     407        if ( ! file_exists( $filename ) ) 
     408                return false; 
     409        if ( ! class_exists( 'DOMDocument' ) ) 
    410410                return false; 
    411411 
    412412        $doc = new DOMDocument(); 
    413         if ( $doc->load($filename) === false ) 
    414                 return false; 
    415         $xpath = new DOMXPath($doc); 
    416         $rules = $xpath->query('/configuration/system.webServer/rewrite/rules/rule[starts-with(@name,\'wordpress\')]'); 
     413        if ( $doc->load( $filename ) === false ) 
     414                return false; 
     415        $xpath = new DOMXPath( $doc ); 
     416        $rules = $xpath->query( '/configuration/system.webServer/rewrite/rules/rule[starts-with(@name,\'wordpress\')]' ); 
    417417        if ( $rules->length == 0 ) 
    418418                return false; 
    419419        else 
    … …  
    428428 * @param string $filename Name of the configuration file 
    429429 * @return bool 
    430430 */ 
    431 function iis7_delete_rewrite_rule($filename) { 
     431function iis7_delete_rewrite_rule( $filename ) { 
    432432        // If configuration file does not exist then rules also do not exist so there is nothing to delete 
    433         if ( ! file_exists($filename) ) 
     433        if ( ! file_exists( $filename ) ) 
    434434                return true; 
    435435 
    436         if ( ! class_exists('DOMDocument') ) 
     436        if ( ! class_exists( 'DOMDocument' ) ) 
    437437                return false; 
    438438 
    439439        $doc = new DOMDocument(); 
    440440        $doc->preserveWhiteSpace = false; 
    441441 
    442         if ( $doc -> load($filename) === false ) 
    443                 return false; 
    444         $xpath = new DOMXPath($doc); 
    445         $rules = $xpath->query('/configuration/system.webServer/rewrite/rules/rule[starts-with(@name,\'wordpress\')]'); 
     442        if ( $doc -> load( $filename ) === false ) 
     443                return false; 
     444        $xpath = new DOMXPath( $doc ); 
     445        $rules = $xpath->query( '/configuration/system.webServer/rewrite/rules/rule[starts-with(@name,\'wordpress\')]' ); 
    446446        if ( $rules->length > 0 ) { 
    447447                $child = $rules->item(0); 
    448448                $parent = $child->parentNode; 
    449                 $parent->removeChild($child); 
     449                $parent->removeChild( $child ); 
    450450                $doc->formatOutput = true; 
    451                 saveDomDocument($doc, $filename); 
     451                saveDomDocument( $doc, $filename ); 
    452452        } 
    453453        return true; 
    454454} 
    … …  
    463463 * @return bool 
    464464 */ 
    465465function iis7_add_rewrite_rule($filename, $rewrite_rule) { 
    466         if ( ! class_exists('DOMDocument') ) 
     466        if ( ! class_exists( 'DOMDocument' ) ) 
    467467                return false; 
    468468 
    469469        // If configuration file does not exist then we create one. 
    470470        if ( ! file_exists($filename) ) { 
    471                 $fp = fopen( $filename, 'w'); 
    472                 fwrite($fp, '<configuration/>'); 
    473                 fclose($fp); 
     471                $fp = fopen( $filename, 'w' ); 
     472                fwrite( $fp, '<configuration/>' ); 
     473                fclose( $fp ); 
    474474        } 
    475475 
    476476        $doc = new DOMDocument(); 
    477477        $doc->preserveWhiteSpace = false; 
    478478 
    479         if ( $doc->load($filename) === false ) 
    480                 return false; 
    481  
    482         $xpath = new DOMXPath($doc); 
     479        if ( $doc->load( $filename ) === false ) 
     480                return false; 
     481 
     482        $xpath = new DOMXPath( $doc ); 
    483483 
    484484        // First check if the rule already exists as in that case there is no need to re-add it 
    485         $wordpress_rules = $xpath->query('/configuration/system.webServer/rewrite/rules/rule[starts-with(@name,\'wordpress\')]'); 
     485        $wordpress_rules = $xpath->query( '/configuration/system.webServer/rewrite/rules/rule[starts-with(@name,\'wordpress\')]' ); 
    486486        if ( $wordpress_rules->length > 0 ) 
    487487                return true; 
    488488 
    489489        // Check the XPath to the rewrite rule and create XML nodes if they do not exist 
    490         $xmlnodes = $xpath->query('/configuration/system.webServer/rewrite/rules'); 
     490        $xmlnodes = $xpath->query( '/configuration/system.webServer/rewrite/rules' ); 
    491491        if ( $xmlnodes->length > 0 ) { 
    492492                $rules_node = $xmlnodes->item(0); 
    493493        } else { 
    494                 $rules_node = $doc->createElement('rules'); 
    495  
    496                 $xmlnodes = $xpath->query('/configuration/system.webServer/rewrite'); 
     494                $rules_node = $doc->createElement( 'rules' ); 
     495 
     496                $xmlnodes = $xpath->query( '/configuration/system.webServer/rewrite' ); 
    497497                if ( $xmlnodes->length > 0 ) { 
    498                         $rewrite_node = $xmlnodes->item(0); 
    499                         $rewrite_node->appendChild($rules_node); 
     498                        $rewrite_node = $xmlnodes->item( 0 ); 
     499                        $rewrite_node->appendChild( $rules_node ); 
    500500                } else { 
    501                         $rewrite_node = $doc->createElement('rewrite'); 
    502                         $rewrite_node->appendChild($rules_node); 
    503  
    504                         $xmlnodes = $xpath->query('/configuration/system.webServer'); 
     501                        $rewrite_node = $doc->createElement( 'rewrite' ); 
     502                        $rewrite_node->appendChild( $rules_node ); 
     503 
     504                        $xmlnodes = $xpath->query( '/configuration/system.webServer' ); 
    505505                        if ( $xmlnodes->length > 0 ) { 
    506                                 $system_webServer_node = $xmlnodes->item(0); 
    507                                 $system_webServer_node->appendChild($rewrite_node); 
     506                                $system_webServer_node = $xmlnodes->item( 0 ); 
     507                                $system_webServer_node->appendChild( $rewrite_node ); 
    508508                        } else { 
    509                                 $system_webServer_node = $doc->createElement('system.webServer'); 
    510                                 $system_webServer_node->appendChild($rewrite_node); 
    511  
    512                                 $xmlnodes = $xpath->query('/configuration'); 
     509                                $system_webServer_node = $doc->createElement( 'system.webServer' ); 
     510                                $system_webServer_node->appendChild( $rewrite_node ); 
     511 
     512                                $xmlnodes = $xpath->query( '/configuration' ); 
    513513                                if ( $xmlnodes->length > 0 ) { 
    514                                         $config_node = $xmlnodes->item(0); 
    515                                         $config_node->appendChild($system_webServer_node); 
     514                                        $config_node = $xmlnodes->item( 0 ); 
     515                                        $config_node->appendChild( $system_webServer_node ); 
    516516                                } else { 
    517                                         $config_node = $doc->createElement('configuration'); 
    518                                         $doc->appendChild($config_node); 
    519                                         $config_node->appendChild($system_webServer_node); 
     517                                        $config_node = $doc->createElement( 'configuration' ); 
     518                                        $doc->appendChild( $config_node ); 
     519                                        $config_node->appendChild( $system_webServer_node ); 
    520520                                } 
    521521                        } 
    522522                } 
    523523        } 
    524524 
    525525        $rule_fragment = $doc->createDocumentFragment(); 
    526         $rule_fragment->appendXML($rewrite_rule); 
    527         $rules_node->appendChild($rule_fragment); 
    528  
    529         $doc->encoding = "UTF-8"; 
     526        $rule_fragment->appendXML( $rewrite_rule ); 
     527        $rules_node->appendChild( $rule_fragment ); 
     528 
     529        $doc->encoding = 'UTF-8'; 
    530530        $doc->formatOutput = true; 
    531         saveDomDocument($doc, $filename); 
     531        saveDomDocument( $doc, $filename ); 
    532532 
    533533        return true; 
    534534} 
    … …  
    541541 * @param DOMDocument $doc 
    542542 * @param string $filename 
    543543 */ 
    544 function saveDomDocument($doc, $filename) { 
     544function saveDomDocument( $doc, $filename ) { 
    545545        $config = $doc->saveXML(); 
    546546        $config = preg_replace("/([^\r])\n/", "$1\r\n", $config); 
    547         $fp = fopen($filename, 'w'); 
    548         fwrite($fp, $config); 
    549         fclose($fp); 
     547        $fp = fopen( $filename, 'w' ); 
     548        fwrite( $fp, $config ); 
     549        fclose( $fp ); 
    550550} 
    551551 
    552552/** 
    … …  
    564564         * see http://bugs.php.net/bug.php?id=30931 
    565565         */ 
    566566 
    567         if ( $path[strlen( $path ) - 1] == '/' ) // recursively return a temporary file path 
    568                 return win_is_writable( $path . uniqid( mt_rand() ) . '.tmp'); 
     567        if ( $path[ strlen( $path ) - 1 ] == '/' ) // recursively return a temporary file path 
     568                return win_is_writable( $path . uniqid( mt_rand() ) . '.tmp' ); 
    569569        else if ( is_dir( $path ) ) 
    570570                return win_is_writable( $path . '/' . uniqid( mt_rand() ) . '.tmp' ); 
    571571        // check tmp file for read/write capabilities 
    572         $should_delete_tmp_file = !file_exists( $path ); 
     572        $should_delete_tmp_file = ! file_exists( $path ); 
    573573        $f = @fopen( $path, 'a' ); 
    574574        if ( $f === false ) 
    575575                return false; 
    … …  
    588588        global $_wp_admin_css_colors, $user_id; ?> 
    589589<fieldset><legend class="screen-reader-text"><span><?php _e('Admin Color Scheme')?></span></legend> 
    590590<?php 
    591 $current_color = get_user_option('admin_color', $user_id); 
    592 if ( empty($current_color) ) 
     591$current_color = get_user_option( 'admin_color', $user_id ); 
     592if ( empty( $current_color ) ) 
    593593        $current_color = 'fresh'; 
    594594foreach ( $_wp_admin_css_colors as $color => $color_info ): ?> 
    595 <div class="color-option"><input name="admin_color" id="admin_color_<?php echo $color; ?>" type="radio" value="<?php echo esc_attr($color) ?>" class="tog" <?php checked($color, $current_color); ?> /> 
     595<div class="color-option"><input name="admin_color" id="admin_color_<?php echo $color; ?>" type="radio" value="<?php echo esc_attr($color) ?>" class="tog" <?php checked( $color, $current_color ); ?> /> 
    596596        <table class="color-palette"> 
    597597        <tr> 
    598598        <?php foreach ( $color_info->colors as $html_color ): ?> 
    599         <td style="background-color: <?php echo $html_color ?>" title="<?php echo $color ?>">&nbsp;</td> 
     599        <td style="background-color: <?php echo $html_color; ?>" title="<?php echo $color ?>">&nbsp;</td> 
    600600        <?php endforeach; ?> 
    601601        </tr> 
    602602        </table> 
    603603 
    604         <label for="admin_color_<?php echo $color; ?>"><?php echo $color_info->name ?></label> 
     604        <label for="admin_color_<?php echo $color; ?>"><?php echo $color_info->name; ?></label> 
    605605</div> 
    606606        <?php endforeach; ?> 
    607607</fieldset>