From decf135295b7ab998a6978c45370fd476cfdeb86 Mon Sep 17 00:00:00 2001
From: ryan <ryan@1a063a9b-81f0-0310-95a4-ce76da25c4cd>
Date: Thu, 21 Jan 2010 06:53:33 +0000
Subject: [PATCH] Update trash and right now links for pages. fixes #11958

git-svn-id: http://svn.automattic.com/wordpress/trunk@12783 1a063a9b-81f0-0310-95a4-ce76da25c4cd
---
 wp-admin/admin-header.php       |  2 +-
 wp-admin/edit.php               |  6 ++---
 wp-admin/includes/dashboard.php |  4 ++--
 wp-admin/includes/template.php  |  5 +++--
 wp-includes/link-template.php   | 40 ++++++++++-----------------------
 wp-includes/post.php            |  2 ++
 6 files changed, 23 insertions(+), 36 deletions(-)

diff --git a/wp-admin/admin-header.php b/wp-admin/admin-header.php
index 652b33b4d6..d6f73bea8d 100644
--- a/wp-admin/admin-header.php
+++ b/wp-admin/admin-header.php
@@ -40,7 +40,7 @@ var ajaxurl = '<?php echo admin_url('admin-ajax.php'); ?>', pagenow = '<?php ech
 </script>
 <?php
 
