-
Notifications
You must be signed in to change notification settings - Fork 332
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open Linker for extension #730
Conversation
@@ -44,6 +45,23 @@ public Linker(ServletContext context, Router router, Class<?> controller, String | |||
} | |||
|
|||
protected String getLink() { | |||
Method method = getMethod(); | |||
return getContextPath() + router.urlFor(controller, method, getArgs(method)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change controller
to getController()
@lucascs it was used by the I18nLinker, but I changed to return the router. |
@dtelaroli if we pass the router on the constructor manually, we don't need to have this |
@lucascs the your comment was pointed on the constructor code, I thought that you have mentioned the properties, so, I will remove the getController too. |
it was indeed to remove unnecessary getters =) |
return getContextPath() + router.urlFor(controller, method, getArgs(method)); | ||
} | ||
|
||
protected String getContextPath() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd rename it to getPrefix()
. ContextPath is the implementation.
thanks @dtelaroli! It'll be released at RC2 version (asap) |
👍 |
In addition to the #702 as dependency on the caelum/vraptor-i18n#32