jenkins-bot has submitted this change and it was merged.
Change subject: Hygiene: Enforce dot notation
......................................................................
Hygiene: Enforce dot notation
Change-Id: I5c613b110aa847a2f9f37c9052d715bbd632ce76
---
M .jscsrc
M resources/mobile.gallery/PhotoListApi.js
M resources/mobile.special.notifications.scripts/notifications.js
M resources/mobile.watchlist/WatchListApi.js
4 files changed, 3 insertions(+), 4 deletions(-)
Approvals:
Bmansurov: Looks good to me, approved
jenkins-bot: Verified
diff --git a/.jscsrc b/.jscsrc
index 33503b8..9a9d529 100644
--- a/.jscsrc
+++ b/.jscsrc
@@ -1,7 +1,6 @@
{
"preset": "wikimedia",
- "requireDotNotation": null,
"requireVarDeclFirst": null,
"requireSpacesInsideBrackets": null,
"disallowDanglingUnderscores": null,
diff --git a/resources/mobile.gallery/PhotoListApi.js
b/resources/mobile.gallery/PhotoListApi.js
index 8936acf..512f869 100644
--- a/resources/mobile.gallery/PhotoListApi.js
+++ b/resources/mobile.gallery/PhotoListApi.js
@@ -123,7 +123,7 @@
} );
if ( resp.hasOwnProperty(
'continue' ) ) {
- self.continueParams =
resp['continue'];
+ self.continueParams =
resp.continue;
} else {
self.canContinue =
false;
}
diff --git a/resources/mobile.special.notifications.scripts/notifications.js
b/resources/mobile.special.notifications.scripts/notifications.js
index 50e8289..50bfdfc 100644
--- a/resources/mobile.special.notifications.scripts/notifications.js
+++ b/resources/mobile.special.notifications.scripts/notifications.js
@@ -79,7 +79,7 @@
} );
} );
- notContinue = notifications['continue'];
+ notContinue = notifications.continue;
if ( unread.length > 0 ) {
markAsRead( unread );
} else {
diff --git a/resources/mobile.watchlist/WatchListApi.js
b/resources/mobile.watchlist/WatchListApi.js
index 283cdbb..a61f323 100644
--- a/resources/mobile.watchlist/WatchListApi.js
+++ b/resources/mobile.watchlist/WatchListApi.js
@@ -69,7 +69,7 @@
url: this.apiUrl
} ).then( function ( data ) {
if ( data.hasOwnProperty( 'continue' ) ) {
- self.continueParams = data['continue'];
+ self.continueParams = data.continue;
} else {
self.canContinue = false;
}
--
To view, visit https://gerrit.wikimedia.org/r/230952
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5c613b110aa847a2f9f37c9052d715bbd632ce76
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <[email protected]>
Gerrit-Reviewer: Bmansurov <[email protected]>
Gerrit-Reviewer: Phuedx <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
