From 85687e807e5e87671263c88009660f1637a86a1b Mon Sep 17 00:00:00 2001
From: Ghislain Loas <ghislo@sandbox.pergame.net>
Date: Fri, 22 Jan 2016 14:20:11 +0100
Subject: [PATCH] hotline #35768 fix rt comments

---
 library/ZendAfi/View/Helper/ComboProfils.php                    | 2 +-
 .../modules/opac/controllers/IndexControllerTest.php            | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/library/ZendAfi/View/Helper/ComboProfils.php b/library/ZendAfi/View/Helper/ComboProfils.php
index 9c0799b772c..4e431b4c630 100644
--- a/library/ZendAfi/View/Helper/ComboProfils.php
+++ b/library/ZendAfi/View/Helper/ComboProfils.php
@@ -89,7 +89,7 @@ class ZendAfi_View_Helper_ComboProfils extends ZendAfi_View_Helper_BaseHelper {
 
 
   protected function getProfilUrl() {
-    return  rtrim($this->view->url(['id_profil' => null], null, true), '/') . '/?id_profil=';
+    return  $this->view->url(['id_profil' => ''], null, true);
   }
 
 
diff --git a/tests/application/modules/opac/controllers/IndexControllerTest.php b/tests/application/modules/opac/controllers/IndexControllerTest.php
index 625762d9b68..97291737ffd 100644
--- a/tests/application/modules/opac/controllers/IndexControllerTest.php
+++ b/tests/application/modules/opac/controllers/IndexControllerTest.php
@@ -124,7 +124,7 @@ class IndexControllerAsAdminWithCssEditorAndNoHeaderCss extends IndexControllerA
 
   /** @test */
   public function comboProfilOnChangeShouldContainsBaseUrl() {
-    $this->assertXPath('//div[@class="footer"]//select[@id="select_clef_profil"][@onchange="window.location=\'/?id_profil=\' + this.value"]');
+    $this->assertXPath('//div[@class="footer"]//select[@id="select_clef_profil"][contains(@onchange, "\'/index/index/id_profil/\' + this.value")]');
   }
 
 
-- 
GitLab