Changeset 7138

Timestamp:

03/03/2008 08:58:06 PM (18 years ago)

Author:

ryan

Message:

Only show "saved at" when the post was actually saved. Props mdawaffe. fixes #6053

Location:

trunk

Files:

• wp-admin/admin-ajax.php (modified) (3 diffs)
• wp-includes/js/autosave.js (modified) (2 diffs)

trunk/wp-admin/admin-ajax.php

@@ -484 +484 @@
     $do_lock = true;
 
-    $data = '<div class="updated"><p>' . sprintf( __('Saved at %s.'), date( __('g:i:s a'), current_time( 'timestamp', true ) ) ) . '</p></div>';
+    $data = '';
+    $message = '<div class="updated"><p>' . sprintf( __('Saved at %s.'), date( __('g:i:s a'), current_time( 'timestamp', true ) ) ) . '</p></div>';
 
     $supplemental = array();
@@ -491 +492 @@
     if($_POST['post_ID'] < 0) {
         $_POST['temp_ID'] = $_POST['post_ID'];
-        if ( $do_autosave )
+        if ( $do_autosave ) {
             $id = wp_write_post();
+            $data = $message;
+        }
     } else {
         $post_ID = (int) $_POST['post_ID'];
@@ -518 +521 @@
                 die(__('You are not allowed to edit this post.'));
         }
-        if ( $do_autosave )
+        if ( $do_autosave ) {
             $id = wp_update_post($_POST);
-        else
+            $data = $message;
+        } else {
             $id = $post->ID;
+        }
     }
 

trunk/wp-includes/js/autosave.js

@@ -18 +18 @@
 // called when autosaving pre-existing post
 function autosave_saved(response) {
+    var oldMessage = jQuery('#autosave').html();
     var res = wpAjax.parseAjaxResponse(response, 'autosave'); // parse the ajax response
     var message = '';
@@ -39 +40 @@
     }
     if ( message ) { jQuery('#autosave').html(message); } // update autosave message
+    else if ( oldMessage && res ) { jQuery('#autosave').html( oldMessage ); }
     autosave_enable_buttons(); // re-enable disabled form buttons
     return res;