Skip to content

Conversation

@barryvdh
Copy link
Collaborator

For example the Laravel error page has hljs now, which doesn't contain SQL. So that will fail otherwise.

@barryvdh barryvdh merged commit f3b0022 into master Dec 15, 2024
16 checks passed
@barryvdh barryvdh deleted the fix-hljs-lang branch December 15, 2024 19:19
barryvdh added a commit that referenced this pull request Dec 15, 2024
@josevv28
Copy link

Hi, It seems the releases are no longer shown in https://packagist.org/packages/maximebf/debugbar
could it be because of maximebf/php-debugbar to php-debugbar/php-debugbar/ change?

@barryvdh
Copy link
Collaborator Author

Sorry webhooks are still a bit wonky. I thinks they are okay now but updated manually now.

barryvdh added a commit that referenced this pull request Feb 7, 2025
* Check if hljs has language (#699)

* Fix PHP 8 warning in `TracedStatement::getSqlWithParameters()` when null was bound

Since PHP 8.0, [built-in functions like `strtr()` will emit a warning when passing `null` to required string parameters](docs):

```
strtr(): Passing null to parameter #1 ($string) of type string is deprecated
```

This can happen when e.g. binding `PDO::PARAM_NULL` to your query.

[docs]: https://www.php.net/manual/en/migration81.deprecated.php#migration81.deprecated.core.null-not-nullable-internal

---------

Co-authored-by: Barry vd. Heuvel <[email protected]>
barryvdh added a commit that referenced this pull request Feb 13, 2025
* Check if hljs has language (#699)

* Fix PHP 8 warning in `TracedStatement::getSqlWithParameters()` when null was bound

Since PHP 8.0, [built-in functions like `strtr()` will emit a warning when passing `null` to required string parameters](docs):

```
strtr(): Passing null to parameter #1 ($string) of type string is deprecated
```

This can happen when e.g. binding `PDO::PARAM_NULL` to your query.

[docs]: https://www.php.net/manual/en/migration81.deprecated.php#migration81.deprecated.core.null-not-nullable-internal

---------

Co-authored-by: Barry vd. Heuvel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants