asgardius/virtualx-engine
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #96206 from AThousandShips/tree_last_child_fix

Browse source 
[Tree] Fix error when removing child from `Tree`
This commit is contained in:
Rémi Verschelde 2024-08-28 13:14:05 +02:00
commit 58a492000f
No known key found for this signature in database
GPG key ID: C3336907360768E1
2 changed files with 27 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
scene/gui/tree.h
View file

@ -178,6 +178,9 @@ private:
if (parent->first_child == this) {
parent->first_child = next;
}
if (parent->last_child == this) {
parent->last_child = prev;
}
}
}

24
tests/scene/test_tree.h
View file

@ -108,6 +108,30 @@ TEST_CASE("[SceneTree][Tree]") {
memdelete(tree);
}
// https://github.com/godotengine/godot/issues/96205
SUBCASE("[Tree] Get last item after removal.") {
Tree *tree = memnew(Tree);
TreeItem *root = tree->create_item();
TreeItem *child1 = tree->create_item(root);
TreeItem *child2 = tree->create_item(root);
CHECK_EQ(root->get_child_count(), 2);
CHECK_EQ(tree->get_last_item(), child2);
root->remove_child(child2);
CHECK_EQ(root->get_child_count(), 1);
CHECK_EQ(tree->get_last_item(), child1);
root->add_child(child2);
CHECK_EQ(root->get_child_count(), 2);
CHECK_EQ(tree->get_last_item(), child2);
memdelete(tree);
}
SUBCASE("[Tree] Previous and Next items.") {
Tree *tree = memnew(Tree);
TreeItem *root = tree->create_item();