]>
Commit | Line | Data |
---|---|---|
eb39fafa DC |
1 | /** |
2 | * @fileoverview A rule to disallow calls to the Object constructor | |
3 | * @author Matt DuVall <http://www.mattduvall.com/> | |
4 | */ | |
5 | ||
6 | "use strict"; | |
7 | ||
56c4a2cb DC |
8 | //------------------------------------------------------------------------------ |
9 | // Requirements | |
10 | //------------------------------------------------------------------------------ | |
11 | ||
12 | const astUtils = require("./utils/ast-utils"); | |
13 | ||
eb39fafa DC |
14 | //------------------------------------------------------------------------------ |
15 | // Rule Definition | |
16 | //------------------------------------------------------------------------------ | |
17 | ||
34eeec05 | 18 | /** @type {import('../shared/types').Rule} */ |
eb39fafa DC |
19 | module.exports = { |
20 | meta: { | |
21 | type: "suggestion", | |
22 | ||
23 | docs: { | |
24 | description: "disallow `Object` constructors", | |
eb39fafa DC |
25 | recommended: false, |
26 | url: "https://eslint.org/docs/rules/no-new-object" | |
27 | }, | |
28 | ||
29 | schema: [], | |
30 | ||
31 | messages: { | |
32 | preferLiteral: "The object literal notation {} is preferrable." | |
33 | } | |
34 | }, | |
35 | ||
36 | create(context) { | |
eb39fafa | 37 | return { |
eb39fafa | 38 | NewExpression(node) { |
56c4a2cb DC |
39 | const variable = astUtils.getVariableByName( |
40 | context.getScope(), | |
41 | node.callee.name | |
42 | ); | |
43 | ||
44 | if (variable && variable.identifiers.length > 0) { | |
45 | return; | |
46 | } | |
47 | ||
eb39fafa DC |
48 | if (node.callee.name === "Object") { |
49 | context.report({ | |
50 | node, | |
51 | messageId: "preferLiteral" | |
52 | }); | |
53 | } | |
54 | } | |
55 | }; | |
eb39fafa DC |
56 | } |
57 | }; |