[longomatch] Fix cameras config config after unloading events
- From: Andoni Morales Alastruey <amorales src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [longomatch] Fix cameras config config after unloading events
- Date: Thu, 2 Apr 2015 14:11:33 +0000 (UTC)
commit 354f47233e1b187a24f052ac36fb97b3ebdb774f
Author: Andoni Morales Alastruey <ylatuya gmail com>
Date: Thu Apr 2 15:34:43 2015 +0200
Fix cameras config config after unloading events
LongoMatch.Services/Services/PlayerController.cs | 4 ++--
Tests/Services/TestPlayerController.cs | 12 ++++++++++++
2 files changed, 14 insertions(+), 2 deletions(-)
---
diff --git a/LongoMatch.Services/Services/PlayerController.cs
b/LongoMatch.Services/Services/PlayerController.cs
index 8904455..236947c 100644
--- a/LongoMatch.Services/Services/PlayerController.cs
+++ b/LongoMatch.Services/Services/PlayerController.cs
@@ -484,12 +484,12 @@ namespace LongoMatch.Services
{
Log.Debug ("Unload current event");
Reset ();
- UpdateCamerasConfig (defaultCamerasVisible, defaultCamerasLayout);
EmitEventUnloaded ();
if (FileSet != defaultFileSet) {
+ UpdateCamerasConfig (defaultCamerasVisible, defaultCamerasLayout);
Open (defaultFileSet);
} else {
- ApplyCamerasConfig ();
+ CamerasVisible = defaultCamerasVisible;
}
}
diff --git a/Tests/Services/TestPlayerController.cs b/Tests/Services/TestPlayerController.cs
index 4285995..1d76110 100644
--- a/Tests/Services/TestPlayerController.cs
+++ b/Tests/Services/TestPlayerController.cs
@@ -568,6 +568,18 @@ namespace Tests.Services
Assert.AreEqual (0, elementLoaded);
// Check that cameras have been restored
Assert.AreEqual (new List<int> { 0, 1 }, player.CamerasVisible);
+
+ /* Change again the cameras visible */
+ player.CamerasVisible = new List<int> { 2, 3 };
+ Assert.AreEqual (evt.CamerasVisible, new List <int> { 0 });
+ player.LoadEvent (mfs, evt, evt.Start, true);
+ Assert.AreEqual (1, elementLoaded);
+ Assert.AreEqual (evt.CamerasVisible, player.CamerasVisible);
+ /* And unload */
+ player.UnloadCurrentEvent ();
+ Assert.AreEqual (0, elementLoaded);
+ // Check that cameras have been restored
+ Assert.AreEqual (new List<int> { 2, 3 }, player.CamerasVisible);
}
[Test ()]
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]