loleaflet/.eslintignore                       |    1 +
 loleaflet/src/control/Control.Presentation.js |    2 +-
 loleaflet/src/control/Control.Styles.js       |    3 ++-
 loleaflet/src/control/Control.Tabs.js         |   15 +++++++++------
 loleaflet/src/control/Parts.js                |   24 ++++++++++++++++--------
 loleaflet/src/core/Socket.js                  |    1 +
 6 files changed, 30 insertions(+), 16 deletions(-)

New commits:
commit 10fcfdd210cbebdb9d5d3d6604aac07ff0c60ea5
Author: Pranav Kant <pran...@collabora.com>
Date:   Fri Feb 19 01:02:17 2016 +0530

    loleaflet: Fix all ESLint errors
    
    And we are now finally in a clean state.
    
    Change-Id: Ifd949d925cc758a007d036e1de9df7dafd911c40

diff --git a/loleaflet/.eslintignore b/loleaflet/.eslintignore
index 546fb41..9474c51 100644
--- a/loleaflet/.eslintignore
+++ b/loleaflet/.eslintignore
@@ -1 +1,2 @@
 src/scrollbar
+src/contextMenu
diff --git a/loleaflet/src/control/Control.Presentation.js 
b/loleaflet/src/control/Control.Presentation.js
index c5b0b9e..e75cdad 100644
--- a/loleaflet/src/control/Control.Presentation.js
+++ b/loleaflet/src/control/Control.Presentation.js
@@ -2,7 +2,7 @@
  * L.Control.Presentation is used for common commands for slides.
  */
 
-/* global vex */
+/* global vex _ */
 L.Control.Presentation = L.Control.extend({
        options: {
                position: 'topleft'
diff --git a/loleaflet/src/control/Control.Styles.js 
b/loleaflet/src/control/Control.Styles.js
index 0691f4f..02965a5 100644
--- a/loleaflet/src/control/Control.Styles.js
+++ b/loleaflet/src/control/Control.Styles.js
@@ -84,8 +84,9 @@ L.Control.Styles = L.Control.extend({
                                                var outlineLevel = 
style.split('Outline')[1];
                                                var localeString = 
'Outline'.toLocaleString() + outlineLevel;
                                                item.innerHTML = localeString;
-                                       } else
+                                       } else {
                                                item.innerHTML = 
style.toLocaleString();
+                                       }
                                });
                        }
                }
diff --git a/loleaflet/src/control/Control.Tabs.js 
b/loleaflet/src/control/Control.Tabs.js
index b8022cb..4a27aa2 100644
--- a/loleaflet/src/control/Control.Tabs.js
+++ b/loleaflet/src/control/Control.Tabs.js
@@ -2,7 +2,7 @@
  * L.Control.Tabs is used to switch sheets in Calc
  */
 
-/* global $ */
+/* global $ vex _ */
 L.Control.Tabs = L.Control.extend({
        onAdd: function (map) {
                this._tabsInitialized = false;
@@ -15,10 +15,12 @@ L.Control.Tabs = L.Control.extend({
                        callback: function(key, options) {
                                var nPos = 
parseInt(options.$trigger.attr('id').split('spreadsheet-tab')[1]);
 
-                               if (key === 'insertsheetbefore')
+                               if (key === 'insertsheetbefore') {
                                        map.insertPage(nPos);
-                               if (key === 'insertsheetafter')
+                               }
+                               if (key === 'insertsheetafter') {
                                        map.insertPage(nPos + 1);
+                               }
                        },
                        items: {
                                'insertsheetbefore': {name: _('Insert sheet 
before this')},
@@ -29,19 +31,20 @@ L.Control.Tabs = L.Control.extend({
                                                        vex.dialog.confirm({
                                                                message: _('Are 
you sure you want to delete this sheet?'),
                                                                callback: 
function(data) {
-                                                                       if 
(data)
+                                                                       if 
(data) {
                                                                                
map.deletePage(nPos);
+                                                                       }
                                                                }
                                                        });
                                                }
                                 },
                                'renamesheet': {name: _('Rename sheet'),
-                                                       callback: function(key, 
options){
+                                                       callback: function(key, 
options) {
                                                        var nPos = 
parseInt(options.$trigger.attr('id').split('spreadsheet-tab')[1]);
                                                        vex.dialog.open({
                                                                message: 
_('Enter new sheet name'),
                                                                input: '<input 
name="sheetname" type="text" required />',
-                                                               callback: 
function(data){
+                                                               callback: 
function(data) {
                                                                        
map.renamePage(data.sheetname, nPos);
                                                                }
                                                        });
diff --git a/loleaflet/src/control/Parts.js b/loleaflet/src/control/Parts.js
index 3ddc9fc..8325e7c 100644
--- a/loleaflet/src/control/Parts.js
+++ b/loleaflet/src/control/Parts.js
@@ -149,8 +149,9 @@ L.Map.include({
        },
 
        insertPage: function(nPos) {
-               if (this.getDocType() === 'presentation')
+               if (this.getDocType() === 'presentation') {
                        this._socket.sendMessage('uno .uno:InsertPage');
+               }
                else if (this.getDocType() === 'spreadsheet') {
                        var command = {
                                'Name': {
@@ -165,8 +166,9 @@ L.Map.include({
 
                        this._socket.sendMessage('uno .uno:Insert ' + 
JSON.stringify(command));
                }
-               else
+               else {
                        return;
+               }
 
                var docLayer = this._docLayer;
 
@@ -178,10 +180,12 @@ L.Map.include({
                docLayer._parts++;
 
                // Since we know which part we want to set, use the index 
(instead of 'next', 'prev')
-               if (typeof nPos === 'number')
+               if (typeof nPos === 'number') {
                        this.setPart(nPos);
-               else
+               }
+               else {
                        this.setPart('next');
+               }
        },
 
        duplicatePage: function() {
@@ -201,8 +205,9 @@ L.Map.include({
        },
 
        deletePage: function (nPos) {
-               if (this.getDocType() === 'presentation' )
+               if (this.getDocType() === 'presentation') {
                        this._socket.sendMessage('uno .uno:DeletePage');
+               }
                else if (this.getDocType() === 'spreadsheet') {
                        var command = {
                                'Index': {
@@ -213,8 +218,9 @@ L.Map.include({
 
                        this._socket.sendMessage('uno .uno:Remove ' + 
JSON.stringify(command));
                }
-               else
+               else {
                        return;
+               }
 
                var docLayer = this._docLayer;
                // TO DO: Deleting all the pages causes problem.
@@ -232,10 +238,12 @@ L.Map.include({
                        docLayer._selectedPart--;
                }
 
-               if (typeof nPos === 'number')
+               if (typeof nPos === 'number') {
                        this.setPart(nPos);
-               else
+               }
+               else {
                        this.setPart(docLayer._selectedPart);
+               }
        },
 
        renamePage: function (name, nPos) {
diff --git a/loleaflet/src/core/Socket.js b/loleaflet/src/core/Socket.js
index af793a6..d3ed85b 100644
--- a/loleaflet/src/core/Socket.js
+++ b/loleaflet/src/core/Socket.js
@@ -2,6 +2,7 @@
  * L.Socket contains methods for the communication with the server
  */
 
+/* global _ */
 L.Socket = L.Class.extend({
        ProtocolVersionNumber: '0.1',
 
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to