ezpublish-legacy icon indicating copy to clipboard operation
ezpublish-legacy copied to clipboard

When using dropdown list as selection list, nodes which are added as add...

Open xmak opened this issue 13 years ago • 3 comments
trafficstars

...itional location to the selection node are not presented in the dropdown list. Therefore main_node_only should be set to false() here.

xmak avatar Mar 03 '12 00:03 xmak

Isn't there a risk that you'll get duplicate nodes ?

bdunogier avatar Mar 15 '12 09:03 bdunogier

If the fetch function is "list" then we wouldn't have such a problem, but now I see that the fetch function used is "tree", which is a little bit strange - why would we want to create a dropdown list from the whole tree of items? If you want to use a dropdown list, then it's not logical to have it populated with items from a tree structure...

Also If you look more carefully at line 20, you'll see that the "list" fetch function is used in the case where there are no class contstraints, which leads to the conclusion that having "tree" in the first case is a bug.

Should we correct this?

xmak avatar Apr 07 '12 00:04 xmak

My expectation was always that it was a "list" fetch; it's very surprising to see that it is in fact a tree fetch. Of course, that brings up a BC issue if "tree" was the expectation for other users.

peterkeung avatar May 26 '12 16:05 peterkeung