From dd883aaf0999e9dc783a1a19bfeeab181c949d55 Mon Sep 17 00:00:00 2001 From: VirtualTam Date: Wed, 8 Nov 2017 20:24:49 +0100 Subject: Improve SessionManager constructor and tests Relates to https://github.com/shaarli/Shaarli/pull/1005 Changed: - pass a copy of the ConfigManager instance instead of a reference - move FakeConfigManager to a dedicated file - update tests Signed-off-by: VirtualTam --- application/SessionManager.php | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'application') diff --git a/application/SessionManager.php b/application/SessionManager.php index 3aa4ddfc..71f0b38d 100644 --- a/application/SessionManager.php +++ b/application/SessionManager.php @@ -12,12 +12,12 @@ class SessionManager * Constructor * * @param array $session The $_SESSION array (reference) - * @param ConfigManager $conf ConfigManager instance (reference) + * @param ConfigManager $conf ConfigManager instance */ - public function __construct(& $session, & $conf) + public function __construct(& $session, $conf) { $this->session = &$session; - $this->conf = &$conf; + $this->conf = $conf; } /** -- cgit v1.2.3