-if ( in_array( $pagenow, array('post.php', 'post-new.php', 'page.php') ) ) {
+if ( in_array( $pagenow, array('post.php', 'post-new.php') ) ) {
 	add_action( 'admin_print_footer_scripts', 'wp_tiny_mce', 25 );
 	wp_enqueue_script('quicktags');
 }
diff --git a/wp-admin/edit.php b/wp-admin/edit.php
index 9a19ede586..950e3b094e 100644
--- a/wp-admin/edit.php
+++ b/wp-admin/edit.php
@@ -65,7 +65,7 @@ if ( isset($_GET['doaction']) || isset($_GET['doaction2']) || isset($_GET['delet
 		$post_ids = isset($_GET['post']) ? array_map( 'intval', (array) $_GET['post'] ) : explode(',', $_GET['ids']);
 		$doaction = ($_GET['action'] != -1) ? $_GET['action'] : $_GET['action2'];
 	} else {
-		wp_redirect( admin_url('edit.php') );
+		wp_redirect( admin_url("edit.php?post_type=$post_type") );
 	}
 
 	switch ( $doaction ) {
@@ -127,7 +127,7 @@ if ( isset($_GET['doaction']) || isset($_GET['doaction2']) || isset($_GET['delet
 	}
 
 	if ( isset($_GET['action']) )
-		$sendback = remove_query_arg( array('action', 'action2', 'cat', 'tags_input', 'post_author', 'comment_status', 'ping_status', '_status',  'post', 'bulk_edit', 'post_view', 'post_type'), $sendback );
+		$sendback = remove_query_arg( array('action', 'action2', 'cat', 'tags_input', 'post_author', 'comment_status', 'ping_status', '_status',  'post', 'bulk_edit', 'post_view'), $sendback );
 
 	wp_redirect($sendback);
 	exit();
@@ -198,7 +198,7 @@ if ( isset($_GET['deleted']) && (int) $_GET['deleted'] ) {
 if ( isset($_GET['trashed']) && (int) $_GET['trashed'] ) {
 	printf( _n( 'Item moved to the trash.', '%s items moved to the trash.', $_GET['trashed'] ), number_format_i18n( $_GET['trashed'] ) );
 	$ids = isset($_GET['ids']) ? $_GET['ids'] : 0;
-	echo ' <a href="' . esc_url( wp_nonce_url( "edit.php?doaction=undo&action=untrash&ids=$ids", "bulk-posts" ) ) . '">' . __('Undo') . '</a><br />';
+	echo ' <a href="' . esc_url( wp_nonce_url( "edit.php?post_type=$post_type&doaction=undo&action=untrash&ids=$ids", "bulk-posts" ) ) . '">' . __('Undo') . '</a><br />';
 	unset($_GET['trashed']);
 }
 
diff --git a/wp-admin/includes/dashboard.php b/wp-admin/includes/dashboard.php
index b05172c21a..74f6d9ec9f 100644
--- a/wp-admin/includes/dashboard.php
+++ b/wp-admin/includes/dashboard.php
@@ -252,8 +252,8 @@ function wp_dashboard_right_now() {
 	$num = number_format_i18n( $num_pages->publish );
 	$text = _n( 'Page', 'Pages', $num_pages->publish );
 	if ( current_user_can( 'edit_pages' ) ) {
-		$num = "<a href='edit-pages.php'>$num</a>";
-		$text = "<a href='edit-pages.php'>$text</a>";
+		$num = "<a href='edit.php?post_type=page'>$num</a>";
+		$text = "<a href='edit.php?post_type=page'>$text</a>";
 	}
 	echo '<td class="first b b_pages">' . $num . '</td>';
 	echo '<td class="t pages">' . $text . '</td>';
diff --git a/wp-admin/includes/template.php b/wp-admin/includes/template.php
index f13a994705..fbc63fe38a 100644
--- a/wp-admin/includes/template.php
+++ b/wp-admin/includes/template.php
@@ -1605,6 +1605,7 @@ function display_page_row( $page, $level = 0 ) {
 	$posts_columns = get_column_headers( $current_screen );
 	$hidden = get_hidden_columns(  $current_screen );
 	$title = _draft_or_post_title();
+	$post_type = $page->post_type;
 ?>
 <tr id="page-<?php echo $id; ?>" class="<?php echo $rowclass; ?> iedit">
 <?php
@@ -1669,11 +1670,11 @@ foreach ($posts_columns as $column_name=>$column_display_name) {
 		}
 		if ( current_user_can('delete_page', $page->ID) ) {
 			if ( $post->post_status == 'trash' )
-				$actions['untrash'] = "<a title='" . esc_attr(__('Remove this page from the Trash')) . "' href='" . wp_nonce_url("page.php?action=untrash&amp;post=$page->ID", 'untrash-page_' . $page->ID) . "'>" . __('Restore') . "</a>";
+				$actions['untrash'] = "<a title='" . esc_attr(__('Remove this page from the Trash')) . "' href='" . wp_nonce_url("post.php?post_type=$post_type&amp;action=untrash&amp;post=$page->ID", 'untrash-page_' . $page->ID) . "'>" . __('Restore') . "</a>";
 			elseif ( EMPTY_TRASH_DAYS )
 				$actions['trash'] = "<a class='submitdelete' title='" . esc_attr(__('Move this page to the Trash')) . "' href='" . get_delete_post_link($page->ID) . "'>" . __('Trash') . "</a>";
 			if ( $post->post_status == 'trash' || !EMPTY_TRASH_DAYS )
-				$actions['delete'] = "<a class='submitdelete' title='" . esc_attr(__('Delete this page permanently')) . "' href='" . wp_nonce_url("page.php?action=delete&amp;post=$page->ID", 'delete-page_' . $page->ID) . "'>" . __('Delete Permanently') . "</a>";
+				$actions['delete'] = "<a class='submitdelete' title='" . esc_attr(__('Delete this page permanently')) . "' href='" . wp_nonce_url("post.php?post_type=$post_type&amp;action=delete&amp;post=$page->ID", 'delete-page_' . $page->ID) . "'>" . __('Delete Permanently') . "</a>";
 		}
 		if ( in_array($post->post_status, array('pending', 'draft')) ) {
 			if ( current_user_can('edit_page', $page->ID) )
diff --git a/wp-includes/link-template.php b/wp-includes/link-template.php
index 9fd303da44..923a9728e4 100644
--- a/wp-includes/link-template.php
+++ b/wp-includes/link-template.php
@@ -737,35 +737,19 @@ function get_delete_post_link($id = 0, $context = 'display') {
 	else
 		$action = 'action=trash&';
 
-	switch ( $post->post_type ) :
-	case 'page' :
-		if ( !current_user_can( 'delete_page', $post->ID ) )
-			return;
-		$file = 'page';
-		$var  = 'post';
-		break;
-	case 'attachment' :
-		if ( !current_user_can( 'delete_post', $post->ID ) )
-			return;
-		$file = 'media';
-		$var  = 'attachment_id';
-		break;
-	case 'revision' :
-		if ( !current_user_can( 'delete_post', $post->ID ) )
-			return;
-		$file = 'revision';
-		$var  = 'revision';
-		$action = '';
-		break;
-	default :
-		if ( !current_user_can( 'edit_post', $post->ID ) )
-			return apply_filters( 'get_delete_post_link', '', $post->ID, $context );
-		$file = 'post';
-		$var  = 'post';
-		break;
-	endswitch;
+	if ( 'display' == $context )
+		$action = '&amp;action=trash';
+	else
+		$action = '&action=trash';
 
-	return apply_filters( 'get_delete_post_link', wp_nonce_url( admin_url("$file.php?{$action}$var=$post->ID"), "trash-{$file}_" . $post->ID ), $context );
+	$post_type_object = get_post_type_object( $post->post_type );
+	if ( !$post_type_object )
+		return;
+
+	if ( !current_user_can( $post_type_object->delete_cap, $post->ID ) )
+		return;
+
+	return apply_filters( 'get_delete_post_link', wp_nonce_url( admin_url( sprintf($post_type_object->_edit_link . $action, $post->ID) ),  "trash-{$post->post_type}_" . $post->ID), $post->ID, $context );
 }
 
 /**
diff --git a/wp-includes/post.php b/wp-includes/post.php
index c168f36c73..73d5fbedae 100644
--- a/wp-includes/post.php
+++ b/wp-includes/post.php
@@ -720,9 +720,11 @@ function register_post_type($post_type, $args = array()) {
 	if ( empty($args->capability_type) ) {
 		$args->edit_cap = '';
 		$args->read_cap = '';
+		$args->delete_cap = '';
 	} else {
 		$args->edit_cap = 'edit_' . $args->capability_type;
 		$args->read_cap = 'read_' . $args->capability_type;
+		$args->delete_cap = 'delete_' . $args->capability_type;
 	}
 
 	if ( !$args->_builtin && $args->public )