Properly actually fixed attachable scripts running when their parent didn't really exist

Added lua binding for SoundContainer::SelectNextSoundSet

Author: garethyr

Entities/Attachable.cpp

@@ -274,7 +274,7 @@ void Attachable::Attach(MOSRotating *pParent)
     // Reset the attachables timers so things that have been sitting in inventory don't make backed up emissions
     ResetAllTimers();
 
-    if (m_pParent != NULL && (!m_pParent->HasAnyScripts() || m_pParent->ObjectScriptsInitialized())) {
+    if (m_pParent != NULL && m_pParent->GetID() != 255) {
         RunScriptedFunctionInAppropriateScripts("OnAttach", false, false, {m_pParent});
     }
 }
@@ -306,7 +306,7 @@ void Attachable::Detach()
 
     m_RestTimer.Reset();
 
-    if (temporaryParent != NULL && (!temporaryParent->HasAnyScripts() || temporaryParent->ObjectScriptsInitialized())) {
+    if (temporaryParent != NULL && temporaryParent->GetID() != 255) {
         RunScriptedFunctionInAppropriateScripts("OnDetach", false, false, {temporaryParent});
     }
 }
@@ -554,7 +554,7 @@ void Attachable::Update()
     MOSRotating::Update();
 
     // If we're attached to something, MoveableMan doesn't own us, and therefore isn't calling our ScriptUpdate (and our parent isn't calling it either), so we should here
-    if (m_pParent != NULL && (!m_pParent->HasAnyScripts() || m_pParent->ObjectScriptsInitialized())) { UpdateScripts(); }
+    if (m_pParent != NULL && m_pParent->GetID() != 255) { UpdateScripts(); }
 }
 
 

Managers/LuaMan.cpp

@@ -568,6 +568,7 @@ int LuaMan::Create()
             .def("AddSound", (void (SoundContainer:: *)(std::string const &soundFilePath, const Vector &offset, float attenuationStartDistance, bool abortGameForInvalidSound)) &SoundContainer::AddSound)
             .def("AddSound", (void (SoundContainer:: *)(std::string const &soundFilePath, unsigned int soundSetIndex, const Vector &offset, float minimumAudibleDistance, float attenuationStartDistance, bool abortGameForInvalidSound)) &SoundContainer::AddSound)
 			.def("SetPosition", &SoundContainer::SetPosition)
+            .def("SelectNextSoundSet", &SoundContainer::SelectNextSoundSet)
 			.property("Loops", &SoundContainer::GetLoopSetting, &SoundContainer::SetLoopSetting)
             .property("Priority", &SoundContainer::GetPriority, &SoundContainer::SetPriority)
             .property("AffectedByGlobalPitch", &SoundContainer::IsAffectedByGlobalPitch, &SoundContainer::SetAffectedByGlobalPitch),