Talking about shipping here, not quoting...
When shipping with UPS, admin is presented with a form generated by uc_ups_fulfill_order(). This form is used to collect information to be used for the shipment, such as weight, insured value, dimensions, and most importantly, which UPS shipping method (Ground, Express, etc.) to use. This information is then sent off to UPS to generate a shipping label.
Currently, the form only allows the admin to choose from among the UPS services enabled for *quoting* at admin/store/settings/quotes/methods/ups/edit. This patch changes the module so that *all* UPS services are available for shipping. The reason this is useful is that shipping doesn't necessarily have anything to do with quoting - the store could be using Flatrate to quote customers but doing the actual shipping through UPS, for instance. There is no reason to restrict the admin's choices on this form.
Secondly, this patch adds informative text on the form to let the admin know what the customer's choice of shipping method was, and how much the customer paid. The admin needs to know this in order to properly select the UPS shipping method to use. If the customer selected and paid for UPS 2nd Day Air, the admin needs to know so he doesn't inadvertently ship via Ground.
Thirdly, if the customer chose a UPS shipping method at checkout, this patch will *default* the form to using the same UPS method that the customer chose. Again, this helps to reduce errors and minimize the work the admin needs to do to ship the order.
Lastly, the patch adds a style to uc_ups.css. The form uses panes from uc_shipping, but the uc_shipping.css is not loaded along with the panes, resulting in a poorly formated form. I copied one style out of uc_shipping.css and put it into uc_ups.css to correct this.
Tested on UC 2.2.
| Comment | File | Size | Author |
|---|---|---|---|
| uc_ups.css_.patch | 194 bytes | tr | |
| uc_ups.module.patch | 1.37 KB | tr |
Comments
Comment #1
tr commentedI think these small changes constitute a major improvement to the ship functionality. Don't you agree?
Comment #2
tr commentedComment #3
Island Usurper commentedGood ideas. Thanks, again.
Committed.