Merge pull request #76854 from CapitaineCookie/fix-change-type

Fix crash when changing node type from PopupMenu to ItemList
This commit is contained in:
Yuri Sizov 2023-06-02 13:03:55 +02:00 committed by GitHub
commit 6a1d3198d1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -2051,8 +2051,13 @@ bool PopupMenu::_get(const StringName &p_name, Variant &r_ret) const {
r_ret = get_item_icon(item_index); r_ret = get_item_icon(item_index);
return true; return true;
} else if (property == "checkable") { } else if (property == "checkable") {
r_ret = this->items[item_index].checkable_type; if (item_index >= 0 && item_index < items.size()) {
return true; r_ret = items[item_index].checkable_type;
return true;
} else {
r_ret = Item::CHECKABLE_TYPE_NONE;
ERR_FAIL_V(true);
}
} else if (property == "checked") { } else if (property == "checked") {
r_ret = is_item_checked(item_index); r_ret = is_item_checked(item_index);
return true; return true;