Changeset 13197 for trunk/wp-admin/includes/template.php

Timestamp:

02/18/2010 07:43:44 PM (16 years ago)

Author:

ryan

Message:

add_settings_error fixes. Props jeremyclarke. fixes #11474

File:

• trunk/wp-admin/includes/template.php (modified) (3 diffs)

trunk/wp-admin/includes/template.php

@@ -3126 +3126 @@
  * 
  * @param string $setting Slug title of the setting to which this error applies
- * @param string $id Slug-name to identify the error. Used as part of 'id' attribute in HTML output.
+ * @param string $code Slug-name to identify the error. Used as part of 'id' attribute in HTML output.
  * @param string $message The formatted message text to display to the user (will be shown inside styled <div> and <p>)
  * @param string $type The type of message it is, controls HTML class. Use 'error' or 'updated'.
  */
-function add_settings_error( $setting, $id, $message, $type = 'error' ) {
+function add_settings_error( $setting, $code, $message, $type = 'error' ) {
     global $wp_settings_errors;
 
@@ -3138 +3138 @@
     $new_error = array(
         'setting' => $setting,
-        'title' => $title, // @todo $title not defined. Use $id instead?
+        'code' => $code,
         'message' => $message,
         'type' => $type
@@ -3226 +3226 @@
     $output = '';
     foreach ( $settings_errors as $key => $details ) {
-        $css_id = 'setting-error-' . $details['title'];
+        $css_id = 'setting-error-' . $details['code'];
         $css_class = $details['type'] . ' fade settings-error';
         $output .= "<div id='$css_id' class='$css_class'> \n";