Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update LogViewerController.php #256

Closed
wants to merge 1 commit into from

Conversation

laurentenhoor
Copy link

https://stackoverflow.com/questions/59674903/trying-to-access-array-offset-on-value-of-type-bool-in-php-7-4

I am getting this error coming from this code:
Notice: Trying to access array offset on value of type bool in

While deleting the log files from Freescout (which is using your dependency).

https://stackoverflow.com/questions/59674903/trying-to-access-array-offset-on-value-of-type-bool-in-php-7-4

I am getting this error coming from this code:
Notice: Trying to access array offset on value of type bool in

While deleting the log files from Freescout (which is using your dependency).
Copy link
Contributor

@mikemand mikemand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While this is a good change, the null coalescing assignment operator is a PHP 7.4+ only feature. Logviewer is targeted at PHP 5.4+, so anything you do must be compatible or @rap2hpoutre has to drop support for anything less than PHP 7.4 - which is out of the scope of this PR.

All that said, I'm pretty sure @rap2hpoutre is no longer supporting Logviewer. See #251.

@rap2hpoutre
Copy link
Owner

Fixed via: 1dfd871
Thank you for your patience!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants