JS: Stop dependeding on getPath() for toString()

asgerf · asgerf · commit a32614939627 · 2025-10-31T09:52:40.000+01:00
diff --git a/javascript/ql/lib/semmle/javascript/ApiGraphs.qll b/javascript/ql/lib/semmle/javascript/ApiGraphs.qll
@@ -613,12 +613,12 @@ module API {
 
   /** A node corresponding to a definition of an API component. */
   class Definition extends Node, Impl::TDef {
-    override string toString() { result = "def " + this.getPath() }
+    override string toString() { result = "def " + this.getInducingNode().toString() }
   }
 
   /** A node corresponding to the use of an API component. */
   class Use extends Node, Impl::TUse {
-    override string toString() { result = "use " + this.getPath() }
+    override string toString() { result = "use " + this.getInducingNode().toString() }
   }
 
   /** Gets the root node. */

Original file line number	Diff line number	Diff line change
`@@ -613,12 +613,12 @@ module API {`
`613`	`613`
`614`	`614`	`/** A node corresponding to a definition of an API component. */`
`615`	`615`	`class Definition extends Node, Impl::TDef {`
`616`		`- override string toString() { result = "def " + this.getPath() }`
	`616`	`+ override string toString() { result = "def " + this.getInducingNode().toString() }`
`617`	`617`	`}`
`618`	`618`
`619`	`619`	`/** A node corresponding to the use of an API component. */`
`620`	`620`	`class Use extends Node, Impl::TUse {`
`621`		`- override string toString() { result = "use " + this.getPath() }`
	`621`	`+ override string toString() { result = "use " + this.getInducingNode().toString() }`
`622`	`622`	`}`
`623`	`623`
`624`	`624`	`/** Gets the root node. */`