#
# You should have received a copy of the GNU General Public License along
# with this program; if not, write to the Free Software Foundation, Inc.,
-# 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
# http://www.gnu.org/copyleft/gpl.html
/**
/**
* Basic setup and applies the limiting factors from the WebRequest object.
* @param WebRequest $request
- * @access private
+ * @private
*/
function setupQuery( $request ) {
$page = $this->db->tableName( 'page' );
$user = $this->db->tableName( 'user' );
- $this->joinClauses = array( "LEFT OUTER JOIN $page ON log_namespace=page_namespace AND log_title=page_title" );
- $this->whereClauses = array( 'user_id=log_user' );
+ $this->joinClauses = array(
+ "LEFT OUTER JOIN $page ON log_namespace=page_namespace AND log_title=page_title",
+ "INNER JOIN $user ON user_id=log_user" );
+ $this->whereClauses = array();
$this->limitType( $request->getVal( 'type' ) );
$this->limitUser( $request->getText( 'user' ) );
/**
* Set the log reader to return only entries of the given type.
* @param string $type A log type ('upload', 'delete', etc)
- * @access private
+ * @private
*/
function limitType( $type ) {
if( empty( $type ) ) {
/**
* Set the log reader to return only entries by the given user.
* @param string $name (In)valid user name
- * @access private
+ * @private
*/
function limitUser( $name ) {
if ( $name == '' )
return false;
- $title = Title::makeTitle( NS_USER, $name );
- if ( is_null( $title ) )
+ $usertitle = Title::makeTitle( NS_USER, $name );
+ if ( is_null( $usertitle ) )
return false;
- $this->user = $title->getText();
- $safename = $this->db->strencode( $this->user );
- $user = $this->db->tableName( 'user' );
- $this->whereClauses[] = "user_name='$safename'";
+ $this->user = $usertitle->getText();
+
+ /* Fetch userid at first, if known, provides awesome query plan afterwards */
+ $userid = $this->db->selectField('user','user_id',array('user_name'=>$this->user));
+ if (!$userid)
+ /* It should be nicer to abort query at all,
+ but for now it won't pass anywhere behind the optimizer */
+ $this->whereClauses[] = "NULL";
+ else
+ $this->whereClauses[] = "log_user=$userid";
}
/**
* Set the log reader to return only entries affecting the given page.
* (For the block and rights logs, this is a user page.)
* @param string $page Title name as text
- * @access private
+ * @private
*/
function limitTitle( $page ) {
$title = Title::newFromText( $page );
* Set the log reader to return only entries in a given time range.
* @param string $time Timestamp of one endpoint
* @param string $direction either ">=" or "<=" operators
- * @access private
+ * @private
*/
function limitTime( $time, $direction ) {
# Direction should be a comparison operator
/**
* Build an SQL query from all the set parameters.
* @return string the SQL query
- * @access private
+ * @private
*/
function getQuery() {
$logging = $this->db->tableName( "logging" );
$user = $this->db->tableName( 'user' );
- $sql = "SELECT log_type, log_action, log_timestamp,
+ $sql = "SELECT /*! STRAIGHT_JOIN */ log_type, log_action, log_timestamp,
log_user, user_name,
log_namespace, log_title, page_id,
- log_comment, log_params FROM $user, $logging ";
- if ($this->type=="" && $this->db->indexExists('logging','times')) {
- $sql .= ' /*! FORCE INDEX (times) */ ';
- }
+ log_comment, log_params FROM $logging ";
if( !empty( $this->joinClauses ) ) {
- $sql .= implode( ',', $this->joinClauses );
+ $sql .= implode( ' ', $this->joinClauses );
}
if( !empty( $this->whereClauses ) ) {
$sql .= " WHERE " . implode( ' AND ', $this->whereClauses );
* @return ResultWrapper result object to return the relevant rows
*/
function getRows() {
- $res = $this->db->query( $this->getQuery() );
+ $res = $this->db->query( $this->getQuery(), 'LogReader::getRows' );
return $this->db->resultObject( $res );
}
/**
* @param Object $s a single row from the result set
* @return string Formatted HTML list item
- * @access private
+ * @private
*/
function logLine( $s ) {
global $wgLang;
/**
* @param OutputPage &$out where to send output
- * @access private
+ * @private
*/
function showHeader( &$out ) {
$type = $this->reader->queryType();
/**
* @param OutputPage &$out where to send output
- * @access private
+ * @private
*/
function showOptions( &$out ) {
global $wgScript;
/**
* @return string Formatted HTML
- * @access private
+ * @private
*/
function getTypeMenu() {
$out = "<select name='type'>\n";
/**
* @return string Formatted HTML
- * @access private
+ * @private
*/
function getUserInput() {
$user = htmlspecialchars( $this->reader->queryUser() );
/**
* @return string Formatted HTML
- * @access private
+ * @private
*/
function getTitleInput() {
$title = htmlspecialchars( $this->reader->queryTitle() );
/**
* @param OutputPage &$out where to send output
- * @access private
+ * @private
*/
function showPrevNext( &$out ) {
global $wgContLang,$wgRequest;