diff --git a/ets2panda/linter/src/lib/TypeScriptLinter.ts b/ets2panda/linter/src/lib/TypeScriptLinter.ts index 0b7067a939d4e9b2293ad9df1c856b5943891888..14e8960641438345541c1f56ed45b2941cdf796c 100644 --- a/ets2panda/linter/src/lib/TypeScriptLinter.ts +++ b/ets2panda/linter/src/lib/TypeScriptLinter.ts @@ -7542,7 +7542,7 @@ export class TypeScriptLinter extends BaseTypeScriptLinter { } const text = stringLiteral.text; - const autofix = this.autofixer?.removeNode(stringLiteral); + const autofix = this.autofixer?.removeNode(stringLiteral.parent); if (text === USE_CONCURRENT) { this.incrementCounters(stringLiteral, FaultID.UseConcurrentDeprecated, autofix); diff --git a/ets2panda/linter/test/concurrent/use_limited_and_concurrent.ets b/ets2panda/linter/test/concurrent/use_limited_and_concurrent.ets index 22ef20b645ce008ca5ad7b59f5a35eed82cac0c0..6e6dfcec0288600fab3ffb50536b034c7b3ccfb3 100644 --- a/ets2panda/linter/test/concurrent/use_limited_and_concurrent.ets +++ b/ets2panda/linter/test/concurrent/use_limited_and_concurrent.ets @@ -39,3 +39,16 @@ switch (variable2) { case 'use shared': break; } + +function refreshToGetFontScale(): number { + 'use concurrent'; + const TAG = 'SCBScreenLock-refreshToGetFontScale'; + const log: LogHelper = LogHelper.getLogHelper(LogDomain.KG, TAG); + const FONT_SCALE_DEFAULT: number = 1; + try { + return uiAppearance.getFontScale(); + } catch (err) { + log.showError(`uiAppearance getFontScale err,code:${err?.code}`); + } + return FONT_SCALE_DEFAULT; +} diff --git a/ets2panda/linter/test/concurrent/use_limited_and_concurrent.ets.arkts2.json b/ets2panda/linter/test/concurrent/use_limited_and_concurrent.ets.arkts2.json index 0a82914a53a6b965b052de59bd9d4bfe1d740554..e660719da97ef9de3000d237e5348d2e8507ae5a 100644 --- a/ets2panda/linter/test/concurrent/use_limited_and_concurrent.ets.arkts2.json +++ b/ets2panda/linter/test/concurrent/use_limited_and_concurrent.ets.arkts2.json @@ -43,6 +43,26 @@ "suggest": "", "rule": "\"use concurrent\" is not supported (arkts-limited-stdlib-no-use-concurrent)", "severity": "ERROR" + }, + { + "line": 44, + "column": 3, + "endLine": 44, + "endColumn": 19, + "problem": "UseConcurrentDeprecated", + "suggest": "", + "rule": "\"use concurrent\" is not supported (arkts-limited-stdlib-no-use-concurrent)", + "severity": "ERROR" + }, + { + "line": 47, + "column": 38, + "endLine": 47, + "endColumn": 39, + "problem": "NumericSemantics", + "suggest": "", + "rule": "Numeric semantics is different for integer values (arkts-numeric-semantic)", + "severity": "ERROR" } ] -} +} \ No newline at end of file diff --git a/ets2panda/linter/test/concurrent/use_limited_and_concurrent.ets.autofix.json b/ets2panda/linter/test/concurrent/use_limited_and_concurrent.ets.autofix.json index 052fc5f382c68d4ab6c6584c022fc23f840cb135..de41dd654130b11016964570784d8cfc8d81e875 100644 --- a/ets2panda/linter/test/concurrent/use_limited_and_concurrent.ets.autofix.json +++ b/ets2panda/linter/test/concurrent/use_limited_and_concurrent.ets.autofix.json @@ -24,7 +24,11 @@ { "start": 605, "end": 617, - "replacementText": "" + "replacementText": "", + "line": 16, + "column": 1, + "endLine": 16, + "endColumn": 13 } ], "suggest": "", @@ -51,12 +55,58 @@ { "start": 687, "end": 703, - "replacementText": "" + "replacementText": "", + "line": 20, + "column": 5, + "endLine": 20, + "endColumn": 21 } ], "suggest": "", "rule": "\"use concurrent\" is not supported (arkts-limited-stdlib-no-use-concurrent)", "severity": "ERROR" + }, + { + "line": 44, + "column": 3, + "endLine": 44, + "endColumn": 19, + "problem": "UseConcurrentDeprecated", + "autofix": [ + { + "start": 1019, + "end": 1036, + "replacementText": "", + "line": 44, + "column": 3, + "endLine": 44, + "endColumn": 19 + } + ], + "suggest": "", + "rule": "\"use concurrent\" is not supported (arkts-limited-stdlib-no-use-concurrent)", + "severity": "ERROR" + }, + { + "line": 47, + "column": 38, + "endLine": 47, + "endColumn": 39, + "problem": "NumericSemantics", + "autofix": [ + { + "start": 1195, + "end": 1196, + "replacementText": "1.0", + "line": 47, + "column": 38, + "endLine": 47, + "endColumn": 39 + } + ], + "suggest": "", + "rule": "Numeric semantics is different for integer values (arkts-numeric-semantic)", + "severity": "ERROR" } ] -} +} \ No newline at end of file diff --git a/ets2panda/linter/test/concurrent/use_limited_and_concurrent.ets.migrate.ets b/ets2panda/linter/test/concurrent/use_limited_and_concurrent.ets.migrate.ets index 5224e3b79f6ce4d369dcfc9ebaa26a63bc1b9219..315ac7533815f4de74ea5396bc59ec687c14d96c 100644 --- a/ets2panda/linter/test/concurrent/use_limited_and_concurrent.ets.migrate.ets +++ b/ets2panda/linter/test/concurrent/use_limited_and_concurrent.ets.migrate.ets @@ -39,3 +39,16 @@ switch (variable2) { case 'use shared': break; } + +function refreshToGetFontScale(): number { + + const TAG = 'SCBScreenLock-refreshToGetFontScale'; + const log: LogHelper = LogHelper.getLogHelper(LogDomain.KG, TAG); + const FONT_SCALE_DEFAULT: number = 1.0; + try { + return uiAppearance.getFontScale(); + } catch (err) { + log.showError(`uiAppearance getFontScale err,code:${err?.code}`); + } + return FONT_SCALE_DEFAULT; +}