Fixes for static analysis

This commit is contained in:
David Bomba 2023-04-27 00:16:07 +10:00
parent 482e74ac5e
commit 759b840dbf
3 changed files with 8 additions and 4 deletions

View File

@ -929,7 +929,7 @@ class BaseController extends Controller
*
* @param Builder $query
*/
protected function listResponse($query)
protected function listResponse(Builder $query)
{
$this->buildManager();

View File

@ -128,9 +128,12 @@ class PreviewController extends BaseController
return (new Phantom)->convertHtmlToPdf($maker->getCompiledHTML(true));
}
/** @var \App\Models\Company $company */
/** @var App\Models\User auth()->user() */
$company = auth()->user()->company();
$user = auth()->user();
/** @var \App\Models\Company $company */
$company = $user->company();
if (config('ninja.invoiceninja_hosted_pdf_generation') || config('ninja.pdf_generator') == 'hosted_ninja') {

View File

@ -28,10 +28,11 @@ trait Excludable
* @param Builder $query
* @param array $columns
*
* @return mixed
* @return Builder
*/
public function scopeExclude($query, $columns): \Illuminate\Database\Eloquent\Builder
{
/** @var Builder $query */
return $query->select(array_diff($this->getTableColumns(), (array) $columns));
}
}