glayzzle#185 - fix get_class($var)::$$$$$property;

Author: ichiriac

src/parser/variable.js

@@ -151,7 +151,7 @@ module.exports = {
         what = what(name, false, false);
         break;
       case "$":
-        this.next().expect(["{", this.tok.T_VARIABLE]);
+        this.next().expect(["$", "{", this.tok.T_VARIABLE]);
         if (this.token === "{") {
           // $obj->${$varname}
           what = this.next().read_expr();

test/debug.js

@@ -17,7 +17,7 @@
 const util = require('util');
 const parser = require("../src/index");
 const ast = parser.parseEval(`
-Foo::$bar['baz']();
+get_class($var)::$$$$$property;
 `, { 
     parser: {
       debug: true

test/snapshot/__snapshots__/variable.test.js.snap

@@ -951,3 +951,55 @@ Program {
   "kind": "program",
 }
 `;
+
+exports[`Test variables valid offset lookup 1`] = `
+Program {
+  "children": Array [
+    ExpressionStatement {
+      "expression": StaticLookup {
+        "kind": "staticlookup",
+        "offset": Variable {
+          "byref": false,
+          "curly": false,
+          "kind": "variable",
+          "name": Variable {
+            "byref": false,
+            "curly": false,
+            "kind": "variable",
+            "name": Variable {
+              "byref": false,
+              "curly": false,
+              "kind": "variable",
+              "name": Variable {
+                "byref": false,
+                "curly": false,
+                "kind": "variable",
+                "name": "property",
+              },
+            },
+          },
+        },
+        "what": Call {
+          "arguments": Array [
+            Variable {
+              "byref": false,
+              "curly": false,
+              "kind": "variable",
+              "name": "var",
+            },
+          ],
+          "kind": "call",
+          "what": ClassReference {
+            "kind": "classreference",
+            "name": "get_class",
+            "resolution": "uqn",
+          },
+        },
+      },
+      "kind": "expressionstatement",
+    },
+  ],
+  "errors": Array [],
+  "kind": "program",
+}
+`;

test/snapshot/variable.test.js

@@ -21,6 +21,11 @@ describe("Test variables", function() {
     expect(parser.parseEval("$var = Foo::{$bar['baz']}();Foo::$bar['baz']();")).toMatchSnapshot();
   });
 
+  it("valid offset lookup", function() {
+    expect(parser.parseEval("get_class($var)::$$$$$property;")).toMatchSnapshot();
+  });
+
+
   it("Class constants", function() {
     expect(parser.parseEval(`
       static::foo();