It helps when you don't break things...

[lhc/web/wiklou.git] / includes / SpecialBooksources.php

diff --git a/includes/SpecialBooksources.php b/includes/SpecialBooksources.php
index 5c047fb..5f10349 100644 (file)
--- a/includes/SpecialBooksources.php
+++ b/includes/SpecialBooksources.php
@@ -4,8 +4,7 @@
  * Special page outputs information on sourcing a book with a particular ISBN
  * The parser creates links to this page when dealing with ISBNs in wikitext
  *
- * @package MediaWiki
- * @subpackage Special pages
+ * @addtogroup SpecialPage
  * @author Rob Church <robchur@gmail.com>
  * @todo Validate ISBNs using the standard check-digit method
  */
@@ -15,29 +14,29 @@ class SpecialBookSources extends SpecialPage {
         * ISBN passed to the page, if any
         */
        private $isbn = '';
-       
+
        /**
         * Constructor
         */
        public function __construct() {
                parent::__construct( 'Booksources' );
        }
-       
+
        /**
         * Show the special page
         *
         * @param $isbn ISBN passed as a subpage parameter
         */
-       public function execute( $isbn = false ) {
+       public function execute( $isbn ) {
                global $wgOut, $wgRequest;
                $this->setHeaders();
                $this->isbn = $this->cleanIsbn( $isbn ? $isbn : $wgRequest->getText( 'isbn' ) );
                $wgOut->addWikiText( wfMsgNoTrans( 'booksources-summary' ) );
                $wgOut->addHtml( $this->makeForm() );
-               if( strlen( $this->isbn) > 0 )
+               if( strlen( $this->isbn ) > 0 )
                        $this->showList();
        }
-       
+
        /**
         * Trim ISBN and remove characters which aren't required
         *
@@ -47,7 +46,7 @@ class SpecialBookSources extends SpecialPage {
        private function cleanIsbn( $isbn ) {
                return trim( preg_replace( '![^0-9X]!', '', $isbn ) );
        }
-       
+
        /**
         * Generate a form to allow users to enter an ISBN
         *
@@ -65,7 +64,7 @@ class SpecialBookSources extends SpecialPage {
                $form .= '</fieldset>';
                return $form;
        }
-       
+
        /**
         * Determine where to get the list of book sources from,
         * format and output them
@@ -74,15 +73,19 @@ class SpecialBookSources extends SpecialPage {
         */
        private function showList() {
                global $wgOut, $wgContLang;
-               
+
+               # Hook to allow extensions to insert additional HTML,
+               # e.g. for API-interacting plugins and so on
+               wfRunHooks( 'BookInformation', array( $this->isbn, &$wgOut ) );
+
                # Check for a local page such as Project:Book_sources and use that if available
-               $title = Title::makeTitleSafe( NS_PROJECT, wfMsg( 'booksources' ) ); # Should this be wfMsgForContent()? -- RC
+               $title = Title::makeTitleSafe( NS_PROJECT, wfMsgForContent( 'booksources' ) ); # Show list in content language
                if( is_object( $title ) && $title->exists() ) {
                        $rev = Revision::newFromTitle( $title );
                        $wgOut->addWikiText( str_replace( 'MAGICNUMBER', $this->isbn, $rev->getText() ) );
                        return true;
                }
-               
+
                # Fall back to the defaults given in the language file
                $wgOut->addWikiText( wfMsgNoTrans( 'booksources-text' ) );
                $wgOut->addHtml( '<ul>' );
@@ -107,4 +110,4 @@ class SpecialBookSources extends SpecialPage {
 
 }
 
-?>
+