Xengine patch to force a persistence save on script creation. This prevents

duplicate delivery of state_entry if a region is restarted just after
saving the script. Changes script state saves to no longer abort long-
running event handlers. Queues the save instead. Adds shutdown handler
to save script state on irderly shutdown

2 files changed, 28 insertions, 6 deletions

diff --git a/OpenSim/Region/ScriptEngine/Shared/Instance/ScriptInstance.cs b/OpenSim/Region/ScriptEngine/Shared/Instance/ScriptInstance.cs
index d35bfa8..08f9545 100644
--- a/OpenSim/Region/ScriptEngine/Shared/Instance/ScriptInstance.cs
+++ b/OpenSim/Region/ScriptEngine/Shared/Instance/ScriptInstance.cs
@@ -73,6 +73,7 @@ namespace OpenSim.Region.ScriptEngine.Shared.Instance
         private string m_CurrentEvent = String.Empty;
         private bool m_InSelfDelete = false;
         private int m_MaxScriptQueue;
+        private bool m_SaveState = true;
 
         private Dictionary<string,IScriptApi> m_Apis = new Dictionary<string,IScriptApi>();
 
@@ -213,6 +214,8 @@ namespace OpenSim.Region.ScriptEngine.Shared.Instance
                 return;
             }
 
+            m_SaveState = true;
+
             string savedState = Path.Combine(Path.GetDirectoryName(assembly),
                     m_ItemID.ToString() + ".state");
             if (File.Exists(savedState))
@@ -257,6 +260,10 @@ namespace OpenSim.Region.ScriptEngine.Shared.Instance
                             // we get new rez events on sim restart, too
                             // but if there is state, then we fire the change
                             // event
+
+                            // We loaded state, don't force a re-save
+                            m_SaveState = false;
+
                             if (stateSource == StateSource.NewRez)
                             {
 //                                m_Engine.Log.Debug("[Script] Posted changed(CHANGED_REGION_RESTART) to script");
@@ -487,6 +494,17 @@ namespace OpenSim.Region.ScriptEngine.Shared.Instance
 
                     m_InEvent = false;
                     m_CurrentEvent = String.Empty;
+
+                    if (m_SaveState)
+                    {
+                        // This will be the very first event we deliver
+                        // (state_entry) in defualt state
+                        //
+
+                        SaveState(m_Assembly);
+
+                        m_SaveState = false;
+                    }
                 }
                 catch (Exception e)
                 {
@@ -589,6 +607,7 @@ namespace OpenSim.Region.ScriptEngine.Shared.Instance
             m_State = "default";
             if (running)
                 Start();
+            m_SaveState = true;
             PostEvent(new EventParams("state_entry",
                     new Object[0], new DetectParams[0]));
         }
@@ -607,6 +626,7 @@ namespace OpenSim.Region.ScriptEngine.Shared.Instance
             async.RemoveScript(m_LocalID, m_ItemID);
             if (m_CurrentEvent != "state_entry")
             {
+                m_SaveState = true;
                 PostEvent(new EventParams("state_entry",
                         new Object[0], new DetectParams[0]));
             }
@@ -640,6 +660,14 @@ namespace OpenSim.Region.ScriptEngine.Shared.Instance
 
         public void SaveState(string assembly)
         {
+            // If we're currently in an event, just tell it to save upon return
+            //
+            if(m_InEvent)
+            {
+                m_SaveState = true;
+                return;
+            }
+
             AsyncCommandManager async = (AsyncCommandManager)m_Engine.AsyncCommands;
             PluginData = async.GetSerializationData(m_ItemID);
 
diff --git a/OpenSim/Region/ScriptEngine/Shared/Instance/ScriptSerializer.cs b/OpenSim/Region/ScriptEngine/Shared/Instance/ScriptSerializer.cs
index bc9b174..a794f02 100644
--- a/OpenSim/Region/ScriptEngine/Shared/Instance/ScriptSerializer.cs
+++ b/OpenSim/Region/ScriptEngine/Shared/Instance/ScriptSerializer.cs
@@ -56,9 +56,6 @@ namespace OpenSim.Region.ScriptEngine.Shared.Instance
         {
             bool running = instance.Running;
 
-            if (running)
-                instance.Stop(50);
-
             XmlDocument xmldoc = new XmlDocument();
 
             XmlNode xmlnode = xmldoc.CreateNode(XmlNodeType.XmlDeclaration,
@@ -182,9 +179,6 @@ namespace OpenSim.Region.ScriptEngine.Shared.Instance
 
             rootElement.AppendChild(plugins);
 
-            if (running)
-                instance.Start();
-
             return xmldoc.InnerXml;
         }