joshtronic/pickles
1
0
Fork 0
Code Issues 8 Pull requests Projects Releases Packages Wiki Activity Actions

Removed session_start in controller, removed exception from Security class

Browse source 
Security class doesn't need to bark so much, just fail and move on
This commit is contained in:
Josh Sherman 2013-02-20 11:38:16 -05:00
parent b4eb28f945
commit 650c16efae
3 changed files with 0 additions and 26 deletions
Download patch file Download diff file Expand all files Collapse all files

12
classes/Controller.php
View file

@ -148,18 +148,6 @@ class Controller extends Object
exit;
}
// Establishes the session
if (ini_get('session.auto_start') == 0)
{
if ($module->session)
{
if (session_id() == '')
{
session_start();
}
}
}
// Validates security level
if ($module->security !== false)
{

1
classes/Security.php
View file

@ -130,7 +130,6 @@ class Security
{
if (session_id() == '')
{
throw new Exception('Sessions must be enabled to use the Security class');
return false;
}
else

13
jar.php
View file

@ -1283,18 +1283,6 @@ class Controller extends Object
exit;
}
// Establishes the session
if (ini_get('session.auto_start') == 0)
{
if ($module->session)
{
if (session_id() == '')
{
session_start();
}
}
}
// Validates security level
if ($module->security !== false)
{
@ -7093,7 +7081,6 @@ class Security
{
if (session_id() == '')
{
throw new Exception('Sessions must be enabled to use the Security class');
return false;
}
else