From 76fdd3c76eb4dbb202657ac3d7cd325edfcb24a4 Mon Sep 17 00:00:00 2001
From: CatChow0 <thetrueshibadoggo@gmail.com>
Date: Wed, 17 Sep 2025 16:47:01 +0200
Subject: [PATCH] Patch - Adds component shutdown functionality - V13.6.1

Implements a shutdown mechanism for components and entities,
ensuring proper resource release and preventing memory leaks.

This change introduces a virtual `Shutdown` method to the
`Component` class, allowing derived components to release
specific resources when they are removed from an entity or
when the entity is destroyed. The `Entity` class now calls
the `Shutdown` method on all its components during its own
release process. The `EntityManager` now calls the
`release()` method when destroying an entity.

This enhancement ensures that resources, such as FMOD sound
objects and channels in the `AudioComponent`, are properly
released, preventing potential resource leaks and improving
the stability of the engine.
---
 .../.idea/workspace.xml                       | 38 +++++++++----------
 enginecustom/src/inc/system/ecs/component.h   |  5 +++
 .../system/ecs/components/audio_component.h   | 21 ++++++++++
 enginecustom/src/inc/system/ecs/entity.h      | 16 ++++++++
 .../src/inc/system/ecs/entity_manager.h       |  1 +
 5 files changed, 62 insertions(+), 19 deletions(-)

diff --git a/.idea/.idea.KhaoticEngineReborn/.idea/workspace.xml b/.idea/.idea.KhaoticEngineReborn/.idea/workspace.xml
index 467c771..2939729 100644
--- a/.idea/.idea.KhaoticEngineReborn/.idea/workspace.xml
+++ b/.idea/.idea.KhaoticEngineReborn/.idea/workspace.xml
@@ -62,26 +62,26 @@
     <option name="hideEmptyMiddlePackages" value="true" />
     <option name="showLibraryContents" value="true" />
   </component>
