Changeset 10957 for trunk/wp-admin/admin-ajax.php

Timestamp:

04/16/2009 09:55:35 PM (16 years ago)

Author:

azaozz

Message:

Show proper error when adding a custom field without a value, props scohoust, fixes #7541

File:

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

trunk/wp-admin/admin-ajax.php

@@ -771 +771 @@
         if ( !current_user_can( 'edit_post', $pid ) )
             die('-1');
-        if ( '#NONE#' == $_POST['metakeyselect'] && empty($_POST['metakeyinput']) )
+        if ( isset($_POST['metakeyselect']) && '#NONE#' == $_POST['metakeyselect'] && empty($_POST['metakeyinput']) )
             die('1');
         if ( $pid < 0 ) {
@@ -785 +785 @@
                     $x->send();
                 }
-                $mid = add_meta( $pid );
+                if ( !$mid = add_meta( $pid ) ) 
+                    die(__('Please provide a custom field value.'));
             } else {
                 die('0');
             }
         } else if ( !$mid = add_meta( $pid ) ) {
-            die('0');
+            die(__('Please provide a custom field value.'));
         }
 
@@ -812 +813 @@
             die('-1');
         if ( !$u = update_meta( $mid, $key, $value ) )
-            die('1'); // We know meta exists; we also know it's unchanged (or DB error, in which case there are bigger problems).
+            die('0'); // We know meta exists; we also know it's unchanged (or DB error, in which case there are bigger problems).
+    
         $key = stripslashes($key);
         $value = stripslashes($value);