From 58a40ef8018eade7bab2419177491e1d96d95472 Mon Sep 17 00:00:00 2001 From: lingenj Date: Thu, 6 Feb 2025 12:07:30 +0100 Subject: [PATCH] Disable parentheses level discovery for method calls for groovy 2 --- .../main/java/org/openrewrite/groovy/GroovyParserVisitor.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/rewrite-groovy/src/main/java/org/openrewrite/groovy/GroovyParserVisitor.java b/rewrite-groovy/src/main/java/org/openrewrite/groovy/GroovyParserVisitor.java index 58594c4b8c2..8ce2c897870 100644 --- a/rewrite-groovy/src/main/java/org/openrewrite/groovy/GroovyParserVisitor.java +++ b/rewrite-groovy/src/main/java/org/openrewrite/groovy/GroovyParserVisitor.java @@ -2458,7 +2458,8 @@ private String sourceSubstring(int beginIndex, String untilDelim) { } else if (rawIpl instanceof Integer) { // On Java 8 _INSIDE_PARENTHESES_LEVEL is a regular Integer return (Integer) rawIpl; - } else if (node instanceof MethodCallExpression) { + } else if (node instanceof MethodCallExpression && !isOlderThanGroovy3()) { + // Only for groovy 3+, because lower versions do always return `-1` for objectExpression.lineNumber / objectExpression.columnNumber MethodCallExpression expr = (MethodCallExpression) node; return determineParenthesisLevel(expr.getObjectExpression().getLineNumber(), expr.getLineNumber(), expr.getObjectExpression().getColumnNumber(), expr.getColumnNumber()); } else if (node instanceof BinaryExpression) {