Hey all. I have the UPS shipping quote module enabled and set to treat all items as they are "all in one" box. This works great until the weight of all items exceeds 150 lbs, UPS's maximum weight per package without an oversized surcharge. At this point, the UPS quote cannot be obtained by the shopper. Shouldn't the module notice that there are multiple items that could be split up into a couple boxes and return a quote?

CommentFileSizeAuthor
#9 all_in_two-d5.patch4.35 KBIsland Usurper
#9 all_in_two.patch4.36 KBIsland Usurper
#2 all_in_two-d5.patch4.27 KBIsland Usurper
#2 all_in_two-d6.patch4.28 KBIsland Usurper

Comments

Island Usurper’s picture

Island Usurper commented
Assigned: Unassigned » Island Usurper
Island Usurper’s picture

Island Usurper commented
Status: Active » Needs review
StatusFileSize
new all_in_two-d6.patch4.28 KB
new all_in_two-d5.patch4.27 KB

I think I've found a way to do it that is just clever enough. I had earlier been counting on the fact that there was only one package in that part of the code, but I think it's been taken care of.

cha0s’s picture

cha0s commented

This requires UPS credentials to review, correct?

Island Usurper’s picture

Island Usurper commented

Yeah, and I can't get UPS to work on the Livetest, either. This issue might be languishing for a while.

cha0s’s picture

cha0s commented

Community, we need ya! =)

aaronp’s picture

aaronp commented

Hey all. I'll try to run this patch this weekend on the site I have running with UPS credentials.

rszrama’s picture

rszrama commented

@Island Usurper - fwiw, UPS quotes are working on the 1.x Livetest, which is the version this issue was reported for anyways. Can you test it there?

aaronp’s picture

aaronp commented

Well Mr. Usurper and crew, same issue with the patch applied. Exact same behavior.

I do have general caching enabled. Is there something that I might have to adjust in order to "reset" the install, so to speak?

Island Usurper’s picture

Island Usurper commented
StatusFileSize
new all_in_two.patch4.36 KB
new all_in_two-d5.patch4.35 KB

OK, I got the patch for D5 to work on the livetest. If aaronp can confirm either works, I'll commit them. I should really figure out why I wasn't getting quotes on the 2.x livetest.

Island Usurper’s picture

Island Usurper commented
Status: Needs review » Fixed

Got these to work on the livetest, so they're committed. The problem was that my browser had autofilled the UPS account ID and password with my user login, and I must have saved it like that at some point. I ought to file a bug report for Chrome....

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.