"Catrope" posted a comment on MediaWiki.r105613.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/105613#c27646

Commit summary for MediaWiki.r105613:

created moodbar user email input / confirm templates and js to post to new api, 
frontend followup to rr105385. added moodbar tool tip module, moodbar status 
bubble icons

Catrope's comment:

<pre>
+                       $(document).ready( function() {
+                               $.ajax( {
+                                       'type': 'POST',
+                                       'url': mw.util.wikiScript( 'api' ),
+                                       'data': query,
+                                       'success': function (data) {
+                                               mb.userData = 
data.query.userinfo;
+                                       },
+                                       'error': function( jqXHR, textStatus, 
errorThrown ) {
+                                               mb.userData = null;
+                                       },
+                                       'dataType': 'json'
+                               } );
+                       });
</pre>
Why is this AJAX call in a document ready wrapper? I mean I'm sure it's 
harmless, but it looks strange to me.

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

Reply via email to