-
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
remove MethodInfo and create ActionParameters, ActionResult (or something like this) #104
Comments
Be aware of performance issues when creating these objects. we could have an no need to prefixes, since we have the packages ;) |
A note: |
We already have More updates are comming. |
Just changing the |
Your PR will be very appreciated =) I think @garcia-jj is working on a kind of |
Well, that is my solution hahaha. Will this branch be merged soon? |
Sorry for long delay. I was very busy in last weeks. @leonardowolter, if you want, you can improve your changes, because I think that I wont have time in this week |
@leonardowolter, @garcia-jj. How is it going? It would be nice do not break compatibility after the RC1. |
I think that is better to change before release RC1. Can we delay the RC1 for some days to think about this change? Valued parameters are very important to keep code more simple. |
vraptor-brutauth project need easy access to parameters names, and it would be much more interesting if instead of putting a new method into
MethodInfo
, create aActionParameter
"component" that besides returning the parameters, already has this behavior (return parameters names).*** not sure if
Action
prefix is really intuitive, any ideas?The text was updated successfully, but these errors were encountered: