Remove implicitGroupby() use in ImageListPager
authorAaron Schulz <aschulz@wikimedia.org>
Sat, 9 Sep 2017 21:05:58 +0000 (14:05 -0700)
committerAaron Schulz <aschulz@wikimedia.org>
Sat, 9 Sep 2017 21:05:58 +0000 (14:05 -0700)
Note this only effects wikis with miser mode disabled, so there
is less risk of a poor query plan.

Change-Id: I692b23f176489b9a0907bcc65b4ca93f10367c48

includes/specials/pagers/ImageListPager.php

index 813d1d4..1c46f4c 100644 (file)
@@ -286,13 +286,8 @@ class ImageListPager extends TablePager {
                        }
                        unset( $field );
 
-                       $dbr = wfGetDB( DB_REPLICA );
-                       if ( $dbr->implicitGroupby() ) {
-                               $options = [ 'GROUP BY' => 'img_name' ];
-                       } else {
-                               $columnlist = preg_grep( '/^img/', array_keys( $this->getFieldNames() ) );
-                               $options = [ 'GROUP BY' => array_merge( [ 'img_user' ], $columnlist ) ];
-                       }
+                       $columnlist = preg_grep( '/^img/', array_keys( $this->getFieldNames() ) );
+                       $options = [ 'GROUP BY' => array_merge( [ 'img_user' ], $columnlist ) ];
                        $join_conds = [ 'oldimage' => [ 'LEFT JOIN', 'oi_name = img_name' ] ];
                }