Completed
Push — master ( 541b76...aec89b )
by Henry
05:33
created

includes/Bootstrap/Session.php (1 issue)

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

1
<?php
2
namespace Redaxscript\Bootstrap;
3
4
/**
5
 * children class to boot the session
6
 *
7
 * @since 3.1.0
8
 *
9
 * @package Redaxscript
10
 * @category Bootstrap
11
 * @author Henry Ruhs
12
 */
13
14
class Session extends BootstrapAbstract
15
{
16
	/**
17
	 * automate run
18
	 *
19
	 * @since 3.1.0
20
	 */
21
22 1
	protected function _autorun()
23
	{
24 1
		session_start();
25 1
		$this->_request->refreshSession();
26
27
		/* prevent hijacking */
28
29 1
		if (!$this->_request->getSession('regenerateId'))
30
		{
31 1
			$this->_request->setSession('regenerateId', session_regenerate_id());
0 ignored issues
show
session_regenerate_id() is of type boolean, but the function expects a string|array|null.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
32
		}
33 1
		$this->_registry->set('sessionStatus', session_status());
34 1
	}
35
}
36