[Refactor] Enhance newline handling for fluent calls in BetterStandardPrinter

MrPunyapal · MrPunyapal · commit bc07f76f79d5 · 2026-02-23T18:30:30.000+05:30
diff --git a/src/PhpParser/Printer/BetterStandardPrinter.php b/src/PhpParser/Printer/BetterStandardPrinter.php
@@ -375,7 +375,10 @@ protected function pExpr_MethodCall(MethodCall $methodCall): string
             return parent::pExpr_MethodCall($methodCall);
         }
 
-        if (SimpleParameterProvider::provideBoolParameter(Option::NEW_LINE_ON_FLUENT_CALL) === false) {
+        $globalFlag = SimpleParameterProvider::provideBoolParameter(Option::NEW_LINE_ON_FLUENT_CALL);
+        $perNodeFlag = (bool) $methodCall->getAttribute(AttributeKey::NEWLINE_ON_FLUENT_CALL, false);
+
+        if ($globalFlag === false && $perNodeFlag === false) {
             return parent::pExpr_MethodCall($methodCall);
         }
 

Original file line number	Diff line number	Diff line change
`@@ -375,7 +375,10 @@ protected function pExpr_MethodCall(MethodCall $methodCall): string`
`375`	`375`	`return parent::pExpr_MethodCall($methodCall);`
`376`	`376`	`}`
`377`	`377`
`378`		`- if (SimpleParameterProvider::provideBoolParameter(Option::NEW_LINE_ON_FLUENT_CALL) === false) {`
	`378`	`+ $globalFlag = SimpleParameterProvider::provideBoolParameter(Option::NEW_LINE_ON_FLUENT_CALL);`
	`379`	`+ $perNodeFlag = (bool) $methodCall->getAttribute(AttributeKey::NEWLINE_ON_FLUENT_CALL, false);`
	`380`	`+`
	`381`	`+ if ($globalFlag === false && $perNodeFlag === false) {`
`379`	`382`	`return parent::pExpr_MethodCall($methodCall);`
`380`	`383`	`}`
`381`	`384`