Amire80 has submitted this change and it was merged.

Change subject: Fixing issues of jshint
......................................................................


Fixing issues of jshint

Change-Id: Ia5c35a19cd8d9fd8b7e98b6d86016c66dc3e6a52
---
M modules/ext.LanguageToolAction.js
M package.json
2 files changed, 10 insertions(+), 10 deletions(-)

Approvals:
  Amire80: Verified; Looks good to me, approved
  Eranroz: Looks good to me, but someone else must approve



diff --git a/modules/ext.LanguageToolAction.js 
b/modules/ext.LanguageToolAction.js
index c06335f..00307ea 100644
--- a/modules/ext.LanguageToolAction.js
+++ b/modules/ext.LanguageToolAction.js
@@ -50,7 +50,7 @@
                var i;
 
                for ( i = 0; i < obj.children.length; i++ ) {
-                       if ( obj.children[i].type === 'text'){
+                       if ( obj.children[i].type === 'text') {
                                nodes.push( obj.children[i] );
                        }
 
@@ -76,13 +76,13 @@
 
        textNodes = this.extract();
        model = ve.init.target.getSurface().getModel();
-       text = "";
+       text = '';
 
        for ( nodeI = 0; nodeI < textNodes.length; nodeI++ ) {
                node = textNodes[nodeI];
                nodeRange = node.getRange();
                nodeText = model.getLinearFragment( nodeRange ).getText();
-               text = text + "\n" + nodeText;
+               text = text + '\n' + nodeText;
        }
 
        // TODO: Get the language from VE's data model
@@ -90,7 +90,7 @@
 
        $.ajax( {
                type: 'POST',
-               //dataType: 'xml',
+               // dataType: 'xml',
                url: 'http://tools.wmflabs.org/languageproofing/',
                data: { language: lang,  text: text }
        } ).done(
@@ -110,15 +110,15 @@
        $( 'body' ).append( windowManager.$element );
        windowManager.addWindows( [ messageDialog ] );
 
-       errors = responseXML.getElementsByTagName( "error" );
+       errors = responseXML.getElementsByTagName( 'error' );
        console.log( errors );
 
-       response = "";
+       response = '';
 
        for ( i = 0; i < errors.length; i++ ) {
-               response = response + "ERROR " + i + " :\n";
-               response = response + "error : " + errors[i].getAttribute( 
'msg' ) + "\n";
-               response = response + "context : " + errors[i].getAttribute( 
'context' ) + "\n";
+               response = response + 'ERROR ' + i + ' :\n';
+               response = response + 'error : ' + errors[i].getAttribute( 
'msg' ) + '\n';
+               response = response + 'context : ' + errors[i].getAttribute( 
'context' ) + '\n';
                messageDialog.setData( 'error', errors[i].getAttribute( 'msg' ) 
);
                messageDialog.setData( 'context', errors[i].getAttribute( 
'context' ));
        }
diff --git a/package.json b/package.json
index a1bfe51..3613055 100644
--- a/package.json
+++ b/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "boilerplate",
+  "name": "LanguageTool",
   "version": "0.0.0",
   "scripts": {
     "test": "grunt test"

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia5c35a19cd8d9fd8b7e98b6d86016c66dc3e6a52
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/LanguageTool
Gerrit-Branch: master
Gerrit-Owner: Eranroz <[email protected]>
Gerrit-Reviewer: Amire80 <[email protected]>
Gerrit-Reviewer: Eranroz <[email protected]>
Gerrit-Reviewer: Fomafix

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to