Merge "Remove use of implicitGroupby() in WikiPage"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Tue, 19 Sep 2017 16:37:46 +0000 (16:37 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Tue, 19 Sep 2017 16:37:47 +0000 (16:37 +0000)
1  2 
includes/page/WikiPage.php

@@@ -213,7 -213,6 +213,7 @@@ class WikiPage implements Page, IDBAcce
         * @todo Move this UI stuff somewhere else
         *
         * @see ContentHandler::getActionOverrides
 +       * @return array
         */
        public function getActionOverrides() {
                return $this->getContentHandler()->getActionOverrides();
  
                $dbr = wfGetDB( DB_REPLICA );
  
-               if ( $dbr->implicitGroupby() ) {
-                       $realNameField = 'user_real_name';
-               } else {
-                       $realNameField = 'MIN(user_real_name) AS user_real_name';
-               }
                $tables = [ 'revision', 'user' ];
  
                $fields = [
                        'user_id' => 'rev_user',
                        'user_name' => 'rev_user_text',
-                       $realNameField,
+                       'user_real_name' => 'MIN(user_real_name)',
                        'timestamp' => 'MAX(rev_timestamp)',
                ];
  
                }
  
                if ( $useParserCache ) {
 -                      $parserOutput = ParserCache::singleton()->get( $this, $parserOptions );
 +                      $parserOutput = MediaWikiServices::getInstance()->getParserCache()
 +                              ->get( $this, $parserOptions );
                        if ( $parserOutput !== false ) {
                                return $parserOutput;
                        }
                                        $wikiPage = $this;
                                        // Trigger post-create hook
                                        $params = [ &$wikiPage, &$user, $content, $summary,
 -                                              $flags & EDIT_MINOR, null, null, &$flags, $revision ];
 +                                                              $flags & EDIT_MINOR, null, null, &$flags, $revision ];
                                        Hooks::run( 'PageContentInsertComplete', $params );
                                        // Trigger post-save hook
 -                                      $params = array_merge( $params, [ &$status, $meta['baseRevId'] ] );
 +                                      $params = array_merge( $params, [ &$status, $meta['baseRevId'], 0 ] );
                                        Hooks::run( 'PageContentSaveComplete', $params );
                                }
                        ),
  
                // Save it to the parser cache.
                // Make sure the cache time matches page_touched to avoid double parsing.
 -              ParserCache::singleton()->save(
 +              MediaWikiServices::getInstance()->getParserCache()->save(
                        $editInfo->output, $this, $editInfo->popts,
                        $revision->getTimestamp(), $editInfo->revid
                );
         * @param array|string &$error Array of errors to append to
         * @param User $user The deleting user
         * @param array $tags Tags to apply to the deletion action
 +       * @param string $logsubtype
         * @return Status Status object; if successful, $status->value is the log_id of the
         *   deletion log entry. If the page couldn't be deleted because it wasn't
         *   found, $status is a non-fatal 'cannotdelete' error