WeSign/unity-application
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 6 Wiki Activity

Merge branch 'WES-XX-persistent-data-controller-backwards-compatible' into 'development'

Browse Source 
Fix bug

See merge request wesign/unity-application!80
This commit was merged in pull request #80.
This commit is contained in:
Dries Van Schuylenbergh
2023-04-08 19:24:46 +00:00
parent 8670fcc4ce f5615bbef3
commit 73c4756f19
2 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
Assets/SystemArchitecture/Scripts/PersistentDataController.cs
View File

@@ -277,7 +277,7 @@ public class PersistentDataController
{
string text = File.ReadAllText(PATH);
SavedDataStructure newJson = JsonUtility.FromJson<SavedDataStructure>(text);
if (newJson == null || newJson.version < VERSION)
if (newJson == null || newJson.version != VERSION)
goto failed;
json = newJson;

8
Assets/SystemArchitecture/Tests/PersistentDataControllerTest.cs
View File

@@ -107,6 +107,14 @@ public class PersistentDataTests
Assert.IsFalse(pdc.Load(false));
}
[Test]
public void Test_PersistentDataController_Load_NewerVersion()
{
string content = $"{{\"version\":{PersistentDataController.VERSION + 1},\"users\":[],\"currentUser\":-1,\"currentMinigame\":0,\"currentCourse\":0,\"currentTheme\":0}}";
File.WriteAllText(PATH, content);
Assert.IsFalse(pdc.Load(false));
}
[Test]
public void Test_PersistentDataController_Load_New()
{