1 | <?php |
||
2 | |||
3 | namespace App\Http\Controllers; |
||
4 | |||
5 | use App\Http\Requests\UpdateDescriptionTask; |
||
6 | use App\Task; |
||
7 | |||
8 | class ApiDescriptionTasksController extends Controller |
||
9 | { |
||
10 | /** |
||
11 | * @param UpdateDescriptionTask $request |
||
12 | * @param Task $task |
||
13 | */ |
||
14 | public function update(UpdateDescriptionTask $request, Task $task) |
||
15 | { |
||
16 | $request->validate([ |
||
0 ignored issues
–
show
|
|||
17 | 'description' => 'required', |
||
18 | 'name' => 'required', |
||
19 | 'user_id' => 'required', |
||
20 | ]); |
||
21 | $task->description = $request->description; |
||
22 | $task->name = $request->name; |
||
23 | $task->user_id = $request->user_id; |
||
24 | $task->save(); |
||
25 | |||
26 | return $task; |
||
27 | |||
28 | } |
||
29 | } |
||
30 |
This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.
If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.