From c9c9ecf2cd96c278825427d1115a524281dd7873 Mon Sep 17 00:00:00 2001
From: martin <martin@martin-ThinkPad-T410.(none)>
Date: Fri, 3 Jun 2011 16:36:13 -0400
Subject: [PATCH] cc-2344: wrong song displayed

-fixed
---
 airtime_mvc/application/models/Dashboard.php | 18 +++++++++++-------
 airtime_mvc/application/models/Schedule.php  | 17 ++++++++++++++---
 2 files changed, 25 insertions(+), 10 deletions(-)

diff --git a/airtime_mvc/application/models/Dashboard.php b/airtime_mvc/application/models/Dashboard.php
index f7d5113b2..d599b5c2b 100644
--- a/airtime_mvc/application/models/Dashboard.php
+++ b/airtime_mvc/application/models/Dashboard.php
@@ -10,14 +10,16 @@ class Application_Model_Dashboard
         //name. Else return the last item from the schedule.
 
         $showInstance = ShowInstance::GetLastShowInstance($p_timeNow);
-        $row = Schedule::GetLastScheduleItem($p_timeNow);
+        $instanceId = $showInstance->getShowId();
+        $row = Schedule::GetLastScheduleItem($p_timeNow, $instanceId);
 
         if (is_null($showInstance)){
             if (count($row) == 0){
                 return null;
             } else {
-                //should never reach here. Doesnt make sense to have
-                //a schedule item not within a show_instance.
+                return array("name"=>$row[0]["artist_name"]." - ".$row[0]["track_title"],
+                    "starts"=>$row[0]["starts"],
+                    "ends"=>$row[0]["ends"]);
             }
         } else {
             if (count($row) == 0){
@@ -47,7 +49,8 @@ class Application_Model_Dashboard
         //name. Else return the last item from the schedule.
 
         $showInstance = ShowInstance::GetCurrentShowInstance($p_timeNow);
-        $row = Schedule::GetCurrentScheduleItem($p_timeNow);
+        $instanceId = $showInstance->getShowId();
+        $row = Schedule::GetCurrentScheduleItem($p_timeNow, $instanceId);
 
         if (is_null($showInstance)){
             if (count($row) == 0){
@@ -62,8 +65,8 @@ class Application_Model_Dashboard
                 return array("name"=>$showInstance->getName(),
                             "starts"=>$showInstance->getShowStart(),
                             "ends"=>$showInstance->getShowEnd(),
-                            "media_item_played"=>false, //TODO
-                            "record"=>$showInstance->isRecorded()); //TODO
+                            "media_item_played"=>false,
+                            "record"=>$showInstance->isRecorded());
             } else {
                  return array("name"=>$row[0]["artist_name"]." - ".$row[0]["track_title"],
                         "starts"=>$row[0]["starts"],
@@ -81,7 +84,8 @@ class Application_Model_Dashboard
         //name. Else return the last item from the schedule.
 
         $showInstance = ShowInstance::GetNextShowInstance($p_timeNow);
-        $row = Schedule::GetNextScheduleItem($p_timeNow);
+        $instanceId = $showInstance->getShowId();
+        $row = Schedule::GetNextScheduleItem($p_timeNow, $instanceId);
 
         if (is_null($showInstance)){
             if (count($row) == 0){
diff --git a/airtime_mvc/application/models/Schedule.php b/airtime_mvc/application/models/Schedule.php
index 812845062..71a6d9674 100644
--- a/airtime_mvc/application/models/Schedule.php
+++ b/airtime_mvc/application/models/Schedule.php
@@ -378,7 +378,7 @@ class Schedule {
             "apiKey"=>$CC_CONFIG['apiKey'][0]);
     }
 
-    public static function GetLastScheduleItem($p_timeNow){
+    public static function GetLastScheduleItem($p_timeNow, $p_instanceId){
         global $CC_CONFIG, $CC_DBC;
 
         $sql = "SELECT *"
@@ -386,6 +386,7 @@ class Schedule {
         ." LEFT JOIN $CC_CONFIG[filesTable] ft"
         ." ON st.file_id = ft.id"
         ." WHERE st.ends < TIMESTAMP '$p_timeNow'"
+        ." AND st.instance_id = $p_instanceId"
         ." ORDER BY st.ends DESC"
         ." LIMIT 1";
 
@@ -394,22 +395,31 @@ class Schedule {
     }
 
 
-    public static function GetCurrentScheduleItem($p_timeNow){
+    public static function GetCurrentScheduleItem($p_timeNow, $p_instanceId){
         global $CC_CONFIG, $CC_DBC;
 
+        /* Note that usually there will be one result returned. In some
+ 	 	 * rare cases two songs are returned. This happens when a track
+ 	 	 * that was overbooked from a previous show appears as if it
+ 	 	 * hasnt ended yet (track end time hasn't been reached yet). For
+ 	 	 * this reason,  we need to get the track that starts later, as
+ 	 	 * this is the *real* track that is currently playing. So this
+ 	 	 * is why we are ordering by track start time. */
         $sql = "SELECT *"
         ." FROM $CC_CONFIG[scheduleTable] st"
         ." LEFT JOIN $CC_CONFIG[filesTable] ft"
         ." ON st.file_id = ft.id"
         ." WHERE st.starts <= TIMESTAMP '$p_timeNow'"
+        ." AND st.instance_id = $p_instanceId"
         ." AND st.ends > TIMESTAMP '$p_timeNow'"
+        ." ORDER BY st.starts DESC"
         ." LIMIT 1";
 
         $row = $CC_DBC->GetAll($sql);
         return $row;
     }
 
-    public static function GetNextScheduleItem($p_timeNow){
+    public static function GetNextScheduleItem($p_timeNow, $p_instanceId){
         global $CC_CONFIG, $CC_DBC;
 
         $sql = "SELECT *"
@@ -417,6 +427,7 @@ class Schedule {
         ." LEFT JOIN $CC_CONFIG[filesTable] ft"
         ." ON st.file_id = ft.id"
         ." WHERE st.starts > TIMESTAMP '$p_timeNow'"
+        ." AND st.instance_id = $p_instanceId"
         ." ORDER BY st.starts"
         ." LIMIT 1";