chore(eslint): fix most of the eslint warnings

Author: richardschneider

js/router.js

@@ -284,10 +284,10 @@ module.exports = Backbone.Router.extend({
 
   launchPageConnectModal: function(config) {
     var defaults = {
-      connectText: 'Connecting...',
-      failedText: 'Unable to Connect.'
-    },
-    deferred = $.Deferred();
+          connectText: 'Connecting...',
+          failedText: 'Unable to Connect.'
+        },
+        deferred = $.Deferred();
 
     if (!(
         config &&

js/views/homeVw.js

@@ -17,8 +17,12 @@ module.exports = pageVw.extend({
   className: "homeView contentWrapper",
 
   events: {
-    'click .js-productsTab': function(){this.setState("products");},
-    'click .js-vendorsTab': function(){this.setState("vendors");},
+    'click .js-productsTab': function() {
+      this.setState("products");
+    },
+    'click .js-vendorsTab': function() {
+      this.setState("vendors");
+    },
     'click .js-homeCreateStore': 'createStore',
     'click .js-homeCreateListing': 'createListing',
     'click .js-homeSearchItemsClear': 'onSearchItemsClear',
@@ -203,11 +207,11 @@ module.exports = pageVw.extend({
         self.listingToggle = self.$('.js-homeListingToggle');
 
         self.setState(self.state, self.searchItemsText);
-        if(self.model.get('page').profile.vendor == true) {
+        if (self.model.get('page').profile.vendor == true) {
           self.$el.find('.js-homeCreateStore').addClass('hide');
           self.$el.find('.js-homeMyPage').addClass('show');
           self.$el.find('.js-homeCreateListing').addClass('show');
-        }else{
+        } else {
           self.$el.find('.js-homeCreateStore').addClass('show');
           self.$el.find('.js-homeCreateListing').addClass('hide');
         }
@@ -216,7 +220,7 @@ module.exports = pageVw.extend({
         self.loadingVendors = true;
         self.socketView.getVendors(self.socketUsersID);
         //set the filter
-        if(localStorage.getItem('homeShowAll') == "yes"){
+        if (localStorage.getItem('homeShowAll') == "yes"){
           self.setListingsAll();
           self.loadAllItems();
         } else {
@@ -246,7 +250,9 @@ module.exports = pageVw.extend({
   renderItem: function(item){
     var self = this,
         blocked,
-        addressCountries = this.userModel.get('shipping_addresses').map(function(address){ return address.country; }),
+        addressCountries = this.userModel.get('shipping_addresses').map(function(address) {
+          return address.country;
+        }),
         userCountry = this.userModel.get('country'),
         contract_type = item.contract_type;
 

js/views/itemEditVw.js

@@ -474,14 +474,36 @@ module.exports = baseVw.extend({
           canvas.height = imgW;
         }
         switch (orientation) {
-          case 2: ctx.translate(imgW, 0);      ctx.scale(-1, 1); break;
-          case 3: ctx.translate(imgW, imgH);   ctx.rotate(Math.PI); break;
-          case 4: ctx.translate(0, imgH);      ctx.scale(1, -1); break;
-          case 5: ctx.rotate(0.5 * Math.PI);   ctx.scale(1, -1); break;
-          case 6: ctx.rotate(0.5 * Math.PI);   ctx.translate(0, -imgH); break;
-          case 7: ctx.rotate(0.5 * Math.PI);   ctx.translate(imgW, -imgH); ctx.scale(-1, 1); break;
-          case 8: ctx.rotate(-0.5 * Math.PI);  ctx.translate(-imgW, 0); break;
-          default: // do nothing
+        case 2:
+          ctx.translate(imgW, 0);
+          ctx.scale(-1, 1);
+          break;
+        case 3:
+          ctx.translate(imgW, imgH);
+          ctx.rotate(Math.PI);
+          break;
+        case 4:
+          ctx.translate(0, imgH);
+          ctx.scale(1, -1);
+          break;
+        case 5:
+          ctx.rotate(0.5 * Math.PI);
+          ctx.scale(1, -1);
+          break;
+        case 6:
+          ctx.rotate(0.5 * Math.PI);
+          ctx.translate(0, -imgH);
+          break;
+        case 7:
+          ctx.rotate(0.5 * Math.PI);
+          ctx.translate(imgW, -imgH);
+          ctx.scale(-1, 1);
+          break;
+        case 8:
+          ctx.rotate(-0.5 * Math.PI);
+          ctx.translate(-imgW, 0);
+          break;
+        default: // do nothing
         }
         ctx.drawImage(newImage, 0, 0, imgW, imgH);
         dataURI = canvas.toDataURL('image/jpeg', 0.7);

js/views/pageNavVw.js

@@ -540,8 +540,7 @@ module.exports = baseVw.extend({
         $target.parents('.popMenu').length ||
         $target.is('[data-popmenu]') ||
         $target.parents('[data-popmenu]').length
-      ))
-    {
+      )) {
       if (!this.suggestionsVw.isVisible()) {
         app.hideOverlay();
       }

js/views/settingsVw.js

@@ -236,7 +236,7 @@ module.exports = pageVw.extend({
         placeholder_text_multiple: window.polyglot.t('chosenJS.placeHolderTextMultiple')
       });
 
-      self.avatarCropper =  self.$('#settings-image-cropper');
+      self.avatarCropper = self.$('#settings-image-cropper');
 
       self.avatarCropper.cropit({
         $preview: self.$('.js-settingsAvatarPreview'),
@@ -313,7 +313,7 @@ module.exports = pageVw.extend({
       editor.subscribe('blur', self.validateDescription);
 
       self.sortableAddresses && self.sortableAddresses.destroy();
-      if(self.$('.js-sortableAddresses').length) {
+      if (self.$('.js-sortableAddresses').length) {
         self.sortableAddresses = Sortable.create(self.$('.js-sortableAddresses')[0], {
           chosenClass: "addressBoxDragging",
           ghostClass: "addressBoxGhost"

js/views/suggestionsVw.js

@@ -88,8 +88,7 @@ module.exports = baseVw.extend({
 
         if (keyCode == 40) {
           this.itemMoveDown();
-        } 
-        else {
+        } else {
           this.itemMoveUp();
         }
       }
@@ -258,4 +257,4 @@ module.exports = baseVw.extend({
   remove: function() {
     baseVw.prototype.remove.apply(this, arguments);
   }
-});
+});

js/views/transactionModalVw.js

@@ -679,7 +679,7 @@ module.exports = baseModal.extend({
 
     if (this.discussionInput.val()) {
       saveToAPI(this.discussionForm, '', this.serverUrl + "dispute_contract", function () {
-        self.status   = 4;
+        self.status = 4;
         self.tabState = "discussion";
         self.$('.js-startDisputeFlag').addClass('hide');
         self.getData();
@@ -769,7 +769,7 @@ module.exports = baseModal.extend({
       refData.order_id = this.orderID;
 
       saveToAPI(null, null, this.serverUrl + "refund", function () {
-        self.status   = 7;
+        self.status = 7;
         self.tabState = "summary";
         self.getData();
       }, function (data) {

main.js

@@ -24,8 +24,8 @@ var fs = require('fs'),
 var launched_from_installer = false;
 var platform = os.platform();
 switch (platform) {
-  case "darwin":
-    platform = "mac";
+case "darwin":
+  platform = "mac";
 }
 var version = app.getVersion();
 var trayMenu = null;
@@ -134,7 +134,7 @@ var kill_local_server = function() {
 };
 
 var start_local_server = function() {
-  if(fs.existsSync(serverPath)) {
+  if (fs.existsSync(serverPath)) {
     if (pendingKill) {
       pendingKill.once('close', startAfterClose = () => {
         start_local_server();
@@ -167,18 +167,18 @@ var start_local_server = function() {
     subpy.stderr.on('data', function (buf) {
       console.log('[STR] stderr "%s"', String(buf));
       fs.appendFile(__dirname + path.sep + "python_error.log", String(buf), function(err) {
-          if(err) {
-              return console.log(err);
-          }
+        if (err) {
+          return console.log(err);
+        }
       });
       stderr += buf;
     });
     subpy.on('error', function (err) {
       console.log('Python error %s', String(err));
       fs.appendFile(__dirname + path.sep + "python_error.log", String(err), function(error) {
-          if(error) {
-              return console.log(error);
-          }
+        if (error) {
+          return console.log(error);
+        }
       });
     });
     subpy.on('close', function (code) {
@@ -192,12 +192,12 @@ var start_local_server = function() {
     mainWindow && mainWindow.webContents.executeJavaScript("console.log('Unable to find OpenBazaar-Server at: '" + serverPath + "')");
   }
   if (fs.existsSync(__dirname + path.sep + '..' + path.sep + 'gpg')) {
-       process.env.PATH = __dirname + path.sep + '..' + path.sep + 'gpg' + path.sep + 'pub' + path.sep + ';' + process.env.PATH;
-   }
+    process.env.PATH = __dirname + path.sep + '..' + path.sep + 'gpg' + path.sep + 'pub' + path.sep + ';' + process.env.PATH;
+  }
 };
 
 // Check if we need to kick off the python server-daemon (Desktop app)
-if(fs.existsSync(__dirname + path.sep + ".." + path.sep + "OpenBazaar-Server" + path.sep + daemon)) {
+if (fs.existsSync(__dirname + path.sep + ".." + path.sep + "OpenBazaar-Server" + path.sep + daemon)) {
   global.launched_from_installer = launched_from_installer = true;
 }
 
@@ -319,11 +319,11 @@ app.on('window-all-closed', function() {
 
 // You can use 'before-quit' instead of (or with) the close event
 app.on('before-quit', function () {
-    // Handle menu-item or keyboard shortcut quit here
-    console.log('Closing Application');
-    if (launched_from_installer) {
-      kill_local_server();
-    }
+  // Handle menu-item or keyboard shortcut quit here
+  console.log('Closing Application');
+  if (launched_from_installer) {
+    kill_local_server();
+  }
 });
 
 app.commandLine.appendSwitch('ignore-certificate-errors', true);
@@ -518,15 +518,15 @@ app.on('ready', function() {
       ]
     },
     {
-    label: 'Window',
-    submenu: [
-      {
-        label: 'Minimize',
-        selector: 'performMiniaturize:',
-        accelerator: 'Command+M'
-      }
-    ]
-  }
+      label: 'Window',
+      submenu: [
+        {
+          label: 'Minimize',
+          selector: 'performMiniaturize:',
+          accelerator: 'Command+M'
+        }
+      ]
+    }
   ]);
   menu.setApplicationMenu(appMenu);
 
@@ -537,8 +537,8 @@ app.on('ready', function() {
   var template = [
     {
       label: 'Start Local Server', type: 'normal', click: function () {
-      start_local_server();
-    }
+        start_local_server();
+      }
     },
     {
       label: 'Shutdown Local Server', type: 'normal', click: function () {
@@ -579,7 +579,7 @@ app.on('ready', function() {
     }}
   ];
 
-  if(launched_from_installer) {
+  if (launched_from_installer) {
     template.push({label: 'View Python Error Log', type: 'normal', click: function() {
       var logPath = __dirname + path.sep + 'python_error.log';
       open(logPath);
@@ -615,7 +615,7 @@ app.on('ready', function() {
   });
 
   // and load the index.html of the app.
-  if(open_url) {
+  if (open_url) {
     mainWindow.loadURL('file://' + __dirname + '/index.html' + open_url);
   } else {
     mainWindow.loadURL('file://' + __dirname + '/index.html');
@@ -666,13 +666,13 @@ app.on('ready', function() {
   });
 
   var feedURL = 'https://updates.openbazaar.org:5001/update/' + platform + '/' + version;
-  autoUpdater.setFeedURL(feedURL)
+  autoUpdater.setFeedURL(feedURL);
   mainWindow.webContents.executeJavaScript("console.log('Checking for new versions at " + feedURL + " ...')");
 
   // Check for updates every hour
   autoUpdater.checkForUpdates();
   setInterval(function () {
-      autoUpdater.checkForUpdates();
+    autoUpdater.checkForUpdates();
   }, 3600000);
 
 });

valiquire-redirect.js

@@ -18,7 +18,9 @@ var electronModules = [
 
 module.exports = function redirect(request) {
   // Tell valiquire to ignore these modules
-  if(~electronModules.indexOf(request)) return null;
+  if (~electronModules.indexOf(request)) {
+    return null;
+  }
 
   // all others we don't redirect
   return request;