Why would I? It's a blatant bug ;)
(OT: feel free to pitch in your PUR-related issues again; so far I'm
still not understand what you needed ;))
-g
On Oct 15, 2009, at 3:36 PM, Bert Leunis wrote:
Ok, thanks Philipp. If nobody has any objections or further
discussion material (Grégory??), I can add this item to jira.
Many regards, Bert
From: [email protected] [mailto:[email protected]
] On Behalf Of Philipp Bärfuss
Sent: donderdag 15 oktober 2009 10:28
To: Magnolia Dev-List
Subject: Re: [magnolia-dev] ContentUtil collectAllChildren differences
I agree
- Philipp Bärfuss
On 15.10.2009, at 09:47, Bert Leunis wrote:
Hello all,
I think I spotted a different approach for two of the
collecAllChildren methods in the info.magnolia.cms.util.ContentUtil
class. The methods collect nodes recursively.
1. collectAllChildren(List nodes, Content node, ContentFilter filter)
In this method the first level of children is collected with the
filter. For the next level of children not only the nodes that match
the filter are taken into account, but the children of ALL nodes of
the first level. So children that match the filter, but who’s
parents do not, are indeed found.
2. collectAllChildren(List nodes, Content node, ItemType[] types)
In this method however, only children that have parents that match
one of the ItemTypes in the array are found. For example: if I am
looking for all nodes of type B and C, but they some of them have a
parent of type A, they will not be found.
This seems inconsistent to me, and I think the first way matches the
expected functionality the best. Do you agree?
Bye, Bert
----------------------------------------------------------------
For list details see
http://www.magnolia-cms.com/home/community/mailing-lists.html
To unsubscribe, E-mail to: <[email protected]>
----------------------------------------------------------------
----------------------------------------------------------------
For list details see
http://www.magnolia-cms.com/home/community/mailing-lists.html
To unsubscribe, E-mail to: <[email protected]>
----------------------------------------------------------------
----------------------------------------------------------------
For list details see
http://www.magnolia-cms.com/home/community/mailing-lists.html
To unsubscribe, E-mail to: <[email protected]>
----------------------------------------------------------------
----------------------------------------------------------------
For list details see
http://www.magnolia-cms.com/home/community/mailing-lists.html
To unsubscribe, E-mail to: <[email protected]>
----------------------------------------------------------------
