Page MenuHomePhabricator

hook SpecialRecentChangesQuery should pass $query_options ; and respect return value
Closed, ResolvedPublic


diff patch

In SpecialRecentChangesQuery function doMainQuery hook parameters should also pass $query_options to enable external modification such as 'group by rc_namespace, rc_title'

wfRunHooks('SpecialRecentChangesQuery', array( &$conds, &$tables, &$join_conds, $opts ) );


wfRunHooks('SpecialRecentChangesQuery', array( &$conds, &$tables, &$join_conds, $opts, &$query_options ) );

The hook call lacks also checking the return value (a false return value should skip/abort further processing in case the extension took over processing).

Version: 1.16.x
Severity: normal

attachment x.diff ignored as obsolete



Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 10:59 PM
bzimport set Reference to bz22315.
bzimport added a subscriber: Unknown Object (MLST).
Wikinaut created this task.Jan 29 2010, 8:10 AM

Created attachment 7048
diff patch


IAlex added a comment.Jan 29 2010, 5:28 PM

Fixed in r61670.