Not HPOS compatible

  • Resolved Pagecode

    (@pagecode)


    7 months, 2 weeks ago

    Hi, I was just checking your code in the plugin and its not HPOS compatible, you’re using :
    delete_post_meta( $order_id, ‘_partial_pay_through_wallet_compleate’ );
    update_post_meta( $order_id, ‘_woo_wallet_partial_payment_refunded’, true );

    Which is not HPOS compatible, you’re also using shop_order in many places

    Need to use

    $order->get_meta(‘metakey’, true);

    $order->update_meta_data(‘meta_key’, ‘value’);
    $order->save();

Viewing 5 replies - 1 through 5 (of 5 total)
  • Ross

    (@rmdglasgow)

    I agree, the plugin is being marked as HPOS compatible, but it is not.

    /woo-wallet/includes/class-woo-wallet-wallet.php:195
    update_post_meta( $order_id, '_partial_pay_through_wallet_compleate', $transaction_id );

    This is causing my network of sites serious issues, and making my plugins behave in an unexpected manor.

    Plugin Author Subrata Mal

    (@subratamal)

    We are working on this and it will be fully compatible with HPOS in our next plugin update.

    Plugin Author Subrata Mal

    (@subratamal)

    Now our plugin is fully WooCommerce HPOS and block cart and checkout supported.

    Thread Starter Pagecode

    (@pagecode)

    Still not resolved. There is a massive issue on HPOS sites where you can get refunded twice for the wallet on partial payment orders, where you can place an order which goes to payment gateways hosted/external page with partial payments. You can then purposely fail the payment and it will refund the wallet credit back twice.

    Looking at the code I can still see saving post meta, you can’t save to post meta in HPOS at all, it has to be update meta data and save.

    A lot of websites are being exploited for this right now, one of them lost 10,000$ because people were purposely failing orders to get double credit back.

    Plugin Author Subrata Mal

    (@subratamal)

    @pagecode The problem has been resolved with the update to plugin version v1.5.4.

Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘Not HPOS compatible’ is closed to new replies.

Tags