* Wrap station podcast feed contents on radio page in displayRssTab check

* Test fix for automatic ingest bug
* Move disable layout code in feeds controller to avoid error case

airtime_mvc/application/common/PodcastManager.php

@@ -51,12 +51,11 @@ class PodcastManager {
         $episodeList = $podcastArray["episodes"];
         $episodes = array();
         // Sort the episodes by publication date to get the most recent
-        usort($episodeList, array(static::class, "_sortByEpisodePubDate"));
+        // usort($episodeList, array(static::class, "_sortByEpisodePubDate"));
         for ($i = 0; $i < sizeof($episodeList); $i++) {
             $episodeData = $episodeList[$i];
             // If the publication date of this episode is before the ingest timestamp, we don't need to ingest it
-            // Since we're sorting by publication date, we can break
+            if (strtotime($episodeData["pub_date"]) < strtotime($podcast->getDbAutoIngestTimestamp())) continue;
-            if (strtotime($episodeData["pub_date"]) < strtotime($podcast->getDbAutoIngestTimestamp())) break;
             $episode = PodcastEpisodesQuery::create()->findOneByDbEpisodeGuid($episodeData["guid"]);
             // Make sure there's no existing episode placeholder or import, and that the data is non-empty
             if (empty($episode) && !empty($episodeData)) {

airtime_mvc/application/controllers/FeedsController.php

@@ -4,14 +4,15 @@ class FeedsController extends Zend_Controller_Action
 {
     public function stationRssAction()
     {
+        $this->view->layout()->disableLayout();
+        $this->_helper->viewRenderer->setNoRender(true);
+
         if (Application_Model_Preference::getStationPodcastPrivacy()
             && $this->getRequest()->getParam("sharing_token") != Application_Model_Preference::getStationPodcastDownloadKey()) {
             $this->getResponse()
                 ->setHttpResponseCode(401);
             return;
         }
-        $this->view->layout()->disableLayout();
-        $this->_helper->viewRenderer->setNoRender(true);
 
         header('Content-Type: text/xml');
 

airtime_mvc/application/controllers/IndexController.php

@@ -46,6 +46,7 @@ class IndexController extends Zend_Controller_Action
         $episodes = $podcastEpisodesService->getPodcastEpisodes($stationPodcastId);
         $this->view->episodes = json_encode($episodes);
 
+        $this->view->displayRssTab = (!Application_Model_Preference::getStationPodcastPrivacy());
     }
 
     public function mainAction()

airtime_mvc/application/views/scripts/index/index.phtml

@@ -52,6 +52,8 @@ document.getElementById(id).width= (newwidth) + "px";
         </form>
     </div>
 
+    <?php
+        if ($this->displayRssTab) {?>
     <div id="tab-4" class="station_rss tab_content">
         <script type="text/javascript">
             $.each(<?php echo $this->episodes ?>, function(index, value){
@@ -70,6 +72,9 @@ document.getElementById(id).width= (newwidth) + "px";
             });
         </script>
     </div>
+    <?php
+        }
+    ?>
 
     <iframe id="player_iframe" frameborder="0" width="100%" style="bottom:0px; left:0px; position:fixed; right:0px;" src=<?php echo $this->stationUrl."embed/player?stream=auto&style=premium";?>></iframe>
 
@@ -79,6 +84,7 @@ document.getElementById(id).width= (newwidth) + "px";
     $(document).ready(function(){
 
         $("#player_iframe").load(function () {
+            // FIXME: Should we hide this somehow when the station is set to private?
             $("#player_iframe").contents().find('.bottom_bar').append("<div class='station_rss_btn button' data-tab='tab-4'></div>");
             $("#player_iframe").contents().find('.bottom_bar').append("<div class='about_us_btn button' data-tab='tab-2'></div>");
             $("#player_iframe").contents().find('.bottom_bar').append("<div class='schedule_btn button current' data-tab='tab-1'></div>");