adulau/cakephp2-php8
1
0
Fork 0
mirror of https://github.com/kamilwylegala/cakephp2-php8.git synced 2024-11-15 03:18:26 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Make allow(null) and deny(null) consistent with no args.

Browse source 
No arguments and a single null should be handled the same.

Fixes #2461
This commit is contained in:
mark_story 2012-01-10 20:32:12 -05:00
parent a8bc916104
commit 7877e7f997
2 changed files with 14 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
lib/Cake/Controller/Component/AuthComponent.php
View file

@ -432,7 +432,7 @@ class AuthComponent extends Component {
*/
public function allow($action = null) {
$args = func_get_args();
if (empty($args)) {
if (empty($args) || $action === null) {
$this->allowedActions = $this->_methods;
} else {
if (isset($args[0]) && is_array($args[0])) {
@ -458,7 +458,7 @@ class AuthComponent extends Component {
*/
public function deny($action = null) {
$args = func_get_args();
if (empty($args)) {
if (empty($args) || $action === null) {
$this->allowedActions = array();
} else {
if (isset($args[0]) && is_array($args[0])) {

12
lib/Cake/Test/Case/Controller/Component/AuthComponentTest.php
View file

@ -654,6 +654,18 @@ class AuthComponentTest extends CakeTestCase {
$this->Controller->request['action'] = 'login';
$this->assertFalse($this->Controller->Auth->startup($this->Controller));
$this->Controller->Auth->deny();
$this->Controller->Auth->allow(null);
$this->Controller->request['action'] = 'camelCase';
$this->assertTrue($this->Controller->Auth->startup($this->Controller));
$this->Controller->Auth->allow();
$this->Controller->Auth->deny(null);
$this->Controller->request['action'] = 'camelCase';
$this->assertFalse($this->Controller->Auth->startup($this->Controller));
}
/**