Parser error on multiline javadoc tag (#5263)

* Test case for parser error

* Capture any whitespace (including newlines) between the dot and the name

* Replicate the same change to the other JavaDoc parsers

---------

Co-authored-by: Tim te Beek <[email protected]>

Author: rlsanders4

rewrite-java-11/src/main/java/org/openrewrite/java/isolated/ReloadableJava11JavadocVisitor.java

@@ -1169,11 +1169,13 @@ public J visitMemberSelect(MemberSelectTree node, Space fmt) {
             JCTree.JCFieldAccess fieldAccess = (JCTree.JCFieldAccess) node;
             Expression selected = (Expression) scan(fieldAccess.selected, Space.EMPTY);
             sourceBefore(".");
+            // Capture any whitespace (including newlines) between the dot and the name
+            String whitespaceAfterDot = whitespaceBeforeAsString();
             cursor += fieldAccess.name.toString().length();
             return new J.FieldAccess(randomId(), fmt, Markers.EMPTY,
                     selected,
                     JLeftPadded.build(new J.Identifier(randomId(),
-                            Space.EMPTY,
+                            Space.build(whitespaceAfterDot, emptyList()),
                             Markers.EMPTY,
                             emptyList(),
                             fieldAccess.name.toString(), null, null)),

rewrite-java-17/src/main/java/org/openrewrite/java/isolated/ReloadableJava17JavadocVisitor.java

@@ -1173,11 +1173,13 @@ public J visitMemberSelect(MemberSelectTree node, Space fmt) {
             JCTree.JCFieldAccess fieldAccess = (JCTree.JCFieldAccess) node;
             Expression selected = (Expression) scan(fieldAccess.selected, Space.EMPTY);
             sourceBefore(".");
+            // Capture any whitespace (including newlines) between the dot and the name
+            String whitespaceAfterDot = whitespaceBeforeAsString();
             cursor += fieldAccess.name.toString().length();
             return new J.FieldAccess(randomId(), fmt, Markers.EMPTY,
                     selected,
                     JLeftPadded.build(new J.Identifier(randomId(),
-                            Space.EMPTY,
+                            Space.build(whitespaceAfterDot, emptyList()),
                             Markers.EMPTY,
                             emptyList(),
                             fieldAccess.name.toString(), null, null)),

rewrite-java-21/src/main/java/org/openrewrite/java/isolated/ReloadableJava21JavadocVisitor.java

@@ -1208,11 +1208,13 @@ public J visitMemberSelect(MemberSelectTree node, Space fmt) {
             JCTree.JCFieldAccess fieldAccess = (JCTree.JCFieldAccess) node;
             Expression selected = (Expression) scan(fieldAccess.selected, Space.EMPTY);
             sourceBefore(".");
+            // Capture any whitespace (including newlines) between the dot and the name
+            String whitespaceAfterDot = whitespaceBeforeAsString();
             cursor += fieldAccess.name.toString().length();
             return new J.FieldAccess(randomId(), fmt, Markers.EMPTY,
                     selected,
                     JLeftPadded.build(new J.Identifier(randomId(),
-                            Space.EMPTY,
+                            Space.build(whitespaceAfterDot, emptyList()),
                             Markers.EMPTY,
                             emptyList(),
                             fieldAccess.name.toString(), null, null)),

rewrite-java-8/src/main/java/org/openrewrite/java/ReloadableJava8JavadocVisitor.java

@@ -1097,11 +1097,13 @@ public J visitMemberSelect(MemberSelectTree node, Space fmt) {
             JCTree.JCFieldAccess fieldAccess = (JCTree.JCFieldAccess) node;
             Expression selected = (Expression) scan(fieldAccess.selected, Space.EMPTY);
             sourceBefore(".");
+            // Capture any whitespace (including newlines) between the dot and the name
+            String whitespaceAfterDot = whitespaceBeforeAsString();
             cursor += fieldAccess.name.toString().length();
             return new J.FieldAccess(randomId(), fmt, Markers.EMPTY,
                     selected,
                     JLeftPadded.build(new J.Identifier(randomId(),
-                            Space.EMPTY,
+                            Space.build(whitespaceAfterDot, emptyList()),
                             Markers.EMPTY,
                             emptyList(), fieldAccess.name.toString(), null, null)),
                     typeMapping.type(node));

rewrite-java-test/src/test/java/org/openrewrite/java/JavaParserTest.java

@@ -391,7 +391,29 @@ public InstallationStatus create(Instant updateTime) {
               }
             }
             """
-          ));
+          )
+        );
+    }
+
+    @Test
+    void multiLinePackageComment() {
+        rewriteRun(
+          // language=java
+          java(
+            """
+              package com.abc;
+              public class Server {
+                /**
+                 * @see com.abc.Server#load(
+                 * java.lang.
+                 * String)
+                 */
+                public void load(String str) {
+                }
+              }
+              """
+          )
+        );
     }
 
     @Issue("https://github.com/openrewrite/rewrite/issues/5445")
@@ -435,5 +457,4 @@ private String mapFoo(final int i) {
           )
         );
     }
-
 }