r77736 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r77735‎ | r77736 | r77737 >
Date:19:01, 4 December 2010
Author:demon
Status:ok
Tags:
Comment:
MFT r77733-77734
Modified paths:
  • /branches/wmf/1.16wmf4/extensions/CodeReview/ui/CodeRevisionTagView.php (modified) (history)

Diff [purge]

Index: branches/wmf/1.16wmf4/extensions/CodeReview/ui/CodeRevisionTagView.php
@@ -19,7 +19,9 @@
2020
2121 function getQueryInfo() {
2222 $info = parent::getQueryInfo();
23 - $info['tables'][] = 'code_tags';
 23+ //Don't change table order, see http://www.mediawiki.org/wiki/Special:Code/MediaWiki/77733
 24+ //Bug in mysql 4 allowed incorrect table ordering joins to work
 25+ $info['tables'] = array_merge( array( 'code_tags' ), $info['tables'] );
2426 $info['conds'][] = 'cr_repo_id=ct_repo_id';
2527 $info['conds'][] = 'cr_id=ct_rev_id';
2628 $info['conds']['ct_tag'] = $this->mTag; // fixme: normalize input?
Property changes on: branches/wmf/1.16wmf4/extensions/CodeReview/ui/CodeRevisionTagView.php
___________________________________________________________________
Added: svn:mergeinfo
2729 Merged /trunk/extensions/CodeReview/ui/CodeRevisionTagView.php:r62838-74821,74985,75235,75559,77698,77700,77704,77707,77733-77734
2830 Merged /trunk/phase3/extensions/CodeReview/ui/CodeRevisionTagView.php:r77555,77558-77560,77563-77565,77573

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r77733Fix table join order bug from change between MySQL 4 to 5.1, noticed by vvv i...reedy18:38, 4 December 2010

Status & tagging log