Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

A few of the "Passing null to parameter" warnings still there #1273

Open
maxxscho opened this issue Jan 12, 2022 · 6 comments
Open

A few of the "Passing null to parameter" warnings still there #1273

maxxscho opened this issue Jan 12, 2022 · 6 comments
Labels

Comments

@maxxscho
Copy link

maxxscho commented Jan 12, 2022

After v3.6.5 fixed many of the Passing null to parameter warnings in the "Messages" tab, a few of them are still there - related to #1261

image
image

@MaxKorlaar
Copy link

I'd like to add that this is still relevant in 3.6.7, caused by ViewCollector.php:

[17:45:28] LOG.warning: substr(): Passing null to parameter #1 ($string) of type string is deprecated in /home/vagrant/code/vendor/barryvdh/laravel-debugbar/src/DataCollector/ViewCollector.php on line 64 [17:45:28] LOG.warning: pathinfo(): Passing null to parameter #1 ($path) of type string is deprecated in /home/vagrant/code/vendor/barryvdh/laravel-debugbar/src/DataCollector/ViewCollector.php on line 67 [17:53:40] LOG.warning: htmlentities(): Passing null to parameter #1 ($string) of type string is deprecated in /home/vagrant/code/vendor/barryvdh/laravel-debugbar/src/DataFormatter/QueryFormatter.php on line 58

This can be fixed by checking if $path is null. If a PR is appreciated be sure to let me know, it's a rather simple fix. I can see that the last warning in QueryFormatter.php has been fixed in master.

@digitall-it
Copy link

I confirm that on DebugBar v3.6.7 running on PHP v8.1 in Laravel v9.11

[2022-05-30 15:31:28] laravel.WARNING: htmlentities(): Passing null to parameter #1 ($string) of type string is deprecated in /var/www/html/vendor/barryvdh/laravel-debugbar/src/DataFormatter/QueryFormatter.php on line 58

@mikedodd
Copy link

I aldo confirm that on DebugBar v3.6.7 running on PHP v8.1 in Laravel v9.11

@wbetalleluzd
Copy link

Any solution...?????

@wbetalleluzd
Copy link

@stale
Copy link

stale bot commented Sep 21, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If this issue is still present on the latest version of this library on supported Laravel versions, please let us know by replying to this issue so we can investigate further.
Thank you for your contribution! Apologies for any delayed response on our side.

@stale stale bot added the stale label Sep 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants