Fix for PHP7

  • Resolved creativeround

    (@creativeround)


    8 years, 10 months ago

    If you are using PHP7 you will need to update the file /Models/Orders/IGFW_Order_CPT.php – Line 213 to:

    private function _log_invoice_number_activity( $new_invoice_number , $existing_invoice_number = null , $post_id ) {

    $existing_invoice_number needs a default value or it will create an error on order save.

Viewing 3 replies - 1 through 3 (of 3 total)
  • rymera01

    (@rymera01)

    8 years, 10 months ago

    Hi @creativeround,

    Thanks for contacting us!

    I’m afraid I can’t reproduce this issue. The function you’re talking about won’t produce any error because the $existing_invoice_number variable uses get_post_meta() function, which will return empty string when the value is not found. This means that $existing_invoice_number will always have value set to it (even though it’s just an empty string).

    Are you using a modified version of the plugin? Because I can see that we have the mentioned function in line 207, not 213. Maybe you have a custom code that produces that error message? In this case, could you please try our released version instead?

    If you have used our released version and still have the issue, could you please let me know the steps to reproduce it?

    Also, could you please let me know the PHP version you have? I’m using PHP 7.0, right now.

    Thanks,
    Okky

    Thread Starter creativeround

    (@creativeround)

    8 years, 10 months ago

    PHP 7.1.3

    rymera01

    (@rymera01)

    8 years, 10 months ago

    Hi @creativeround,

    Thanks for the reply!

    I’ve tested it again with PHP 7.1.4, and still can’t reproduce the issue. Could you please make sure that you are not using the modified version of Invoice Gateway For WooCommerce? Could you please let me know the steps to reproduce the issue?

    It seems you also contacted us via email. In this case, I’ll close this topic and we can move the discussion via email instead.

    Thanks,
    Okky

Viewing 3 replies - 1 through 3 (of 3 total)

The topic ‘Fix for PHP7’ is closed to new replies.