View GitHub Repository laravel-inbox

Schedule Inspection
Subscribe

Issues (42)

src/routes.php (1 issue)

← previous file next file →
Labels
Severity
Introduced By
Code
1 
<?php
2 
/**
3 
 * Created by PhpStorm.
4 
 * User: me
5 
 * Date: 09.07.2017
6 
 * Time: 19:37
7 
 */
8 
Route::group(['middleware' => ['web', 'auth']], function () {
0 ignored issues
– show
Unused Code introduced by
The call to Illuminate\Support\Facades\Route::group() has too many arguments starting with function(...) { /* ... */ }. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

8 
Route::/** @scrutinizer ignore-call */ 
9 
       group(['middleware' => ['web', 'auth']], function () {

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.

Loading history...
9 
    Route::get('conversation', 'Evilnet\Inbox\InboxController@create');
10 
    Route::post('conversation', 'Evilnet\Inbox\InboxController@store');
11 
    Route::get('conversation/{id}', 'Evilnet\Inbox\InboxController@show');
12 
    Route::post('message/{id}', 'Evilnet\Inbox\InboxController@addMessage');
13 
    Route::get('inbox', 'Evilnet\Inbox\InboxController@index')->name('inbox');
14 
    Route::delete('/conversation/{id}', '\Evilnet\Inbox\InboxController@destroy');
15 
});