Sweep pending podcast episodes after an hour; css tweaks to My Podcast page

airtime_mvc/application/services/MediaService.php

@@ -133,17 +133,23 @@ class Application_Service_MediaService
             ->filterByDbImportStatus(CcFiles::IMPORT_STATUS_PENDING)
             ->filterByDbUtime($oneHourAgo, Criteria::LESS_EQUAL)
             ->find();
-        return !empty(self::$_pendingFiles);
+        $pendingEpisodes = Application_Service_PodcastEpisodeService::getStuckPendingImports();
+        return !empty(self::$_pendingFiles) && !empty($pendingEpisodes);
     }
 
     /**
      * Clean up stuck imports by changing their import status to Failed
      */
     public static function clearStuckPendingImports() {
-        foreach(self::$_pendingFiles as $file) {
+        $pendingEpisodes = Application_Service_PodcastEpisodeService::getStuckPendingImports();
+        foreach (self::$_pendingFiles as $file) {
             /** @var $file CcFiles */
             $file->setDbImportStatus(CcFiles::IMPORT_STATUS_FAILED)->save();
         }
+        foreach ($pendingEpisodes as $episode) {
+            /** @var $episode PodcastEpisodes */
+            $episode->delete();
+        }
     }
 
 }

airtime_mvc/application/services/PodcastEpisodeService.php

@@ -12,6 +12,8 @@ class Application_Service_PodcastEpisodeService extends Application_Service_Thir
 
     const DOWNLOAD = 'download';
 
+    const PENDING_EPISODE_TIMEOUT_SECONDS = 3600;
+
     /**
      * @var string service name to store in ThirdPartyTrackReferences database
      */
@@ -143,7 +145,11 @@ class Application_Service_PodcastEpisodeService extends Application_Service_Thir
             'callback_url'  => Application_Common_HTTPHelper::getStationUrl() . '/rest/media',
             'api_key'       => $apiKey = $CC_CONFIG["apiKey"][0],
         );
