Deprecation warning: Markdown_Parser has a deprecated constructor

  • Resolved ethereumicoio

    (@ethereumicoio)


    5 years, 3 months ago

    <br />
<b>Deprecated</b>:  Methods with the same name as their class will not be constructors in a future version of PHP; Markdown_Parser has a deprecated constructor in <b>/var/www/html/wp-content/plugins/simba-plugin-updates-manager/vendor/php-markdown-extra/markdown.php</b> on line <b>71</b><br />
<br />
<b>Deprecated</b>:  Methods with the same name as their class will not be constructors in a future version of PHP; MarkdownExtra_Parser has a deprecated constructor in <b>/var/www/html/wp-content/plugins/simba-plugin-updates-manager/vendor/php-markdown-extra/markdown.php</b> on line <b>1533</b><br />

    It looks like this packaage is used: https://packagist.org/packages/michelf/php-markdown
    I had a look at it and it looks like that in latest versions this is fixed. I recommend you to delete the old version and instal the new one via composer.

    If you have a plugin repo on github I can make a PR.

    The page I need help with: [log in to see the link]

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author David Anderson

    (@davidanderson)

    Hi,

    Thank you. I’m aware of this. Deprecation just means “will go away in future”, in this case, PHP 8, which is at least 16 months in the future, so, I’m not treating it as urgent. There’s no public repo but you are welcome to send a patch via another method if you are motivated before then.

    David

    Thread Starter ethereumicoio

    (@ethereumicoio)

    The problem is that these deprecation warnings are outputted in the JSON file generated. The one under this title: “Updates URL for this plugin (used inside the plugin)”. Making it broken.

    Yes, I’m ready to send you my patch. Just say how/where I can do that.

    Plugin Author David Anderson

    (@davidanderson)

    You can post a link here to wherever you want to put the patch.

    The problem is that these deprecation warnings are outputted in the JSON file generated.

    This means that your PHP install is configured to send PHP log messages to output, instead of to a log file. You’ll not want that, as you’ll always be at the mercy of anything and everything in your WP install that might one day produce a PHP log message.

    Thread Starter ethereumicoio

    (@ethereumicoio)

    Thank you!
    It is my development server, so, it is intentionally configured to output everything.

    The patched plugin archive:
    https://drive.google.com/file/d/1IPRl4I1C3JO5YlMj87fwO_BrtxPxCBm6/view?usp=sharing

    I’ve updated other composer libs versions to latest stable ones too.

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘Deprecation warning: Markdown_Parser has a deprecated constructor’ is closed to new replies.