Making WordPress.org

Changeset 5318


Ignore:
Timestamp:
04/11/2017 06:37:04 AM (7 years ago)
Author:
dd32
Message:

Plugin Directory: Introduce the 'new' post_status to replace 'draft', this will allow a better more well-defined statuses for plugins.

See #2716

Location:
sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory
Files:
10 edited

Legend:

Unmodified
Added
Removed
  • sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/class-customizations.php

    r5308 r5318  
    131131
    132132        if ( empty( $query->query['post_status'] ) ) {
    133             $query->query_vars['post_status'] = array( 'publish', 'future', 'draft', 'pending', 'disabled', 'closed', 'rejected', 'approved' );
     133            $query->query_vars['post_status'] = array( 'publish', 'future', 'new', 'pending', 'disabled', 'closed', 'rejected', 'approved' );
    134134        }
    135135
     
    176176            'post_type'      => 'plugin',
    177177            'post__in'       => array_map( 'absint', $_REQUEST['post'] ),
    178             'post_status'    => array( 'draft', 'pending' ),
     178            'post_status'    => array( 'new', 'pending' ),
    179179            'posts_per_page' => count( $_REQUEST['post'] ),
    180180        ) );
     
    238238
    239239        switch ( $post->post_status ) {
    240             case 'draft':
     240            case 'new':
    241241                $message = __( 'This plugin is newly requested and has not yet been reviewed.', 'wporg-plugins' );
    242242                $type    = 'notice-info';
     
    366366        );
    367367
    368         if ( 'draft' !== $post->post_status ) {
     368        if ( 'new' !== $post->post_status && 'pending' != $post->post_status ) {
    369369            add_meta_box(
    370370                'plugin-committers',
     
    381381
    382382        // Remove slug metabox unless the slug is editable for the current user.
    383         if ( ! in_array( $post->post_status, array( 'draft', 'pending' ) ) || ! current_user_can( 'plugin_approve', $post ) ) {
     383        if ( ! in_array( $post->post_status, array( 'new', 'pending' ) ) || ! current_user_can( 'plugin_approve', $post ) ) {
    384384            remove_meta_box( 'slugdiv', 'plugin', 'normal' );
    385385        }
  • sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/class-status-transitions.php

    r5313 r5318  
    5757
    5858        // ...or it's a white-listed status for plugin reviewers.
    59         if ( current_user_can( 'plugin_review', $postarr['ID'] ) && in_array( $postarr['post_status'], array( 'draft', 'pending' ) ) ) {
     59        if ( current_user_can( 'plugin_review', $postarr['ID'] ) && in_array( $postarr['post_status'], array( 'new', 'pending' ) ) ) {
    6060            return $data;
    6161        }
  • sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/list-table/class-plugin-posts.php

    r5037 r5318  
    8080        }
    8181
    82         if ( current_user_can( 'plugin_reject' ) && ( empty( $_REQUEST['post_status'] ) || in_array( $_REQUEST['post_status'], array( 'draft', 'pending' ) ) ) ) {
     82        if ( current_user_can( 'plugin_reject' ) && ( empty( $_REQUEST['post_status'] ) || in_array( $_REQUEST['post_status'], array( 'new', 'pending' ) ) ) ) {
    8383            $actions['plugin_reject'] = __( 'Reject', 'wporg-plugins' );
    8484        }
     
    407407            }
    408408
    409             if ( ! current_user_can( 'plugin_approve' ) && ! in_array( $status_name, array( 'draft', 'pending' ) ) ) {
     409            if ( ! current_user_can( 'plugin_approve' ) && ! in_array( $status_name, array( 'new', 'pending' ) ) ) {
    410410                continue;
    411411            }
  • sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/metabox/class-author-card.php

    r5274 r5318  
    149149
    150150                    $plugin_slug = $plugin->post_name;
    151                     if ( in_array( $plugin->post_status, array( 'draft', 'pending' ) ) ) {
     151                    if ( in_array( $plugin->post_status, array( 'new', 'pending' ) ) ) {
    152152                        $extra .= ' (requested ' . human_time_diff( strtotime( $last_updated ) ) . ' ago)';
    153153                        $tooltips[] = 'Requested, remains unapproved.';
  • sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/metabox/class-controls.php

    r4212 r5318  
    4141        }
    4242
    43         $statuses = array( 'draft', 'pending' );
     43        $statuses = array( 'new', 'pending' );
    4444        if ( current_user_can( 'plugin_approve', $post ) ) {
    45             if ( in_array( $post->post_status, array( 'draft', 'pending', 'rejected', 'approved' ) ) ) {
     45            if ( in_array( $post->post_status, array( 'new', 'draft', 'pending', 'rejected', 'approved' ) ) ) {
    4646                $statuses = array_merge( $statuses, array( 'approved', 'rejected' ) );
    4747            } else {
    48                 $statuses = array( 'publish', 'disabled', 'closed' );
     48                $statuses = array( 'publish', 'disabled', 'closed', 'pending' );
    4949            }
    5050        }
  • sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/metabox/class-review-tools.php

    r5267 r5318  
    1111        $post = get_post();
    1212
    13         $zip_url = '';
     13        $zip_files = array();
    1414        foreach ( get_attached_media( 'application/zip', $post ) as $zip_file ) {
    15             $zip_url = wp_get_attachment_url( $zip_file->ID );
    16             break;
     15            $zip_files[ $zip_file->post_date ] = wp_get_attachment_url( $zip_file->ID );
    1716        }
    18         if ( ! $zip_url ) {
    19             $zip_url = get_post_meta( $post->ID, '_submitted_zip', true );
     17        uksort( $zip_files, function( $a, $b ) {
     18            return strtotime( $a ) < strtotime( $b );
     19        } );
     20
     21        if ( $zip_url = get_post_meta( $post->ID, '_submitted_zip', true ) ) {
     22            // Back-compat only.
     23            $zip_files[ 'User provided URL' ] = $zip_url;
    2024        }
    2125
    22         if ( $zip_url ) {
     26        foreach ( $zip_files as $zip_date => $zip_url ) {
    2327            printf( '<p>' . __( '<strong>Zip file:</strong> %s', 'wporg-plugins' ) . '</p>',
    24                 sprintf( '<a href="%s">%s</a>', esc_url( $zip_url ), esc_html( $zip_url ) )
     28                sprintf( '%s <a href="%s">%s</a>', esc_html( $zip_date ), esc_url( $zip_url ), esc_html( $zip_url ) )
    2529            );
    2630        }
    2731
    28         if ( 'pending' != $post->post_status ) {
     32        if ( 'pending' != $post->post_status && 'new' != $post->post_status ) {
    2933            echo "<ul>
    3034                <li><a href='https://plugins.trac.wordpress.org/log/{$post->post_name}/'>" . __( 'Development Log', 'wporg-plugins' ) . "</a></li>
     
    3337            </ul>';
    3438        }
    35         if ( $post->post_excerpt && in_array( $post->post_status, array( 'pending', 'approved' ) ) ) {
     39        if ( $post->post_excerpt && in_array( $post->post_status, array( 'new', 'pending', 'approved' ) ) ) {
    3640            echo '<p>' . strip_tags( $post->post_excerpt ) . '</p>';
    3741        }
     
    4145           
    4246            $type   = 'Notice';
    43             if ( $post->post_status == 'draft' || $post->post_status == 'pending' ) {
     47            if ( $post->post_status == 'new' || $post->post_status == 'pending' ) {
    4448                $type = 'Request';
    4549            }
  • sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/tools/class-stats-report.php

    r5312 r5318  
    108108        // # of plugins currently in the queue that are drafts (have not been processed/replied to yet)
    109109        $stats['in_queue_draft'] = $wpdb->get_var(
    110             "SELECT COUNT(*) FROM $wpdb->posts WHERE `post_type` = 'plugin' AND `post_status` = 'draft'"
     110            "SELECT COUNT(*) FROM $wpdb->posts WHERE `post_type` = 'plugin' AND `post_status` = 'new'"
    111111        );
    112112
     
    121121        // # of plugins currently in the queue submitted during the specified time window
    122122        $stats['in_queue_from_time_window'] = $wpdb->get_var( $wpdb->prepare(
    123             "SELECT COUNT(*) FROM $wpdb->posts WHERE `post_type` = 'plugin' AND `post_status` IN ( 'draft','pending' ) AND post_date < %s AND post_date > DATE_SUB( %s, INTERVAL %d DAY )",
     123            "SELECT COUNT(*) FROM $wpdb->posts WHERE `post_type` = 'plugin' AND `post_status` IN ( 'new','pending' ) AND post_date < %s AND post_date > DATE_SUB( %s, INTERVAL %d DAY )",
    124124            $args['date'],
    125125            $args['date'],
     
    129129        // # of plugins currently in the queue that are older than "recently"
    130130        $stats['in_queue_old'] = $wpdb->get_var( $wpdb->prepare(
    131             "SELECT COUNT(*) FROM $wpdb->posts WHERE `post_type` = 'plugin' AND `post_status` IN ( 'draft','pending' ) AND post_date < DATE_SUB( %s, INTERVAL %d DAY )",
     131            "SELECT COUNT(*) FROM $wpdb->posts WHERE `post_type` = 'plugin' AND `post_status` IN ( 'new','pending' ) AND post_date < DATE_SUB( %s, INTERVAL %d DAY )",
    132132            $args['date'],
    133133            absint( $args['recentdays'] ) + 1
  • sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/class-plugin-directory.php

    r5316 r5318  
    284284        ) );
    285285
     286        register_post_status( 'new', array(
     287            'label'                     => _x( 'Pending Initial Review', 'plugin status', 'wporg-plugins' ),
     288            'public'                    => false,
     289            'show_in_admin_status_list' => current_user_can( 'plugin_review' ),
     290            'label_count'               => _n_noop( 'Pending Initial Review <span class="count">(%s)</span>', 'Pending Initial Review <span class="count">(%s)</span>', 'wporg-plugins' ),
     291        ) );
    286292        register_post_status( 'pending', array(
    287293            'label'                     => _x( 'Pending', 'plugin status', 'wporg-plugins' ),
     
    793799        if ( $viewing_own_author_archive || ( is_user_logged_in() && !empty( $wp_query->query_vars['name'] ) ) ) {
    794800
    795             $wp_query->query_vars['post_status'] = array( 'pending', 'approved', 'publish', 'closed', 'disabled' );
     801            $wp_query->query_vars['post_status'] = array( 'approved', 'publish', 'closed', 'disabled' );
    796802
    797803            add_filter( 'posts_results', function( $posts, $this_wp_query ) use( $wp_query ) {
     
    11931199                'post_type'   => 'plugin',
    11941200                'name'        => $plugin_slug,
    1195                 'post_status' => array( 'publish', 'pending', 'disabled', 'closed', 'draft', 'approved' ),
     1201                'post_status' => array( 'publish', 'pending', 'disabled', 'closed', 'new', 'draft', 'approved' ),
    11961202            ) );
    11971203
  • sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/shortcodes/class-upload-handler.php

    r5264 r5318  
    9797
    9898        // Is there already a plugin by a different author?
    99         if ( $plugin_post instanceof \WP_Post && $plugin_post->post_author != get_current_user_id() ) {
     99        if ( $plugin_post && $plugin_post->post_author != get_current_user_id() ) {
    100100            /* translators: %s: plugin slug */
    101101            return sprintf( __( 'There is already a plugin called %s by a different author. Please change the name of your plugin in the plugin header file and upload it again.', 'wporg-plugins' ),
     102                '<code>' . $this->plugin_slug . '</code>'
     103            );
     104        }
     105
     106        // Check the plugin can accept uploads (New submissions, or pending further review).
     107        if ( $plugin_post && ! in_array( $plugin_post->post_status, array( 'new', 'pending' ) ) ) {
     108            /* translators: %s: plugin slug */
     109            return sprintf( __( 'There is already a plugin called %s. Please change the name of your plugin in the plugin header file and upload it again.', 'wporg-plugins' ),
    102110                '<code>' . $this->plugin_slug . '</code>'
    103111            );
     
    171179                'post_title'   => $this->plugin['Name'],
    172180                'post_name'    => $this->plugin_slug,
    173                 'post_status'  => 'draft',
     181                'post_status'  => 'new',
    174182                'post_content' => $content,
    175183                'post_excerpt' => $this->plugin['Description'],
     
    205213                    'usage'                    => array(),
    206214                    '_author_ip'               => preg_replace( '/[^0-9a-fA-F:., ]/', '', $_SERVER['REMOTE_ADDR'] ),
     215                    '_submitted_date'          => time(),
    207216                ),
    208217            ) );
  • sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/shortcodes/class-upload.php

    r4817 r5318  
    3232                    <p>
    3333                        <?php
    34                         printf( _n( 'Currently there is %1$s plugin in the review queue.', 'Currently there are %1$s plugins in the review queue, %2$s of which are awaiting their initial review.', ( $plugins->draft + $plugins->pending ), 'wporg-plugins' ),
    35                             '<strong>' . ( $plugins->draft + $plugins->pending ) . '</strong>',
    36                             '<strong>' . $plugins->draft . '</strong>'
     34                        printf( _n( 'Currently there is %1$s plugin in the review queue.', 'Currently there are %1$s plugins in the review queue, %2$s of which are awaiting their initial review.', ( $plugins->new + $plugins->pending ), 'wporg-plugins' ),
     35                            '<strong>' . ( $plugins->new + $plugins->pending ) . '</strong>',
     36                            '<strong>' . $plugins->new . '</strong>'
    3737                        );
    3838                        ?>
    3939                    </p>
    4040                </div>
    41                 <?php
    42                 if ( '/plugins-wp' === parse_url( home_url(), PHP_URL_PATH ) ) {
    43                     echo '<div class="notice notice-error notice-alt">
    44                         <p>Please submit all new plugin requests through the existing form <a href="https://wordpress.org/plugins/add/">available here</a>, You\'re currently viewing the beta version of the upcoming plugin directory, and this form is only for testing purposes.</p>
    45                     </div>';
    46                 }
    47                 ?>
    4841
    4942            <?php endif; ?>
Note: See TracChangeset for help on using the changeset viewer.