Fix linting errors

.eslintrc.json

@@ -57,7 +57,7 @@
         // best practice
         "block-scoped-var": "error",
         "curly": "error",
-        "default-case": "error",
+        "default-case": "warn",
         "dot-location": ["error", "property"],
         "dot-notation": "error",
         "eqeqeq": ["error", "smart"],
@@ -162,7 +162,7 @@
         "space-before-blocks": "error",
         "space-before-function-paren": ["error", "never"],
         "space-in-parens": "warn",
-        "space-infix-ops": "error",
+        "space-infix-ops": "warn",
         "space-unary-ops": "error",
         "spaced-comment": "warn",
 

src/components/accounts/actions.js

@@ -1,5 +1,6 @@
 import { routeActions } from 'react-router-redux';
 
+import { sessionStorage } from 'services/localStorage';
 import authentication from 'services/api/authentication';
 import { updateUser, setGuest } from 'components/user/actions';
 import { setLocale } from 'components/i18n/actions';

src/services/logger/abbreviate.js

@@ -14,10 +14,8 @@ const STRING_MAX_LENGTH = 128*1024;
  *
  * @see https://github.com/ftlabs/js-abbreviate
  */
-function abbreviate(obj, options) {
+function abbreviate(obj, options = {}) {
-    if (!options) options = {};
+    var filter = options.filter || function(key, value) {return value;};
-
-    var filter = options.filter || function(k,v){return v;};
     var maxDepth = options.depth || 10;
     var maxSize = options.maxSize || 1 * 1024 * 1024;
 
@@ -54,7 +52,9 @@ function abbreviateRecursive(key, obj, filter, state, maxDepth) {
                 var newobj = Array.isArray(obj) ? [] : {};
                 for (var i in obj) {
                     newobj[i] = abbreviateRecursive(i, obj[i], filter, state, maxDepth-1);
-                    if (state.sizeLeft < 0) break;
+                    if (state.sizeLeft < 0) {
+                        break;
+                    }
                 }
                 return newobj;
 
@@ -68,36 +68,38 @@ function abbreviateRecursive(key, obj, filter, state, maxDepth) {
             case 'undefined':
                 return obj;
         }
-    } catch(e) {/* fall back to inspect*/}
+    } catch (err) {/* fall back to inspect*/}
 
     try {
         obj = limitStringLength('' + obj);
         state.sizeLeft -= obj.length;
         return obj;
-    } catch(e) {
+    } catch (err) {
-        return "**non-serializable**";
+        return '**non-serializable**';
     }
 }
 
 function commonFilter(key, val) {
-    if ('function' === typeof val) {
+    if (typeof val === 'function') {
         return undefined;
     }
 
     if (val instanceof Date) {
-        return "**Date** " + val;
+        return '**Date** ' + val;
     }
 
     if (val instanceof Error) {
-        var err = {
+        const err = {
             // These properties are implemented as magical getters and don't show up in for in
             stack: val.stack,
             message: val.message,
             name: val.name,
         };
-        for(var i in val) {
+
+        for (let i in val) {
             err[i] = val[i];
         }
+
         return err;
     }
 
@@ -150,5 +152,5 @@ export {
 export default function(obj) {
     return abbreviate(obj, {
         filter: browserFilter
-    })
+    });
-};
+}