Changeset 11807 for trunk/wp-admin/page.php

Timestamp:

08/12/2009 10:57:15 AM (17 years ago)

Author:

azaozz

Message:

Fix bulk edit, some cleanup and message fixes for moving in/out of the trash, see #4529

File:

• trunk/wp-admin/page.php (modified) (9 diffs)

trunk/wp-admin/page.php

@@ -53 +53 @@
         $location = explode('#', $location);
         $location = $location[0] . '#postcustom';
-    } elseif ($action == 'editattachment') {
-        $location = 'attachments.php';
     } else {
         $location = add_query_arg( 'message', 1, get_edit_post_link( $page_ID, 'url' ) );
@@ -83 +81 @@
     $post = get_post_to_edit($page_ID);
 
-    if ( empty($post->ID) ) wp_die( __('You attempted to edit a page that doesn’t exist. Perhaps it was deleted?') );
-    if ( $post->post_status == 'trash' ) wp_die( __('You can’t edit this page because it is in the Trash. Please move it out of the Trash and try again.') );
+    if ( empty($post->ID) )
+        wp_die( __('You attempted to edit a page that doesn’t exist. Perhaps it was deleted?') );
+
+    if ( !current_user_can('edit_page', $page_ID) )
+        wp_die( __('You are not allowed to edit this page.') );
+
+    if ( 'trash' == $post->post_status )
+        wp_die( __('You can’t edit this page because it is in the Trash. Please move it out of the Trash and try again.') );
 
     if ( 'page' != $post->post_type ) {
@@ -98 +102 @@
     wp_enqueue_script('word-count');
 
-    if ( current_user_can('edit_page', $page_ID) ) {
-        if ( $last = wp_check_post_lock( $post->ID ) ) {
-            $last_user = get_userdata( $last );
-            $last_user_name = $last_user ? $last_user->display_name : __('Somebody');
-            $message = sprintf( __( 'Warning: %s is currently editing this page' ), esc_html( $last_user_name ) );
-            $message = str_replace( "'", "\'", "<div class='error'><p>$message</p></div>" );
-            add_action('admin_notices', create_function( '', "echo '$message';" ) );
-        } else {
-            wp_set_post_lock( $post->ID );
-            wp_enqueue_script('autosave');
-        }
-    }
-
-    if ( !current_user_can('edit_page', $page_ID) )
-        die ( __('You are not allowed to edit this page.') );
+    if ( $last = wp_check_post_lock( $post->ID ) ) {
+        $last_user = get_userdata( $last );
+        $last_user_name = $last_user ? $last_user->display_name : __('Somebody');
+        $message = sprintf( __( 'Warning: %s is currently editing this page' ), esc_html( $last_user_name ) );
+        $message = str_replace( "'", "\'", "<div class='error'><p>$message</p></div>" );
+        add_action('admin_notices', create_function( '', "echo '$message';" ) );
+    } else {
+        wp_set_post_lock( $post->ID );
+        wp_enqueue_script('autosave');
+    }
 
     include('edit-page-form.php');
@@ -143 +142 @@
 
 case 'trash':
-    $post_id = (isset($_GET['post']))  ? intval($_GET['post']) : intval($_POST['post_ID']);
+    $post_id = isset($_GET['post']) ? intval($_GET['post']) : intval($_POST['post_ID']);
     check_admin_referer('trash-page_' . $post_id);
 
@@ -152 +151 @@
 
     if ( !wp_trash_post($post_id) )
-        wp_die( __('Error in removing from trash...') );
+        wp_die( __('Error in moving to trash...') );
 
     $sendback = wp_get_referer();
-    if (strpos($sendback, 'page.php') !== false) $sendback = admin_url('edit-pages.php?trashed=1');
-    elseif (strpos($sendback, 'attachments.php') !== false) $sendback = admin_url('attachments.php');
-    else $sendback = add_query_arg('trashed', 1, $sendback);
+    if ( strpos($sendback, 'page.php') !== false )
+        $sendback = admin_url('edit-pages.php?trashed=1');
+    else
+        $sendback = add_query_arg('trashed', 1, $sendback);
+
     wp_redirect($sendback);
     exit();
@@ -163 +164 @@
 
 case 'untrash':
-    $post_id = (isset($_GET['post']))  ? intval($_GET['post']) : intval($_POST['post_ID']);
+    $post_id = isset($_GET['post']) ? intval($_GET['post']) : intval($_POST['post_ID']);
     check_admin_referer('untrash-page_' . $post_id);
 
@@ -169 +170 @@
 
     if ( !current_user_can('delete_page', $page_id) )
-        wp_die( __('You are not allowed to remove this page form the trash.') );
+        wp_die( __('You are not allowed to move this page out of the trash.') );
 
     if ( !wp_untrash_post($post_id) )
-        wp_die( __('Error in removing from trash...') );
+        wp_die( __('Error in restoring from trash...') );
 
     $sendback = wp_get_referer();
-    if (strpos($sendback, 'page.php') !== false) $sendback = admin_url('edit-pages.php?untrashed=1');
-    elseif (strpos($sendback, 'attachments.php') !== false) $sendback = admin_url('attachments.php');
-    else $sendback = add_query_arg('untrashed', 1, $sendback);
+    if ( strpos($sendback, 'page.php') !== false )
+        $sendback = admin_url('edit-pages.php?untrashed=1');
+    else
+        $sendback = add_query_arg('untrashed', 1, $sendback);
+
     wp_redirect($sendback);
     exit();
@@ -183 +186 @@
 
 case 'delete':
-    $page_id = (isset($_GET['post']))  ? intval($_GET['post']) : intval($_POST['post_ID']);
+    $page_id = isset($_GET['post']) ? intval($_GET['post']) : intval($_POST['post_ID']);
     check_admin_referer('delete-page_' .  $page_id);
 
@@ -200 +203 @@
 
     $sendback = wp_get_referer();
-    if (strpos($sendback, 'page.php') !== false) $sendback = admin_url('edit-pages.php?deleted=1');
-    elseif (strpos($sendback, 'attachments.php') !== false) $sendback = admin_url('attachments.php');
-    else $sendback = add_query_arg('deleted', 1, $sendback);
+    if ( strpos($sendback, 'page.php') !== false )
+        $sendback = admin_url('edit-pages.php?deleted=1');
+    else
+        $sendback = add_query_arg('deleted', 1, $sendback);
+
     wp_redirect($sendback);
     exit();