dépôts
/
lhc
/
web
/
wiklou.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Merge "Add mw-changeslist-(log|ns)-* class to OldChangesList as well"
[lhc/web/wiklou.git]
/
includes
/
specials
/
SpecialBooksources.php
diff --git
a/includes/specials/SpecialBooksources.php
b/includes/specials/SpecialBooksources.php
index
91a148d
..
fe90a4f
100644
(file)
--- a/
includes/specials/SpecialBooksources.php
+++ b/
includes/specials/SpecialBooksources.php
@@
-34,9
+34,6
@@
class SpecialBookSources extends SpecialPage {
*/
private $isbn = '';
*/
private $isbn = '';
- /**
- * Constructor
- */
public function __construct() {
parent::__construct( 'Booksources' );
}
public function __construct() {
parent::__construct( 'Booksources' );
}
@@
-49,9
+46,9
@@
class SpecialBookSources extends SpecialPage {
public function execute( $isbn ) {
$this->setHeaders();
$this->outputHeader();
public function execute( $isbn ) {
$this->setHeaders();
$this->outputHeader();
- $this->isbn = self::cleanIsbn( $isbn ?
$isbn
: $this->getRequest()->getText( 'isbn' ) );
+ $this->isbn = self::cleanIsbn( $isbn ?: $this->getRequest()->getText( 'isbn' ) );
$this->getOutput()->addHTML( $this->makeForm() );
$this->getOutput()->addHTML( $this->makeForm() );
- if (
strlen( $this->isbn ) > 0
) {
+ if (
$this->isbn !== ''
) {
if ( !self::isValidISBN( $this->isbn ) ) {
$this->getOutput()->wrapWikiMsg(
"<div class=\"error\">\n$1\n</div>",
if ( !self::isValidISBN( $this->isbn ) ) {
$this->getOutput()->wrapWikiMsg(
"<div class=\"error\">\n$1\n</div>",
@@
-63,7
+60,8
@@
class SpecialBookSources extends SpecialPage {
}
/**
}
/**
- * Returns whether a given ISBN (10 or 13) is valid. True indicates validity.
+ * Return whether a given ISBN (10 or 13) is valid.
+ *
* @param string $isbn ISBN passed for check
* @return bool
*/
* @param string $isbn ISBN passed for check
* @return bool
*/
@@
-124,10
+122,10
@@
class SpecialBookSources extends SpecialPage {
$form = Html::openElement( 'fieldset' ) . "\n";
$form .= Html::element(
'legend',
$form = Html::openElement( 'fieldset' ) . "\n";
$form .= Html::element(
'legend',
-
array()
,
+
[]
,
$this->msg( 'booksources-search-legend' )->text()
) . "\n";
$this->msg( 'booksources-search-legend' )->text()
) . "\n";
- $form .= Html::openElement( 'form',
array( 'method' => 'get', 'action' => wfScript() )
) . "\n";
+ $form .= Html::openElement( 'form',
[ 'method' => 'get', 'action' => wfScript() ]
) . "\n";
$form .= Html::hidden( 'title', $this->getPageTitle()->getPrefixedText() ) . "\n";
$form .= '<p>' . Xml::inputLabel(
$this->msg( 'booksources-isbn' )->text(),
$form .= Html::hidden( 'title', $this->getPageTitle()->getPrefixedText() ) . "\n";
$form .= '<p>' . Xml::inputLabel(
$this->msg( 'booksources-isbn' )->text(),
@@
-135,12
+133,12
@@
class SpecialBookSources extends SpecialPage {
'isbn',
20,
$this->isbn,
'isbn',
20,
$this->isbn,
- array( 'autofocus' => true, 'class' => 'mw-ui-input-inline' )
+ [ 'autofocus' => '', 'class' => 'mw-ui-input-inline' ]
);
$form .= ' ' . Html::submitButton(
$this->msg( 'booksources-search' )->text(),
);
$form .= ' ' . Html::submitButton(
$this->msg( 'booksources-search' )->text(),
- array(), array( 'mw-ui-progressive' )
+ [], [ 'mw-ui-progressive' ]
) . "</p>\n";
$form .= Html::closeElement( 'form' ) . "\n";
) . "</p>\n";
$form .= Html::closeElement( 'form' ) . "\n";
@@
-154,14
+152,14
@@
class SpecialBookSources extends SpecialPage {
* format and output them
*
* @throws MWException
* format and output them
*
* @throws MWException
- * @return
string
+ * @return
bool
*/
private function showList() {
global $wgContLang;
# Hook to allow extensions to insert additional HTML,
# e.g. for API-interacting plugins and so on
*/
private function showList() {
global $wgContLang;
# Hook to allow extensions to insert additional HTML,
# e.g. for API-interacting plugins and so on
- Hooks::run( 'BookInformation',
array( $this->isbn, $this->getOutput() )
);
+ Hooks::run( 'BookInformation',
[ $this->isbn, $this->getOutput() ]
);
# Check for a local page such as Project:Book_sources and use that if available
$page = $this->msg( 'booksources' )->inContentLanguage()->text();
# Check for a local page such as Project:Book_sources and use that if available
$page = $this->msg( 'booksources' )->inContentLanguage()->text();
@@
-171,7
+169,7
@@
class SpecialBookSources extends SpecialPage {
$content = $rev->getContent();
if ( $content instanceof TextContent ) {
$content = $rev->getContent();
if ( $content instanceof TextContent ) {
- //XXX: in the future, this could be stored as structured data, defining a list of book sources
+ //
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 ) );
$text = $content->getNativeData();
$this->getOutput()->addWikiText( str_replace( 'MAGICNUMBER', $this->isbn, $text ) );
@@
-204,8
+202,9
@@
class SpecialBookSources extends SpecialPage {
private function makeListItem( $label, $url ) {
$url = str_replace( '$1', $this->isbn, $url );
private function makeListItem( $label, $url ) {
$url = str_replace( '$1', $this->isbn, $url );
- return Html::rawElement( 'li', array(),
- Html::element( 'a', array( 'href' => $url, 'class' => 'external' ), $label ) );
+ return Html::rawElement( 'li', [],
+ Html::element( 'a', [ 'href' => $url, 'class' => 'external' ], $label )
+ );
}
protected function getGroupName() {
}
protected function getGroupName() {