From 1e297f021eb2961f4e538417307832913b404928 Mon Sep 17 00:00:00 2001
From: efalcy <efalcy@afi-sa.fr>
Date: Thu, 4 Jun 2015 16:24:52 +0200
Subject: [PATCH] hotline #25724: forward viewselection on Rss link

---
 application/modules/opac/controllers/CmsController.php   | 8 +++++++-
 .../modules/opac/controllers/CmsControllerTest.php       | 9 +++++++++
 2 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/application/modules/opac/controllers/CmsController.php b/application/modules/opac/controllers/CmsController.php
index 7bffbe12996..adeab4cf474 100644
--- a/application/modules/opac/controllers/CmsController.php
+++ b/application/modules/opac/controllers/CmsController.php
@@ -200,13 +200,19 @@ class CmsController extends Zend_Controller_Action {
 	}
 
 
-	/**
+  /**
 	 * @see ZendAfi_View_Helper_Accueil_MenuVertical
 	 */
 	public function viewsummaryAction() {
 		$this->_viewArticlesByPreferences($this->_getAllParams());
 	}
 
+	/*
+	 *  prevent rss broken links, tix #25724
+	 */
+	public function viewselectionAction() {
+		$this->_forward('articleviewselection');
+	}
 
 	/**
 	 * @see ZendAfi_View_Helper_Accueil_News
diff --git a/tests/application/modules/opac/controllers/CmsControllerTest.php b/tests/application/modules/opac/controllers/CmsControllerTest.php
index c41e13cf516..04ae42f332a 100644
--- a/tests/application/modules/opac/controllers/CmsControllerTest.php
+++ b/tests/application/modules/opac/controllers/CmsControllerTest.php
@@ -1333,6 +1333,7 @@ class CmsControllerViewSummaryTest extends CmsControllerListTestCase {
 
 
 
+
 class CmsControllerArticleViewSelectionTest extends CmsControllerListTestCase {
 	protected function _dispatchHook() {
 		$this->dispatch('/cms/articleviewselection');
@@ -1359,6 +1360,14 @@ class CmsControllerArticleViewSelectionTest extends CmsControllerListTestCase {
 
 
 
+class CmsControllerArticleViewSelectionRssBrokenLinkForwardTest extends CmsControllerArticleViewSelectionTest {
+	// forward to prevent broken link on rss, tix #25724
+	protected function _dispatchHook() {
+		$this->dispatch('/cms/viewselection');
+	}
+
+}
+
 
 class CmsControllerArticleViewSelectionFromHorizontalMenuBoxTest extends AbstractControllerTestCase {
 	public function setUp() {
-- 
GitLab