Posted by Turgrid on April 17, 2009 at 8:39pm
| Project: | Ubercart |
| Version: | 6.x-2.0-rc1 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Island Usurper |
| Status: | closed (fixed) |
Issue Summary
I believe this affects all versions (uc1 and uc2).
At least in uc_ups_fulfill_order_validate function and based on my results with UPS shipping tools,$error->errorseverity[0]->data() is set to 'Hard' when an error is present and not 'HardError' as the current code checks against.
In other words, I believe the statement:if ($error->errorseverity[0]->data() == 'HardError') {
should be:if ($error->errorseverity[0]->data() == 'Hard') {
This may also affect uc_ups_quote function, but I have not tested it.
Comments
#1
#2
I feel like this has gone back and forth several times. And rather than figure out if the two services have different specifications, or the documentation is wrong, I'm just going to change the code to handle both situations. If it works, I'll backport it to Ubercart 1.x as well.
#3
The validate function wasn't doing what I intended anyway, which is to set a form error and return to the shipment form. New patch fixes that (as tested on the Livetest) and is committed.
#4
Automatically closed -- issue fixed for 2 weeks with no activity.