Merge pull request #140 from ahmed-shariff/fix/test_tracker_data_count

jackbrookes · web-flow · commit 06b9d2601178 · 2022-08-17T19:11:33.000+01:00
Avoid writing empty UXFDataTables
diff --git a/Assets/UXF/Scripts/Etc/Trial.cs b/Assets/UXF/Scripts/Etc/Trial.cs
@@ -264,7 +264,10 @@ private void SaveData()
                 try
                 {
                     tracker.StopRecording();
-                    SaveDataTable(tracker.Data, tracker.DataName, dataType: UXFDataType.Trackers);
+                    if (tracker.Data.CountRows() > 0)
+                    {
+                        SaveDataTable(tracker.Data, tracker.DataName, dataType: UXFDataType.Trackers);
+                    }
                 }
                 catch (NullReferenceException)
                 {

Original file line number	Diff line number	Diff line change
`@@ -264,7 +264,10 @@ private void SaveData()`
`264`	`264`	`try`
`265`	`265`	`{`
`266`	`266`	`tracker.StopRecording();`
`267`		`- SaveDataTable(tracker.Data, tracker.DataName, dataType: UXFDataType.Trackers);`
	`267`	`+ if (tracker.Data.CountRows() > 0)`
	`268`	`+ {`
	`269`	`+ SaveDataTable(tracker.Data, tracker.DataName, dataType: UXFDataType.Trackers);`
	`270`	`+ }`
`268`	`271`	`}`
`269`	`272`	`catch (NullReferenceException)`
`270`	`273`	`{`