• Resolved BeycanPress LLC

    (@beycanpress)


    I think in future updates you can add a exists check before calling the relevant method. Thanks.

    Error Details
    
    =============
    
    An error of type E_ERROR was caused in line 596 of the file /woocommerce-paypal-payments/modules/ppcp-subscription/src/SubscriptionModule.php. Error message: Uncaught Error: Call to undefined method WP_Post::get_meta() in /woocommerce-paypal-payments/modules/ppcp-subscription/src/SubscriptionModule.php:596
    
    Stack trace:
    
    #0 /public_html/wp-includes/class-wp-hook.php(310): WooCommerce\PayPalCommerce\Subscription\SubscriptionModule->WooCommerce\PayPalCommerce\Subscription\{closure}()
    
    #1 /public_html/wp-includes/class-wp-hook.php(334): WP_Hook->apply_filters()
    
    #2 /public_html/wp-includes/plugin.php(517): WP_Hook->do_action()
    
    #3 /woocommerce-subscriptions/vendor/woocommerce/subscriptions-core/includes/admin/meta-boxes/class-wcs-meta-box-subscription-data.php(417): do_action()
    
    #4 /public_html/wp-includes/class-wp-hook.php(310): WCS_Meta_Box_Subscription_Data::save()
Viewing 11 replies - 1 through 11 (of 11 total)
  • Plugin Support Syde Joost

    (@joostvandevijver)

    Hello @beycanpress

    Thank you for reaching out to us, we are here to help.

    I searched our backlog and see that a bug was already opened to correct this behavior. I am still waiting for a developer to pick it up and provide a response back. If they require additional details from you or need you to test anything, I will reach out to you.

    Kind regards,
    Joost

    Thread Starter BeycanPress LLC

    (@beycanpress)

    Hello, I’m always here. Since I don’t know the entire code, I think just adding a condition will solve the problem. But of course, those working on it may have a better solution. However, if you are open to external participation, I can examine the problem a little more and add a condition as I mentioned or work on another solution. Because I get this error on my current site, I can work on it.

    Plugin Support Syde Joost

    (@joostvandevijver)

    Hello @beycanpress

    I have informed the team, and we will reach out if they want to take you up on your offer to help, after our devs have taken an initial look at this situation.

    Thanks again for the report and the offer to help!

    Kind regards,
    Joost

    I am getting the exact same error with the latest update. If I manually change a subscription it gives the fatal error.

    Thread Starter BeycanPress LLC

    (@beycanpress)

    Hey hi again, i created a pull request for this problem. Can you check?

    See: https://github.com/woocommerce/woocommerce-paypal-payments/pull/1742

    I created a ticket at WooCommerce last week about this and haven’t received a reply. Still getting the error, here’s my stack trace:

    Error Details
    =============
    An error of type E_ERROR was caused in line 596 of the file /nas/content/live/thornebottomfa/wp-content/plugins/woocommerce-paypal-payments/modules/ppcp-subscription/src/SubscriptionModule.php. Error message: Uncaught Error: Call to undefined method WP_Post::get_meta() in /nas/content/live/thornebottomfa/wp-content/plugins/woocommerce-paypal-payments/modules/ppcp-subscription/src/SubscriptionModule.php:596
    Stack trace:
    #0 /nas/content/live/thornebottomfa/wp-includes/class-wp-hook.php(310): WooCommerce\PayPalCommerce\Subscription\SubscriptionModule->WooCommerce\PayPalCommerce\Subscription\{closure}(7284, Object(WP_Post))
    #1 /nas/content/live/thornebottomfa/wp-includes/class-wp-hook.php(334): WP_Hook->apply_filters('', Array)
    #2 /nas/content/live/thornebottomfa/wp-includes/plugin.php(517): WP_Hook->do_action(Array)
    #3 /nas/content/live/thornebottomfa/wp-content/plugins/woocommerce-subscriptions/vendor/woocommerce/subscriptions-core/includes/admin/meta-boxes/class-wcs-meta-box-subscription-data.php(417): do_action('woocommerce_pro...', 7284, Object(WP_Post))
    #4 /nas/content/live/thornebottomfa/wp-includes/class-wp-hook.php(310): WCS_Meta_Box_Subscription_Data::save(7284, Object(WP_Post))
    #5 /nas/content/live/thornebottomfa/wp-includes/class-wp-hook.php(334): WP_Hook->apply_filters(NULL, Array)
    #6 /nas/content/live/thornebottomfa/wp-includes/plugin.php(517): WP_Hook->do_action(Array)
    #7 /nas/content/live/thornebottomfa/wp-content/plugins/woocommerce/includes/admin/class-wc-admin-meta-boxes.php(263): do_action('woocommerce_pro...', 7284, Object(WP_Post))
    #8 /nas/content/live/thornebottomfa/wp-includes/class-wp-hook.php(312): WC_Admin_Meta_Boxes->save_meta_boxes(7284, Object(WP_Post))
    #9 /nas/content/live/thornebottomfa/wp-includes/class-wp-hook.php(334): WP_Hook->apply_filters('', Array)
    #10 /nas/content/live/thornebottomfa/wp-includes/plugin.php(517): WP_Hook->do_action(Array)
    #11 /nas/content/live/thornebottomfa/wp-includes/post.php(4751): do_action('save_post', 7284, Object(WP_Post), true)
    #12 /nas/content/live/thornebottomfa/wp-includes/post.php(4853): wp_insert_post(Array, false, true)
    #13 /nas/content/live/thornebottomfa/wp-admin/includes/post.php(445): wp_update_post(Array)
    #14 /nas/content/live/thornebottomfa/wp-admin/post.php(227): edit_post()
    #15 {main}
      thrown

    Chris

    Plugin Support Syde Joost

    (@joostvandevijver)

    Hello all,

    we are working on this issue and currently have a test package available for you to try. We prefer not to share these packages openly on this forum, so if you want to try our test package, please open a private engagement here and ask for the test package for dev-ticket: PCP-2093

    Our QA team tested this fix this morning and informed us that it resolved the issue for them.

    Please let us know if you want to try this test package.

    Kind regards,
    Joost

    Plugin Support Syde Joost

    (@joostvandevijver)

    Hello,

    this issue should be resolved with our next release. Details on this bug/fix can be found here on GitHub. The fix we put in place is currently being tested for the official release, but a beta version can be tested already via this release candidate. The official release should take place within the next 2 weeks.

    If you are still encountering this issue after using this package, please open a new thread and we will take another look.

    Kind regards,
    Joost

    Hey,

    I used the release candidate, version 2.4.0-rc1, and it fixed the issue reported here. However, now I get a message in every page that says:

    Apple Pay Validation Error. Please check the requirements.

    It happened right after replacing the plugin version.

    Plugin Support Syde Joost

    (@joostvandevijver)

    Hello @ixistudio

    Thank you for your feedback. This issue should be also solved with the official release that should become available in about 1 week. For now, you can use the rc2 beta version that is available here that also includes a fix for the Apple notification that you mentioned.

    Let me know if you still encounter this or any other issue with that version.

    Kind regards,
    Joost

    Thanks @joostvandevijver,

    It looks like that’s working. Thanks for the quick reply.

    Cheers!

Viewing 11 replies - 1 through 11 (of 11 total)
  • The topic ‘Error when updating a subscription’ is closed to new replies.