Commit 954ac9c1 authored by Patrick Barroca's avatar Patrick Barroca 🐧

Merge branch…

Merge branch 'hotline#94842_en_attente_de_reponse_philharmoniedeparis_a_faire_cette_semaine_28_changement_adresse_sso_philharmonie_a_la_demande' into 'hotline'

hotline #94842 : update philarmoniedeparis.fr SSO

See merge request !3181
parents 080d77eb eff5cd1c
Pipeline #7693 failed with stage
in 31 minutes
- ticket #94842 : Connecteur Cité de la musique : Maintenance SSO
\ No newline at end of file
......@@ -21,6 +21,8 @@
class Class_AdminVar_CiteDeLaMusique {
const SSO_BASE_URL = 'https://admin-pad.philharmoniedeparis.fr/logon/';
/**
* @return bool
*/
......@@ -40,5 +42,18 @@ class Class_AdminVar_CiteDeLaMusique {
public function getSets() {
return array_filter(explode(';', Class_AdminVar::get('CITEDELAMUSIQUE_SETS')));
}
public function getSsoUrl() {
return $this->isEnabled()
? (static::SSO_BASE_URL . $this->getLibraryId())
: '';
}
public function getAlbumSsoUrl($album) {
return $this->isEnabled()
? ($this->getSsoUrl() . '?' . http_build_query(['backUrl' => $album->getExternalUri()]))
: '';
}
}
?>
\ No newline at end of file
......@@ -20,7 +20,6 @@
*/
class Class_Systeme_ModulesMenu_CiteDeLaMusique extends Class_Systeme_ModulesMenu_SSOAbstract {
protected
$_group = Class_Systeme_ModulesMenu::GROUP_MENU_ABONNES,
$_type_module = 'CITE_DE_LA_MUSIQUE',
......@@ -46,7 +45,7 @@ class Class_Systeme_ModulesMenu_CiteDeLaMusique extends Class_Systeme_ModulesMen
if (!$user->hasRightAccessCiteDeLaMusique())
return '';
return 'https://media.citedelamusique.fr/medias/logon/' . $cite_musique->getLibraryId();
return $cite_musique->getSsoUrl();
}
......
......@@ -27,16 +27,12 @@ class ZendAfi_View_Helper_Album_CiteDeLaMusique extends ZendAfi_View_Helper_TagR
return '';
if (!$this->canAccessRessourceNumerique())
return $this->view->tag('p',
$this->view->_('Vous devez être connecté sous un compte avec abonnement valide pour pouvoir accéder à la ressource numérique'));
return $this->_tag('p',
$this->_('Vous devez être connecté sous un compte avec abonnement valide pour pouvoir accéder à la ressource numérique'));
$url = 'http://media.citedelamusique.fr/extranet/autologon/autologon.ashx?'
. http_build_query(['bib' => $cite_musique->getLibraryId(),
'backUrl' => $album->getExternalUri()]);
return $this->view->tagAnchor($url,
$this->view->_('Accéder à la ressource numérique'));
return $this->view->tagAnchor($cite_musique->getAlbumSsoUrl($album),
$this->_('Accéder à la ressource numérique'));
}
......@@ -44,4 +40,3 @@ class ZendAfi_View_Helper_Album_CiteDeLaMusique extends ZendAfi_View_Helper_TagR
return $user->hasRightAccessCiteDeLaMusique();
}
}
?>
\ No newline at end of file
......@@ -145,7 +145,7 @@ class CiteDeLaMusiqueModulesControllerTest extends AbstractControllerTestCase {
public function citeDeLaMusiqueActionShouldRedirectToCiteDeLaMusiqueLogon() {
$this->dispatch('/modules/cite-de-la-musique', true);
$this->assertXPathContentContains('//script',
'document.location.href="https://media.citedelamusique.fr/medias/logon/34-mylib"');
'document.location.href="https://admin-pad.philharmoniedeparis.fr/logon/34-mylib"');
}
......@@ -197,7 +197,7 @@ class CiteDeLaMusiqueRenderAlbumTest extends ViewHelperTestCase {
public function htmlShouldContainsSsoLinkToCiteDeLaMusiqueWhenActivated() {
CiteDeLaMusiqueFixtures::activate();
$expected_url = 'http://media.citedelamusique.fr/extranet/autologon/autologon.ashx?bib=34-mylib&backUrl=http%3A%2F%2Fmedia.citedelamusique.fr%2Fmedias%2Fdoc%2FEXTRANET%2FCIMU%2F1038311%2FDETAIL';
$expected_url = 'https://admin-pad.philharmoniedeparis.fr/logon/34-mylib?backUrl=http%3A%2F%2Fmedia.citedelamusique.fr%2Fmedias%2Fdoc%2FEXTRANET%2FCIMU%2F1038311%2FDETAIL';
$this->assertXPathContentContains($this->_renderAlbum(),
'//a[@href="' . $expected_url . '"]',
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment