laravel-debugbar icon indicating copy to clipboard operation
laravel-debugbar copied to clipboard

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

Open maxxscho opened this issue 3 years ago • 6 comments

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

maxxscho avatar Jan 12 '22 15:01 maxxscho

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.

MaxKorlaar avatar Mar 30 '22 15:03 MaxKorlaar

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

digitall-it avatar May 30 '22 15:05 digitall-it

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

mikedodd avatar May 30 '22 17:05 mikedodd

Any solution...?????

wbetalleluzd avatar Jun 16 '22 14:06 wbetalleluzd

Solution: https://bytemeta.vip/repo/barryvdh/laravel-debugbar/issues/1296

wbetalleluzd avatar Jun 17 '22 05:06 wbetalleluzd

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[bot] avatar Sep 21 '22 02:09 stale[bot]