\n$1\n" . $titles . "
",
- array( 'protect-cascadeon', count( $cascadeSources ) )
+ [ 'protect-cascadeon', count( $cascadeSources ) ]
);
}
@@ -270,8 +270,8 @@ class ProtectionForm {
$user = $this->mContext->getUser();
$out = $this->mContext->getOutput();
$token = $request->getVal( 'wpEditToken' );
- if ( !$user->matchEditToken( $token, array( 'protect', $this->mTitle->getPrefixedDBkey() ) ) ) {
- $this->show( array( 'sessionfailure' ) );
+ if ( !$user->matchEditToken( $token, [ 'protect', $this->mTitle->getPrefixedDBkey() ] ) ) {
+ $this->show( [ 'sessionfailure' ] );
return false;
}
@@ -283,18 +283,18 @@ class ProtectionForm {
} elseif ( $reasonstr == 'other' ) {
$reasonstr = $this->mReason;
}
- $expiry = array();
+ $expiry = [];
foreach ( $this->mApplicableTypes as $action ) {
$expiry[$action] = $this->getExpiry( $action );
if ( empty( $this->mRestrictions[$action] ) ) {
continue; // unprotected
}
if ( !$expiry[$action] ) {
- $this->show( array( 'protect_expiry_invalid' ) );
+ $this->show( [ 'protect_expiry_invalid' ] );
return false;
}
if ( $expiry[$action] < wfTimestampNow() ) {
- $this->show( array( 'protect_expiry_old' ) );
+ $this->show( [ 'protect_expiry_old' ] );
return false;
}
}
@@ -321,9 +321,9 @@ class ProtectionForm {
* you can also return an array of message name and its parameters
*/
$errorMsg = '';
- if ( !Hooks::run( 'ProtectionForm::save', array( $this->mArticle, &$errorMsg, $reasonstr ) ) ) {
+ if ( !Hooks::run( 'ProtectionForm::save', [ $this->mArticle, &$errorMsg, $reasonstr ] ) ) {
if ( $errorMsg == '' ) {
- $errorMsg = array( 'hookaborted' );
+ $errorMsg = [ 'hookaborted' ];
}
}
if ( $errorMsg != '' ) {
@@ -351,14 +351,14 @@ class ProtectionForm {
if ( !$this->disabled ) {
$output->addModules( 'mediawiki.legacy.protect' );
$output->addJsConfigVars( 'wgCascadeableLevels', $cascadingRestrictionLevels );
- $out .= Xml::openElement( 'form', array( 'method' => 'post',
+ $out .= Xml::openElement( 'form', [ 'method' => 'post',
'action' => $this->mTitle->getLocalURL( 'action=protect' ),
- 'id' => 'mw-Protect-Form' ) );
+ 'id' => 'mw-Protect-Form' ] );
}
$out .= Xml::openElement( 'fieldset' ) .
Xml::element( 'legend', null, $context->msg( 'protect-legend' )->text() ) .
- Xml::openElement( 'table', array( 'id' => 'mwProtectSet' ) ) .
+ Xml::openElement( 'table', [ 'id' => 'mwProtectSet' ] ) .
Xml::openElement( 'tbody' );
$scExpiryOptions = wfMessage( 'protect-expiry-options' )->inContentLanguage()->text();
@@ -372,7 +372,7 @@ class ProtectionForm {
$out .= "| ";
$mProtectexpiry = Xml::label(
@@ -384,7 +384,16 @@ class ProtectionForm {
"mwProtect-$action-expires"
);
- $expiryFormOptions = '';
+ $expiryFormOptions = new XmlSelect(
+ "wpProtectExpirySelection-$action",
+ "mwProtectExpirySelection-$action",
+ $this->mExpirySelection[$action]
+ );
+ $expiryFormOptions->setAttribute( 'tabindex', '2' );
+ if ( $this->disabled ) {
+ $expiryFormOptions->setAttribute( 'disabled', 'disabled' );
+ }
+
if ( $this->mExistingExpiry[$action] ) {
if ( $this->mExistingExpiry[$action] == 'infinity' ) {
$existingExpiryMessage = $context->msg( 'protect-existing-expiry-infinity' );
@@ -392,31 +401,27 @@ class ProtectionForm {
$timestamp = $lang->userTimeAndDate( $this->mExistingExpiry[$action], $user );
$d = $lang->userDate( $this->mExistingExpiry[$action], $user );
$t = $lang->userTime( $this->mExistingExpiry[$action], $user );
- $existingExpiryMessage = $context->msg( 'protect-existing-expiry', $timestamp, $d, $t );
+ $existingExpiryMessage = $context->msg(
+ 'protect-existing-expiry',
+ $timestamp,
+ $d,
+ $t
+ );
}
- $expiryFormOptions .=
- Xml::option(
- $existingExpiryMessage->text(),
- 'existing',
- $this->mExpirySelection[$action] == 'existing'
- ) . "\n";
+ $expiryFormOptions->addOption( $existingExpiryMessage->text(), 'existing' );
}
- $expiryFormOptions .= Xml::option(
+ $expiryFormOptions->addOption(
$context->msg( 'protect-othertime-op' )->text(),
- "othertime"
- ) . "\n";
+ 'othertime'
+ );
foreach ( explode( ',', $scExpiryOptions ) as $option ) {
if ( strpos( $option, ":" ) === false ) {
$show = $value = $option;
} else {
list( $show, $value ) = explode( ":", $option );
}
- $expiryFormOptions .= Xml::option(
- $show,
- htmlspecialchars( $value ),
- $this->mExpirySelection[$action] === $value
- ) . "\n";
+ $expiryFormOptions->addOption( $show, htmlspecialchars( $value ) );
}
# Add expiry dropdown
if ( $showProtectOptions && !$this->disabled ) {
@@ -426,17 +431,12 @@ class ProtectionForm {
{$mProtectexpiry}
|
" .
- Xml::tags( 'select',
- array(
- 'id' => "mwProtectExpirySelection-$action",
- 'name' => "wpProtectExpirySelection-$action",
- 'tabindex' => '2' ) + $this->disabledAttrib,
- $expiryFormOptions ) .
+ $expiryFormOptions->getHTML() .
" |
";
}
# Add custom expiry field
- $attribs = array( 'id' => "mwProtect-$action-expires" ) + $this->disabledAttrib;
+ $attribs = [ 'id' => "mwProtect-$action-expires" ] + $this->disabledAttrib;
$out .= "
| " .
$mProtectother .
@@ -451,13 +451,13 @@ class ProtectionForm {
" |
";
}
# Give extensions a chance to add items to the form
- Hooks::run( 'ProtectionForm::buildForm', array( $this->mArticle, &$out ) );
+ Hooks::run( 'ProtectionForm::buildForm', [ $this->mArticle, &$out ] );
$out .= Xml::closeElement( 'tbody' ) . Xml::closeElement( 'table' );
// JavaScript will add another row with a value-chaining checkbox
if ( $this->mTitle->exists() ) {
- $out .= Xml::openElement( 'table', array( 'id' => 'mw-protect-table2' ) ) .
+ $out .= Xml::openElement( 'table', [ 'id' => 'mw-protect-table2' ] ) .
Xml::openElement( 'tbody' );
$out .= '
|
@@ -491,7 +491,7 @@ class ProtectionForm {
$this->mReasonSelection,
'mwProtect-reason', 4 );
- $out .= Xml::openElement( 'table', array( 'id' => 'mw-protect-table3' ) ) .
+ $out .= Xml::openElement( 'table', [ 'id' => 'mw-protect-table3' ] ) .
Xml::openElement( 'tbody' );
$out .= "
@@ -507,8 +507,8 @@ class ProtectionForm {
{$mProtectreason}
| " .
- Xml::input( 'mwProtect-reason', 60, $this->mReason, array( 'type' => 'text',
- 'id' => 'mwProtect-reason', 'maxlength' => 180 ) ) .
+ Xml::input( 'mwProtect-reason', 60, $this->mReason, [ 'type' => 'text',
+ 'id' => 'mwProtect-reason', 'maxlength' => 180 ] ) .
// Limited maxlength as the database trims at 255 bytes and other texts
// chosen by dropdown menus on this page are also included in this database field.
// The byte limit of 180 bytes is enforced in javascript
@@ -532,7 +532,7 @@ class ProtectionForm {
| " .
Xml::submitButton(
$context->msg( 'confirm' )->text(),
- array( 'id' => 'mw-Protect-submit' )
+ [ 'id' => 'mw-Protect-submit' ]
) .
" |
\n";
@@ -544,8 +544,8 @@ class ProtectionForm {
$link = Linker::linkKnown(
$context->msg( 'protect-dropdown' )->inContentLanguage()->getTitle(),
$context->msg( 'protect-edit-reasonlist' )->escaped(),
- array(),
- array( 'action' => 'edit' )
+ [],
+ [ 'action' => 'edit' ]
);
$out .= '' . $link . '
';
}
@@ -553,7 +553,7 @@ class ProtectionForm {
if ( !$this->disabled ) {
$out .= Html::hidden(
'wpEditToken',
- $user->getEditToken( array( 'protect', $this->mTitle->getPrefixedDBkey() ) )
+ $user->getEditToken( [ 'protect', $this->mTitle->getPrefixedDBkey() ] )
);
$out .= Xml::closeElement( 'form' );
}
@@ -576,18 +576,18 @@ class ProtectionForm {
);
$id = 'mwProtect-level-' . $action;
- $attribs = array(
- 'id' => $id,
- 'name' => $id,
- 'size' => count( $levels ),
- ) + $this->disabledAttrib;
- $out = Xml::openElement( 'select', $attribs );
+ $select = new XmlSelect( $id, $id, $selected );
+ $select->setAttribute( 'size', count( $levels ) );
+ if ( $this->disabled ) {
+ $select->setAttribute( 'disabled', 'disabled' );
+ }
+
foreach ( $levels as $key ) {
- $out .= Xml::option( $this->getOptionLabel( $key ), $key, $key == $selected );
+ $select->addOption( $this->getOptionLabel( $key ), $key );
}
- $out .= Xml::closeElement( 'select' );
- return $out;
+
+ return $select->getHTML();
}
/**
@@ -621,6 +621,6 @@ class ProtectionForm {
$out->addHTML( Xml::element( 'h2', null, $protectLogPage->getName()->text() ) );
LogEventsList::showLogExtract( $out, 'protect', $this->mTitle );
# Let extensions add other relevant log extracts
- Hooks::run( 'ProtectionForm::showLogExtract', array( $this->mArticle, $out ) );
+ Hooks::run( 'ProtectionForm::showLogExtract', [ $this->mArticle, $out ] );
}
}