Fixing record list.

plugin/openmeetings/lib/openmeetings.class.php

@@ -423,24 +423,18 @@ class OpenMeetings
                 $meetingDb['closed_at'] = (!empty($meetingDb['closed_at']) ? api_get_local_time($meetingDb['closed_at']):'');
                 // Fixed value for now
                 $meetingDb['participantCount'] = 40;
-                $rec = $this->gateway->getFlvRecordingsByRoomId($meetingDb['room_id']);
-                $recordings = array();
+                $rec = $this->gateway->getFlvRecordingByRoomId($meetingDb['room_id']);
                 $links = array();
-                $i = 0;
                 // Links to videos look like these:
                 // http://video2.openmeetings.com:5080/openmeetings/DownloadHandler?fileName=flvRecording_4.avi&moduleName=lzRecorderApp&parentPath=&room_id=&sid=dfc0cac396d384f59242aa66e5a9bbdd
                 $link = $this->url.'/DownloadHandler?fileName=%s&moduleName=lzRecorderApp&parentPath=&room_id=%s&sid=%s';
                 if (!empty($rec)) {
-                    foreach ($rec as $info) {
-                        $recordings[$i]['filename'] = $info['fileHash'];
-                        $recordings[$i]['image'] = $info['previewImage'];
-                        $recordings[$i]['link1'] = sprintf($link, $recordings[$i]['filename'], $meetingDb['room_id'], $this->sessionId);
-                        $recordings[$i]['link2'] = sprintf($link, $info['alternateDownload'], $meetingDb['room_id'], $this->sessionId);
-                        $recordings[$i]['end'] = $info['recordEnd'];
-                        $links[] = $info['fileName'].' '.
-                            \Display::url('[1]', $recordings[$i]['link1'], array('target' => '_blank')).' '.
-                            \Display::url('[2]', $recordings[$i]['link2'], array('target' => '_blank'));
-                    }
+                    $link1 = sprintf($link, $rec['filename'], $meetingDb['room_id'], $this->sessionId);
+                    $link2 = sprintf($link, $rec['alternateDownload'], $meetingDb['room_id'], $this->sessionId);
+                    $links[] = $rec['fileName'].' '.
+                        \Display::url('[1]', $link1, array('target' => '_blank')).' '.
+                        \Display::url('[2]', $link2, array('target' => '_blank'));
+
                 }
                 $item['show_links']  = implode('<br />', $links);
 

plugin/openmeetings/lib/openmeetings_gateway.php

@@ -441,11 +441,11 @@ class OpenMeetingsGateway
     }
 
     /**
-     * Get list of available recordings made for the given room
+     * Get the recording from the room
      * @param   int $id Room ID
      * @return array
      */
-    public function getFlvRecordingsByRoomId($id)
+    public function getFlvRecordingByRoomId($id)
     {
         $url = $this->getRestUrl("RoomService")
             . "getFlvRecordingByRoomId?"

plugin/openmeetings/listing.php

@@ -79,17 +79,19 @@ if ($teacher) {
 }
 
 $meetings = $om->getCourseMeetings();
+$openMeeting = false;
+$users_online = 0;
+
 if (!empty($meetings)) {
     $meetings = array_reverse($meetings);
-}
-$openMeeting = false;
-foreach ($meetings as $meeting) {
-    if ($meeting['status'] == 1) {
-        $openMeeting = true;
+    foreach ($meetings as $meeting) {
+        if ($meeting['status'] == 1) {
+            $openMeeting = true;
+        }
     }
+    $users_online = $meetings->participantCount;
 }
 
-$users_online = $meetings->participantCount;
 $status = $om->isServerRunning();
 $show_join_button = false;
 if ($openMeeting || $teacher) {
@@ -103,8 +105,6 @@ $tpl->assign('conference_url', $conference_url);
 $tpl->assign('users_online', $users_online);
 $tpl->assign('openmeetings_status', $status);
 $tpl->assign('show_join_button', $show_join_button);
-
-//$tpl->assign('actions', $actions);
 $tpl->assign('message', $message);
 $listing_tpl = 'openmeetings/listing.tpl';
 $content = $tpl->fetch($listing_tpl);