Remove comment about possibly removing the extra debugging info. Only the site admin...
[lhc/web/wiklou.git] / includes / specials / SpecialNewimages.php
index 88e34f4..cecd7df 100644 (file)
@@ -1,10 +1,29 @@
 <?php
 /**
+ * Implements Special:Newimages
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * 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.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * http://www.gnu.org/copyleft/gpl.html
+ *
  * @file
  * @ingroup SpecialPage
- * FIXME: this code is crap, should use Pager and Database::select().
  */
 
+/**
+ * @todo FIXME: this code is crap, should use Pager and Database::select().
+ */
 function wfSpecialNewimages( $par, $specialPage ) {
        global $wgUser, $wgOut, $wgLang, $wgRequest, $wgMiserMode;
 
@@ -40,7 +59,8 @@ function wfSpecialNewimages( $par, $specialPage ) {
        if ($hidebotsql) {
                $sql .= "$hidebotsql WHERE ug_group IS NULL";
        }
-       $sql .= ' ORDER BY img_timestamp DESC LIMIT 1';
+       $sql .= ' ORDER BY img_timestamp DESC';
+       $sql = $dbr->limitResult($sql, 1, false);
        $res = $dbr->query( $sql, __FUNCTION__ );
        $row = $dbr->fetchRow( $res );
        if( $row !== false ) {
@@ -48,41 +68,40 @@ function wfSpecialNewimages( $par, $specialPage ) {
        } else {
                $ts = false;
        }
-       $dbr->freeResult( $res );
-       $sql = '';
 
        # If we were clever, we'd use this to cache.
        $latestTimestamp = wfTimestamp( TS_MW, $ts );
 
        # Hardcode this for now.
        $limit = 48;
-
-       if ( $parval = intval( $par ) ) {
+       $parval = intval( $par );
+       if ( $parval ) {
                if ( $parval <= $limit && $parval > 0 ) {
                        $limit = $parval;
                }
        }
 
        $where = array();
-       $searchpar = '';
+       $searchpar = array();
        if ( $wpIlMatch != '' && !$wgMiserMode) {
-               $nt = Title::newFromUrl( $wpIlMatch );
+               $nt = Title::newFromURL( $wpIlMatch );
                if( $nt ) {
-                       $m = $dbr->escapeLike( strtolower( $nt->getDBkey() ) );
-                       $where[] = "LOWER(img_name) LIKE '%{$m}%'";
-                       $searchpar = '&wpIlMatch=' . urlencode( $wpIlMatch );
+                       $where[] = 'LOWER(img_name) ' .  $dbr->buildLike( $dbr->anyString(), strtolower( $nt->getDBkey() ), $dbr->anyString() );
+                       $searchpar['wpIlMatch'] = $wpIlMatch;
                }
        }
 
        $invertSort = false;
-       if( $until = $wgRequest->getVal( 'until' ) ) {
+       $until = $wgRequest->getVal( 'until' );
+       if( $until ) {
                $where[] = "img_timestamp < '" . $dbr->timestamp( $until ) . "'";
        }
-       if( $from = $wgRequest->getVal( 'from' ) ) {
+       $from = $wgRequest->getVal( 'from' );
+       if( $from ) {
                $where[] = "img_timestamp >= '" . $dbr->timestamp( $from ) . "'";
                $invertSort = true;
        }
-       $sql='SELECT img_size, img_name, img_user, img_user_text,'.
+       $sql = 'SELECT img_size, img_name, img_user, img_user_text,'.
             "img_description,img_timestamp FROM $image";
 
        if( $hidebotsql ) {
@@ -93,21 +112,20 @@ function wfSpecialNewimages( $par, $specialPage ) {
                $sql .= ' WHERE ' . $dbr->makeList( $where, LIST_AND );
        }
        $sql.=' ORDER BY img_timestamp '. ( $invertSort ? '' : ' DESC' );
-       $sql.=' LIMIT ' . ( $limit + 1 );
+       $sql = $dbr->limitResult($sql, ( $limit + 1 ), false);
        $res = $dbr->query( $sql, __FUNCTION__ );
 
        /**
         * We have to flip things around to get the last N after a certain date
         */
        $images = array();
-       while ( $s = $dbr->fetchObject( $res ) ) {
+       foreach ( $res as $s ) {
                if( $invertSort ) {
                        array_unshift( $images, $s );
                } else {
                        array_push( $images, $s );
                }
        }
-       $dbr->freeResult( $res );
 
        $gallery = new ImageGallery();
        $firstTimestamp = null;
@@ -127,7 +145,7 @@ function wfSpecialNewimages( $par, $specialPage ) {
                $nt = Title::newFromText( $name, NS_FILE );
                $ul = $sk->link( Title::makeTitle( NS_USER, $ut ), $ut );
 
-               $gallery->add( $nt, "$ul<br />\n<i>".$wgLang->timeanddate( $s->img_timestamp, true )."</i><br />\n" );
+               $gallery->add( $nt, "$ul<br />\n<i>".htmlspecialchars($wgLang->timeanddate( $s->img_timestamp, true ))."</i><br />\n" );
 
                $timestamp = wfTimestamp( TS_MW, $s->img_timestamp );
                if( empty( $firstTimestamp ) ) {
@@ -162,29 +180,72 @@ function wfSpecialNewimages( $par, $specialPage ) {
 
        # If we change bot visibility, this needs to be carried along.
        if( !$hidebots ) {
-               $botpar = '&hidebots=0';
+               $botpar = array( 'hidebots' => 0 );
        } else {
-               $botpar = '';
+               $botpar = array();
        }
        $now = wfTimestampNow();
        $d = $wgLang->date( $now, true );
        $t = $wgLang->time( $now, true );
-       $dateLink = $sk->makeKnownLinkObj( $titleObj, wfMsgHtml( 'sp-newimages-showfrom', $d, $t ), 
-               'from='.$now.$botpar.$searchpar );
+       $query = array_merge(
+               array( 'from' => $now ),
+               $botpar,
+               $searchpar
+       );
+
+       $dateLink = $sk->linkKnown(
+               $titleObj,
+               htmlspecialchars( wfMsg( 'sp-newimages-showfrom', $d, $t ) ),
+               array(),
+               $query
+       );
 
-       $botLink = $sk->makeKnownLinkObj($titleObj, wfMsgHtml( 'showhidebots', 
-               ($hidebots ? wfMsgHtml('show') : wfMsgHtml('hide'))),'hidebots='.($hidebots ? '0' : '1').$searchpar);
+       $query = array_merge(
+               array( 'hidebots' => ( $hidebots ? 0 : 1 ) ),
+               $searchpar
+       );
 
+       $showhide = $hidebots ? wfMsg( 'show' ) : wfMsg( 'hide' );
+
+       $botLink = $sk->linkKnown(
+               $titleObj,
+               htmlspecialchars( wfMsg( 'showhidebots', $showhide ) ),
+               array(),
+               $query
+       );
 
        $opts = array( 'parsemag', 'escapenoentities' );
        $prevLink = wfMsgExt( 'pager-newer-n', $opts, $wgLang->formatNum( $limit ) );
        if( $firstTimestamp && $firstTimestamp != $latestTimestamp ) {
-               $prevLink = $sk->makeKnownLinkObj( $titleObj, $prevLink, 'from=' . $firstTimestamp . $botpar . $searchpar );
+               $query = array_merge(
+                       array( 'from' => $firstTimestamp ),
+                       $botpar,
+                       $searchpar
+               );
+
+               $prevLink = $sk->linkKnown(
+                       $titleObj,
+                       $prevLink,
+                       array(),
+                       $query
+               );
        }
 
        $nextLink = wfMsgExt( 'pager-older-n', $opts, $wgLang->formatNum( $limit ) );
-       if( $shownImages > $limit && $lastTimestamp ) {
-               $nextLink = $sk->makeKnownLinkObj( $titleObj, $nextLink, 'until=' . $lastTimestamp.$botpar.$searchpar );
+       if( $invertSort || ( $shownImages > $limit && $lastTimestamp ) ) {
+               $query = array_merge(
+                       array( 'until' => ( $lastTimestamp ? $lastTimestamp : "" ) ),
+                       $botpar,
+                       $searchpar
+               );
+
+               $nextLink = $sk->linkKnown(
+                       $titleObj,
+                       $nextLink,
+                       array(),
+                       $query
+               );
+
        }
 
        $prevnext = '<p>' . $botLink . ' '. wfMsgHtml( 'viewprevnext', $prevLink, $nextLink, $dateLink ) .'</p>';