adulau/cakephp2-php8
1
0
Fork 0
mirror of https://github.com/kamilwylegala/cakephp2-php8.git synced 2024-11-25 16:27:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fixed passing arguments to invokeArgs when resolving controller's action.

Browse source
This commit is contained in:
Kamil Wylegala 2022-03-08 19:55:03 +01:00
parent ea13b9f3d1
commit a5a0ccd551
2 changed files with 10 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
README.md
View file

@ -55,6 +55,10 @@ It means that composer will look at `master` branch of repository configured und
## Changelog ## Changelog
### 2022-03-08
- Fixed passing `params["pass"]` argument to `invokeArgs` when resolving controller action - `array_values` used to avoid problems with named parameters.
### 2022-03-03 ### 2022-03-03
- Removed `String` class. - Removed `String` class.

7
lib/Cake/Controller/Controller.php
View file

@ -496,8 +496,13 @@ class Controller extends CakeObject implements CakeEventListener {
'action' => $request->params['action'] 'action' => $request->params['action']
)); ));
} }
return $method->invokeArgs($this, $request->params['pass']);
return $method->invokeArgs(
$this,
is_array($request->params['pass'])
? array_values($request->params['pass'])
: []
);
} catch (ReflectionException $e) { } catch (ReflectionException $e) {
if ($this->scaffold !== false) { if ($this->scaffold !== false) {
return $this->_getScaffold($request); return $this->_getScaffold($request);