Как поправить ошибку в коде ESLint?

Question

[Александр Фелисон]

Здравствуйте, столкнулся с тем что у меня в консоле выбивает вот такую ошибку:

Вот сам код:

$(document).ready(function () {
  'use strict'

  // Sidebar
  if ($('.videos-sidebar').length) {
    $('.mobile-sidebar-trigger').on('click', function () {
      $('.videos-sidebar').addClass('is-active')
    })
    $('.close-videos-sidebar').on('click', function () {
      $(this).closest('.videos-sidebar').removeClass('is-active')
    })

    function handleMobileSidebar () {
      if (window.matchMedia('(max-width: 767px)').matches) {
        $('.videos-sidebar').removeClass('is-active')
      } else if (window.matchMedia('(max-width: 768px)').matches) {
        $('.videos-sidebar').removeClass('is-active')
      } else {
        $('.videos-sidebar').addClass('is-active')
      }
    }

    handleMobileSidebar()

    $(window).on('resize', () => {
      handleMobileSidebar()
    })
  }

})

Заранее спасибо

Comments

[Александр Фелисон]

Баг решил)

// Sidebar
  if ($('.videos-sidebar').length) {
    $('.mobile-sidebar-trigger').on('click', function () {
      $('.videos-sidebar').addClass('is-active')
    })
    $('.close-videos-sidebar').on('click', function () {
      $(this).closest('.videos-sidebar').removeClass('is-active')
    })

    let fn

    fn = function handleMobileSidebar () {
      if (window.matchMedia('(max-width: 767px)').matches) {
        $('.videos-sidebar').removeClass('is-active')
      } else if (window.matchMedia('(max-width: 768px)').matches) {
        $('.videos-sidebar').removeClass('is-active')
      } else {
        $('.videos-sidebar').addClass('is-active')
      }
    }

    fn()

    $(window).on('resize', () => {
      fn()
    })
  }

[historydev]

Александр Фелисон, это костыль, а не решение, у вас функция инициализирована внутри условия, вынесите её за условие и всё будет прекрасно работать.

[Александр Фелисон]

Владимир, понял, исправил. Спасибо

Answer 1

[casoer]

Вынеси декларацию handleMobileSidebar с анонимной функции

function handleMobileSidebar () {
      if (window.matchMedia('(max-width: 767px)').matches) {
        $('.videos-sidebar').removeClass('is-active')
      } else if (window.matchMedia('(max-width: 768px)').matches) {
        $('.videos-sidebar').removeClass('is-active')
      } else {
        $('.videos-sidebar').addClass('is-active')
      }
    }
$(document).ready(function () {
  'use strict'

  // Sidebar
  if ($('.videos-sidebar').length) {
    $('.mobile-sidebar-trigger').on('click', function () {
      $('.videos-sidebar').addClass('is-active')
    })
    $('.close-videos-sidebar').on('click', function () {
      $(this).closest('.videos-sidebar').removeClass('is-active')
    })

    

    handleMobileSidebar()

    $(window).on('resize', () => {
      handleMobileSidebar()
    })
  }

})

Или отключи no-inner-declarations в eslint

Comments

[Александр Фелисон]

тоже спасибо)