Skip to content

Commit

Permalink
Fix for the script reset that was occuring exactly once on region sta…
Browse files Browse the repository at this point in the history
…rtup. If you are loading script state from a region that was not previously running experiences a variable would be null and the null needed to be tested for when parsing a UUID. After this initial load the experience entry is saved and exists on future restarts. Also bumped the version number to 9037
  • Loading branch information
mdickson committed Sep 28, 2024
1 parent bb19c86 commit 0281a9f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion OpenSim/Framework/VersionInfo.cs
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ public class VersionInfo
{
public const string VersionNumber = "0.9.3";
public const string AssemblyVersionNumber = "0.9.3";
public const string Release = "9032";
public const string Release = "9037";

public static string Version
{
Expand Down
3 changes: 2 additions & 1 deletion OpenSim/Region/ScriptEngine/YEngine/XMRInstCtor.cs
Original file line number Diff line number Diff line change
Expand Up @@ -545,7 +545,8 @@ private void LoadScriptState(XmlDocument doc)
}

XmlElement experienceKey = (XmlElement)scriptStateN.SelectSingleNode("ExperienceKey");
m_Item.ExperienceID = UUID.Parse(experienceKey.InnerText);
if (experienceKey is not null)
m_Item.ExperienceID = UUID.Parse(experienceKey.InnerText);

XmlElement permissionsN = (XmlElement)scriptStateN.SelectSingleNode("Permissions");
m_Item.PermsGranter = new UUID(permissionsN.GetAttribute("granter"));
Expand Down

0 comments on commit 0281a9f

Please sign in to comment.