PHP 8.1+ Support

  • Resolved davecunnington

    (@davecunnington)


    1 month, 2 weeks ago

    Hey Jeff, thanks for your plugin and supporting the WordPress community!

    Running the plugin on some sites in PHP 8.1+ throws some warnings surrounding the use of explode() when the string is null. Specifically, plugin-core.php:407, method: disable_gutenberg_explode($string) when trimming. Recommend either adding a default value $string = '' or some quick error checking in the method before using the variable on line 409.

    Happy to submit a PR!

Viewing 5 replies - 1 through 5 (of 5 total)
  • Plugin Author Jeff Starr

    (@specialk)

    Thanks for the report, Dave. Will get this fixed up for the next release.

    Plugin Author Jeff Starr

    (@specialk)

    This is resolved in version 3.2, available later today.

    Thanks again for reporting.

    Thread Starter davecunnington

    (@davecunnington)

    Thanks again for providing this plugin to the community!

    Thread Starter davecunnington

    (@davecunnington)

    Sorry Jeff, I gave bad information for the fix. After checking the docs I see that “passing?null?does?not?assign the default value” so the fix should be in the method instead of adding the default value to the method parametres.

    If you can’t use the null coalesce for backwards compatibility then the method will need something like:

    if (!$string) {
    $string = "";
    }

    Plugin Author Jeff Starr

    (@specialk)

    Thanks Dave, just released 3.2.1 which hopefully should resolve this.

Viewing 5 replies - 1 through 5 (of 5 total)
  • You must be logged in to reply to this topic.