for testing and deploying your application
for finding and fixing issues
for empowering human code reviews
<?php namespace App\Modules\Notifications\Repositories;
use App\Modules\Core\AbstractRepositories\AbstractRepository;
class NotificationRepository extends AbstractRepository
{
/**
* Return the model full namespace.
*
* @return string
*/
protected function getModel()
$apiVersion = \Request::header('api-version') ?: 1;
$apiVersion
This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.
$myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; }
Both the $myVar assignment in line 1 and the $higher assignment in line 2 are dead. The first because $myVar is never used and the second because $higher is always overwritten for every possible time line.
$myVar
$higher
return 'App\Modules\Notifications\Notification';
}
* Set the notification notified to true.
* @param integer $id
* @return object
public function notified($id)
return $this->save(['id' => $id, 'notified' => 1]);
* Set the notification notified to all.
* @return void
public function notifyAll()
\Core::notifications()->update(false, ['notified' => 1], 'notified');
SherifElfadaly /
Laravel-Api-Skeleton
This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.
Both the
$myVarassignment in line 1 and the$higherassignment in line 2 are dead. The first because$myVaris never used and the second because$higheris always overwritten for every possible time line.