Skip to content

Commit 0e31cad

Browse files
author
Esben Sparre Andreasen
committed
JS: simplify this.getStringValue() to getStringValue()
1 parent ee106cc commit 0e31cad

File tree

5 files changed

+6
-6
lines changed

5 files changed

+6
-6
lines changed

javascript/ql/src/semmle/javascript/AMD.qll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,7 @@ private class AmdDependencyPath extends PathExprCandidate {
189189
private class ConstantAmdDependencyPathElement extends PathExprInModule, ConstantString {
190190
ConstantAmdDependencyPathElement() { this = any(AmdDependencyPath amd).getAPart() }
191191

192-
override string getValue() { result = this.getStringValue() }
192+
override string getValue() { result = getStringValue() }
193193
}
194194

195195
/**

javascript/ql/src/semmle/javascript/ES2015Modules.qll

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ class ImportDeclaration extends Stmt, Import, @importdeclaration {
8282
private class LiteralImportPath extends PathExprInModule, ConstantString {
8383
LiteralImportPath() { exists(ImportDeclaration req | this = req.getChildExpr(-1)) }
8484

85-
override string getValue() { result = this.getStringValue() }
85+
override string getValue() { result = getStringValue() }
8686
}
8787

8888
/**
@@ -596,7 +596,7 @@ abstract class ReExportDeclaration extends ExportDeclaration {
596596
private class LiteralReExportPath extends PathExprInModule, ConstantString {
597597
LiteralReExportPath() { exists(ReExportDeclaration bred | this = bred.getImportedPath()) }
598598

599-
override string getValue() { result = this.getStringValue() }
599+
override string getValue() { result = getStringValue() }
600600
}
601601

602602
/**

javascript/ql/src/semmle/javascript/Expr.qll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2600,7 +2600,7 @@ private class LiteralDynamicImportPath extends PathExprInModule, ConstantString
26002600
exists(DynamicImportExpr di | this.getParentExpr*() = di.getSource())
26012601
}
26022602

2603-
override string getValue() { result = this.getStringValue() }
2603+
override string getValue() { result = getStringValue() }
26042604
}
26052605

26062606
/**

javascript/ql/src/semmle/javascript/NodeJS.qll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -269,7 +269,7 @@ private class RequirePath extends PathExprCandidate {
269269
private class ConstantRequirePathElement extends PathExprInModule, ConstantString {
270270
ConstantRequirePathElement() { this = any(RequirePath rp).getAPart() }
271271

272-
override string getValue() { result = this.getStringValue() }
272+
override string getValue() { result = getStringValue() }
273273
}
274274

275275
/** A `__dirname` path expression. */

javascript/ql/src/semmle/javascript/TypeScript.qll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -222,7 +222,7 @@ private class LiteralExternalModulePath extends PathExprInModule, ConstantString
222222
exists(ExternalModuleReference emr | this.getParentExpr*() = emr.getExpression())
223223
}
224224

225-
override string getValue() { result = this.getStringValue() }
225+
override string getValue() { result = getStringValue() }
226226
}
227227

228228
/** A TypeScript "export-assign" declaration. */

0 commit comments

Comments
 (0)