Remove use of deprecated methods (#2173)

sindresorhus · Jul 11, 2023 · 980d98b · 980d98b
1 parent b2d7378
commit 980d98b
Show file tree

Hide file tree

Showing 9 changed files with 13 additions and 12 deletions.
diff --git a/rules/filename-case.js b/rules/filename-case.js
@@ -144,7 +144,7 @@ const create = context => {
 		return new RegExp(item, 'u');
 	});
 	const chosenCasesFunctions = chosenCases.map(case_ => ignoreNumbers(cases[case_].fn));
-	const filenameWithExtension = context.getPhysicalFilename();
+	const filenameWithExtension = context.physicalFilename;
 
 	if (filenameWithExtension === '<input>' || filenameWithExtension === '<text>') {
 		return;

diff --git a/rules/no-empty-file.js b/rules/no-empty-file.js
@@ -17,7 +17,7 @@ const hasTripeSlashDirectives = comments =>
 
 /** @param {import('eslint').Rule.RuleContext} context */
 const create = context => {
-	const filename = context.getPhysicalFilename();
+	const filename = context.physicalFilename;
 
 	if (!/\.(?:js|mjs|cjs|jsx|ts|mts|cts|tsx)$/i.test(filename)) {
 		return;

diff --git a/rules/no-instanceof-array.js b/rules/no-instanceof-array.js
@@ -27,8 +27,8 @@ const create = context => {
 			const {left, right} = node;
 			let tokenStore = sourceCode;
 			let instanceofToken = tokenStore.getTokenAfter(left, isInstanceofToken);
-			if (!instanceofToken && context.parserServices.getTemplateBodyTokenStore) {
-				tokenStore = context.parserServices.getTemplateBodyTokenStore();
+			if (!instanceofToken && sourceCode.parserServices.getTemplateBodyTokenStore) {
+				tokenStore = sourceCode.parserServices.getTemplateBodyTokenStore();
 				instanceofToken = tokenStore.getTokenAfter(left, isInstanceofToken);
 			}
 

diff --git a/rules/no-useless-undefined.js b/rules/no-useless-undefined.js
@@ -81,7 +81,7 @@ const isFunctionBindCall = node =>
 	&& node.callee.property.name === 'bind';
 
 const isTypeScriptFile = context =>
-	/\.(?:ts|mts|cts|tsx)$/i.test(context.getPhysicalFilename());
+	/\.(?:ts|mts|cts|tsx)$/i.test(context.physicalFilename);
 
 /** @param {import('eslint').Rule.RuleContext} context */
 const create = context => {

diff --git a/rules/prefer-module.js b/rules/prefer-module.js
@@ -215,7 +215,7 @@ function fixModuleExports(node, sourceCode) {
 }
 
 function create(context) {
-	const filename = context.getFilename().toLowerCase();
+	const filename = context.filename.toLowerCase();
 
 	if (filename.endsWith('.cjs')) {
 		return;

diff --git a/rules/prefer-top-level-await.js b/rules/prefer-top-level-await.js
@@ -61,7 +61,7 @@ const isInPromiseMethods = node =>
 
 /** @param {import('eslint').Rule.RuleContext} context */
 function create(context) {
-	if (context.getFilename().toLowerCase().endsWith('.cjs')) {
+	if (context.filename.toLowerCase().endsWith('.cjs')) {
 		return;
 	}
 

diff --git a/rules/prevent-abbreviations.js b/rules/prevent-abbreviations.js
@@ -338,7 +338,7 @@ const isInternalImport = node => {
 /** @param {import('eslint').Rule.RuleContext} context */
 const create = context => {
 	const options = prepareOptions(context.options[0]);
-	const filenameWithExtension = context.getPhysicalFilename();
+	const filenameWithExtension = context.physicalFilename;
 
 	// A `class` declaration produces two variables in two scopes:
 	// the inner class scope, and the outer one (wherever the class is declared).

diff --git a/rules/utils/rule.js b/rules/utils/rule.js
@@ -137,12 +137,13 @@ function checkVueTemplate(create, options) {
 
 	const wrapped = context => {
 		const listeners = create(context);
+		const {parserServices} = context.sourceCode;
 
 		// `vue-eslint-parser`
-		if (context.parserServices?.defineTemplateBodyVisitor) {
+		if (parserServices?.defineTemplateBodyVisitor) {
 			return visitScriptBlock
-				? context.parserServices.defineTemplateBodyVisitor(listeners, listeners)
-				: context.parserServices.defineTemplateBodyVisitor(listeners);
+				? parserServices.defineTemplateBodyVisitor(listeners, listeners)
+				: parserServices.defineTemplateBodyVisitor(listeners);
 		}
 
 		return listeners;

diff --git a/scripts/internal-rules/index.js b/scripts/internal-rules/index.js
@@ -26,7 +26,7 @@ module.exports = {
 				{
 					...rule,
 					create(context) {
-						const filename = context.getPhysicalFilename();
+						const filename = context.physicalFilename;
 						if (directories.every(directory => !isFileInsideDirectory(filename, directory))) {
 							return {};
 						}