Skip to content
PHP Static Analysis (Larastan)

(Add) Use shouldSend for notifications #4084

Sign in to view logs

(Add) Use shouldSend for notifications

(Add) Use shouldSend for notifications #4084

Triggered via pull request February 1, 2025 13:06
@Obi-WanaObi-Wana
synchronize #4374
Obi-Wana:use-shouldSend-for-notifications
Status Failure
Total duration 2m 57s
Artifacts –

larastan.yml

on: pull_request
Matrix: test
Fit to window
Zoom out
Zoom in

Annotations

3 errors
php 8.4 on ubuntu-22.04: app/Notifications/NewRequestFill.php#L51
Access to an undefined property App\Models\User::$user_id.
php 8.4 on ubuntu-22.04: app/Notifications/NewRequestFillApprove.php#L51
Access to an undefined property App\Models\User::$user_id.
php 8.4 on ubuntu-22.04
Process completed with exit code 1.