jenkins-bot has submitted this change and it was merged.
Change subject: Fix inaccurate comment and indentation
......................................................................
Fix inaccurate comment and indentation
Change-Id: Iedaebb9491b6700bd51cecbbcc4483612ea49be4
---
M modules/ext.MassMessage.special.js
1 file changed, 20 insertions(+), 20 deletions(-)
Approvals:
Yuvipanda: Looks good to me, approved
Legoktm: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/ext.MassMessage.special.js
b/modules/ext.MassMessage.special.js
index ddfbca8..1f31c1b 100644
--- a/modules/ext.MassMessage.special.js
+++ b/modules/ext.MassMessage.special.js
@@ -22,27 +22,27 @@
'prop': 'info',
'indexpageids': true
} ).done( function ( data ) {
- if ( data && data.query &&
- // If the page exists
and has a supported content model
- ( data.query.pageids[0]
!== '-1' &&
-
data.query.pages[data.query.pageids[0]].contentmodel === 'wikitext' ||
-
data.query.pages[data.query.pageids[0]].contentmodel ===
'MassMessageListContent' ) ||
- // Or if the text
refers to a category
-
data.query.pages[data.query.pageids[0]].ns === 14
- ) {
- // No error message is
displayed
- $spamliststatus
- .removeClass(
'invalid' )
- .text( '' );
- } else {
- // Otherwise, display
an error notice
- $spamliststatus
- .addClass(
'invalid' )
- .text(
mw.message( 'massmessage-parse-badpage', pagetitle ).text() );
- }
- } );
+ if ( data && data.query &&
+ // If the page exists and has a
supported content model
+ ( data.query.pageids[0] !==
'-1' &&
+
data.query.pages[data.query.pageids[0]].contentmodel === 'wikitext' ||
+
data.query.pages[data.query.pageids[0]].contentmodel ===
'MassMessageListContent' ) ||
+ // Or if the text refers to a
category
+
data.query.pages[data.query.pageids[0]].ns === 14
+ ) {
+ // No error message is displayed
+ $spamliststatus
+ .removeClass( 'invalid'
)
+ .text( '' );
+ } else {
+ // Otherwise, display an error
notice
+ $spamliststatus
+ .addClass( 'invalid' )
+ .text( mw.message(
'massmessage-parse-badpage', pagetitle ).text() );
+ }
+ } );
} else {
- // If no text is entered or if the text refers
to a category, don't display any warning
+ // If no text is entered, don't display any
warning
$spamliststatus
.removeClass( 'invalid' )
.text( '' );
--
To view, visit https://gerrit.wikimedia.org/r/172016
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iedaebb9491b6700bd51cecbbcc4483612ea49be4
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MassMessage
Gerrit-Branch: master
Gerrit-Owner: Wctaiwan <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: Wctaiwan <[email protected]>
Gerrit-Reviewer: Yuvipanda <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
