diff options
Diffstat (limited to 'includes/specials/SpecialBooksources.php')
-rw-r--r-- | includes/specials/SpecialBooksources.php | 81 |
1 files changed, 51 insertions, 30 deletions
diff --git a/includes/specials/SpecialBooksources.php b/includes/specials/SpecialBooksources.php index bf7de3f5..5ad961c3 100644 --- a/includes/specials/SpecialBooksources.php +++ b/includes/specials/SpecialBooksources.php @@ -46,15 +46,15 @@ class SpecialBookSources extends SpecialPage { /** * Show the special page * - * @param $isbn string ISBN passed as a subpage parameter + * @param string $isbn ISBN passed as a subpage parameter */ public function execute( $isbn ) { $this->setHeaders(); $this->outputHeader(); $this->isbn = self::cleanIsbn( $isbn ? $isbn : $this->getRequest()->getText( 'isbn' ) ); $this->getOutput()->addHTML( $this->makeForm() ); - if( strlen( $this->isbn ) > 0 ) { - if( !self::isValidISBN( $this->isbn ) ) { + if ( strlen( $this->isbn ) > 0 ) { + if ( !self::isValidISBN( $this->isbn ) ) { $this->getOutput()->wrapWikiMsg( "<div class=\"error\">\n$1\n</div>", 'booksources-invalid-isbn' ); } $this->showList(); @@ -62,46 +62,47 @@ class SpecialBookSources extends SpecialPage { } /** - * Returns whether a given ISBN (10 or 13) is valid. True indicates validity. - * @param isbn string ISBN passed for check + * Returns whether a given ISBN (10 or 13) is valid. True indicates validity. + * @param string $isbn ISBN passed for check * @return bool */ public static function isValidISBN( $isbn ) { $isbn = self::cleanIsbn( $isbn ); $sum = 0; - if( strlen( $isbn ) == 13 ) { - for( $i = 0; $i < 12; $i++ ) { - if($i % 2 == 0) { + if ( strlen( $isbn ) == 13 ) { + for ( $i = 0; $i < 12; $i++ ) { + if ( $i % 2 == 0 ) { $sum += $isbn[$i]; } else { $sum += 3 * $isbn[$i]; } } - $check = (10 - ($sum % 10)) % 10; - if ($check == $isbn[12]) { + $check = ( 10 - ( $sum % 10 ) ) % 10; + if ( $check == $isbn[12] ) { return true; } - } elseif( strlen( $isbn ) == 10 ) { - for($i = 0; $i < 9; $i++) { - $sum += $isbn[$i] * ($i + 1); + } elseif ( strlen( $isbn ) == 10 ) { + for ( $i = 0; $i < 9; $i++ ) { + $sum += $isbn[$i] * ( $i + 1 ); } $check = $sum % 11; - if($check == 10) { + if ( $check == 10 ) { $check = "X"; } - if($check == $isbn[9]) { + if ( $check == $isbn[9] ) { return true; } } + return false; } /** * Trim ISBN and remove characters which aren't required * - * @param $isbn string Unclean ISBN + * @param string $isbn Unclean ISBN * @return string */ private static function cleanIsbn( $isbn ) { @@ -116,13 +117,14 @@ class SpecialBookSources extends SpecialPage { private function makeForm() { global $wgScript; - $form = '<fieldset><legend>' . $this->msg( 'booksources-search-legend' )->escaped() . '</legend>'; - $form .= Xml::openElement( 'form', array( 'method' => 'get', 'action' => $wgScript ) ); - $form .= Html::hidden( 'title', $this->getTitle()->getPrefixedText() ); - $form .= '<p>' . Xml::inputLabel( $this->msg( 'booksources-isbn' )->text(), 'isbn', 'isbn', 20, $this->isbn ); - $form .= ' ' . Xml::submitButton( $this->msg( 'booksources-go' )->text() ) . '</p>'; - $form .= Xml::closeElement( 'form' ); - $form .= '</fieldset>'; + $form = Html::openElement( 'fieldset' ) . "\n"; + $form .= Html::element( 'legend', array(), $this->msg( 'booksources-search-legend' )->text() ) . "\n"; + $form .= Html::openElement( 'form', array( 'method' => 'get', 'action' => $wgScript ) ) . "\n"; + $form .= Html::hidden( 'title', $this->getTitle()->getPrefixedText() ) . "\n"; + $form .= '<p>' . Xml::inputLabel( $this->msg( 'booksources-isbn' )->text(), 'isbn', 'isbn', 20, $this->isbn, array( 'autofocus' => true ) ); + $form .= ' ' . Xml::submitButton( $this->msg( 'booksources-go' )->text() ) . "</p>\n"; + $form .= Html::closeElement( 'form' ) . "\n"; + $form .= Html::closeElement( 'fieldset' ) . "\n"; return $form; } @@ -130,6 +132,7 @@ class SpecialBookSources extends SpecialPage { * Determine where to get the list of book sources from, * format and output them * + * @throws MWException * @return string */ private function showList() { @@ -142,31 +145,49 @@ class SpecialBookSources extends SpecialPage { # Check for a local page such as Project:Book_sources and use that if available $page = $this->msg( 'booksources' )->inContentLanguage()->text(); $title = Title::makeTitleSafe( NS_PROJECT, $page ); # Show list in content language - if( is_object( $title ) && $title->exists() ) { + if ( is_object( $title ) && $title->exists() ) { $rev = Revision::newFromTitle( $title, false, Revision::READ_NORMAL ); - $this->getOutput()->addWikiText( str_replace( 'MAGICNUMBER', $this->isbn, $rev->getText() ) ); - return true; + $content = $rev->getContent(); + + if ( $content instanceof TextContent ) { + //XXX: in the future, this could be stored as structured data, defining a list of book sources + + $text = $content->getNativeData(); + $this->getOutput()->addWikiText( str_replace( 'MAGICNUMBER', $this->isbn, $text ) ); + + return true; + } else { + throw new MWException( "Unexpected content type for book sources: " . $content->getModel() ); + } } # Fall back to the defaults given in the language file $this->getOutput()->addWikiMsg( 'booksources-text' ); $this->getOutput()->addHTML( '<ul>' ); $items = $wgContLang->getBookstoreList(); - foreach( $items as $label => $url ) + foreach ( $items as $label => $url ) { $this->getOutput()->addHTML( $this->makeListItem( $label, $url ) ); + } $this->getOutput()->addHTML( '</ul>' ); + return true; } /** * Format a book source list item * - * @param $label string Book source label - * @param $url string Book source URL + * @param string $label Book source label + * @param string $url Book source URL * @return string */ private function makeListItem( $label, $url ) { $url = str_replace( '$1', $this->isbn, $url ); - return '<li><a href="' . htmlspecialchars( $url ) . '" class="external">' . htmlspecialchars( $label ) . '</a></li>'; + + return Html::rawElement( 'li', array(), + Html::element( 'a', array( 'href' => $url, 'class' => 'external' ), $label ) ); + } + + protected function getGroupName() { + return 'other'; } } |