style: fix eslint issues with old files

Author: marcosmoura

src/components/MdApp/MdApp.vue

@@ -14,6 +14,22 @@
     return componentOptions && componentTypes.includes(componentOptions.tag)
   }
 
+  function isRightDrawer ({ mdRight }) {
+    return mdRight === '' || !!mdRight
+  }
+
+  function createRightDrawer (isMdRight) {
+    if (isMdRight) {
+      const drawerRightPrevious = createElement(MdDrawerRightPrevious, { props: {...child.data.attrs}})
+      drawerRightPrevious.data.slot = 'md-app-drawer-right-previous'
+      slots.push(drawerRightPrevious)
+    }
+  }
+
+  function shouldRenderSlot (data, componentOptions) {
+    rreturn (data && componentTypes.includes(data.slot)) || isValidChild(componentOptions)
+  }
+
   function buildSlots (children, context, functionalContext, options, createElement) {
     let slots = []
 
@@ -24,29 +40,25 @@
         const data = child.data
         const componentOptions = child.componentOptions
 
-        if ((data && componentTypes.includes(data.slot)) || isValidChild(componentOptions)) {
+        if (shouldRenderSlot(data, componentOptions)) {
           child.data.slot = data.slot || componentOptions.tag
 
           if (componentOptions.tag === 'md-app-drawer') {
+            const isRightDrawer = isRightDrawer(componentOptions.propsData)
+
             if (hasDrawer) {
               Vue.util.warn(`There shouldn't be more than one drawer in a MdApp at one time.`)
               return
             }
 
             hasDrawer = true
-            let nativeMdRight = componentOptions.propsData.mdRight
-            let mdRight = nativeMdRight === '' || !!nativeMdRight
-            child.data.slot += `-${mdRight ? 'right' : 'left'}`
+            child.data.slot += `-${isRightDrawer ? 'right' : 'left'}`
             child.key = JSON.stringify({
               'persistent': child.data.attrs['md-persistent'],
               'permanent': child.data.attrs['md-permanent']
             })
 
-            if (mdRight) {
-              const drawerRightPrevious = createElement(MdDrawerRightPrevious, { props: {...child.data.attrs}})
-              drawerRightPrevious.data.slot = 'md-app-drawer-right-previous'
-              slots.push(drawerRightPrevious)
-            }
+            createRightDrawer(isRightDrawer)
           }
 
           child.data.provide = options.Ctor.options.provide

src/components/MdBadge/MdBadge.vue

@@ -40,12 +40,7 @@
         return !!this.$slots.default
       },
       badgeClasses () {
-        const staticClass = this.$vnode.data.staticClass ?
-          this.$vnode.data.staticClass.split(' ').filter(val => val).reduce((result, key) => {
-          result[key] = true
-          return result
-        }, {}) : {}
-
+        const staticClass = this.getStaticClass()
         const dynamicClass = this.$vnode.data.class
 
         return {
@@ -63,6 +58,20 @@
           ...style
         }
       }
+    },
+    methods: {
+      getStaticClass () {
+        const staticClass = this.$vnode.data.staticClass
+
+        function filterClasses () {
+          staticClass.split(' ').filter(val => val).reduce((result, key) => {
+            result[key] = true
+            return result
+          }, {})
+        }
+
+        return staticClass ? filterClasses() : {}
+      }
     }
   })
 </script>

src/components/MdMenu/MdMenuContent.vue

@@ -193,14 +193,21 @@
 
         return Boolean(alignTrigger || offsetY || offsetX)
       },
+      isMenu ({ target }) {
+        return this.MdMenu.$el ? this.MdMenu.$el.contains(target) : false
+      },
+      isMenuContentEl ({ target }) {
+        return this.$refs.menu ? this.$refs.menu.contains(target) : false
+      },
+      isBackdropExpectMenu ($event) {
+        return !this.$el.contains($event.target) && !this.isMenu($event)
+      },
       createClickEventObserver () {
         if (document) {
           this.MdMenu.bodyClickObserver = new MdObserveEvent(document.body, 'click', $event => {
             $event.stopPropagation()
-            let isMdMenu = this.MdMenu.$el ? this.MdMenu.$el.contains($event.target) : false
-            let isMenuContentEl = this.$refs.menu ? this.$refs.menu.contains($event.target) : false
-            let onBackdropExpectMenu = !this.$el.contains($event.target) && !isMenuContentEl
-            if (!isMdMenu && (this.MdMenu.closeOnClick || onBackdropExpectMenu)) {
+
+            if (!this.isMenu($event) && (this.MdMenu.closeOnClick || this.isBackdropExpectMenu($event))) {
               this.MdMenu.active = false
               this.MdMenu.bodyClickObserver.destroy()
               this.MdMenu.windowResizeObserver.destroy()
@@ -217,19 +224,26 @@
       },
       keyNavigation (event) {
         switch (event.key) {
-          case 'ArrowUp':
-            event.preventDefault()
-            this.setHighlight('up')
-            break
-          case 'ArrowDown':
-            event.preventDefault()
-            this.setHighlight('down')
-            break
-          case 'Enter': this.setSelection()
-            break
-          case 'Space': this.setSelection()
-            break
-          case 'Escape': this.onEsc()
+        case 'ArrowUp':
+          event.preventDefault()
+          this.setHighlight('up')
+          break
+
+        case 'ArrowDown':
+          event.preventDefault()
+          this.setHighlight('down')
+          break
+
+        case 'Enter':
+          this.setSelection()
+          break
+
+        case 'Space':
+          this.setSelection()
+          break
+
+        case 'Escape':
+          this.onEsc()
         }
       },
       createResizeObserver () {

src/components/MdTable/MdTable.vue

@@ -235,16 +235,17 @@
       },
       'MdTable.selectedItems' (val, old) {
         let changed = (() => {
-          let isValEmpty = !val || val.length === 0
-          let isOldEmpty = !old || old.length === 0
+          let isValEmpty = this.isEmpty(val)
+          let isOldEmpty = this.isEmpty(old)
+          let hasValues = isValEmpty && isOldEmpty
 
-          if (isValEmpty && isOldEmpty) {
+          if (hasValues) {
             return false
-          } else if (!isValEmpty && !isOldEmpty) {
+          } else if (!hasValues) {
             return (val.length !== old.length) ? true : !val.every((item, index) => item == old[index])
-          } else {
-            return true
           }
+
+          return true
         })()
 
         if (changed) {
@@ -261,6 +262,9 @@
       }
     },
     methods: {
+      isEmpty (value) {
+        return !value || value.length === 0
+      },
       emitEvent (eventName, value) {
         this.$emit(eventName, value)
       },