Skip to content
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

More than one argument in events should be supported #21

Closed
GoogleCodeExporter opened this issue Jul 1, 2015 · 8 comments
Closed

More than one argument in events should be supported #21

GoogleCodeExporter opened this issue Jul 1, 2015 · 8 comments

Comments

@GoogleCodeExporter
Copy link

If you declare event with more than one argument the
com.mvp4g.util.exception.loader.Mvp4gAnnotationException is thrown with
message: Event method must not have more than 1 argument.

Original issue reported on code.google.com by michal.k...@gmail.com on 28 Jan 2010 at 10:48

@GoogleCodeExporter
Copy link
Author

This could be a nice improvement but it's not that easy to implement because of 
History Converter.

I will look into it for the next version after 1.1.0

Original comment by plcoir...@gmail.com on 29 Jan 2010 at 2:29

  • Changed state: Accepted
  • Added labels: Type-Enhancement
  • Removed labels: Type-Defect

@GoogleCodeExporter
Copy link
Author

I would be very much interested too into that .

Original comment by googelyb...@gmail.com on 7 Feb 2010 at 3:43

@GoogleCodeExporter
Copy link
Author

For now, a workaround is to create a bean and throw this bean with your event.

I will definitely improve this in the next version.

Original comment by plcoir...@gmail.com on 9 Feb 2010 at 3:51

@GoogleCodeExporter
Copy link
Author

Issue 23 has been merged into this issue.

Original comment by plcoir...@gmail.com on 18 Mar 2010 at 5:18

@GoogleCodeExporter
Copy link
Author

Issue 24 has been merged into this issue.

Original comment by plcoir...@gmail.com on 18 Mar 2010 at 5:19

@GoogleCodeExporter
Copy link
Author

Just to let you know that this issue will be the first feature developed for 
1.2.0

Original comment by plcoir...@gmail.com on 18 Mar 2010 at 5:20

@GoogleCodeExporter
Copy link
Author

great, can't wait to get rid of my wrapper objects ;-)

Original comment by dennis.r...@gmail.com on 18 Mar 2010 at 5:23

@GoogleCodeExporter
Copy link
Author

I added this feature in 1.2.0-SNAPSHOT that is now available. You can look at 
this 
message to know the impact: 
http://groups.google.com/group/mvp4g/browse_thread/thread/135e9d0226291d15?hl=en
_US

Let me know if you have any problem,
Thanks

Original comment by plcoir...@gmail.com on 30 Mar 2010 at 12:41

  • Changed state: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant