Skip to content
PHP Static Analysis (Larastan)

(Add) Use shouldSend for notifications #3807

Sign in to view logs

(Add) Use shouldSend for notifications

(Add) Use shouldSend for notifications #3807

Triggered via pull request December 20, 2024 14:35
@Obi-WanaObi-Wana
synchronize #4374
Obi-Wana:use-shouldSend-for-notifications
Status Failure
Total duration 2m 45s
Artifacts –

larastan.yml

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

Annotations

3 errors
php 8.3 on ubuntu-22.04: app/Notifications/NewComment.php#L80
Access to an undefined property App\Models\Comment::$staff_id.
php 8.3 on ubuntu-22.04: app/Notifications/NewComment.php#L85
Unreachable statement - code above always terminates.
php 8.3 on ubuntu-22.04
Process completed with exit code 1.