-  <component name="PropertiesComponent"><![CDATA[{
-  "keyToString": {
-    "ASKED_SHARE_PROJECT_CONFIGURATION_FILES": "true",
-    "C++ Project.enginecustom.executor": "Run",
-    "C/C++ Project.KhaoticDemo.executor": "Run",
-    "C/C++ Project.enginecustom.executor": "Run",
-    "RunOnceActivity.ShowReadmeOnStart": "true",
-    "RunOnceActivity.git.unshallow": "true",
-    "SHARE_PROJECT_CONFIGURATION_FILES": "true",
-    "git-widget-placeholder": "main",
-    "ignore.virus.scanning.warn.message": "true",
-    "node.js.detected.package.eslint": "true",
-    "node.js.detected.package.tslint": "true",
-    "node.js.selected.package.eslint": "(autodetect)",
-    "node.js.selected.package.tslint": "(autodetect)",
-    "nodejs_package_manager_path": "npm",
-    "settings.editor.selected.configurable": "preferences.pluginManager",
-    "vue.rearranger.settings.migration": "true"
+  <component name="PropertiesComponent">{
+  &quot;keyToString&quot;: {
+    &quot;ASKED_SHARE_PROJECT_CONFIGURATION_FILES&quot;: &quot;true&quot;,
+    &quot;C++ Project.enginecustom.executor&quot;: &quot;Run&quot;,
+    &quot;C/C++ Project.KhaoticDemo.executor&quot;: &quot;Run&quot;,
+    &quot;C/C++ Project.enginecustom.executor&quot;: &quot;Run&quot;,
+    &quot;RunOnceActivity.ShowReadmeOnStart&quot;: &quot;true&quot;,
+    &quot;RunOnceActivity.git.unshallow&quot;: &quot;true&quot;,
+    &quot;SHARE_PROJECT_CONFIGURATION_FILES&quot;: &quot;true&quot;,
+    &quot;git-widget-placeholder&quot;: &quot;main&quot;,
+    &quot;ignore.virus.scanning.warn.message&quot;: &quot;true&quot;,
+    &quot;node.js.detected.package.eslint&quot;: &quot;true&quot;,
+    &quot;node.js.detected.package.tslint&quot;: &quot;true&quot;,
+    &quot;node.js.selected.package.eslint&quot;: &quot;(autodetect)&quot;,
+    &quot;node.js.selected.package.tslint&quot;: &quot;(autodetect)&quot;,
+    &quot;nodejs_package_manager_path&quot;: &quot;npm&quot;,
+    &quot;settings.editor.selected.configurable&quot;: &quot;preferences.pluginManager&quot;,
+    &quot;vue.rearranger.settings.migration&quot;: &quot;true&quot;
   }
-}]]></component>
+}</component>
   <component name="RunManager" selected="C/C++ Project.enginecustom">
     <configuration name="KhaoticDemo" type="CppProject" factoryName="C++ Project">
       <configuration_1 setup="1">
diff --git a/enginecustom/src/inc/system/ecs/component.h b/enginecustom/src/inc/system/ecs/component.h
index b4f74cb..3d80dea 100644
--- a/enginecustom/src/inc/system/ecs/component.h
+++ b/enginecustom/src/inc/system/ecs/component.h
@@ -32,6 +32,11 @@ public:
      */
     virtual void Initialize() {}
 
+    /**
+     *Virtual function to shutdown the component.
+     */
+    virtual void Shutdown() {}
+
     /**
      * Virtual function to update the component.
      * @param deltaTime Time since the last update.
diff --git a/enginecustom/src/inc/system/ecs/components/audio_component.h b/enginecustom/src/inc/system/ecs/components/audio_component.h
index ed1938b..ebcd9fe 100644
--- a/enginecustom/src/inc/system/ecs/components/audio_component.h
+++ b/enginecustom/src/inc/system/ecs/components/audio_component.h
@@ -30,6 +30,27 @@ public:
         // Ne pas libérer m_system ici car il peut être partagé
     }
 
+    /**
+     * Shutdown the audio component by releasing the sound and channel.
+     * This method is called when the component is removed from an entity or when the entity is destroyed.
+     */
+    void Shutdown() override {
+        if (m_channel) {
+            bool isPlaying = false;
+            m_channel->isPlaying(&isPlaying);
+            if (isPlaying) {
+                m_channel->stop();
+            }
+            m_channel = nullptr;
+        }
+        if (m_sound) {
+            m_sound->release();
+            m_sound = nullptr;
+        }
+        m_system = nullptr; // Ne pas libérer m_system ici car il peut être partagé
+        m_camera = nullptr;
+    }
+
     /**
      * Initialize the audio component by setting up the FMOD system and camera from the parent entity.
      * This method is called when the component is added to an entity.
diff --git a/enginecustom/src/inc/system/ecs/entity.h b/enginecustom/src/inc/system/ecs/entity.h
index cec0898..6a45a9f 100644
--- a/enginecustom/src/inc/system/ecs/entity.h
+++ b/enginecustom/src/inc/system/ecs/entity.h
@@ -163,6 +163,22 @@ public:
      */
     FMOD::System* GetSoundSystem() const { return m_soundSystem; }
 
+    /**
+     * Release all data associated with the entity.
+     * This includes clearing all components and resetting the camera and sound system pointers.
+     */
+    void release()
+    {
+        // call shutdown on all components if needed
+        for (auto& [typeID, component] : m_Components) {
+            // If components had a shutdown method, we would call it here
+            component->Shutdown();
+        }
+        m_Components.clear();
+        m_camera = nullptr;
+        m_soundSystem = nullptr;
+    }
+
     void SetDevice (ID3D11Device* dev) { device = dev; }
     void SetContext(ID3D11DeviceContext* ctx) { context = ctx; }
     ID3D11Device* GetDevice() const { return device; }
diff --git a/enginecustom/src/inc/system/ecs/entity_manager.h b/enginecustom/src/inc/system/ecs/entity_manager.h
index be5852b..f641876 100644
--- a/enginecustom/src/inc/system/ecs/entity_manager.h
+++ b/enginecustom/src/inc/system/ecs/entity_manager.h
@@ -48,6 +48,7 @@ public:
     void DestroyEntity(EntityID id) {
         auto it = m_Entities.find(id);
         if (it != m_Entities.end()) {
+            it->second->release();
             m_Entities.erase(it);
             m_FreeIDs.push(id); // Recycler l'ID
         }