ezpublish-legacy
ezpublish-legacy copied to clipboard
When using dropdown list as selection list, nodes which are added as add...
...itional location to the selection node are not presented in the dropdown list. Therefore main_node_only should be set to false() here.
Isn't there a risk that you'll get duplicate nodes ?
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?
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.