• BoUk

    (@bouk)


    Hello,

    I’ve just wanted to quickly ask if there’s any plan to make this plugin compatible with the new High-Performance Order Storage recently introduced into WooCommerce plugin.

    Thanks a lot.

Viewing 11 replies - 1 through 11 (of 11 total)
  • Plugin Author gopayplugins

    (@gopayplugins)

    According to the information, the publisher will try to update the free open source GoPay for WooCommerce plugin in the future if possible.

    HPOS support is not something optional.

    HPOS is a default storage as of Woo 8.2, so your plugin is incompactible with all new Woo installations, and also with all the installations that do have HPOS enabled manually.

    Also, the plugin is not compatible with Block Checkout, which is default from Woo 8.3 and again makes the plugin unusable for the new installations.

    This needs to be fixed, if you need help with this, let me know.

    Plugin Author gopayplugins

    (@gopayplugins)

    There will be a new version of the free open source plugin released in the future that should resolve the situation, but it can not say anything further at this time.

    hexcross

    (@hexcross)

    Any news about this issue? It is still not compatible with latest WooCommerce.

    Plugin Author gopayplugins

    (@gopayplugins)

    Hello,
    We’re sorry, but we don’t have more information on the payment gateway side.

    Thread Starter BoUk

    (@bouk)

    Hi,

    It’s been nearly 5 months since the initial question. Any update on that?

    Thanks a lot.

    Plugin Author gopayplugins

    (@gopayplugins)

    Hello,
    We’re sorry, but we don’t have new information on the payment gateway side.
    For information about the release of a new version please follow GoPay for WooCommerce.

    Plugin Author gopayplugins

    (@gopayplugins)

    Please be informed that a new version of the plugin has been released that supports HPOS.

    Thread Starter BoUk

    (@bouk)

    Good morning @gopayplugins,

    Thanks for the update, but I am not sure this problem has been fully resolved by the latest changes introduced.

    Looking into the codebase, I see there’s been added following piece of code:

    add_action( 'before_woocommerce_init', function() {
    if ( class_exists( \Automattic\WooCommerce\Utilities\FeaturesUtil::class ) ) {
    \Automattic\WooCommerce\Utilities\FeaturesUtil::declare_compatibility( 'custom_order_tables', __FILE__, true );
    }
    } );

    This indeed does solve the problem in terms of getting rid of error message in admin and allowing to enable HPOS, but I don’t think the actual problematic parts in codebase has been updated to work reliably with HPOS.

    Please see the following docs – part about ‘Functions for getting/setting order & order meta‘ – https://webkul.com/blog/woocommerce-plugin-high-performance-order-storage-compatible/

    They are clearing saying there that functions such as update_post_meta, add_post_meta, delete_post_meta, get_post_meta should be replaced.

    So for instance class-gopay-gateway-api.php line 447:

    $gopay_transaction_id = get_post_meta( $order_id, 'GoPay_Transaction_id', true );

    I believe this should be replaced by something like this (correct me if I am wrong @vasikgreif ):

    $order                = wc_get_order( $order_id );
    $gopay_transaction_id = $order->get_meta( 'GoPay_Transaction_id', true );

    Thanks for checking again.

    • This reply was modified 5 months, 2 weeks ago by BoUk.
    Plugin Author gopayplugins

    (@gopayplugins)

    Thank you for your feedback. We have forwarded it to the plugin publisher.

    Plugin Author gopayplugins

    (@gopayplugins)

    @bouk
    Please disregard the previous answer as it was posted by an ill-informed representative and WordPress does not let us delete/modify it.

    Thank you for informing us about the details and we are working on fixing the issue. We’ll publish an update once it is resolved.

    Moreover, you can contribute a PR on our git repository: https://github.com/gopaycommunity/gopay-woocommerce-integration

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