mirror of
https://github.com/danbulant/oxc
synced 2026-05-24 20:32:10 +00:00
feat(linter) eslint plugin unicorn: prefer dom node dataset (#1257)
This commit is contained in:
Cameron
GitHub
parent
76d4690218
commit
c0d42b8a33
3 changed files with 690 additions and 0 deletions
|
|
@ -170,6 +170,7 @@ mod unicorn {
|
|||
pub mod prefer_code_point;
|
||||
pub mod prefer_date_now;
|
||||
pub mod prefer_dom_node_append;
|
||||
pub mod prefer_dom_node_dataset;
|
||||
pub mod prefer_logical_operator_over_ternary;
|
||||
pub mod prefer_optional_catch_binding;
|
||||
pub mod prefer_query_selector;
|
||||
|
|
@ -321,6 +322,7 @@ oxc_macros::declare_all_lint_rules! {
|
|||
unicorn::prefer_code_point,
|
||||
unicorn::prefer_date_now,
|
||||
unicorn::prefer_dom_node_append,
|
||||
unicorn::prefer_dom_node_dataset,
|
||||
unicorn::prefer_logical_operator_over_ternary,
|
||||
unicorn::prefer_optional_catch_binding,
|
||||
unicorn::prefer_query_selector,
|
||||
|
|
|
|||
263
crates/oxc_linter/src/rules/unicorn/prefer_dom_node_dataset.rs
Normal file
263
crates/oxc_linter/src/rules/unicorn/prefer_dom_node_dataset.rs
Normal file
|
|
@ -0,0 +1,263 @@
|
|||
use oxc_ast::{
|
||||
ast::{Argument, Expression},
|
||||
AstKind,
|
||||
};
|
||||
use oxc_diagnostics::{
|
||||
miette::{self, Diagnostic},
|
||||
thiserror::Error,
|
||||
};
|
||||
use oxc_macros::declare_oxc_lint;
|
||||
use oxc_span::Span;
|
||||
|
||||
use crate::{context::LintContext, rule::Rule, AstNode};
|
||||
|
||||
#[derive(Debug, Error, Diagnostic)]
|
||||
enum PreferDomNodeDatasetDiagnostic {
|
||||
#[error("eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.")]
|
||||
#[diagnostic(
|
||||
severity(warning),
|
||||
help("Access the `.dataset` object directly: `element.dataset.{1} = ...;`")
|
||||
)]
|
||||
Set(#[label] Span, String),
|
||||
#[error("eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `getAttribute`.")]
|
||||
#[diagnostic(
|
||||
severity(warning),
|
||||
help("Access the `.dataset` object directly: `element.dataset.{1}`")
|
||||
)]
|
||||
Get(#[label] Span, String),
|
||||
#[error("eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `hasAttribute`.")]
|
||||
#[diagnostic(
|
||||
severity(warning),
|
||||
help("Check the `dataset` object directly: `Object.hasOwn(element.dataset, '{1}')")
|
||||
)]
|
||||
Has(#[label] Span, String),
|
||||
#[error("eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `removeAttribute`.")]
|
||||
#[diagnostic(
|
||||
severity(warning),
|
||||
help("Access the `.dataset` object directly: `delete element.dataset.{1};")
|
||||
)]
|
||||
Remove(#[label] Span, String),
|
||||
}
|
||||
|
||||
#[derive(Debug, Default, Clone)]
|
||||
pub struct PreferDomNodeDataset;
|
||||
|
||||
declare_oxc_lint!(
|
||||
/// ### What it does
|
||||
///
|
||||
/// Use [`.dataset`](https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/dataset) on DOM elements over `getAttribute(…)`, `.setAttribute(…)`, `.removeAttribute(…)` and `.hasAttribute(…)`.
|
||||
///
|
||||
/// ### Why is this bad?
|
||||
///
|
||||
/// The `dataset` property is a map of strings that contains all the `data-*` attributes from the element. It is a convenient way to access all of them at once.
|
||||
///
|
||||
/// ### Example
|
||||
/// ```javascript
|
||||
/// // Bad
|
||||
/// element.setAttribute('data-unicorn', '🦄');
|
||||
///
|
||||
/// // Good
|
||||
/// element.dataset.unicorn = '🦄';
|
||||
/// ```
|
||||
PreferDomNodeDataset,
|
||||
pedantic
|
||||
);
|
||||
|
||||
impl Rule for PreferDomNodeDataset {
|
||||
fn run<'a>(&self, node: &AstNode<'a>, ctx: &LintContext<'a>) {
|
||||
let AstKind::CallExpression(call_expr) = node.kind() else {
|
||||
return;
|
||||
};
|
||||
|
||||
let Some(member_expr) = call_expr.callee.get_member_expr() else { return };
|
||||
|
||||
if member_expr.is_computed() {
|
||||
return;
|
||||
}
|
||||
|
||||
let Some((span, method_name)) = member_expr.static_property_info() else { return };
|
||||
|
||||
match method_name {
|
||||
"setAttribute" => {
|
||||
if call_expr.arguments.len() != 2 {
|
||||
return;
|
||||
}
|
||||
}
|
||||
"getAttribute" | "removeAttribute" | "hasAttribute" => {
|
||||
if call_expr.arguments.len() != 1 {
|
||||
return;
|
||||
}
|
||||
}
|
||||
_ => return,
|
||||
}
|
||||
|
||||
let Argument::Expression(Expression::StringLiteral(string_lit)) = &call_expr.arguments[0]
|
||||
else {
|
||||
return;
|
||||
};
|
||||
|
||||
let Some(dataset_property_name) = strip_data_prefix(&string_lit.value) else {
|
||||
return;
|
||||
};
|
||||
|
||||
match method_name {
|
||||
"setAttribute" => {
|
||||
ctx.diagnostic(PreferDomNodeDatasetDiagnostic::Set(span, dataset_property_name));
|
||||
}
|
||||
"getAttribute" => {
|
||||
ctx.diagnostic(PreferDomNodeDatasetDiagnostic::Get(span, dataset_property_name));
|
||||
}
|
||||
|
||||
"removeAttribute" => ctx.diagnostic(PreferDomNodeDatasetDiagnostic::Remove(
|
||||
string_lit.span,
|
||||
dataset_property_name,
|
||||
)),
|
||||
|
||||
"hasAttribute" => {
|
||||
ctx.diagnostic(PreferDomNodeDatasetDiagnostic::Has(span, dataset_property_name));
|
||||
}
|
||||
|
||||
_ => unreachable!(),
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
fn strip_data_prefix(s: &str) -> Option<String> {
|
||||
let prefix = "data-";
|
||||
if s.len() >= prefix.len() && s[..prefix.len()].eq_ignore_ascii_case(prefix) {
|
||||
Some(s[prefix.len()..].to_string())
|
||||
} else {
|
||||
None
|
||||
}
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn test() {
|
||||
use crate::tester::Tester;
|
||||
|
||||
let pass = vec![
|
||||
r#"element.dataset.unicorn = '🦄';"#,
|
||||
r#"element.dataset['unicorn'] = '🦄';"#,
|
||||
r#"new element.setAttribute('data-unicorn', '🦄');"#,
|
||||
r#"setAttribute('data-unicorn', '🦄');"#,
|
||||
r#"element['setAttribute']('data-unicorn', '🦄');"#,
|
||||
r#"element[setAttribute]('data-unicorn', '🦄');"#,
|
||||
r#"element.foo('data-unicorn', '🦄');"#,
|
||||
r#"element.setAttribute('data-unicorn', '🦄', 'extra');"#,
|
||||
r#"element.setAttribute('data-unicorn');"#,
|
||||
r#"element.setAttribute(...argumentsArray, ...argumentsArray2)"#,
|
||||
r#"element.setAttribute(`data-unicorn`, '🦄');"#,
|
||||
r#"element.setAttribute(0, '🦄');"#,
|
||||
r#"element.setAttribute('foo-unicorn', '🦄');"#,
|
||||
r#"element.setAttribute('data', '🦄');"#,
|
||||
r#"delete element.dataset.unicorn;"#,
|
||||
r#"delete element.dataset["unicorn"];"#,
|
||||
r#"new element.removeAttribute("data-unicorn");"#,
|
||||
r#"removeAttribute("data-unicorn");"#,
|
||||
r#"element["removeAttribute"]("data-unicorn");"#,
|
||||
r#"element[removeAttribute]("data-unicorn");"#,
|
||||
r#"element.foo("data-unicorn");"#,
|
||||
r#"element.removeAttribute("data-unicorn", "extra");"#,
|
||||
r#"element.removeAttribute();"#,
|
||||
r#"element.removeAttribute(...argumentsArray, ...argumentsArray2)"#,
|
||||
r#"element.removeAttribute(`data-unicorn`);"#,
|
||||
r#"element.removeAttribute(0);"#,
|
||||
r#"element.removeAttribute("foo-unicorn");"#,
|
||||
r#"element.removeAttribute("data");"#,
|
||||
r#""unicorn" in element.dataset"#,
|
||||
r#"element.dataset.hasOwnProperty("unicorn")"#,
|
||||
r#"Object.prototype.hasOwnProperty.call(element.dataset, "unicorn")"#,
|
||||
r#"Object.hasOwn(element.dataset, "unicorn")"#,
|
||||
r#"Reflect.has(element.dataset, "unicorn")"#,
|
||||
r#"new element.hasAttribute("data-unicorn");"#,
|
||||
r#"hasAttribute("data-unicorn");"#,
|
||||
r#"element["hasAttribute"]("data-unicorn");"#,
|
||||
r#"element[hasAttribute]("data-unicorn");"#,
|
||||
r#"element.foo("data-unicorn");"#,
|
||||
r#"element.hasAttribute("data-unicorn", "extra");"#,
|
||||
r#"element.hasAttribute();"#,
|
||||
r#"element.hasAttribute(...argumentsArray, ...argumentsArray2)"#,
|
||||
r#"element.hasAttribute(`data-unicorn`);"#,
|
||||
r#"element.hasAttribute(0);"#,
|
||||
r#"element.hasAttribute("foo-unicorn");"#,
|
||||
r#"element.hasAttribute("data");"#,
|
||||
r#"element.dataset.unicorn"#,
|
||||
r#"new element.getAttribute("data-unicorn");"#,
|
||||
r#"getAttribute("data-unicorn");"#,
|
||||
r#"element["getAttribute"]("data-unicorn");"#,
|
||||
r#"element[getAttribute]("data-unicorn");"#,
|
||||
r#"element.foo("data-unicorn");"#,
|
||||
r#"element.getAttribute("data-unicorn", "extra");"#,
|
||||
r#"element.getAttribute();"#,
|
||||
r#"element.getAttribute(...argumentsArray, ...argumentsArray2)"#,
|
||||
r#"element.getAttribute(`data-unicorn`);"#,
|
||||
r#"element.getAttribute(0);"#,
|
||||
r#"element.getAttribute("foo-unicorn");"#,
|
||||
r#"element.getAttribute("data");"#,
|
||||
];
|
||||
|
||||
let fail = vec![
|
||||
r#"element.setAttribute('data-unicorn', '🦄');"#,
|
||||
r#"element.setAttribute('data-🦄', '🦄');"#,
|
||||
r#"element.setAttribute('data-ゆ', 'ゆ');"#,
|
||||
r#"element.setAttribute('data-foo2', '🦄');"#,
|
||||
r#"element.setAttribute('data-foo:bar', 'zaz');"#,
|
||||
r#"element.setAttribute("data-foo:bar", "zaz");"#,
|
||||
r#"element.setAttribute('data-foo.bar', 'zaz');"#,
|
||||
r#"element.setAttribute('data-foo-bar', 'zaz');"#,
|
||||
r#"element.setAttribute('data-foo', /* comment */ 'bar');"#,
|
||||
r#"element.querySelector('#selector').setAttribute('data-AllowAccess', true);"#,
|
||||
r#"element.setAttribute("data-", "🦄");"#,
|
||||
r#"element.setAttribute("data--foo", "🦄");"#,
|
||||
r#"element.setAttribute("DATA--FOO", "🦄");"#,
|
||||
r#"element.setAttribute("DATA- ", "🦄");"#,
|
||||
r#"element.setAttribute("DATA-Foo-bar", "🦄");"#,
|
||||
r#"optional?.element.setAttribute("data-unicorn", "🦄");"#,
|
||||
r#"console.log(element.setAttribute("data-unicorn", "🦄"))"#,
|
||||
r#"element.removeAttribute('data-unicorn');"#,
|
||||
r#"element.removeAttribute("data-unicorn");"#,
|
||||
r#"element.removeAttribute("data-unicorn",);"#,
|
||||
r#"element.removeAttribute("data-🦄");"#,
|
||||
r#"element.removeAttribute("data-ゆ");"#,
|
||||
r#"element.removeAttribute("data-foo2");"#,
|
||||
r#"element.removeAttribute("data-foo:bar");"#,
|
||||
r#"element.removeAttribute("data-foo:bar");"#,
|
||||
r#"element.removeAttribute("data-foo.bar");"#,
|
||||
r#"element.removeAttribute("data-foo-bar");"#,
|
||||
r#"element.removeAttribute("data-foo");"#,
|
||||
r##"element.querySelector("#selector").removeAttribute("data-AllowAccess");"##,
|
||||
r#"element.removeAttribute("data-");"#,
|
||||
r#"optional?.element.removeAttribute("data-unicorn");"#,
|
||||
r#"element.removeAttribute("data-unicorn")?.property"#,
|
||||
r#"element.hasAttribute('data-unicorn');"#,
|
||||
r#"element.hasAttribute("data-unicorn");"#,
|
||||
r#"element.hasAttribute("data-unicorn",);"#,
|
||||
r#"element.hasAttribute("data-🦄");"#,
|
||||
r#"element.hasAttribute("data-ゆ");"#,
|
||||
r#"element.hasAttribute("data-foo2");"#,
|
||||
r#"element.hasAttribute("data-foo:bar");"#,
|
||||
r#"element.hasAttribute("data-foo:bar");"#,
|
||||
r#"element.hasAttribute("data-foo.bar");"#,
|
||||
r#"element.hasAttribute("data-foo-bar");"#,
|
||||
r#"element.hasAttribute("data-foo");"#,
|
||||
r##"element.querySelector("#selector").hasAttribute("data-AllowAccess");"##,
|
||||
r#"optional?.element.hasAttribute("data-unicorn");"#,
|
||||
r#"element.hasAttribute("data-unicorn").toString()"#,
|
||||
r#"element.getAttribute('data-unicorn');"#,
|
||||
r#"element.getAttribute("data-unicorn");"#,
|
||||
r#"element.getAttribute("data-unicorn",);"#,
|
||||
r#"element.getAttribute("data-🦄");"#,
|
||||
r#"element.getAttribute("data-ゆ");"#,
|
||||
r#"element.getAttribute("data-foo2");"#,
|
||||
r#"element.getAttribute("data-foo:bar");"#,
|
||||
r#"element.getAttribute("data-foo:bar");"#,
|
||||
r#"element.getAttribute("data-foo.bar");"#,
|
||||
r#"element.getAttribute("data-foo-bar");"#,
|
||||
r#"element.getAttribute("data-foo");"#,
|
||||
r##"element.querySelector("#selector").getAttribute("data-AllowAccess");"##,
|
||||
r#"optional?.element.getAttribute("data-unicorn");"#,
|
||||
r#"element.getAttribute("data-unicorn").toString()"#,
|
||||
];
|
||||
|
||||
Tester::new_without_config(PreferDomNodeDataset::NAME, pass, fail).test_and_snapshot();
|
||||
}
|
||||
425
crates/oxc_linter/src/snapshots/prefer_dom_node_dataset.snap
Normal file
425
crates/oxc_linter/src/snapshots/prefer_dom_node_dataset.snap
Normal file
|
|
@ -0,0 +1,425 @@
|
|||
---
|
||||
source: crates/oxc_linter/src/tester.rs
|
||||
expression: prefer_dom_node_dataset
|
||||
---
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.setAttribute('data-unicorn', '🦄');
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.unicorn = ...;`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.setAttribute('data-🦄', '🦄');
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.🦄 = ...;`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.setAttribute('data-ゆ', 'ゆ');
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.ゆ = ...;`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.setAttribute('data-foo2', '🦄');
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.foo2 = ...;`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.setAttribute('data-foo:bar', 'zaz');
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.foo:bar = ...;`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.setAttribute("data-foo:bar", "zaz");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.foo:bar = ...;`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.setAttribute('data-foo.bar', 'zaz');
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.foo.bar = ...;`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.setAttribute('data-foo-bar', 'zaz');
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.foo-bar = ...;`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.setAttribute('data-foo', /* comment */ 'bar');
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.foo = ...;`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.querySelector('#selector').setAttribute('data-AllowAccess', true);
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.AllowAccess = ...;`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.setAttribute("data-", "🦄");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset. = ...;`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.setAttribute("data--foo", "🦄");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.-foo = ...;`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.setAttribute("DATA--FOO", "🦄");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.-FOO = ...;`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.setAttribute("DATA- ", "🦄");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset. = ...;`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.setAttribute("DATA-Foo-bar", "🦄");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.Foo-bar = ...;`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ optional?.element.setAttribute("data-unicorn", "🦄");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.unicorn = ...;`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `setAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ console.log(element.setAttribute("data-unicorn", "🦄"))
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.unicorn = ...;`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `removeAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.removeAttribute('data-unicorn');
|
||||
· ──────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `delete element.dataset.unicorn;
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `removeAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.removeAttribute("data-unicorn");
|
||||
· ──────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `delete element.dataset.unicorn;
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `removeAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.removeAttribute("data-unicorn",);
|
||||
· ──────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `delete element.dataset.unicorn;
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `removeAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.removeAttribute("data-🦄");
|
||||
· ─────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `delete element.dataset.🦄;
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `removeAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.removeAttribute("data-ゆ");
|
||||
· ─────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `delete element.dataset.ゆ;
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `removeAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.removeAttribute("data-foo2");
|
||||
· ───────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `delete element.dataset.foo2;
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `removeAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.removeAttribute("data-foo:bar");
|
||||
· ──────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `delete element.dataset.foo:bar;
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `removeAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.removeAttribute("data-foo:bar");
|
||||
· ──────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `delete element.dataset.foo:bar;
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `removeAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.removeAttribute("data-foo.bar");
|
||||
· ──────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `delete element.dataset.foo.bar;
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `removeAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.removeAttribute("data-foo-bar");
|
||||
· ──────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `delete element.dataset.foo-bar;
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `removeAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.removeAttribute("data-foo");
|
||||
· ──────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `delete element.dataset.foo;
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `removeAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.querySelector("#selector").removeAttribute("data-AllowAccess");
|
||||
· ──────────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `delete element.dataset.AllowAccess;
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `removeAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.removeAttribute("data-");
|
||||
· ───────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `delete element.dataset.;
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `removeAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ optional?.element.removeAttribute("data-unicorn");
|
||||
· ──────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `delete element.dataset.unicorn;
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `removeAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.removeAttribute("data-unicorn")?.property
|
||||
· ──────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `delete element.dataset.unicorn;
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `hasAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.hasAttribute('data-unicorn');
|
||||
· ────────────
|
||||
╰────
|
||||
help: Check the `dataset` object directly: `Object.hasOwn(element.dataset, 'unicorn')
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `hasAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.hasAttribute("data-unicorn");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Check the `dataset` object directly: `Object.hasOwn(element.dataset, 'unicorn')
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `hasAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.hasAttribute("data-unicorn",);
|
||||
· ────────────
|
||||
╰────
|
||||
help: Check the `dataset` object directly: `Object.hasOwn(element.dataset, 'unicorn')
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `hasAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.hasAttribute("data-🦄");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Check the `dataset` object directly: `Object.hasOwn(element.dataset, '🦄')
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `hasAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.hasAttribute("data-ゆ");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Check the `dataset` object directly: `Object.hasOwn(element.dataset, 'ゆ')
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `hasAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.hasAttribute("data-foo2");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Check the `dataset` object directly: `Object.hasOwn(element.dataset, 'foo2')
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `hasAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.hasAttribute("data-foo:bar");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Check the `dataset` object directly: `Object.hasOwn(element.dataset, 'foo:bar')
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `hasAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.hasAttribute("data-foo:bar");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Check the `dataset` object directly: `Object.hasOwn(element.dataset, 'foo:bar')
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `hasAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.hasAttribute("data-foo.bar");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Check the `dataset` object directly: `Object.hasOwn(element.dataset, 'foo.bar')
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `hasAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.hasAttribute("data-foo-bar");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Check the `dataset` object directly: `Object.hasOwn(element.dataset, 'foo-bar')
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `hasAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.hasAttribute("data-foo");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Check the `dataset` object directly: `Object.hasOwn(element.dataset, 'foo')
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `hasAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.querySelector("#selector").hasAttribute("data-AllowAccess");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Check the `dataset` object directly: `Object.hasOwn(element.dataset, 'AllowAccess')
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `hasAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ optional?.element.hasAttribute("data-unicorn");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Check the `dataset` object directly: `Object.hasOwn(element.dataset, 'unicorn')
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `hasAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.hasAttribute("data-unicorn").toString()
|
||||
· ────────────
|
||||
╰────
|
||||
help: Check the `dataset` object directly: `Object.hasOwn(element.dataset, 'unicorn')
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `getAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.getAttribute('data-unicorn');
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.unicorn`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `getAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.getAttribute("data-unicorn");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.unicorn`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `getAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.getAttribute("data-unicorn",);
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.unicorn`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `getAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.getAttribute("data-🦄");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.🦄`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `getAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.getAttribute("data-ゆ");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.ゆ`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `getAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.getAttribute("data-foo2");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.foo2`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `getAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.getAttribute("data-foo:bar");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.foo:bar`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `getAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.getAttribute("data-foo:bar");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.foo:bar`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `getAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.getAttribute("data-foo.bar");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.foo.bar`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `getAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.getAttribute("data-foo-bar");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.foo-bar`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `getAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.getAttribute("data-foo");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.foo`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `getAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.querySelector("#selector").getAttribute("data-AllowAccess");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.AllowAccess`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `getAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ optional?.element.getAttribute("data-unicorn");
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.unicorn`
|
||||
|
||||
⚠ eslint-plugin-unicorn(prefer-dom-node-dataset): Prefer using `dataset` over `getAttribute`.
|
||||
╭─[prefer_dom_node_dataset.tsx:1:1]
|
||||
1 │ element.getAttribute("data-unicorn").toString()
|
||||
· ────────────
|
||||
╰────
|
||||
help: Access the `.dataset` object directly: `element.dataset.unicorn`
|
||||
|
||||
|
||||
Loading…
Reference in a new issue