Make WordPress Core

Ticket #46619: 46619.5.diff

File 46619.5.diff, 9.6 KB (added by garrett-eclipse, 5 years ago)

Update to sentence case for privacy actions.

  • src/wp-admin/includes/class-wp-privacy-data-export-requests-list-table.php

     
    5050                $request_id      = $item->ID;
    5151                $nonce           = wp_create_nonce( 'wp-privacy-export-personal-data-' . $request_id );
    5252
    53                 $download_data_markup = '<div class="export-personal-data" ' .
     53                $download_data_markup = '<span class="export-personal-data" ' .
    5454                        'data-exporters-count="' . esc_attr( $exporters_count ) . '" ' .
    5555                        'data-request-id="' . esc_attr( $request_id ) . '" ' .
    5656                        'data-nonce="' . esc_attr( $nonce ) .
    5757                        '">';
    5858
    59                 $download_data_markup .= '<span class="export-personal-data-idle"><button type="button" class="button-link export-personal-data-handle">' . __( 'Download Personal Data' ) . '</button></span>' .
    60                         '<span class="export-personal-data-processing hidden">' . __( 'Downloading Data...' ) . ' <span class="export-progress"></span></span>' .
    61                         '<span class="export-personal-data-success hidden"><button type="button" class="button-link export-personal-data-handle">' . __( 'Download Personal Data Again' ) . '</button></span>' .
     59                $download_data_markup .= '<span class="export-personal-data-idle"><button type="button" class="button-link export-personal-data-handle">' . __( 'Download personal data' ) . '</button></span>' .
     60                        '<span class="export-personal-data-processing hidden">' . __( 'Downloading data...' ) . ' <span class="export-progress"></span></span>' .
     61                        '<span class="export-personal-data-success hidden"><button type="button" class="button-link export-personal-data-handle">' . __( 'Download personal data again' ) . '</button></span>' .
    6262                        '<span class="export-personal-data-failed hidden">' . __( 'Download failed.' ) . ' <button type="button" class="button-link">' . __( 'Retry' ) . '</button></span>';
    6363
    64                 $download_data_markup .= '</div>';
     64                $download_data_markup .= '</span>';
    6565
     66                $complete_request_markup = '';
     67                if ( 'request-completed' !== $status ) {
     68                        $complete_request_markup .= '<span>';
     69                        $complete_request_markup .= sprintf(
     70                                '<a href="%s" class="complete-request" aria-label="%s">%s</a>',
     71                                esc_url(
     72                                        wp_nonce_url(
     73                                                add_query_arg(
     74                                                        array(
     75                                                                'action'     => 'complete',
     76                                                                'request_id' => array( $request_id ),
     77                                                        ),
     78                                                        admin_url( 'export-personal-data.php' )
     79                                                ),
     80                                                'bulk-privacy_requests'
     81                                        )
     82                                ),
     83                                esc_attr(
     84                                        sprintf(
     85                                                /* translators: %s: Request email. */
     86                                                __( 'Mark export request for &#8220;%s&#8221; as completed.' ),
     87                                                $item->email
     88                                        )
     89                                ),
     90                                __( 'Complete request' )
     91                        );
     92                        $complete_request_markup .= '</span>';
     93                }
     94
    6695                $row_actions = array(
    67                         'download-data' => $download_data_markup,
     96                        'download-data'    => $download_data_markup,
     97                        'complete-request' => $complete_request_markup,
    6898                );
    6999
    70100                return sprintf( '<a href="%1$s">%2$s</a> %3$s', esc_url( 'mailto:' . $item->email ), $item->email, $this->row_actions( $row_actions ) );
  • src/wp-admin/includes/class-wp-privacy-data-removal-requests-list-table.php

     
    4747                $row_actions = array();
    4848
    4949                // Allow the administrator to "force remove" the personal data even if confirmation has not yet been received.
    50                 $status = $item->status;
     50                $status      = $item->status;
     51                $row_actions = array();
     52                $request_id  = $item->ID;
    5153                if ( 'request-confirmed' !== $status ) {
    5254                        /** This filter is documented in wp-admin/includes/ajax-actions.php */
    5355                        $erasers       = apply_filters( 'wp_privacy_personal_data_erasers', array() );
    5456                        $erasers_count = count( $erasers );
    55                         $request_id    = $item->ID;
    5657                        $nonce         = wp_create_nonce( 'wp-privacy-erase-personal-data-' . $request_id );
    5758
    58                         $remove_data_markup = '<div class="remove-personal-data force-remove-personal-data" ' .
     59                        $remove_data_markup = '<span class="remove-personal-data force-remove-personal-data" ' .
    5960                                'data-erasers-count="' . esc_attr( $erasers_count ) . '" ' .
    6061                                'data-request-id="' . esc_attr( $request_id ) . '" ' .
    6162                                'data-nonce="' . esc_attr( $nonce ) .
    6263                                '">';
    6364
    64                         $remove_data_markup .= '<span class="remove-personal-data-idle"><button type="button" class="button-link remove-personal-data-handle">' . __( 'Force Erase Personal Data' ) . '</button></span>' .
    65                                 '<span class="remove-personal-data-processing hidden">' . __( 'Erasing Data...' ) . ' <span class="erasure-progress"></span></span>' .
     65                        $remove_data_markup .= '<span class="remove-personal-data-idle"><button type="button" class="button-link remove-personal-data-handle">' . __( 'Force erase personal data' ) . '</button></span>' .
     66                                '<span class="remove-personal-data-processing hidden">' . __( 'Erasing data...' ) . ' <span class="erasure-progress"></span></span>' .
    6667                                '<span class="remove-personal-data-success hidden">' . __( 'Erasure completed.' ) . '</span>' .
    67                                 '<span class="remove-personal-data-failed hidden">' . __( 'Force Erasure has failed.' ) . ' <button type="button" class="button-link remove-personal-data-handle">' . __( 'Retry' ) . '</button></span>';
     68                                '<span class="remove-personal-data-failed hidden">' . __( 'Force erasure has failed.' ) . ' <button type="button" class="button-link remove-personal-data-handle">' . __( 'Retry' ) . '</button></span>';
    6869
    69                         $remove_data_markup .= '</div>';
     70                        $remove_data_markup .= '</span>';
    7071
    71                         $row_actions = array(
    72                                 'remove-data' => $remove_data_markup,
     72                        $row_actions['remove-data'] = $remove_data_markup;
     73                }
     74
     75                $complete_request_markup = '';
     76                if ( 'request-completed' !== $status ) {
     77                        $complete_request_markup .= '<span>';
     78                        $complete_request_markup .= sprintf(
     79                                '<a href="%s" class="complete-request" aria-label="%s">%s</a>',
     80                                esc_url(
     81                                        wp_nonce_url(
     82                                                add_query_arg(
     83                                                        array(
     84                                                                'action'     => 'complete',
     85                                                                'request_id' => array( $request_id ),
     86                                                        ),
     87                                                        admin_url( 'erase-personal-data.php' )
     88                                                ),
     89                                                'bulk-privacy_requests'
     90                                        )
     91                                ),
     92                                esc_attr(
     93                                        sprintf(
     94                                                /* translators: %s: Request email. */
     95                                                __( 'Mark export request for &#8220;%s&#8221; as completed.' ),
     96                                                $item->email
     97                                        )
     98                                ),
     99                                __( 'Complete request' )
    73100                        );
     101                        $complete_request_markup .= '</span>';
    74102                }
    75103
     104                $row_actions['complete-request'] = $complete_request_markup;
     105
    76106                return sprintf( '<a href="%1$s">%2$s</a> %3$s', esc_url( 'mailto:' . $item->email ), $item->email, $this->row_actions( $row_actions ) );
    77107        }
    78108
  • src/wp-admin/includes/class-wp-privacy-requests-table.php

     
    4343                        'email'             => __( 'Requester' ),
    4444                        'status'            => __( 'Status' ),
    4545                        'created_timestamp' => __( 'Requested' ),
    46                         'next_steps'        => __( 'Next Steps' ),
     46                        'next_steps'        => __( 'Next steps' ),
    4747                );
    4848                return $columns;
    4949        }
     
    210210         */
    211211        protected function get_bulk_actions() {
    212212                return array(
    213                         'delete' => __( 'Delete Requests' ),
    214                         'resend' => __( 'Resend Confirmation Requests' ),
     213                        'resend'   => __( 'Resend confirmation requests' ),
     214                        'complete' => __( 'Mark requests as completed' ),
     215                        'delete'   => __( 'Delete requests' ),
    215216                );
    216217        }
    217218
     
    219220         * Process bulk actions.
    220221         *
    221222         * @since 4.9.6
     223         * @since 5.6.0 Added support for the `complete` action.
    222224         */
    223225        public function process_bulk_action() {
    224226                $action      = $this->current_action();
     
    231233                }
    232234
    233235                switch ( $action ) {
    234                         case 'delete':
     236                        case 'resend':
    235237                                foreach ( $request_ids as $request_id ) {
    236                                         if ( wp_delete_post( $request_id, true ) ) {
    237                                                 $count ++;
     238                                        $resend = _wp_privacy_resend_request( $request_id );
     239
     240                                        if ( $resend && ! is_wp_error( $resend ) ) {
     241                                                $count++;
    238242                                        }
    239243                                }
    240244
     
    242246                                        'bulk_action',
    243247                                        'bulk_action',
    244248                                        /* translators: %d: Number of requests. */
    245                                         sprintf( _n( 'Deleted %d request.', 'Deleted %d requests.', $count ), $count ),
     249                                        sprintf( _n( 'Re-sent %d request.', 'Re-sent %d requests.', $count ), $count ),
    246250                                        'success'
    247251                                );
    248252                                break;
    249                         case 'resend':
     253                        case 'complete':
    250254                                foreach ( $request_ids as $request_id ) {
    251                                         $resend = _wp_privacy_resend_request( $request_id );
     255                                        $result = _wp_privacy_completed_request( $request_id );
    252256
    253                                         if ( $resend && ! is_wp_error( $resend ) ) {
     257                                        if ( $result && ! is_wp_error( $result ) ) {
    254258                                                $count++;
    255259                                        }
    256260                                }
     
    259263                                        'bulk_action',
    260264                                        'bulk_action',
    261265                                        /* translators: %d: Number of requests. */
    262                                         sprintf( _n( 'Re-sent %d request.', 'Re-sent %d requests.', $count ), $count ),
     266                                        sprintf( _n( '%d request marked as complete.', '%d requests marked as complete.', $count ), $count ),
    263267                                        'success'
    264268                                );
    265269                                break;
     270                        case 'delete':
     271                                foreach ( $request_ids as $request_id ) {
     272                                        if ( wp_delete_post( $request_id, true ) ) {
     273                                                $count ++;
     274                                        }
     275                                }
     276
     277                                add_settings_error(
     278                                        'bulk_action',
     279                                        'bulk_action',
     280                                        /* translators: %d: Number of requests. */
     281                                        sprintf( _n( 'Deleted %d request.', 'Deleted %d requests.', $count ), $count ),
     282                                        'success'
     283                                );
     284                                break;
    266285                }
    267286        }
    268287