jenkins-bot has submitted this change and it was merged.

Change subject: Add doc block and re-order curUserIsInList()
......................................................................


Add doc block and re-order curUserIsInList()

Change-Id: I82fc2460146511d50d547765a3247632c2389040
---
M modules/ext.CollaborationKit.list.edit.js
1 file changed, 16 insertions(+), 11 deletions(-)

Approvals:
  Brian Wolff: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/ext.CollaborationKit.list.edit.js 
b/modules/ext.CollaborationKit.list.edit.js
index f7f2d8b..593fe17 100644
--- a/modules/ext.CollaborationKit.list.edit.js
+++ b/modules/ext.CollaborationKit.list.edit.js
@@ -18,6 +18,22 @@
                windowManager.openWindow( dialog );
        };
 
+       /**
+        * Find if the current user is already is in list.
+        *
+        * @return {boolean}
+        */
+       curUserIsInList = function curUserIsInList() {
+               var titleObj, escapedText;
+               titleObj = mw.Title.newFromText( mw.config.get( 'wgUserName' ), 
2 );
+               escapedText = titleObj.getPrefixedText();
+               escapedText = escapedText.replace( /\\/g, '\\\\' );
+               escapedText = escapedText.replace( /"/g, '\\"' );
+               query = '.mw-collabkit-list-item[data-collabkit-item-title="' +
+                       escapedText + '"]';
+               return $( query ).length > 0;
+       };
+
        modifyExistingItem = function ( itemName ) {
                getCurrentJson( mw.config.get( 'wgArticleId' ), function ( res 
) {
                        var done = false;
@@ -582,16 +598,5 @@
                }
 
        } );
-
-       curUserIsInList = function curUserIsInList() {
-               var titleObj, escapedText;
-               titleObj = mw.Title.newFromText( mw.config.get( 'wgUserName' ), 
2 );
-               escapedText = titleObj.getPrefixedText();
-               escapedText = escapedText.replace( /\\/g, '\\\\' );
-               escapedText = escapedText.replace( /"/g, '\\"' );
-               query = '.mw-collabkit-list-item[data-collabkit-item-title="' +
-                       escapedText + '"]';
-               return $( query ).length > 0;
-       };
 
 } )( jQuery, mediaWiki, OO );

-- 
To view, visit https://gerrit.wikimedia.org/r/315190
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I82fc2460146511d50d547765a3247632c2389040
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/CollaborationKit
Gerrit-Branch: master
Gerrit-Owner: Brian Wolff <bawolff...@gmail.com>
Gerrit-Reviewer: Brian Wolff <bawolff...@gmail.com>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to