Declarative linking should use uriInfo to fill Links

[glassfishrobot]

Right now, all parameters of the resource have to be defined in the Bindings of the @InjectLink annotation. It comes with 2 issues :

• It assumes all parameters can be retrieved through instance/entity/resource. This may not always be the case
• It forces to redeclare in Bindings what is already present in uriInfo which is a burden and makes the annotation declaration big and hard to maintain when you have a lot of parameters.

The fix would be to get the pathParameters and queryParameters and add them by default to the list of values. Those values can be overridden if the name is set in one of the Bindings

Affected Versions

[2.17]

[glassfishrobot]

Reported by jccounio

[glassfishrobot]

Was assigned to michalgajdos

[glassfishrobot]

jccounio said:
Patch is available here
https://gist.github.com/anonymous/bb457425728291ae7e7c

[glassfishrobot]

@AdamLindenthal said:
Hi,
just to make things easier to find, there is also a pull request you've created:
https://github.com/jersey/jersey/pull/163

I will move the issue to backlog now, as we are waiting for your OCA to get approved, the patch to get reviewed and finally merged.

Thank you,
Adam

[glassfishrobot]

Marked as fixed on Wednesday, May 20th 2015, 8:05:41 am

[glassfishrobot]

This issue was imported from java.net JIRA JERSEY-2863