As suggested to me by email: very active members of a wiki community can end up recei...
[lhc/web/wiklou.git] / includes / specials / SpecialWhatlinkshere.php
index 7af8ac8..3740442 100644 (file)
@@ -1,33 +1,53 @@
 <?php
 /**
- * @todo Use some variant of Pager or something; the pagination here is lousy.
+ * Implements Special:Whatlinkshere
+ *
+ * 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
+ * @todo Use some variant of Pager or something; the pagination here is lousy.
  */
 
 /**
- * implements Special:Whatlinkshere
+ * Implements Special:Whatlinkshere
+ *
  * @ingroup SpecialPage
  */
 class SpecialWhatLinksHere extends SpecialPage {
 
-       // Stored objects
-       protected $opts, $target, $selfTitle;
+       /**
+        * @var FormOptions
+        */
+       protected $opts;
+
+       protected $selfTitle;
 
-       // Stored globals
-       protected $skin;
+       /**
+        * @var Title
+        */
+       protected $target;
 
        protected $limits = array( 20, 50, 100, 250, 500 );
 
        public function __construct() {
                parent::__construct( 'Whatlinkshere' );
-               global $wgUser;
-               $this->skin = $wgUser->getSkin();
        }
 
        function execute( $par ) {
-               global $wgOut, $wgRequest;
+               $out = $this->getOutput();
 
                $this->setHeaders();
 
@@ -43,7 +63,7 @@ class SpecialWhatLinksHere extends SpecialPage {
                $opts->add( 'hidelinks', false );
                $opts->add( 'hideimages', false );
 
-               $opts->fetchValuesFromRequest( $wgRequest );
+               $opts->fetchValuesFromRequest( $this->getRequest() );
                $opts->validateIntBounds( 'limit', 0, 5000 );
 
                // Give precedence to subpage syntax
@@ -56,29 +76,32 @@ class SpecialWhatLinksHere extends SpecialPage {
 
                $this->target = Title::newFromURL( $opts->getValue( 'target' ) );
                if( !$this->target ) {
-                       $wgOut->addHTML( $this->whatlinkshereForm() );
+                       $out->addHTML( $this->whatlinkshereForm() );
                        return;
                }
 
-               $this->selfTitle = SpecialPage::getTitleFor( 'Whatlinkshere', $this->target->getPrefixedDBkey() );
+               $this->getSkin()->setRelevantTitle( $this->target );
+
+
+               $this->selfTitle = $this->getTitle( $this->target->getPrefixedDBkey() );
 
-               $wgOut->setPageTitle( wfMsg( 'whatlinkshere-title', $this->target->getPrefixedText() ) );
-               $wgOut->setSubtitle( wfMsg( 'whatlinkshere-backlink', $this->skin->link( $this->target, $this->target->getPrefixedText(), array(), array( 'redirect' => 'no'  ) ) ) );
+               $out->setPageTitle( wfMsg( 'whatlinkshere-title', $this->target->getPrefixedText() ) );
+               $out->setSubtitle( wfMsg( 'whatlinkshere-backlink', Linker::link( $this->target, $this->target->getPrefixedText(), array(), array( 'redirect' => 'no'  ) ) ) );
 
                $this->showIndirectLinks( 0, $this->target, $opts->getValue( 'limit' ),
                        $opts->getValue( 'from' ), $opts->getValue( 'back' ) );
        }
 
        /**
-        * @param $level  int     Recursion level
+        * @param $level int     Recursion level
         * @param $target Title   Target title
-        * @param $limit  int     Number of entries to display
-        * @param $from   Title   Display from this article ID
-        * @param $back   Title   Display from this article ID at backwards scrolling
-        * @private
+        * @param $limit int     Number of entries to display
+        * @param $from Title   Display from this article ID
+        * @param $back Title   Display from this article ID at backwards scrolling
         */
        function showIndirectLinks( $level, $target, $limit, $from = 0, $back = 0 ) {
-               global $wgOut, $wgMaxRedirectLinksRetrieved;
+               global $wgMaxRedirectLinksRetrieved;
+               $out = $this->getOutput();
                $dbr = wfGetDB( DB_SLAVE );
                $options = array();
 
@@ -155,14 +178,14 @@ class SpecialWhatLinksHere extends SpecialPage {
 
                if( ( !$fetchlinks || !$dbr->numRows($plRes) ) && ( $hidetrans || !$dbr->numRows($tlRes) ) && ( $hideimages || !$dbr->numRows($ilRes) ) ) {
                        if ( 0 == $level ) {
-                               $wgOut->addHTML( $this->whatlinkshereForm() );
+                               $out->addHTML( $this->whatlinkshereForm() );
 
                                // Show filters only if there are links
                                if( $hidelinks || $hidetrans || $hideredirs || $hideimages )
-                                       $wgOut->addHTML( $this->getFilterPanel() );
+                                       $out->addHTML( $this->getFilterPanel() );
 
                                $errMsg = is_int($namespace) ? 'nolinkshere-ns' : 'nolinkshere';
-                               $wgOut->addWikiMsg( $errMsg, $this->target->getPrefixedText() );
+                               $out->addWikiMsg( $errMsg, $this->target->getPrefixedText() );
                        }
                        return;
                }
@@ -171,29 +194,25 @@ class SpecialWhatLinksHere extends SpecialPage {
                // templatelinks comes second so that the templatelinks row overwrites the
                // pagelinks row, so we get (inclusion) rather than nothing
                if( $fetchlinks ) {
-                       while ( $row = $dbr->fetchObject( $plRes ) ) {
+                       foreach ( $plRes as $row ) {
                                $row->is_template = 0;
                                $row->is_image = 0;
                                $rows[$row->page_id] = $row;
                        }
-                       $dbr->freeResult( $plRes );
-
                }
                if( !$hidetrans ) {
-                       while ( $row = $dbr->fetchObject( $tlRes ) ) {
+                       foreach ( $tlRes as $row ) {
                                $row->is_template = 1;
                                $row->is_image = 0;
                                $rows[$row->page_id] = $row;
                        }
-                       $dbr->freeResult( $tlRes );
                }
                if( !$hideimages ) {
-                       while ( $row = $dbr->fetchObject( $ilRes ) ) {
+                       foreach ( $ilRes as $row ) {
                                $row->is_template = 0;
                                $row->is_image = 1;
                                $rows[$row->page_id] = $row;
                        }
-                       $dbr->freeResult( $ilRes );
                }
 
                // Sort by key and then change the keys to 0-based indices
@@ -216,39 +235,42 @@ class SpecialWhatLinksHere extends SpecialPage {
                $prevId = $from;
 
                if ( $level == 0 ) {
-                       $wgOut->addHTML( $this->whatlinkshereForm() );
-                       $wgOut->addHTML( $this->getFilterPanel() );
-                       $wgOut->addWikiMsg( 'linkshere', $this->target->getPrefixedText() );
+                       $out->addHTML( $this->whatlinkshereForm() );
+                       $out->addHTML( $this->getFilterPanel() );
+                       $out->addWikiMsg( 'linkshere', $this->target->getPrefixedText() );
 
                        $prevnext = $this->getPrevNext( $prevId, $nextId );
-                       $wgOut->addHTML( $prevnext );
+                       $out->addHTML( $prevnext );
                }
 
-               $wgOut->addHTML( $this->listStart() );
+               $out->addHTML( $this->listStart( $level ) );
                foreach ( $rows as $row ) {
                        $nt = Title::makeTitle( $row->page_namespace, $row->page_title );
 
                        if ( $row->page_is_redirect && $level < 2 ) {
-                               $wgOut->addHTML( $this->listItem( $row, $nt, true ) );
+                               $out->addHTML( $this->listItem( $row, $nt, true ) );
                                $this->showIndirectLinks( $level + 1, $nt, $wgMaxRedirectLinksRetrieved );
-                               $wgOut->addHTML( Xml::closeElement( 'li' ) );
+                               $out->addHTML( Xml::closeElement( 'li' ) );
                        } else {
-                               $wgOut->addHTML( $this->listItem( $row, $nt ) );
+                               $out->addHTML( $this->listItem( $row, $nt ) );
                        }
                }
 
-               $wgOut->addHTML( $this->listEnd() );
+               $out->addHTML( $this->listEnd() );
 
                if( $level == 0 ) {
-                       $wgOut->addHTML( $prevnext );
+                       $out->addHTML( $prevnext );
                }
        }
 
-       protected function listStart() {
-               return Xml::openElement( 'ul', array ( 'id' => 'mw-whatlinkshere-list' ) );
+       protected function listStart( $level ) {
+               return Xml::openElement( 'ul', ( $level ? array() : array( 'id' => 'mw-whatlinkshere-list' ) ) );
        }
 
        protected function listItem( $row, $nt, $notClose = false ) {
+               global $wgLang;
+               $dirmark = $wgLang->getDirMark();
+
                # local message cache
                static $msgcache = null;
                if ( $msgcache === null ) {
@@ -266,7 +288,7 @@ class SpecialWhatLinksHere extends SpecialPage {
                        $query = array();
                }
 
-               $link = $this->skin->linkKnown(
+               $link = Linker::linkKnown(
                        $nt,
                        null,
                        array(),
@@ -292,8 +314,8 @@ class SpecialWhatLinksHere extends SpecialPage {
                $wlh = Xml::wrapClass( "($wlhLink)", 'mw-whatlinkshere-tools' );
 
                return $notClose ?
-                       Xml::openElement( 'li' ) . "$link $propsText $wlh\n" :
-                       Xml::tags( 'li', null, "$link $propsText $wlh" ) . "\n";
+                       Xml::openElement( 'li' ) . "$link $propsText $dirmark $wlh\n" :
+                       Xml::tags( 'li', null, "$link $propsText $dirmark $wlh" ) . "\n";
        }
 
        protected function listEnd() {
@@ -303,9 +325,9 @@ class SpecialWhatLinksHere extends SpecialPage {
        protected function wlhLink( Title $target, $text ) {
                static $title = null;
                if ( $title === null )
-                       $title = SpecialPage::getTitleFor( 'Whatlinkshere' );
+                       $title = $this->getTitle();
 
-               return $this->skin->linkKnown(
+               return Linker::linkKnown(
                        $title,
                        $text,
                        array(),
@@ -314,7 +336,7 @@ class SpecialWhatLinksHere extends SpecialPage {
        }
 
        function makeSelfLink( $text, $query ) {
-               return $this->skin->linkKnown(
+               return Linker::linkKnown(
                        $this->selfTitle,
                        $text,
                        array(),
@@ -366,11 +388,11 @@ class SpecialWhatLinksHere extends SpecialPage {
 
                # Build up the form
                $f = Xml::openElement( 'form', array( 'action' => $wgScript ) );
-               
+
                # Values that should not be forgotten
-               $f .= Xml::hidden( 'title', SpecialPage::getTitleFor( 'Whatlinkshere' )->getPrefixedText() );
+               $f .= Html::hidden( 'title', $this->getTitle()->getPrefixedText() );
                foreach ( $this->opts->getUnconsumedValues() as $name => $value ) {
-                       $f .= Xml::hidden( $name, $value );
+                       $f .= Html::hidden( $name, $value );
                }
 
                $f .= Xml::fieldset( wfMsg( 'whatlinkshere' ) );
@@ -398,7 +420,7 @@ class SpecialWhatLinksHere extends SpecialPage {
 
        /**
         * Create filter panel
-        * 
+        *
         * @return string HTML fieldset and filter panel with the show/hide links
         */
        function getFilterPanel() {