tdf#142294 drop placeholder on-demand node on directly populating node

Initially there's a <placeholder> entry to indicate the load-on-demand
state.  It needs to be there for the expander indicator to be
drawn/shown when there are no "real" entries yet.

Typically the user clicks on the expander, the expansion remove the
<placeholder> entry and calls the provided expander-callback which has
the change to populate the node.

Here RequestingChildrenHdl is the expander-callback (set via
connect_expanding) and populates the node like it should.

But then there are two places where it is called directly outside the
expander-callback so it merrily appends entries into the node, but
because it's called outside the expander-callback the <placeholder>
entry was never removed which is then a problem because of the resulting
inconsistency.

Change-Id: Ie7d034e3f41832591438127c82eba85deb610952
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/115949
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>

0 files changed, 0 insertions, 0 deletions