PHP 7.2 Errors

  • Resolved npeacher

    (@npeacher)


    5 years, 11 months ago

    Not sure if anyone else has seen this error or not when trying to upgrade to PHP 7.2….

    /plugins/woocommerce-shipstation/includes/api-requests/class-wc-shipstation-api-shipnotify.php

    122 | ERROR | Global variable ‘$HTTP_RAW_POST_DATA’ is deprecated since PHP 5.6 and removed since PHP 7.0; Use php://input instead
    124 | ERROR | Global variable ‘$HTTP_RAW_POST_DATA’ is deprecated since PHP 5.6 and removed since PHP 7.0; Use php://input instead
    125 | ERROR | Global variable ‘$HTTP_RAW_POST_DATA’ is deprecated since PHP 5.6 and removed since PHP 7.0; Use php://input instead
    128 | ERROR | Global variable ‘$HTTP_RAW_POST_DATA’ is deprecated since PHP 5.6 and removed since PHP 7.0; Use php://input instead

Viewing 4 replies - 1 through 4 (of 4 total)
  • Erica K. a11n

    (@piratepenpen)

    Automattic Happiness Engineer

    Hey there!

    I’m sorry to hear you’re experiencing those errors. Allow me to explain what is going on here.

    Our developers have looked into this for us previously. The PHP compatibility checker doesn’t check which version of PHP the code was written for — $HTTP_RAW_POST_DATA was kept in the code in order to maintain backward compatibility with older versions of PHP, however when the code is run on PHP 7+ it does actually use php://input instead. Because of this, it should not cause any errors.

    This follows the same method that is used in WordPress core as well.

    If there are any errors caused when the plugin is actually being run, please do come back and let us know and we’ll be happy to take a closer look. Otherwise, if they’re only coming from a compatibility checker, please keep in mind they won’t affect how the site functions.

    Cheers!

    Erica K. a11n

    (@piratepenpen)

    Automattic Happiness Engineer

    Hey @npeacher,

    We haven’t heard back from you in a while, so I’m going to mark this as resolved – if you have any further questions, please start a new thread.

    Thanks!

    So basically you’re saying that this is a false-positive? I’m getting the same when I run a php 7.2 compatibility check.

    Thanks

    Thread Starter npeacher

    (@npeacher)

    Hey @piratepenpen. Thanks for the quick response and reassurance!

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘PHP 7.2 Errors’ is closed to new replies.