I came to the conclusion that using J2EEApplication for wrapping non-ear modules is a bug. E.g. a war inside a sar that describes the parent .sar as being a J2EEApplication is not correct.
So I've removed this wrapping here: http://jira.jboss.com/jira/browse/JBAS-2026
A J2EEApplication with a non-null value really refers to an .ear. The TCK tests seem to agree with that.
Based on this, the org.jboss.test.jmx.test.JarInSarJSR77UnitTestCase doesn't make sense, so I'll disable it. This is in Branch_4_2 and HEAD.
We need to rethink our jsr77 view of location and scoping and the discussion in this thread is a good start.
View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4034690#4034690
Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4034690
"adrian(a)jboss.org" wrote : If you are going to do property editors then you should use the TypeInfo.convertValue()
| instead of creating a whole new place to fix when somebody wants to change the logic
| or finds a bug.
Sure, but how to get a TypeInfo instance of the clazz in AbstractAliasMetaData?
Static [Introspection]TypeInfoFactory instance + context classloader in AbstractAliasMetaData?
View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4034682#4034682
Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4034682
"adrian(a)jboss.org" wrote :
| Why did you add support for PropertyEditors?
|
It's optional. ;-)
"adrian(a)jboss.org" wrote :
| A bean's name or an injection name doesn't support them so isn't this useless?
|
So why is name and aliases then Object? :-)
"adrian(a)jboss.org" wrote :
| And probably confusing if somebody tries it? :-)
| "12345".equals(new Integer(12345)) == false
When we get a user that is confused with this, I'll personally make sure he get the right info. ;-)
View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4034675#4034675
Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4034675