Bug in automatic/ftp plugin update

[Harald Nesland]

Hi,

Sorry if this isn't the right place, or if it is already fixed.

Just a minor spelling error ;-)


Index: wp-admin/includes/file.php
===================================================================
--- wp-admin/includes/file.php  (revision 12151)
+++ wp-admin/includes/file.php  (working copy)
@@ -840,8 +840,8 @@
         $disabled = defined('FTP_SSL') || defined('FTP_SSH') ? '' : '
disabled="disabled"';

         foreach ( $types as $name => $text ) : ?>
-       <label for="<?php echo esc_atr($name) ?>">
-               <input type="radio" name="connection_type" id="<?php
echo esc_atr($name) ?>" value="<?php echo esc_atr($name) ?>" <?php
checked($name, $connection_type); echo $disabled; ?>/>
+       <label for="<?php echo esc_attr($name) ?>">
+               <input type="radio" name="connection_type" id="<?php
echo esc_attr($name) ?>" value="<?php echo esc_attr($name) ?>" <?php
checked($name, $connection_type); echo $disabled; ?>/>
                 <?php echo $text ?>
         </label>
         <?php endforeach; ?>


_______________________________________________
wp-hackers mailing list
wp-hackers@...
http://lists.automattic.com/mailman/listinfo/wp-hackers

[Oliver Schloebe]

Better open a new ticket here (if not already submitted):
http://core.trac.wordpress.org/


On 06.11.2009 10:24, Harald Nesland wrote:

> Hi,
>
> Sorry if this isn't the right place, or if it is already fixed.
>
> Just a minor spelling error ;-)
>
>
> Index: wp-admin/includes/file.php
> ===================================================================
> --- wp-admin/includes/file.php  (revision 12151)
> +++ wp-admin/includes/file.php  (working copy)
> @@ -840,8 +840,8 @@
>         $disabled = defined('FTP_SSL') || defined('FTP_SSH') ? '' : '
> disabled="disabled"';
>
>         foreach ( $types as $name => $text ) : ?>
> - <label for="<?php echo esc_atr($name) ?>">
> - <input type="radio" name="connection_type" id="<?php echo
> esc_atr($name) ?>" value="<?php echo esc_atr($name) ?>" <?php
> checked($name, $connection_type); echo $disabled; ?>/>
> + <label for="<?php echo esc_attr($name) ?>">
> + <input type="radio" name="connection_type" id="<?php echo
> esc_attr($name) ?>" value="<?php echo esc_attr($name) ?>" <?php
> checked($name, $connection_type); echo $disabled; ?>/>
> <?php echo $text ?>
> </label>
> <?php endforeach; ?>
>
>
> _______________________________________________
> wp-hackers mailing list
> wp-hackers@...
> http://lists.automattic.com/mailman/listinfo/wp-hackers
>

--
Regards,
Oliver Schlöbe

http://wpseek.com/
http://wpworldmap.net/
http://www.schloebe.de/firefox-extensions/wordpress-helper/#english

_______________________________________________
wp-hackers mailing list
wp-hackers@...
http://lists.automattic.com/mailman/listinfo/wp-hackers

[Matt Martz-2]

> Better open a new ticket here (if not already submitted):
> http://core.trac.wordpress.org/
>
>> Hi,
>>
>> Sorry if this isn't the right place, or if it is already fixed.
>>
>> Just a minor spelling error ;-)

Meant to mention this earlier,  But Andrew Ozz saw this message and
already committed the fix.

http://core.trac.wordpress.org/changeset/12152

--
Matt Martz
matt@...
http://sivel.net/
_______________________________________________
wp-hackers mailing list
wp-hackers@...
http://lists.automattic.com/mailman/listinfo/wp-hackers