diff options
author | VirtualTam <virtualtam@flibidi.net> | 2017-11-08 20:24:49 +0100 |
---|---|---|
committer | VirtualTam <virtualtam@flibidi.net> | 2017-11-08 20:26:03 +0100 |
commit | dd883aaf0999e9dc783a1a19bfeeab181c949d55 (patch) | |
tree | 8cd606b71d846c65cf1b8ad4b577c86c0171987e /application | |
parent | b14d34d2c7b7a7360adcc823472a43f64b579e1c (diff) | |
download | Shaarli-dd883aaf0999e9dc783a1a19bfeeab181c949d55.tar.gz Shaarli-dd883aaf0999e9dc783a1a19bfeeab181c949d55.tar.zst Shaarli-dd883aaf0999e9dc783a1a19bfeeab181c949d55.zip |
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 <virtualtam@flibidi.net>
Diffstat (limited to 'application')
-rw-r--r-- | application/SessionManager.php | 6 |
1 files changed, 3 insertions, 3 deletions
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 | |||
12 | * Constructor | 12 | * Constructor |
13 | * | 13 | * |
14 | * @param array $session The $_SESSION array (reference) | 14 | * @param array $session The $_SESSION array (reference) |
15 | * @param ConfigManager $conf ConfigManager instance (reference) | 15 | * @param ConfigManager $conf ConfigManager instance |
16 | */ | 16 | */ |
17 | public function __construct(& $session, & $conf) | 17 | public function __construct(& $session, $conf) |
18 | { | 18 | { |
19 | $this->session = &$session; | 19 | $this->session = &$session; |
20 | $this->conf = &$conf; | 20 | $this->conf = $conf; |
21 | } | 21 | } |
22 | 22 | ||
23 | /** | 23 | /** |