You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I would like to switch one constructor call for an other with JavaTemplate. E.g. if a class A offers two constructors A(int foo), A(int foo, double ba) I would like to be able to insert a new argument after, or replace the list of arguments altogether.
Describe the solution you'd like
API offers suitable methods through getCoordinates.
Have you considered any alternatives or workarounds?
Replacing the entire constructor call should work in theory. withArguments() works as well, but JavaTemplates seem to be the way to go.
Additional context
Currently a constructor call can be replaced entirely, but it would be nice to be able to operate it without putting "new A" into the template, which requires me to find out the class name at runtime. J.MethodInvocation already has getCoordinates().replaceArguments(), that could just be implemented in newClass as well.
Thanks for the suggestion @timo-abele ; I imagine this to be very similar to how we handle J.MethodInvocation, which I think should make for a nice addition.
What problem are you trying to solve?
I would like to switch one constructor call for an other with JavaTemplate. E.g. if a class
A
offers two constructorsA(int foo)
,A(int foo, double ba)
I would like to be able to insert a new argument after, or replace the list of arguments altogether.Describe the solution you'd like
API offers suitable methods through getCoordinates.
Have you considered any alternatives or workarounds?
Replacing the entire constructor call should work in theory.
withArguments()
works as well, but JavaTemplates seem to be the way to go.Additional context
Currently a constructor call can be replaced entirely, but it would be nice to be able to operate it without putting "new A" into the template, which requires me to find out the class name at runtime.
J.MethodInvocation
already hasgetCoordinates().replaceArguments()
, that could just be implemented innewClass
as well.Are you interested in contributing this feature to OpenRewrite?
probably won't find the time
The text was updated successfully, but these errors were encountered: