tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 12:05:37 +00:00
Code Issues Projects Releases Wiki Activity

convert to php 5.4+ array syntax

Browse Source
This commit is contained in:
maks 2018-08-17 15:03:46 +02:00 committed by GitHub
parent 706f114aec
commit aa1e576a3f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 13 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
src/Control/RequestHandler.php
View File

@ -87,9 +87,9 @@ class RequestHandler extends ViewableData
* the named parameter of the parsed URL wil be used to determine the method name. * the named parameter of the parsed URL wil be used to determine the method name.
* @config * @config
*/ */
private static $url_handlers = array( private static $url_handlers = [
'$Action' => '$Action', '$Action' => '$Action',
); ];
/** /**
@ -97,7 +97,7 @@ class RequestHandler extends ViewableData
* The variable should be an array of action names. This sample shows the different values that it can contain: * The variable should be an array of action names. This sample shows the different values that it can contain:
* *
* <code> * <code>
* array( * [
* // someaction can be accessed by anyone, any time * // someaction can be accessed by anyone, any time
* 'someaction', * 'someaction',
* // So can otheraction * // So can otheraction
@ -105,8 +105,8 @@ class RequestHandler extends ViewableData
* // restrictedaction can only be people with ADMIN privilege * // restrictedaction can only be people with ADMIN privilege
* 'restrictedaction' => 'ADMIN', * 'restrictedaction' => 'ADMIN',
* // complexaction can only be accessed if $this->canComplexAction() returns true * // complexaction can only be accessed if $this->canComplexAction() returns true
* 'complexaction' '->canComplexAction' * 'complexaction' '->canComplexAction',
* ); * ];
* </code> * </code>
* *
* Form getters count as URL actions as well, and should be included in allowed_actions. * Form getters count as URL actions as well, and should be included in allowed_actions.
@ -196,7 +196,7 @@ class RequestHandler extends ViewableData
if (!$this->hasAction($action)) { if (!$this->hasAction($action)) {
return $this->httpError(404, "Action '$action' isn't available $classMessage."); return $this->httpError(404, "Action '$action' isn't available $classMessage.");
} }
if (!$this->checkAccessAction($action) || in_array(strtolower($action), array('run', 'doInit'))) { if (!$this->checkAccessAction($action) || in_array(strtolower($action), ['run', 'doInit'])) {
return $this->httpError(403, "Action '$action' isn't allowed $classMessage."); return $this->httpError(403, "Action '$action' isn't allowed $classMessage.");
} }
$result = $this->handleAction($request, $action); $result = $this->handleAction($request, $action);
@ -271,7 +271,10 @@ class RequestHandler extends ViewableData
); );
} }
return array('rule' => $rule, 'action' => $action); return [
'rule' => $rule,
'action' => $action,
];
} }
} }
} }
@ -383,7 +386,7 @@ class RequestHandler extends ViewableData
} }
// Don't allow access to any non-public methods (inspect instance plus all extensions) // Don't allow access to any non-public methods (inspect instance plus all extensions)
$insts = array_merge(array($this), (array) $this->getExtensionInstances()); $insts = array_merge([$this] (array) $this->getExtensionInstances());
foreach ($insts as $inst) { foreach ($insts as $inst) {
if (!method_exists($inst, $action)) { if (!method_exists($inst, $action)) {
continue; continue;
@ -430,7 +433,7 @@ class RequestHandler extends ViewableData
$action = strtolower($actionOrigCasing); $action = strtolower($actionOrigCasing);
$definingClass = null; $definingClass = null;
$insts = array_merge(array($this), (array) $this->getExtensionInstances()); $insts = array_merge([$this], (array) $this->getExtensionInstances());
foreach ($insts as $inst) { foreach ($insts as $inst) {
if (!method_exists($inst, $action)) { if (!method_exists($inst, $action)) {
continue; continue;
@ -472,7 +475,7 @@ class RequestHandler extends ViewableData
} elseif (substr($test, 0, 2) == '->') { } elseif (substr($test, 0, 2) == '->') {
// Determined by custom method with "->" prefix // Determined by custom method with "->" prefix
list($method, $arguments) = ClassInfo::parse_class_spec(substr($test, 2)); list($method, $arguments) = ClassInfo::parse_class_spec(substr($test, 2));
$isAllowed = call_user_func_array(array($this, $method), $arguments); $isAllowed = call_user_func_array([$this, $method], $arguments);
} else { } else {
// Value is a permission code to check the current member against // Value is a permission code to check the current member against
$isAllowed = Permission::check($test); $isAllowed = Permission::check($test);