Move the remaining order status change code out of uc_cart.module
rszrama - January 7, 2008 - 20:36
| Project: | Ubercart |
| Version: | 5.x-1.0-alpha8 |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | postponed |
Jump to:
Description
Lines 1717-1721 of uc_cart.module include updating the order status upon checkout completion. This code should be considered for removal in favor of a default enabled workflow configuration.
An acceptable fix will include a description of testing done to verify the change works and a patch that removes these lines and adds code to uc_cart_workflow.inc for the appropriate default configuration.
#1
Postponing this issue. Right now the way it's setup we can ensure the order of when the order status gets updated. I'm not sure if moving this into a workflow configuration will cause any conflicts if the order status is still "in checkout" for a particular event/action (for example, sending an e-mail notification). If someone feels this should be readdressed, feel free to reactivate this.