Default Shipping Method Problem

[DrByte]

At the present time, once a shipping method is selected, it remembers that selection until the customer changes it or completes checkout or logs out.

If the selection is not valid for checkout, the customer is not allowed to complete checkout without selecting a valid option.

[toeyt]

Will there be any fixes?
Or, this is how it suppose to work...

[DrByte]

There may be a "fix" to ensure that an error message is properly displayed if the previously-selected choice is no longer valid ... I'm surprised you're not seeing any error/warning about that.

What template are you using?
What customizations have you made to the templates used for checkout pages?

[toeyt]

I have my own template. But I use default tempaltes for all checkout process. There is no customization on the checkout process.

I also tested on a demo site, http://demo.opensourcecms.com/zencart/ , and I got the same result.

I looked at the header.php of checkout_shipping page, but I couldn't find any code that sends out error. Not sure if I look at the right place...

While waiting for the fix, I am planing to make a hack to it. I am thinking that if the selected shipping module is no longer exist, I will default to the cheapest one. (This is what my client wants.):

on line 172 of /includes/modules/pages/checkout_shipping/header.php.

 
// if no shipping method has been selected, automatically select the cheapest method.
// if the modules status was changed when none were available, to save on implementing
// a javascript force-selection method, also automatically select the cheapest shipping
// method if more than one module is now enabled
  if ( !$_SESSION['shipping'] || ( $_SESSION['shipping'] && ($_SESSION['shipping'] == false) && (zen_count_shipping_modules() > 1) ) ) {
   $_SESSION['shipping'] = $shipping_modules->cheapest();
  } else {
 $x3 = false;
 foreach ($quotes as $x1) {
  if ($x1["id"] == $_SESSION['shipping']['module']) {
   $x3 = true;
  }
 }
 if (!$x3) {
  $_SESSION['shipping'] = $shipping_modules->cheapest();
 }
}

Does it look ok? Is it the right place to do the hack?

Thanks

[Alex Clarke]

I too am looking for a fix/solution to this issue.

Toeyt, did your fix work?

[Alex Clarke]

I also noticed that if changing the address (on the checkout step 1 page), to one with a different shipping zone, the cheapest new shipping method is automatically selected. I'm going to look at this code now and see if it can be copied for use elsewhere.

[toeyt]

I too am looking for a fix/solution to this issue.

Toeyt, did your fix work?

I tested it in test environment, and it worked fine.
But I didn't implement it on the live site.
The owner of the site didn't want to hack the site. He can live with the issue...

Good luck
Toey

[Alex Clarke]

I've got this working on my live site and it's running fine.

Thanks for this bit of code, it's very useful. :)