Skip to content
This repository was archived by the owner on May 12, 2025. It is now read-only.

Commit 031a6de

Browse files
Fix JavaScriptVisitor bug (#217)
* Fix `JavaScriptVisitor` bug The `JS.ArrowFunction#body` property was visited twice. * Polish
1 parent 77ae582 commit 031a6de

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

rewrite-javascript/src/main/java/org/openrewrite/javascript/JavaScriptVisitor.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -93,16 +93,15 @@ public J visitArrowFunction(JS.ArrowFunction arrowFunction, P p) {
9393
)
9494
);
9595
a = a.withParameters(
96-
a.getParameters().getPadding().withParams(
97-
Objects.requireNonNull(ListUtils.map(a.getParameters().getPadding().getParams(),
96+
a.getParameters().getPadding().withParameters(
97+
Objects.requireNonNull(ListUtils.map(a.getParameters().getPadding().getParameters(),
9898
param -> visitRightPadded(param, JRightPadded.Location.LAMBDA_PARAM, p)
9999
))
100100
)
101101
);
102102
a = a.withParameters(Objects.requireNonNull(visitAndCast(a.getParameters(), p)));
103103
a = a.withReturnTypeExpression(visitAndCast(a.getReturnTypeExpression(), p));
104104
a = a.getPadding().withBody(Objects.requireNonNull(visitLeftPadded(a.getPadding().getBody(), JsLeftPadded.Location.LAMBDA_ARROW, p)));
105-
a = a.withBody(Objects.requireNonNull(visitAndCast(a.getBody(), p)));
106105
a = a.withType(visitType(a.getType(), p));
107106
return a;
108107
}

0 commit comments

Comments
 (0)