DO NOT REPLY [Bug 41647] ConditionBase and TaskContainer do not work together since ant 1.6.0

2008-10-27 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=41647


Stefan Bodewig <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Comment #3 from Stefan Bodewig <[EMAIL PROTECTED]>  2008-10-27 07:49:11 PST 
---
OK, I've combined my code from revision 707398 ( 
https://svn.apache.org/viewcvs.cgi?view=rev&rev=707398 ) with my second option 
of comment
#1.  Fixed in svn revision 708204 ( 
https://svn.apache.org/viewcvs.cgi?view=rev&rev=708204 ).


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 41647] ConditionBase and TaskContainer do not work together since ant 1.6.0

2008-10-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=41647


Stefan Bodewig <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Target Milestone|--- |1.8.0




--- Comment #2 from Stefan Bodewig <[EMAIL PROTECTED]>  2008-10-23 08:47:56 PST 
---
I've added a more thorough implementation of supportsNestedElement that is now
used by UnknownElement#handleChild.

I tried to deal with the createDynamicElement case as well, but failed.  Maybe
you have an idea how we could check whether createDynamicElement works without
actually invoking it (I think it is impossible).

svn revision 707398.


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 41647] ConditionBase and TaskContainer do not work together since ant 1.6.0

2008-09-16 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=41647





--- Comment #1 from Stefan Bodewig <[EMAIL PROTECTED]>  2008-09-16 05:43:41 PST 
---
ConditionBase has public void add(Condition) since Ant 1.6 and thus the related
IntrospectionHelper instance returns true in supportsNestedElement
(addTypeMethods.size() == 1).

This in turn makes UnknownElement's handleChild method assume a child by that
name must exist and invoke getElementCreator on IntrospectionHelper which does
a more thorough check in createAddTypeCreator and realizes that  is not a
condition.

This again makes handleChild throw an exception instead of returning false
which would trigger the addTask logic in handleChildren.

I see two options: (1) make IntrospectionHelper#supportsNestedElement only
return true when getElementCreator would return something or (2) catch the
exception in UnknownElement#handleChild and return false.

(1) would probably be cleaner but I'm afraid the method doesn't get all the
context it would need to be able to tell the truth.  You, Peter, should be at
least as familiar as myself with the code so I'll wait for your input ;-)


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.