From aeb2977830c4b6cf92ba4e3d435ee51038a27b96 Mon Sep 17 00:00:00 2001
From: llaffont <llaffont@afi-sa.fr>
Date: Wed, 23 Jul 2014 16:13:32 +0200
Subject: [PATCH] dev #15293 add data to cas server 1.0 validate
---
.../modules/opac/controllers/CasServerV10Controller.php | 4 +++-
.../modules/opac/controllers/CasServerControllerTest.php | 8 +++++---
2 files changed, 8 insertions(+), 4 deletions(-)
diff --git a/application/modules/opac/controllers/CasServerV10Controller.php b/application/modules/opac/controllers/CasServerV10Controller.php
index 1c451f6781e..bc3a90808df 100644
--- a/application/modules/opac/controllers/CasServerV10Controller.php
+++ b/application/modules/opac/controllers/CasServerV10Controller.php
@@ -21,7 +21,9 @@
class CasServerV10Controller extends CasServerController {
// see http://www.jasig.org/cas/protocol#validate-cas-1.0
public function returnValidTicketResponse($user, $ticket) {
- $this->getResponse()->setBody('yes'.chr(10));
+ $this->getResponse()->setBody(
+ 'yes'.chr(10).$user->getNomAff().'|'.$user->getLogin().chr(10)
+ );
}
diff --git a/tests/application/modules/opac/controllers/CasServerControllerTest.php b/tests/application/modules/opac/controllers/CasServerControllerTest.php
index 055acafccef..986a49592c8 100644
--- a/tests/application/modules/opac/controllers/CasServerControllerTest.php
+++ b/tests/application/modules/opac/controllers/CasServerControllerTest.php
@@ -28,7 +28,9 @@ class CasServerControllerValidateActionTest extends AbstractControllerTestCase {
parent::setUp();
$user = new StdClass();
$user->ID_USER=300;
- Class_Users::newInstanceWithId(300);
+ Class_Users::newInstanceWithId(300,
+ ['login' => '87364',
+ 'pseudo' => 'georges']);
Zend_Registry::get('cache')->save('300',
md5(Zend_Session::getId().'300'));
}
@@ -82,11 +84,11 @@ class CasServerControllerValidateActionTest extends AbstractControllerTestCase {
* see http://www.jasig.org/cas/protocol#validate-cas-1.0
* @test
*/
- public function validateOnCasOneZeroWithValidTicketShouldAnswerYesLF() {
+ public function validateOnCasOneZeroWithValidTicketShouldAnswerYesLFUsernameLogin() {
$this->dispatch(
'/opac/cas-server-v10/validate?ticket=ST-'.md5(Zend_Session::getId().'300').'&service=http://test.com',
true);
- $this->assertEquals('yes'.chr(10), $this->_response->getBody());
+ $this->assertEquals('yes'.chr(10).'georges|87364'.chr(10), $this->_response->getBody());
}
--
GitLab