From b04565c9dcb557766f2b5be4095d0141df620265 Mon Sep 17 00:00:00 2001
From: Kah Wei <msntkw@msn.com>
Date: Mon, 31 Oct 2022 15:07:10 +0800
Subject: [PATCH] Added implementation for scripts and component functions for
 scripts

---
 SHADE_Managed/src/Components/Component.cxx |  5 ++++
 SHADE_Managed/src/Components/Component.hxx |  9 ++++++++
 SHADE_Managed/src/Scripts/Script.cxx       | 27 +++++++++++-----------
 SHADE_Managed/src/Scripts/Script.hxx       |  9 ++++++++
 4 files changed, 37 insertions(+), 13 deletions(-)

diff --git a/SHADE_Managed/src/Components/Component.cxx b/SHADE_Managed/src/Components/Component.cxx
index a6afc5cc..7f56fad3 100644
--- a/SHADE_Managed/src/Components/Component.cxx
+++ b/SHADE_Managed/src/Components/Component.cxx
@@ -67,6 +67,11 @@ namespace SHADE
         ScriptStore::RemoveScript<T>(owner.GetEntity());
     }
 
+    BaseComponent::operator bool(BaseComponent^ c)
+    {
+        return c != nullptr;
+    }
+
     /*---------------------------------------------------------------------------------*/
     /* Constructors                                                                    */
     /*---------------------------------------------------------------------------------*/
diff --git a/SHADE_Managed/src/Components/Component.hxx b/SHADE_Managed/src/Components/Component.hxx
index 670e4e21..5ffa3952 100644
--- a/SHADE_Managed/src/Components/Component.hxx
+++ b/SHADE_Managed/src/Components/Component.hxx
@@ -101,6 +101,15 @@ namespace SHADE
         generic<typename T> where T : ref class, Script
         void RemoveScript();
 
+        /*-----------------------------------------------------------------------------*/
+        /* Operator Overloads                                                          */
+        /*-----------------------------------------------------------------------------*/
+        /// <summary>
+        /// Implicit conversion operator to enable checking if a component is null.
+        /// </summary>
+        /// <param name="c">Component to check.</param>
+        static operator bool(BaseComponent^ c);
+
       protected:
         /*-----------------------------------------------------------------------------*/
         /* Constructors                                                                */
diff --git a/SHADE_Managed/src/Scripts/Script.cxx b/SHADE_Managed/src/Scripts/Script.cxx
index ecd27325..d4004e03 100644
--- a/SHADE_Managed/src/Scripts/Script.cxx
+++ b/SHADE_Managed/src/Scripts/Script.cxx
@@ -18,6 +18,8 @@ of DigiPen Institute of Technology is prohibited.
 #include "Script.hxx"
 // Project Headers
 #include "Utility/Debug.hxx"
+#include "ScriptStore.hxx"
+#include "Engine/ECS.hxx"
 
 namespace SHADE
 {
@@ -49,8 +51,7 @@ namespace SHADE
     generic <typename T>
     void Script::RemoveComponent()
     {
-        throw gcnew System::NotImplementedException;
-        //ECS::RemoveComponent<T>(owner.GetNativeEntity());
+        owner.RemoveComponent<T>();
     }
 
     /*---------------------------------------------------------------------------------*/
@@ -59,37 +60,37 @@ namespace SHADE
     generic <typename T>
     T Script::AddScript()
     {
-        throw gcnew System::NotImplementedException;
-        //return ScriptStore::AddScript<T>(owner.GetEntity());
+        return ScriptStore::AddScript<T>(owner.GetEntity());
     }
     generic <typename T>
     T Script::GetScript()
     {
-        throw gcnew System::NotImplementedException;
-        //return ScriptStore::GetScript<T>(owner.GetEntity());
+        return ScriptStore::GetScript<T>(owner.GetEntity());
     }
 
     generic <typename T>
     T Script::GetScriptInChildren()
     {
-        throw gcnew System::NotImplementedException;
-        //return ScriptStore::GetScriptInChildren<T>(owner.GetEntity());
+        return ScriptStore::GetScriptInChildren<T>(owner.GetEntity());
     }
 
     generic <typename T>
     System::Collections::Generic::IEnumerable<T>^ Script::GetScripts()
     {
-        throw gcnew System::NotImplementedException;
-        //return ScriptStore::GetScripts<T>(owner.GetEntity());
+        return ScriptStore::GetScripts<T>(owner.GetEntity());
     }
 
     generic <typename T>
     void Script::RemoveScript()
     {
-        throw gcnew System::NotImplementedException;
-        //ScriptStore::RemoveScript<T>(owner.GetEntity());
+        ScriptStore::RemoveScript<T>(owner.GetEntity());
     }
-    
+
+    Script::operator bool(Script^ s)
+    {
+        return s != nullptr;
+    }
+
     /*---------------------------------------------------------------------------------*/
     /* "All-time" Lifecycle Functions                                                  */
     /*---------------------------------------------------------------------------------*/
diff --git a/SHADE_Managed/src/Scripts/Script.hxx b/SHADE_Managed/src/Scripts/Script.hxx
index cef9f4cd..bd336726 100644
--- a/SHADE_Managed/src/Scripts/Script.hxx
+++ b/SHADE_Managed/src/Scripts/Script.hxx
@@ -153,6 +153,15 @@ namespace SHADE
         generic<typename T> where T : ref class, Script
         void RemoveScript();
 
+        /*-----------------------------------------------------------------------------*/
+        /* Operator Overloads                                                          */
+        /*-----------------------------------------------------------------------------*/
+        /// <summary>
+        /// Implicit conversion operator to enable checking if a component is null.
+        /// </summary>
+        /// <param name="c">Component to check.</param>
+        static operator bool(Script^ s);
+
       internal:
         /*-----------------------------------------------------------------------------*/
         /* "All-Time" Lifecycle Functions                                              */