Correct handler for @AllArgsConstructorHandler

The `onConstructor` parameter is specified as `onConstructor_` on Java 1.8+ (as in the case of `J.Modifier` for example).
openrewrite · Nov 7, 2024 · 6b7d843 · 6b7d843
1 parent 1007940
commit 6b7d843
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/rewrite-java-lombok/src/main/java/org/openrewrite/java/lombok/AllArgsConstructorHandler.java b/rewrite-java-lombok/src/main/java/org/openrewrite/java/lombok/AllArgsConstructorHandler.java
@@ -38,7 +38,9 @@ public void handle(AnnotationValues<AllArgsConstructor> annotationValues, JCTree
                 if (originalArg instanceof JCTree.JCAssign && ((JCTree.JCAssign) originalArg).getVariable() instanceof JCTree.JCIdent) {
                     JCTree.JCAssign assign = (JCTree.JCAssign) originalArg;
                     JCTree.JCIdent ident = (JCTree.JCIdent) assign.getVariable();
-                    if ("onConstructor".equals(ident.getName().toString())) {
+                    String name = ident.getName().toString();
+                    if (name.equals("onConstructor") || name.equals("onConstructor_")) {
+                        // In Java 1.8+ the parameter is `onConstructor_`
                         continue;
                     }
                 }