Eslint дает ошибку Assignment to property of function parameter 'element' no-param-reassign?

Question

[Norton_bike]

Eslint дает ошибку Assignment to property of function parameter 'element' no-param-reassign, как ее можно исправить код, не трогая правила самого Eslint, что бы он принял.

container.querySelectorAll('[data-title]').forEach((element) => {
        element.closest('[data-card]').style.display = element.innerText.toLowerCase().includes(val)
            ? 'block'
            : 'none';
    });

Answer 1

[Aetae]

Как обычно: // eslint-disable-next-line no-param-reassign

Но если хочется, чтоб было красиво, надо договориться по проекту, что аргумент с именем element всегда будет dom-элементом и добавить в правило
ignorePropertyModificationsFor: ['element']

Comments

[Norton_bike]

та вот договорится не получится. нужно код изменить что б приняло, а как понять пока не могу.

[Aetae]

Robert_Norton, я написал как первой сточкой. =\

container.querySelectorAll('[data-title]').forEach((element) => {
        // eslint-disable-next-line no-param-reassign
        element.closest('[data-card]').style.display = element.innerText.toLowerCase().includes(val)
            ? 'block'
            : 'none';
});

Если хочешь прям переписать код, то можешь сделать так:

for(const element of container.querySelectorAll('[data-title]')) {
  element.closest('[data-card]').style.display = element.innerText.toLowerCase().includes(val)
            ? 'block'
            : 'none';
}

Только если у тебя те настройки линтера о которых я думаю, то тут вылезет ещё одно дебильное древнее и бесполезное правило.)

[Norton_bike]

Aetae, походу про эту ошибку "iterators/generators require regenerator-runtime, which is too heavyweight for this guide to allow them. Separately, loops should be avoided in favor of array iterations no-restricted-syntax" . очень мало работаю с eslint, а он меня уже так бесит. Мне кажется там много дурных и бесполезных правил.
буду пробовать.
Спасибо тебе большое за помощь.

[Aetae]

Robert_Norton, ну тут не столько eslint виноват, сколько конкретный набор правил и его бездумное включение в проект так как "все его используют".
А тем временем набор правил airbnb распространяется и поддерживается компанией собственно Airbnb(аренда жилья, а не флагман фронт-энтда, как можно было бы подумать), которая в свою очередь ещё IE8 поддерживает, из-за чего там и есть такие правила как последнее. На просьбы обновить правила, они закономерно говорят: "это наши правила, сделайте свой набор, чего пристали".)

[Norton_bike]

Aetae, Я понял, хочешь хорошие актуальные правила, сидите и пишите, исправляйте сами.
Правило таки откоректировали и роботу приняло.
Спасибо тебе еще раз. буду набираться опыта)