ictvmt / mvp4g

Automatically exported from code.google.com/p/mvp4g
0 stars 0 forks source link

More than one argument in events should be supported #21

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
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 commented 8 years ago
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

GoogleCodeExporter commented 8 years ago
I would be very much interested too into that .

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

GoogleCodeExporter commented 8 years ago
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 commented 8 years ago
Issue 23 has been merged into this issue.

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

GoogleCodeExporter commented 8 years ago
Issue 24 has been merged into this issue.

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

GoogleCodeExporter commented 8 years ago
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 commented 8 years ago
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 commented 8 years ago
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