r67775 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r67774‎ | r67775 | r67776 >
Date:06:15, 10 June 2010
Author:werdna
Status:deferred
Tags:
Comment:
Include LiquidThreads posts in the definition of "edits" for ActiveStrategy
Modified paths:
  • /trunk/extensions/StrategyWiki/ActiveStrategy/ActiveStrategy_body.php (modified) (history)

Diff [purge]

Index: trunk/extensions/StrategyWiki/ActiveStrategy/ActiveStrategy_body.php
@@ -131,8 +131,18 @@
132132 $tables[] = 'revision';
133133 $joinConds['revision'] =
134134 array( 'left join',
135 - array( 'rev_page=page_id', "rev_timestamp > $cutoff" ) );
136 - $fields[] = 'count(distinct rev_id) as value';
 135+ array( 'rev_page=page_id',
 136+ "rev_timestamp > $cutoff",
 137+ "rev_page IS NOT NULL" ) );
 138+ $fields[] = 'count(distinct rev_id) + count(distinct thread_id) as value';
 139+
 140+ // Include LQT posts
 141+ $tables[] = 'thread';
 142+ $joinConds['thread'] =
 143+ array( 'left join',
 144+ array( 'thread.thread_article_title=page.page_title',
 145+ "thread.thread_modified > $cutoff" )
 146+ );
137147 } elseif ( $sortField == 'ranking' ) {
138148 $tables[] = 'pagelinks';
139149 $joinConds['pagelinks'] = array( 'left join',

Follow-up revisions

RevisionCommit summaryAuthorDate
r67780Merge r67775 and r67778werdna06:29, 10 June 2010

Status & tagging log