Fix (unreported): Outliner Data API tree subtree expansion

The changes in rB70151e41dc02 broke subtree expansion in the Data API
display mode because the closed subtrees are empty lists. Move the empty
subtree check from `outliner_item_openclose` to the walk navigation
code to prevent the issue.

2 files changed, 5 insertions, 5 deletions

diff --git a/source/blender/editors/space_outliner/outliner_edit.c b/source/blender/editors/space_outliner/outliner_edit.c
index 99eb9a04c0e..8567dd4da13 100644
--- a/source/blender/editors/space_outliner/outliner_edit.c
+++ b/source/blender/editors/space_outliner/outliner_edit.c
@@ -152,11 +152,8 @@ void OUTLINER_OT_highlight_update(wmOperatorType *ot)
 /* Open or close a tree element, optionally toggling all children recursively */
 void outliner_item_openclose(TreeElement *te, bool open, bool toggle_all)
 {
-  if (BLI_listbase_is_empty(&te->subtree)) {
-    return;
-  }
-
   TreeStoreElem *tselem = TREESTORE(te);
+
   if (open) {
     tselem->flag &= ~TSE_CLOSED;
   }
diff --git a/source/blender/editors/space_outliner/outliner_select.c b/source/blender/editors/space_outliner/outliner_select.c
index 28f7fc70618..3c7e4506e5d 100644
--- a/source/blender/editors/space_outliner/outliner_select.c
+++ b/source/blender/editors/space_outliner/outliner_select.c
@@ -1661,7 +1661,10 @@ static TreeElement *outliner_walk_right(SpaceOutliner *space_outliner,
   if (!toggle_all && TSELEM_OPEN(tselem, space_outliner) && !BLI_listbase_is_empty(&te->subtree)) {
     te = te->subtree.first;
   }
-  else {
+  /* Prevent opening leaf elements in the tree.
+   * This cannot be done in `outliner_item_openclose` because the Data API display mode subtrees
+   * are empty unless expanded. */
+  else if (!BLI_listbase_is_empty(&te->subtree)) {
     outliner_item_openclose(te, true, toggle_all);
   }