WordPress.org
Get WordPress

Making WordPress.org

Changeset 11369


Ignore:
Timestamp:
12/14/2021 04:39:01 AM (3 years ago)
Author:
dd32
Message:

SVN Watcher: Reports: Avoid some PHP Notices, and compress the versions contributed to table. Instead of showing 3.0, 3.1, .... 5.8 just show 3.0-5.8.

See #5978.

Location:
sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-trac-watcher/admin
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-trac-watcher/admin/list-table.php

    r11362 r11369  
    5757
    5858        if ( ! empty( $args['version'] ) ) {
    59             $where .= $wpdb->prepare( ' AND version LIKE %s', $wpdb->esc_like( $args['version'] ) . '%' );
     59            $where .= $wpdb->prepare( ' AND r.version LIKE %s', $wpdb->esc_like( $args['version'] ) . '%' );
    6060        }
    6161        if ( ! empty( $args['branch'] ) ) {
    62             $where .= $wpdb->prepare( ' AND branch LIKE %s', $wpdb->esc_like( $args['branch'] ) );
    63         }
    64 
    65         if ( ! empty( $args['revisions'] ) && preg_match( '!(?P<start>\d+):(?P<end>\d+)!', $args['revisions'], $m ) ) {
    66             $where .= $wpdb->prepare( ' AND id BETWEEN %d AND %d', $m['start'], $m['end'] );
     62            $where .= $wpdb->prepare( ' AND r.branch LIKE %s', $wpdb->esc_like( $args['branch'] ) );
     63        }
     64
     65        if ( ! empty( $args['revisions'] ) && preg_match( '!(?P<start>\d+)[:-](?P<end>(HEAD|\d+))!', $args['revisions'], $m ) ) {
     66            if ( 'HEAD' === $m['end'] ) {
     67                $where .= $wpdb->prepare( ' AND r.id > %d', $m['start'] );
     68            } else {
     69                $where .= $wpdb->prepare( ' AND r.id BETWEEN %d AND %d', $m['start'], $m['end'] );
     70            }
    6771        }
    6872        if ( ! empty( $args['revisions'] ) && preg_match( '!^[\d,]+$!', $args['revisions'], $m ) ) {
    6973            $ids = implode(',', array_map( 'intval', explode( ',', $args['revisions'] ) ) );
    70             $where .= " AND id IN({$ids})";
     74            $where .= " AND r.id IN({$ids})";
    7175        }
    7276
    7377        if ( ! empty( $args['author'] ) ) {
    74             $where .= $wpdb->prepare( ' AND author = %s', $args['author'] );
     78            $where .= $wpdb->prepare( ' AND r.author = %s', $args['author'] );
    7579        }
    7680
    … …  
    8084            }
    8185            $where .= $wpdb->prepare(
    82                 ' AND ( message LIKE %s OR p.prop_name LIKE %s )',
     86                ' AND ( r.message LIKE %s OR p.prop_name LIKE %s )',
    8387                '%' . $wpdb->esc_like( $args['s'] ) . '%',
    8488                '%' . $wpdb->esc_like( $args['s'] ) . '%',
    8589            );
    8690        }
    87 
    88     //      $where .= ' AND r.id IN( 46290, 51195, 50933, 50810 )';
    8991
    9092        $this->items = $wpdb->get_results(
    … …  
    117119        $revisions = implode( ',', array_map( 'intval', $revisions ) );
    118120
    119         $props_list = $wpdb->get_results( $wpdb->prepare(
     121        $props_list = $wpdb->get_results(
    120122            "SELECT revision,user_id,prop_name
    121123            FROM {$props_table}
    122124            WHERE revision IN({$revisions})
    123             ORDER BY LENGTH(prop_name) DESC
    124             "
    125         ) );
     125            ORDER BY LENGTH(prop_name) DESC"
     126        );
    126127
    127128        foreach ( $this->items as $i => $details ) {
    … …  
    215216        $views = [
    216217            'all' => '<a href="' . esc_url( $url ) . '">All</a>',
    217             'unknown-props' => '<a href="' . esc_url( add_query_arg( [ 'unknown-props' => 1 ], $url ) ) . '">Unknown Props</a>',
     218            'unknown-props' => '<a href="' . esc_url( add_query_arg( 'unknown-props', 1, $url ) ) . '">Unknown Props</a>',
    218219        ];
    219220
    220221        if ( defined( 'WP_CORE_LATEST_RELEASE' ) && 'core' === $this->svn['slug'] ) {
    221222            $v = sprintf( '%.1f', ((float)WP_CORE_LATEST_RELEASE+0.1) );
    222             $views['commits-to-trunk'] = '<a href="' . esc_url( add_query_arg( [ 'version' => $v ], $url ) ) . '">Commits to ' . $v .'</a>';
     223            $views['commits-to-trunk'] = '<a href="' . esc_url( add_query_arg( 'version', $v, $url ) ) . '">Commits to ' . $v .'</a>';
    223224        }
    224225
    … …  
    345346                        '<span class="user" data-prop="%s" data-user="%s">',
    346347                        esc_attr( $prop ),
    347                         esc_attr( $user->user_login )
     348                        esc_attr( $user->user_login ?? '' )
    348349                    );
    349350                    if ( $user ) {

  • sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-trac-watcher/admin/reports-page.php

    r11367 r11369  
    66    $url       = add_query_arg( 'page', $_REQUEST['page'], admin_url( 'admin.php' ) );
    77    $what      = $_REQUEST['what'] ?? '';
    8     $version   = $_REQUEST['version'] ?? (WP_CORE_LATEST_RELEASE+0.1);
     8    $version   = $_REQUEST['version'] ?? null;
    99    $revisions = $_REQUEST['revisions'] ?? '';
    1010    $branch    = $_REQUEST['branch'] ?? '';
     11    $is_core   = ( 'core' === $details['slug'] );
     12
     13    // Default to the latest version for core.
     14    if ( $is_core && is_null( $version ) ) {
     15        $version = sprintf( '%.1f', WP_CORE_LATEST_RELEASE + 0.1 );
     16    }
    1117
    1218    $url = add_query_arg( 'version', $version, $url );
    … …  
    2127            <li><a href="<?php echo $url; ?>&what=raw-contributors-and-committers">All Props+Committers matching filter grouped together</a></li>
    2228           
     29            <?php if ( $is_core ) { ?>
    2330            <li><a href="<?php echo $url; ?>&what=versions-contributed">Versions which users have contributed to. Ignores filter.</a></li>
     31            <?php } ?>
    2432        </ol>
    2533
    … …  
    2937
    3038        <?php
    31         if ( 'core' === $details['slug'] ) {
     39        if ( $is_core ) {
    3240            echo '<select name="version"><option value="">Version</option>';
    3341            foreach ( get_wordpress_versions() as $v ) {
    … …  
    5765
    5866        // Revision range.
    59         echo '<input type="text" name="revisions" placeholder="Revs: 1:50 or 1,2,4,5" value="' . esc_attr( $revisions ) .'">';
     67        echo '<input type="text" name="revisions" placeholder="Revs: 1:HEAD or 1,2,4,5" value="' . esc_attr( $revisions ) .'">';
    6068
    6169        echo '<input type="submit" class="button button-primary" value="Filter">';
    … …  
    7078        // Revisions.
    7179        if ( ! empty( $revisions ) ) {
    72             if ( false !== strpos( $revisions, ':' ) || false !== strpos( $revisions, '-' )  ) {
    73                 $where .= $wpdb->prepare( ' AND r.id BETWEEN %d AND %d', preg_split( '![-:]!', $revisions ) );
     80            if (  preg_match( '!(?P<start>\d+)[:-](?P<end>(HEAD|\d+))!', $args['revisions'], $m ) ) {
     81                if ( 'HEAD' === $m['end'] ) {
     82                    $where .= $wpdb->prepare( ' AND r.id > %d', $m['start'] );
     83                } else {
     84                    $where .= $wpdb->prepare( ' AND r.id BETWEEN %d AND %d', $m['start'], $m['end'] );
     85                }
    7486            } elseif ( false !== strpos( $revisions, ',' ) ) {
    7587                $ids = implode( ',', array_map( 'intval', explode( ',', $revisions ) ) );
    … …  
    201213                    );
    202214
     215                    // Compress the versions list down to a smaller range.
     216                    $compress = function( $versions ) {
     217                        $in = $versions;
     218                        $out = [];
     219                        if ( ! is_array( $versions ) ) {
     220                            $versions = preg_split( '![,\s]+!', $versions );
     221                        }
     222
     223                        // Don't try to compress only a few versions.
     224                        if ( count( $versions ) <= 2 ) {
     225                            return $in;
     226                        }
     227
     228                        // [ X.Y => 0, X.Y+1 => 1, ... ]
     229                        $versions = array_flip( $versions );
     230
     231                        $not_a_version = [
     232                            '0.9', '1.1', '1.2', '1.4', '1.7', '1.8', '1.9',
     233                        ];
     234
     235                        // 2.4 + 2.5 are special. 2.4 was skipped, and many users only have a 2.4 or a 2.5 prop. Give them both versions if they have either.
     236                        if ( isset( $versions['2.5'] ) || isset( $versions['2.4'] ) ) {
     237                            $versions['2.5'] = $version['2.4'] = 1;
     238                        }
     239
     240                        $i = 0; // This counter is just here to protect against infinite loops should something go wrong.
     241                        while ( $versions && $i++ < 40 ) {
     242                            $min   = sprintf( '%.1f', min( array_keys( $versions ) ) );
     243                            $max   = sprintf( '%.1f', max( array_keys( $versions ) ) );
     244                            if ( $min === $max ) {
     245                                $out[] = $min;
     246                                break;
     247                            } elseif ( $max - $min < 0.2 ) {
     248                                $out[] = "{$min}-{$max}";
     249                                break;
     250                            }
     251
     252                            foreach ( range( $min, $max+0.1, 0.1 ) as $v ) {
     253                                $v = sprintf( '%.1f', $v );
     254
     255                                if ( in_array( $v, $not_a_version ) ) {
     256                                    unset( $versions[ $v ] );
     257                                    continue;
     258                                }
     259
     260                                if ( ! isset( $versions[ $v ] ) ) {
     261                                    $last = sprintf( '%.1f', $v - 0.1 );
     262
     263                                    if ( $last == $min ) {
     264                                        $out[] = $last;
     265                                        unset( $versions[ $v ] );
     266                                        break;
     267                                    } elseif ( $last - $min < 0.15 ) {
     268                                        // No point doing 1-2, just do 1, 2
     269                                        $out[] = $min;
     270                                        $out[] = $last;
     271                                    } else {
     272                                        $out[] = "{$min}-{$last}";
     273                                    }
     274                                    break;
     275                                } elseif ( $v == $max ) {
     276                                    $out[] = "{$min}-{$max}"; // (5)";
     277                                    unset( $versions[ $v ] );
     278                                    break;
     279                                } else {
     280                                    // no break. We're between a start, and end version.
     281                                }
     282                                unset( $versions[ $v ] );
     283                            }
     284                        }
     285
     286                        return implode( ', ', $out );
     287                    };
     288
    203289                    echo '<table class="widefat striped">';
    204290                    echo '<thead><tr><th>Prop</th><th>Count</th><th>Versions</th></tr></thead>';
    … …  
    223309                            $p->count,
    224310                            esc_attr( $p->versions ),
    225                             $p->versions
     311                            $compress( $p->versions )
    226312                        );
    227313                    }

  • sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-trac-watcher/admin/ui.php

    r11362 r11369  
    7878
    7979function display_list_table( $details ) {
     80    $url   = add_query_arg( 'page', $_REQUEST['page'], admin_url( 'admin.php' ) );
    8081    $table = new Commits_List_Table( $details );
    8182    $table->prepare_items( $_REQUEST );
    … …  
    8384        <div class="wrap propstable">
    8485            <h2><?php echo esc_html( $details['name'] ); ?> Props</h2>
    85             <form method="GET" action="<?php echo esc_url( add_query_arg() ); ?>">
     86            <form method="GET" action="<?php echo esc_url( $url ); ?>">
    8687                <input type="hidden" name="page" value="<?php echo esc_attr( $_REQUEST['page'] ); ?>" />
    8788                <?php $table->search_box( 'Search', 's' ); ?>
Note: See TracChangeset for help on using the changeset viewer.