[jboss-user] [JBoss jBPM] - Re: Is this a bug or is it intentional

2008-06-20 Thread bentins
I agree both scanarios should be addressed. Addressing the interface issue is 
not hard as we can do
type.isInterface()


However this leaves two problems. Both are not common. use cases.
1. Abstract classes which if someone uses it is imposible to get it to work, 
since it will fail on 'Field.set'. 

2. Another problem is when a user uses his own sub interface.

I think the code should address the first two issues and state in the 
documentation not to use abstract classes or sub interfaces of Collection, Set, 
List and Map for field mapping. (Maybe add an clear exception message)

View the original post : 
http://www.jboss.com/index.html?module=bbop=viewtopicp=4159483#4159483

Reply to the post : 
http://www.jboss.com/index.html?module=bbop=postingmode=replyp=4159483
___
jboss-user mailing list
jboss-user@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jboss-user


[jboss-user] [JBoss jBPM] - Re: Is this a bug or is it intentional

2008-06-20 Thread kukeltje
look in the jira, I think Alex already has a solution there

View the original post : 
http://www.jboss.com/index.html?module=bbop=viewtopicp=4159651#4159651

Reply to the post : 
http://www.jboss.com/index.html?module=bbop=postingmode=replyp=4159651
___
jboss-user mailing list
jboss-user@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jboss-user


[jboss-user] [JBoss jBPM] - Re: Is this a bug or is it intentional

2008-06-19 Thread [EMAIL PROTECTED]
I see problems with both the current code and your proposed fix.

The current code fails for the declaration below, because it will try to assign 
a HashMap to a Hashtable.
Hashtable keyCodes;

The proposed fix actually makes things worse because the following declaration 
is the common case. Class.newInstance() fails for interfaces.
Map keyCodes;

The definitive solution should address both scenarios. I'll reopen the issue 
and add this comment.

View the original post : 
http://www.jboss.com/index.html?module=bbop=viewtopicp=4159378#4159378

Reply to the post : 
http://www.jboss.com/index.html?module=bbop=postingmode=replyp=4159378
___
jboss-user mailing list
jboss-user@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jboss-user