#6009: ModelNodeBase subtypes need to implement createRenderableSurfa…

…ces to populate the renderable surface collection
codereader · Oct 29, 2022 · f4a5b59 · f4a5b59
1 parent 84248c4
commit f4a5b59
Show file tree

Hide file tree

Showing 6 changed files with 35 additions and 19 deletions.
diff --git a/radiantcore/model/ModelNodeBase.cpp b/radiantcore/model/ModelNodeBase.cpp
@@ -17,6 +17,13 @@ void ModelNodeBase::renderHighlights(IRenderableCollector& collector, const Volu
     }
 }
 
+void ModelNodeBase::destroyRenderableSurfaces()
+{
+    detachFromShaders();
+
+    _renderableSurfaces.clear();
+}
+
 void ModelNodeBase::detachFromShaders()
 {
     // Detach any existing surfaces. In case we need them again,

diff --git a/radiantcore/model/ModelNodeBase.h b/radiantcore/model/ModelNodeBase.h
@@ -26,6 +26,12 @@ class ModelNodeBase :
     void renderHighlights(IRenderableCollector& collector, const VolumeTest& volume) override;
 
 protected:
+    // To be implemented by subclasses, this should populate the _renderableSurfaces collection
+    virtual void createRenderableSurfaces() = 0;
+
+    // Detaches all surfaces from their shaders and clears the _renderableSurfaces collection
+    virtual void destroyRenderableSurfaces();
+
     void attachToShaders();
     void detachFromShaders();
     void queueRenderableUpdate();

diff --git a/radiantcore/model/StaticModelNode.cpp b/radiantcore/model/StaticModelNode.cpp
@@ -20,11 +20,8 @@ StaticModelNode::StaticModelNode(const StaticModelPtr& picoModel) :
     skinChanged("");
 }
 
-void StaticModelNode::onInsertIntoScene(scene::IMapRootNode& root)
+void StaticModelNode::createRenderableSurfaces()
 {
-    _model->connectUndoSystem(root.getUndoSystem());
-
-    // Renderables will acquire their shaders in onPreRender
     _model->foreachSurface([&](const StaticModelSurface& surface)
     {
         if (surface.getVertexArray().empty() || surface.getIndexArray().empty())
@@ -34,6 +31,14 @@ void StaticModelNode::onInsertIntoScene(scene::IMapRootNode& root)
 
         _renderableSurfaces.emplace_back(std::make_shared<RenderableModelSurface>(surface, _renderEntity, localToWorld()));
     });
+}
+
+void StaticModelNode::onInsertIntoScene(scene::IMapRootNode& root)
+{
+    _model->connectUndoSystem(root.getUndoSystem());
+
+    // Renderables will acquire their shaders in onPreRender
+    createRenderableSurfaces();
 
     Node::onInsertIntoScene(root);
 }
@@ -42,12 +47,7 @@ void StaticModelNode::onRemoveFromScene(scene::IMapRootNode& root)
 {
     _model->disconnectUndoSystem(root.getUndoSystem());
 
-    for (auto& surface : _renderableSurfaces)
-    {
-        surface->detach();
-    }
-
-    _renderableSurfaces.clear();
+    destroyRenderableSurfaces();
 
     Node::onRemoveFromScene(root);
 }

diff --git a/radiantcore/model/StaticModelNode.h b/radiantcore/model/StaticModelNode.h
@@ -90,6 +90,8 @@ class StaticModelNode final :
     void onModelScaleApplied();
 
 protected:
+    void createRenderableSurfaces() override;
+
 	void _onTransformationChanged() override;
 	void _applyTransformation() override;
     void onVisibilityChanged(bool isVisibleNow) override;

diff --git a/radiantcore/model/md5/MD5ModelNode.cpp b/radiantcore/model/md5/MD5ModelNode.cpp
@@ -72,9 +72,8 @@ scene::INode::Type MD5ModelNode::getNodeType() const
     return Type::Model;
 }
 
-void MD5ModelNode::onInsertIntoScene(scene::IMapRootNode& root)
+void MD5ModelNode::createRenderableSurfaces()
 {
-    // Renderables will acquire their shaders in onPreRender
     _model->foreachSurface([&](const MD5Surface& surface)
     {
         if (surface.getVertexArray().empty() || surface.getIndexArray().empty())
@@ -86,20 +85,21 @@ void MD5ModelNode::onInsertIntoScene(scene::IMapRootNode& root)
             std::make_shared<model::RenderableModelSurface>(surface, _renderEntity, localToWorld())
         );
     });
+}
+
+void MD5ModelNode::onInsertIntoScene(scene::IMapRootNode& root)
+{
+    // Renderables will acquire their shaders in onPreRender
+    createRenderableSurfaces();
 
     Node::onInsertIntoScene(root);
 }
 
 void MD5ModelNode::onRemoveFromScene(scene::IMapRootNode& root)
 {
-    Node::onRemoveFromScene(root);
-
-    for (auto& surface : _renderableSurfaces)
-    {
-        surface->detach();
-    }
+    destroyRenderableSurfaces();
 
-    _renderableSurfaces.clear();
+    Node::onRemoveFromScene(root);
 }
 
 void MD5ModelNode::testSelect(Selector& selector, SelectionTest& test)

diff --git a/radiantcore/model/md5/MD5ModelNode.h b/radiantcore/model/md5/MD5ModelNode.h
@@ -78,6 +78,7 @@ class MD5ModelNode :
     void transformChangedLocal() override;
 
 protected:
+    void createRenderableSurfaces() override;
     void onVisibilityChanged(bool isVisibleNow) override;
 
 private: