Knah-Tsaeb/MyShaarli
Knah-Tsaeb/MyShaarli
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Fix compatiliby issue on login with PHP 7.1

Browse source 
session_set_cookie_params does not return any value in PHP 7.1
This commit is contained in:
ArthurHoaro 2020-10-28 14:02:08 +01:00
parent 156061d445
commit d3f6d52525
2 changed files with 6 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
application/render/PageBuilder.php
View file

@ -160,7 +160,7 @@ private function initialize()
$this->tpl->assign('formatter', $this->conf->get('formatter', 'default')); $this->tpl->assign('formatter', $this->conf->get('formatter', 'default'));
$this->tpl->assign('links_per_page', $this->session['LINKS_PER_PAGE']); $this->tpl->assign('links_per_page', $this->session['LINKS_PER_PAGE'] ?? 20);
// To be removed with a proper theme configuration. // To be removed with a proper theme configuration.
$this->tpl->assign('conf', $this->conf); $this->tpl->assign('conf', $this->conf);

7
application/security/SessionManager.php
View file

@ -293,9 +293,12 @@ public function start(): bool
return session_start(); return session_start();
} }
public function cookieParameters(int $lifeTime, string $path, string $domain): bool /**
* Be careful, return type of session_set_cookie_params() changed between PHP 7.1 and 7.2.
*/
public function cookieParameters(int $lifeTime, string $path, string $domain): void
{ {
return session_set_cookie_params($lifeTime, $path, $domain); session_set_cookie_params($lifeTime, $path, $domain);
} }
public function regenerateId(bool $deleteOldSession = false): bool public function regenerateId(bool $deleteOldSession = false): bool