Missing return statement in method/function PHP-E1004
Bug risk
4 months ago
—
a year old
Method App\Console\Commands\RecalculateWorkshopBalances::handle() should return int but return statement is missing
41 {
42 $semesters = Semester::all()
43 ->where('tag', '<=', Semester::current()->tag);
44 foreach ($semesters as $semester) {45 WorkshopBalance::generateBalances($semester->id);
46 }
47 }Method App\Console\Commands\RefreshIdeHelper::handle() should return int but return statement is missing
41 Artisan::call('ide-helper:generate');
42 Artisan::call('ide-helper:meta');
43 Artisan::call('ide-helper:models --nowrite');
44 Artisan::call('ide-helper:eloquent');45 }
46}Method App\Http\Middleware\Authenticate::redirectTo() should return string but return statement is missing
14 */
15 protected function redirectTo($request)
16 {
17 if (! $request->expectsJson()) {18 return route('login');
19 }
20 }
Description
This issue is raised if a method with a return type does not have a return statement of an appropriate type.
Bad practice
class User
{
public function greet(): string
{
$message = 'Hello, there!';
// missing return statement
}
}
class User
{
public function greet()
{
$message = function (): string {
$message = 'Hello, there!';
// missing return statement
};
return $message();
}
}
Recommended
class User
{
public function greet(): string
{
$message = 'Hello, there!';
return $message;
}
}
class User
{
public function greet()
{
$message = function (): string {
$message = 'Hello, there!';
return $message;
};
return $message();
}
}