Edit Orders v4.0 Support Thread

[lat9]

v5.0.3 of Edit Orders is now available for download: https://www.zen-cart.com/downloads.php?do=file&id=2400

This release provides updates for the 15 issues referenced by this GitHub link: https://github.com/lat9/edit_orders/...stone%3Av5.0.3

[swguy]

Getting an issue with 5.0.3 when run with Zen Cart 2.1.0.

[07-Jul-2026 05:16:19 America/New_York] PHP Fatal error: Uncaught TypeError: EditOrdersAdminObserver::notify_order_query_add_product(): Argument #5 ($ordered_product) must be of type array, null given, called in /Users/scott/Sites/client/includes/classes/traits/NotifierManager.php on line 106 and defined in /Users/scott/Sites/client/zc_plugins/EditOrders/v5.0.3/admin/includes/classes/observers/EditOrdersAdminObserver.php:298
Stack trace:
#0 /Users/scott/Sites/client/includes/classes/traits/NotifierManager.php(106): EditOrdersAdminObserver->notify_order_query_add_product(Object(order), 'NOTIFY_ORDER_QU...', Array, 0, NULL, NULL, NULL, NULL, NULL, NULL, NULL)
#1 /Users/scott/Sites/client/includes/classes/order.php(364): base->notify('NOTIFY_ORDER_QU...', Array, 0)
#2 /Users/scott/Sites/client/includes/classes/order.php(152): order->query(53215)
#3 /Users/scott/Sites/client/zc_plugins/EditOrders/v5.0.3/admin/edit_orders.php(43): order->__construct(53215)
#4 /Users/scott/Sites/client/admin/index.php(28): require('/Users/scott/Si...')
#5 {main}
thrown in /Users/scott/Sites/client/zc_plugins/EditOrders/v5.0.3/admin/includes/classes/observers/EditOrdersAdminObserver.php on line 298


Making the final parameter of notify_order_query_add_product optional

?array &$ordered_product

seems to fix this but I was uncertain if that was the right way to go.

[lat9]

Right, that last parameter (which isn't used by the observer) was added in zc220. The correction I've pushed to EO's GitHub is to simply remove that $ordered_product parameter.

[swguy]

Would it be possible for edit orders to recompute the shipping after an order change? Or is this just too complicated?

[lat9]

Would it be possible for edit orders to recompute the shipping after an order change? Or is this just too complicated?

As you surmised, that's just too complicated. It's bad enough getting order-total modules involved in the recalculation; shipping modules just present another layer of complexity upon an already complex plugin.

[swguy]

Fair enough. These things can be simulated using the Shipping Estimator anyway. Thanks for the quick response.