-        $this->_executeTask(static::$_CELERY_TASKS[self::DOWNLOAD], $data);
+        $task = $this->_executeTask(static::$_CELERY_TASKS[self::DOWNLOAD], $data);
+        // Get the created ThirdPartyTaskReference and set the episode ID so
+        // we can remove the placeholder if the import ends up stuck in a pending state
+        $ref = ThirdPartyTrackReferencesQuery::create()->findPk($task->getDbTrackReference());
+        $ref->setDbForeignId($id)->save();
     }
 
     /**
@@ -161,6 +167,7 @@ class Application_Service_PodcastEpisodeService extends Application_Service_Thir
      */
     public function updateTrackReference($task, $episodeId, $episode, $status) {
         $ref = parent::updateTrackReference($task, $episodeId, $episode, $status);
+        $ref->setDbForeignId($episode->episodeid)->save();
         $dbEpisode = PodcastEpisodesQuery::create()->findOneByDbId($episode->episodeid);
 
         try {
@@ -231,6 +238,32 @@ class Application_Service_PodcastEpisodeService extends Application_Service_Thir
         return (int) $stationPodcast->hasEpisodeForFile($fileId);
     }
 
+    /**
+     * Find any episode placeholders that have been stuck pending (empty file ID) for over an hour
+     *
+     * @return array the episode imports stuck in pending
+     */
+    public static function getStuckPendingImports() {
+        $oneHourAgo = gmdate(DEFAULT_TIMESTAMP_FORMAT, (microtime(true) - self::PENDING_EPISODE_TIMEOUT_SECONDS));
+        $episodes = PodcastEpisodesQuery::create()
+            ->filterByDbFileId()
+            ->find();
+        $stuckImports = array();
+        foreach ($episodes as $episode) {
+            $ref = ThirdPartyTrackReferencesQuery::create()
+                ->findOneByDbForeignId(strval($episode->getDbId()));
+            if (!empty($ref)) {
+                $task = CeleryTasksQuery::create()
+                    ->filterByDbDispatchTime($oneHourAgo, Criteria::LESS_EQUAL)
+                    ->findOneByDbTrackReference($ref->getDbId());
+                if (!empty($task)) {
+                    array_push($stuckImports, $episode);
+                }
+            }
+        }
+        return $stuckImports;
+    }
+
     /**
      * @param $episodeId
      * @return array

airtime_mvc/application/services/ThirdPartyCeleryService.php

@@ -20,15 +20,20 @@ abstract class Application_Service_ThirdPartyCeleryService extends Application_S
      * @param string $taskName the name of the celery task to execute
      * @param array $data      the data array to send as task parameters
      * @param int $fileId      the unique identifier for the file involved in the task
+     *
+     * @return CeleryTasks the created task
+     *
+     * @throws Exception
      */
     protected function _executeTask($taskName, $data, $fileId = null) {
         try {
             $brokerTaskId = CeleryManager::sendCeleryMessage($taskName,
                                                              static::$_CELERY_EXCHANGE_NAME,
                                                              $data);
-            $this->_createTaskReference($fileId, $brokerTaskId, $taskName);
+            return $this->_createTaskReference($fileId, $brokerTaskId, $taskName);
         } catch (Exception $e) {
-            Logging::info("Invalid request: " . $e->getMessage());
+            Logging::error("Invalid request: " . $e->getMessage());
+            throw $e;
         }
     }
 
@@ -41,6 +46,8 @@ abstract class Application_Service_ThirdPartyCeleryService extends Application_S
      *                             receive completed task messages
      * @param $taskName     string broker task name
      *
+     * @return CeleryTasks the created task
+     *
      * @throws Exception
      * @throws PropelException
      */
@@ -54,6 +61,7 @@ abstract class Application_Service_ThirdPartyCeleryService extends Application_S
         $task->setDbStatus(CELERY_PENDING_STATUS);
         $task->setDbTrackReference($trackReferenceId);
         $task->save();
+        return $task;
     }
 
     /**

airtime_mvc/application/services/ThirdPartyService.php

@@ -30,12 +30,10 @@ abstract class Application_Service_ThirdPartyService {
     public function createTrackReference($fileId) {
         // First, check if the track already has an entry in the database
         // If the file ID given is null, create a new reference
-        $ref = is_null($fileId) ? null : ThirdPartyTrackReferencesQuery::create()
-            ->filterByDbService(static::$_SERVICE_NAME)
-            ->findOneByDbFileId($fileId);
-        if (is_null($ref)) {
-            $ref = new ThirdPartyTrackReferences();
-        }
+        $ref = is_null($fileId) ? new ThirdPartyTrackReferences()
+                                : ThirdPartyTrackReferencesQuery::create()
+                                    ->filterByDbService(static::$_SERVICE_NAME)
+                                    ->findOneByDbFileId($fileId);
         $ref->setDbService(static::$_SERVICE_NAME);
         $ref->setDbFileId($fileId);
         $ref->save();

airtime_mvc/application/views/scripts/podcast/podcast.phtml

@@ -1,5 +1,3 @@
-
-
 <div class="angular_wrapper" ng-controller="Podcast">
     <div class="inner_editor_title">
         <h2>

airtime_mvc/application/views/scripts/podcast/station.phtml

@@ -1,6 +1,6 @@
 <div id="station_podcast">
     <div class="angular_wrapper" ng-controller="StationPodcast">
-        <div class="station_podcast_wrapper">
+        <div class="station_podcast_wrapper content-pane">
             <div class="inner_editor_title">
                 <h2>
                     <span class="title_obj_name"><?php echo _("My Podcast") ?></span>

airtime_mvc/public/css/dashboard.css

@@ -739,7 +739,11 @@ div.ColVis_collectionBackground {
 
 .fg-toolbar .btn-toolbar {
     margin: 0;
-    padding: 6px 6px 5px;
+    padding: 5px;
+}
+
+.fg-toolbar .btn-toolbar .ColVis_MasterButton {
+    margin: 0;
 }
 /* ~~~~~~~~~~~~~~~~ */
 

airtime_mvc/public/css/station_podcast.css

@@ -31,7 +31,7 @@
 }
 
 .station_podcast_wrapper {
-    padding-right: 4px;
+    padding: 8px;
     overflow-x: hidden;
 }