PHP 8.2 support

  • Hi! PHP 8.2 is throwing notices about deprecated dynamic properties in the SyntaxHighlighter class.

    Defining the following properties in the class solves this issue:

    class SyntaxHighlighter {
    // ...
    public $brush_names;
    public $specialchars;
    // ...
}
Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Support Stef (a11n)

    (@erania-pinnera)

    Hi there, @ideag,

    Usually, when you see is deprecated?it means it doesn’t break anything but will in a future PHP version. If you don’t want to see those messages, you can decrease the debug log to ignore deprecation notices.

    That said, it seems that you solve the problem and are sharing your solution. Did I get this right, or do you still help with that?

    Thread Starter Arunas Liuiza

    (@ideag)

    Because hiding error messages under a rug is a sound strategy? ?? We know we can hide them, but we keep the debug log verbose so we can find and address issues early.

    We could fix this in our own codebase, but that would mean we’ll need to re-apply the patch anytime there is an update. So we would love to see this fixed upstream in the plugin itself.

    It also fixes the issue to all the other 30,000+ users of this plugin. If there is a repo I can contribute to, I’d be happy to supply a patch/PR for this.

    Plugin Support Stef (a11n)

    (@erania-pinnera)

    Hi @ideag,

    Thanks for graciously offering your help!

    You’re very are welcome to contribute here:??https://github.com/Automattic/syntaxhighlighter

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘PHP 8.2 support’ is closed to new replies.