WordPress.org
Get WordPress

Make WordPress Core

Ticket #23022: 23022.diff

File 23022.diff, 8.5 KB (added by johnbillion, 4 years ago)

  • src/wp-admin/edit.php

     
    134134                        break;
    135135                case 'untrash':
    136136                        $untrashed = 0;
     137
     138                        if ( isset( $_GET['doaction'] ) && ( 'undo' === $_GET['doaction'] ) ) {
     139                                add_filter( 'wp_untrash_post_status', 'wp_untrash_post_set_previous_status', 10, 3 );
     140                        }
     141
    137142                        foreach ( (array) $post_ids as $post_id ) {
    138143                                if ( ! current_user_can( 'delete_post', $post_id ) ) {
    139144                                        wp_die( __( 'Sorry, you are not allowed to restore this item from the Trash.' ) );
    … …  
    146151                                $untrashed++;
    147152                        }
    148153                        $sendback = add_query_arg( 'untrashed', $untrashed, $sendback );
     154
     155                        remove_filter( 'wp_untrash_post_status', 'wp_untrash_post_set_previous_status', 10, 3 );
     156
    149157                        break;
    150158                case 'delete':
    151159                        $deleted = 0;
    … …  
    419427                $ids        = preg_replace( '/[^0-9,]/', '', $_REQUEST['ids'] );
    420428                $messages[] = '<a href="' . esc_url( wp_nonce_url( "edit.php?post_type=$post_type&doaction=undo&action=untrash&ids=$ids", 'bulk-posts' ) ) . '">' . __( 'Undo' ) . '</a>';
    421429        }
     430
     431        if ( 'untrashed' === $message && isset( $_REQUEST['ids'] ) ) {
     432                $ids = explode( ',', $_REQUEST['ids'] );
     433
     434                if ( 1 === count( $ids ) && current_user_can( 'edit_post', $ids[0] ) ) {
     435                        $messages[] = sprintf(
     436                                '<a href="%1$s">%2$s</a>',
     437                                esc_url( get_edit_post_link( $ids[0] ) ),
     438                                esc_html( get_post_type_object( get_post_type( $ids[0] ) )->labels->edit_item )
     439                        );
     440                }
     441        }
    422442}
    423443
    424444if ( $messages ) {

  • src/wp-admin/post.php

     
    291291                        wp_die( __( 'Error in restoring the item from Trash.' ) );
    292292                }
    293293
    294                 wp_redirect( add_query_arg( 'untrashed', 1, $sendback ) );
     294                $sendback = add_query_arg(
     295                        array(
     296                                'untrashed' => 1,
     297                                'ids'       => $post_id,
     298                        ),
     299                        $sendback
     300                );
     301                wp_redirect( $sendback );
    295302                exit;
    296303
    297304        case 'delete':

  • src/wp-includes/functions.php

     
    11931193                'error',
    11941194                'hotkeys_highlight_first',
    11951195                'hotkeys_highlight_last',
     1196                'ids',
    11961197                'locked',
    11971198                'message',
    11981199                'same',

  • src/wp-includes/post.php

     
    32323232 * Restore a post or page from the Trash.
    32333233 *
    32343234 * @since 2.9.0
     3235 * @since 5.6.0 An untrashed post is now returned to `draft` status by default.
    32353236 *
    32363237 * @param int $post_id Optional. Post ID. Default is ID of the global $post.
    32373238 * @return WP_Post|false|null Post data on success, false or null on failure.
    … …  
    32433244                return $post;
    32443245        }
    32453246
     3247        $post_id = $post->ID;
     3248
    32463249        if ( 'trash' !== $post->post_status ) {
    32473250                return false;
    32483251        }
    32493252
     3253        $previous_status = get_post_meta( $post_id, '_wp_trash_meta_status', true );
     3254
    32503255        /**
    32513256         * Filters whether a post untrashing should take place.
    32523257         *
    32533258         * @since 4.9.0
     3259         * @since 5.6.0 The `$previous_status` parameter was added.
    32543260         *
    3255          * @param bool|null $untrash Whether to go forward with untrashing.
    3256          * @param WP_Post   $post    Post object.
     3261         * @param bool|null $untrash         Whether to go forward with untrashing.
     3262         * @param WP_Post   $post            Post object.
     3263         * @param string    $previous_status The status of the post at the point where it was trashed.
    32573264         */
    32583265        $check = apply_filters( 'pre_untrash_post', null, $post );
    32593266        if ( null !== $check ) {
    … …  
    32643271         * Fires before a post is restored from the Trash.
    32653272         *
    32663273         * @since 2.9.0
     3274         * @since 5.6.0 The `$previous_status` parameter was added.
    32673275         *
    3268          * @param int $post_id Post ID.
     3276         * @param int    $post_id         Post ID.
     3277         * @param string $previous_status The status of the post at the point where it was trashed.
    32693278         */
    3270         do_action( 'untrash_post', $post_id );
     3279        do_action( 'untrash_post', $post_id, $previous_status );
    32713280
    3272         $post_status = get_post_meta( $post_id, '_wp_trash_meta_status', true );
     3281        /**
     3282         * Filter the status that a post gets when it is restored from the trash (untrashed).
     3283         *
     3284         * By default, posts that are restored will be assigned a status of 'draft'. Return the value of `$previous_status`
     3285         * in order to assign the status that the post had before it was trashed. The `wp_untrash_post_set_previous_status()`
     3286         * function is available for this.
     3287         *
     3288         * Prior to WordPress 5.6, restored posts were given their original status.
     3289         *
     3290         * @since 5.6.0
     3291         *
     3292         * @param string $new_status      The new status of the post being restored.
     3293         * @param int    $post_id         The ID of the post being restored.
     3294         * @param string $previous_status The status of the post at the point where it was trashed.
     3295         */
     3296        $post_status = apply_filters( 'wp_untrash_post_status', 'draft', $post_id, $previous_status );
    32733297
    32743298        delete_post_meta( $post_id, '_wp_trash_meta_status' );
    32753299        delete_post_meta( $post_id, '_wp_trash_meta_time' );
    … …  
    32913315         * Fires after a post is restored from the Trash.
    32923316         *
    32933317         * @since 2.9.0
     3318         * @since 5.6.0 The `$previous_status` parameter was added.
    32943319         *
    3295          * @param int $post_id Post ID.
     3320         * @param int    $post_id         Post ID.
     3321         * @param string $previous_status The status of the post at the point where it was trashed.
    32963322         */
    3297         do_action( 'untrashed_post', $post_id );
     3323        do_action( 'untrashed_post', $post_id, $previous_status );
    32983324
    32993325        return $post;
    33003326}
    … …  
    74957521         */
    74967522        return apply_filters( 'wp_get_original_image_url', $original_image_url, $attachment_id );
    74977523}
     7524
     7525/**
     7526 * Filter callback which sets the status of an untrashed post to its previous status.
     7527 *
     7528 * This can be used as a callback on the `wp_untrash_post_status` filter.
     7529 *
     7530 * @since 5.6.0
     7531 *
     7532 * @param string $new_status      The new status of the post being restored.
     7533 * @param int    $post_id         The ID of the post being restored.
     7534 * @param string $previous_status The status of the post at the point where it was trashed.
     7535 * @return string The status of the post at the point where it was trashed.
     7536 */
     7537function wp_untrash_post_set_previous_status( $new_status, $post_id, $previous_status ) {
     7538        return $previous_status;
     7539}

  • tests/phpunit/tests/post/wpInsertPost.php

     
    159159                );
    160160
    161161                wp_untrash_post( $about_page_id );
     162                wp_update_post(
     163                        array(
     164                                'ID'          => $about_page_id,
     165                                'post_status' => 'publish',
     166                        )
     167                );
    162168
    163169                $this->assertSame( 'about', get_post( $another_about_page_id )->post_name );
    164170                $this->assertSame( 'about-2', get_post( $about_page_id )->post_name );
    165171        }
    166172
    167173        /**
     174         * @ticket 23022
     175         * @dataProvider data_various_post_statuses
     176         */
     177        function test_untrashing_a_post_should_always_restore_it_to_draft_status( $post_status ) {
     178                $page_id = self::factory()->post->create(
     179                        array(
     180                                'post_type'   => 'page',
     181                                'post_status' => $post_status,
     182                        )
     183                );
     184
     185                wp_trash_post( $page_id );
     186                wp_untrash_post( $page_id );
     187
     188                $this->assertSame( 'draft', get_post( $page_id )->post_status );
     189        }
     190
     191        /**
     192         * @ticket 23022
     193         * @dataProvider data_various_post_statuses
     194         */
     195        function test_wp_untrash_post_status_filter_restores_post_to_correct_status( $post_status ) {
     196                add_filter( 'wp_untrash_post_status', 'wp_untrash_post_set_previous_status', 10, 3 );
     197
     198                $page_id = self::factory()->post->create(
     199                        array(
     200                                'post_type'   => 'page',
     201                                'post_status' => $post_status,
     202                        )
     203                );
     204
     205                wp_trash_post( $page_id );
     206                wp_untrash_post( $page_id );
     207
     208                remove_filter( 'wp_untrash_post_status', 'wp_untrash_post_set_previous_status', 10, 3 );
     209
     210                $this->assertSame( $post_status, get_post( $page_id )->post_status );
     211        }
     212
     213        /**
    168214         * Data for testing the ability for users to set the post slug.
    169215         *
    170216         * @return array Array of test arguments.
    … …  
    184230        }
    185231
    186232        /**
     233         * Data for testing post statuses.
     234         *
     235         * @return array Array of test arguments.
     236         */
     237        function data_various_post_statuses() {
     238                return array(
     239                        array(
     240                                'draft',
     241                        ),
     242                        array(
     243                                'pending',
     244                        ),
     245                        array(
     246                                'private',
     247                        ),
     248                        array(
     249                                'publish',
     250                        ),
     251                );
     252        }
     253
     254        /**
    187255         * Test contributor making changes to the pending post slug.
    188256         *
    189257         * @ticket 42464