Correction accès RSS profil supprimé

application/modules/opac/controllers/RssController.php

@@ -353,15 +353,19 @@ class RssController extends Zend_Controller_Action
 	function kiosqueAction() {
 		$id_profil = (int)$this->_request->getParam('id_profil');
 		$id_module = (int)$this->_request->getParam('id_module');
-		$profil = Class_Profil::getLoader()->find($id_profil);
-		$preferences = $profil->getModuleAccueilPreferences($id_module);
+		if (!$profil = Class_Profil::getLoader()->find($id_profil)) {
+			$notices = [];
+			$titre = 'Non trouvé';
+		} else {
+			$preferences = $profil->getModuleAccueilPreferences($id_module);
+			$catalogue=new Class_Catalogue();
+			$preferences["aleatoire"] = 0; // les dernières seulement
+			$notices = $catalogue->getNoticesByPreferences($preferences,"url");
+			$titre = $preferences['titre'];
+		}
 
-		$catalogue=new Class_Catalogue();
-		$preferences["aleatoire"] = 0; // les dernières seulement
-		$notices=$catalogue->getNoticesByPreferences($preferences,"url");
 
 		$entries = array();
-
 		foreach($notices as $notice) {
 			$entries []= array(
 												 'title'       => $notice["titre"].', '.$notice["auteur"],
@@ -370,10 +374,10 @@ class RssController extends Zend_Controller_Action
 												 'lastUpdate'	 => strtotime($notice['date_creation']));
 		}
 		$rss_array = array(
-											'title' 	=> $preferences['titre'],
+											'title' 	=> $titre,
 											'link'  	=> 'http://' . $_SERVER['SERVER_NAME'].BASE_URL,
 											'charset'	  => 'utf-8',
-											'description' => $preferences['titre'],
+											'description' => $titre,
 											'lastUpdate'  => time(),
 											'entries' => $entries);
 

tests/application/modules/opac/controllers/RssControllerTest.php

@@ -26,6 +26,9 @@ class MockZendHttpClient extends Zend_Http_Client {
 	}
 }
 
+
+
+
 class RssControllerViewRawRssTest extends AbstractControllerTestCase {
 	public function setUp() {
 		parent::setUp();
@@ -97,6 +100,23 @@ class RssControllerViewRawRssTest extends AbstractControllerTestCase {
 
 
 
+
+class RssControllerKiosqueInexistingProfilTest extends AbstractControllerTestCase {
+	public function setUp() {
+		parent::setUp();
+		$this->dispatch('/rss/kiosque/id_profil/-1/id_module/12', true);
+	}
+
+	
+	/** @test */
+	public function rssShouldBeEmpty() {
+		$this->assertNotXPath('//item');
+	}
+}
+
+
+
+
 class RssFixtures {
 	public static function lemondeRSS() {
 		return