Deprecation in Html::cleanCssIdentifier()

@prudloff Question: The method expects a CSS identifier as argument. Why are we passing the output of file_get_contents()?

The file contains an example of a bogus unicode string that can be used to reproduce the deprecation.
I tried to paste the content directly in my PHP example, but it would lose some of the buggy invisible characters.

Thanks for the explanation.

Hi @prudloff i am trying to reproduce the issue in php8.3 but failed to do so.
I have provided the attatchment .

Is your PHP configured to display/log deprecations?

If I try to reproduce with drush php-eval, it seems I need to call it with -v to see the deprecation.

@prudloff i can only see error when i am providing $content=null

Here is an easier way to reproduce:

\Drupal\Component\Utility\Html::cleanCssIdentifier("\x80\x81");

Binoli Lalani → made their first commit to this issue’s fork.

Hello,

I have added test coverage for this PHP deprecation error along with this error fix. Please review.

Thank you!

Should find where null is being passed in. This change could be masking a larger issue.

Hi @smustgrave ,
In this particular situation error is coming after line 127 , in this when using preg_replace function it is stripping of all the character which are not in the list and that 's why it is getting null.