From 29dea68baba6409d789e689343ddd2af928a6ac3 Mon Sep 17 00:00:00 2001 From: clayjohn Date: Fri, 2 Dec 2022 23:20:39 -0800 Subject: Index override surface name starting from 1 to match surface name --- scene/3d/mesh_instance_3d.cpp | 19 +++++++++++++++---- scene/3d/mesh_instance_3d.h | 1 + 2 files changed, 16 insertions(+), 4 deletions(-) diff --git a/scene/3d/mesh_instance_3d.cpp b/scene/3d/mesh_instance_3d.cpp index 04d164ba88..aee1138316 100644 --- a/scene/3d/mesh_instance_3d.cpp +++ b/scene/3d/mesh_instance_3d.cpp @@ -52,11 +52,22 @@ bool MeshInstance3D::_set(const StringName &p_name, const Variant &p_value) { if (p_name.operator String().begins_with("surface_material_override/")) { int idx = p_name.operator String().get_slicec('/', 1).to_int(); - if (idx >= surface_override_materials.size() || idx < 0) { + + // This is a bit of a hack to ensure compatibility with older material + // overrides that start indexing at 0. + // We assume that idx 0 is always read first, if its not, this won't work. + if (idx == 0) { + old_surface_index = true; + } + if (old_surface_index) { + idx++; + } + + if (idx > surface_override_materials.size() || idx < 0) { return false; } - set_surface_override_material(idx, p_value); + set_surface_override_material(idx - 1, p_value); return true; } @@ -75,7 +86,7 @@ bool MeshInstance3D::_get(const StringName &p_name, Variant &r_ret) const { } if (p_name.operator String().begins_with("surface_material_override/")) { - int idx = p_name.operator String().get_slicec('/', 1).to_int(); + int idx = p_name.operator String().get_slicec('/', 1).to_int() - 1; if (idx >= surface_override_materials.size() || idx < 0) { return false; } @@ -98,7 +109,7 @@ void MeshInstance3D::_get_property_list(List *p_list) const { } if (mesh.is_valid()) { - for (int i = 0; i < mesh->get_surface_count(); i++) { + for (int i = 1; i <= mesh->get_surface_count(); i++) { p_list->push_back(PropertyInfo(Variant::OBJECT, vformat("%s/%d", PNAME("surface_material_override"), i), PROPERTY_HINT_RESOURCE_TYPE, "BaseMaterial3D,ShaderMaterial", PROPERTY_USAGE_DEFAULT | PROPERTY_USAGE_DEFERRED_SET_RESOURCE)); } } diff --git a/scene/3d/mesh_instance_3d.h b/scene/3d/mesh_instance_3d.h index caf7d25616..9e6dbaf160 100644 --- a/scene/3d/mesh_instance_3d.h +++ b/scene/3d/mesh_instance_3d.h @@ -57,6 +57,7 @@ protected: bool _set(const StringName &p_name, const Variant &p_value); bool _get(const StringName &p_name, Variant &r_ret) const; void _get_property_list(List *p_list) const; + bool old_surface_index = false; void _notification(int p_what); static void _bind_methods(); -- cgit v1.2.3