JAXRS code lens participant based on method name range

[jeffmaury]

When generating the code lens, the JAXRS participant is computing the range based on the method name element which may lead to strange behavior for methods that are declared on several lines:

So on the IntelliJ implementation, I have used the whole method element to compute the range which gives this:

The fix is a single line of code change so let me know if you're interested by a PR

[rgrunber]

CC'ing @angelozerr . I think it makes sense! getParent() on the ASTNode to get the MethodDeclaration start line instead of the SimpleName ?

In the "method-on-single-line" case, it looks like we placed the codelens above the declaration but below the annotations. Does it make sense to preserve that behaviour ? Does anyone care ?

[angelozerr]

I think it makes sense! getParent() on the ASTNode to get the MethodDeclaration start line instead of the SimpleName ?

Indeed we should improve that, but...

Does it make sense to preserve that behaviour

It should be nice to keep that.

Does anyone care ?

If I have time, I will do it.

[datho7561]

@JessicaJHee What do you think about this issue? If you are interested and are looking for an issue that won't take too long, I think it could be good. If you're not interested, I could also take a look into implementing it.