Fix: broken method lookup on closure (#1678)

Author: Vovan-VE

src/needs-parens.js

@@ -205,7 +205,17 @@ function needsParens(path) {
           return false;
       }
     case "closure":
-      return parent.kind === "call" && name === "what" && parent.what === node;
+      switch (parent.kind) {
+        case "call":
+          return name === "what" && parent.what === node;
+
+        // https://github.com/prettier/plugin-php/issues/1675
+        case "propertylookup":
+          return true;
+
+        default:
+          return false;
+      }
     case "silence":
     case "cast":
       // TODO: bug https://github.com/glayzzle/php-parser/issues/172

tests/parens/__snapshots__/jsfmt.spec.js.snap

@@ -2339,6 +2339,13 @@ $var = (($var)->bar);
 $var = ($var)->bar();
 $var = (($var)->bar());
 
+$var = (function () {
+    return $this->foo;
+})->bindTo($var, A::class)();
+$var = (((function () {
+    return $this->foo;
+})))->bindTo($var, A::class)();
+
 =====================================output=====================================
 <?php
 
@@ -2424,6 +2431,13 @@ $var = $var->bar;
 $var = $var->bar();
 $var = $var->bar();
 
+$var = (function () {
+    return $this->foo;
+})->bindTo($var, A::class)();
+$var = (function () {
+    return $this->foo;
+})->bindTo($var, A::class)();
+
 ================================================================================
 `;
 

tests/parens/lookups.php

@@ -81,3 +81,10 @@
 $var = (($var)->bar);
 $var = ($var)->bar();
 $var = (($var)->bar());
+
+$var = (function () {
+    return $this->foo;
+})->bindTo($var, A::class)();
+$var = (((function () {
+    return $this->foo;
+})))->bindTo($var, A::class)();