checkout_shipping template problem

[BlessIsaacola]

Did you fix this error? I am not seeing it .

Linda, the problem is not fixed. You created an account in our test environment domain/zcv1371 which is why you're not seeing the problem there.

The problem is in an production environment at www.clevershoppers.com

[Ajeh]

Of course it is ...

[Ajeh]

Set the sort order to 100 on Free Shipping Options freeoptions ...

I know there is an issue on the invisible box with Zones when conditions exist and that has been fixed in v1.3.8 ...

I have not seen it on Free Shipping Options freeoptions ...

[BlessIsaacola]

1. I'm not sure I understand why your production site and your test site wouldn't be using the same code. Thus, I don't understand why you continue to only compare the files you suspect are the cause rather than comparing everything.
Look for differences, and for missing files, and for extra files.

2. If you switch your site to classic template, do you still encounter the same problem? (assuming you've updated the template_default version of the template fix Ajeh mentioned earlier).
Are you sure you don't have an "extra" file in an overrides folder that's overriding your expected results ?

DrByte, switching the template to classic or template default (by renaming classic as Linda suggested) still shows the problem. I do not believe it's a template problem because it would have been different when switching between on template or the other.

Which update template_default version fix are you referring to?

As far as comparing the whole site, I can do that as a last resort I was hoping I won't have to compare 1,000s of file one at a time.

The fact that even when free shipping option is enabled with other option, we are seeing the wrong text for shipping methods makes me wonder which file is responsible for what text to display on checkout_shipping.php. I thought there are going to be some obvious files to compare for differences.

I abandoned this problem months ago because I wasn't getting anywhere. I might just have to change the border to white for that page (I hope that's an option). At least that will take care of the look problem since the functionality works as intended.

[BlessIsaacola]

Set the sort order to 100 on Free Shipping Options freeoptions ...

I know there is an issue on the invisible box with Zones when conditions exist and that has been fixed in v1.3.8 ...

I have not seen it on Free Shipping Options freeoptions ...

You're going to love this Linda...

When I did as suggested (changed sort order to 100) the Free Shipping Options moves to the bottom of the list on that page but the empty box is there without the title.

However, the Shipping Method text displayed reads: Please select the preferred shipping method to use on this order.

If I change the sort order back to 1 the error still exists but the shipping method text in the middle of the screen reads: This is currently the only shipping method available to use on this order.

If I change all shipping options sort order to 0 I get the same effect as changing Free Shipping Option to 100.

The only zone that we currently have is one tax zone for Georgia and it's set up similar to the default Florida Tax zone in the test environment except for different tax rate.

[Ajeh]

Number your Sort Orders by 10s or set ALL to 0s ...

If that does not fix it, try editing the tpl_checkout_shipping_default.php and add the changes:

<?php
    } else {
      $radio_buttons = 0;
      for ($i=0, $n=sizeof($quotes); $i<$n; $i++) {
      // bof: field set
      if ($quotes[$i]['module'] != '') {
?>
<fieldset>
<legend><?php echo $quotes[$i]['module']; ?>&nbsp;<?php if (isset($quotes[$i]['icon']) && zen_not_null($quotes[$i]['icon'])) { echo $quotes[$i]['icon']; } ?></legend>

<?php
        if (isset($quotes[$i]['error'])) {
?>
      <div><?php echo $quotes[$i]['error']; ?></div>
<?php
        } else {
          for ($j=0, $n2=sizeof($quotes[$i]['methods']); $j<$n2; $j++) {
// set the radio button to be checked if it is the method chosen
            $checked = (($quotes[$i]['id'] . '_' . $quotes[$i]['methods'][$j]['id'] == $_SESSION['shipping']['id']) ? true : false);

            if ( ($checked == true) || ($n == 1 && $n2 == 1) ) {
              //echo '      <div id="defaultSelected" class="moduleRowSelected">' . "\n";
            //} else {
              //echo '      <div class="moduleRow">' . "\n";
            }
?>
<?php
            if ( ($n > 1) || ($n2 > 1) ) {
?>
<div class="important forward"><?php echo $currencies->format(zen_add_tax($quotes[$i]['methods'][$j]['cost'], (isset($quotes[$i]['tax']) ? $quotes[$i]['tax'] : 0))); ?></div>
<?php
            } else {
?>
<div class="important forward"><?php echo $currencies->format(zen_add_tax($quotes[$i]['methods'][$j]['cost'], $quotes[$i]['tax'])) . zen_draw_hidden_field('shipping', $quotes[$i]['id'] . '_' . $quotes[$i]['methods'][$j]['id']); ?></div>
<?php
            }
?>

<?php echo zen_draw_radio_field('shipping', $quotes[$i]['id'] . '_' . $quotes[$i]['methods'][$j]['id'], $checked, 'id="ship-'.$quotes[$i]['id'] . '-' . $quotes[$i]['methods'][$j]['id'].'"'); ?>
<label for="ship-<?php echo $quotes[$i]['id'] . '-' . $quotes[$i]['methods'][$j]['id']; ?>" class="checkboxLabel" ><?php echo $quotes[$i]['methods'][$j]['title']; ?></label>
<!--</div>-->
<br class="clearBoth" />
<?php
            $radio_buttons++;
          }
        }
?>

</fieldset>
<?php
    }
// eof: field set
      }
    }
?>

[BlessIsaacola]

Number your Sort Orders by 10s or set ALL to 0s ...

If that does not fix it, try editing the tpl_checkout_shipping_default.php and add the changes:

<?php
    } else {
      $radio_buttons = 0;
      for ($i=0, $n=sizeof($quotes); $i<$n; $i++) {
      // bof: field set
      if ($quotes[$i]['module'] != '') {
?>
<fieldset>
<legend><?php echo $quotes[$i]['module']; ?>&nbsp;<?php if (isset($quotes[$i]['icon']) && zen_not_null($quotes[$i]['icon'])) { echo $quotes[$i]['icon']; } ?></legend>

<?php
        if (isset($quotes[$i]['error'])) {
?>
      <div><?php echo $quotes[$i]['error']; ?></div>
<?php
        } else {
          for ($j=0, $n2=sizeof($quotes[$i]['methods']); $j<$n2; $j++) {
// set the radio button to be checked if it is the method chosen
            $checked = (($quotes[$i]['id'] . '_' . $quotes[$i]['methods'][$j]['id'] == $_SESSION['shipping']['id']) ? true : false);

            if ( ($checked == true) || ($n == 1 && $n2 == 1) ) {
              //echo '      <div id="defaultSelected" class="moduleRowSelected">' . "\n";
            //} else {
              //echo '      <div class="moduleRow">' . "\n";
            }
?>
<?php
            if ( ($n > 1) || ($n2 > 1) ) {
?>
<div class="important forward"><?php echo $currencies->format(zen_add_tax($quotes[$i]['methods'][$j]['cost'], (isset($quotes[$i]['tax']) ? $quotes[$i]['tax'] : 0))); ?></div>
<?php
            } else {
?>
<div class="important forward"><?php echo $currencies->format(zen_add_tax($quotes[$i]['methods'][$j]['cost'], $quotes[$i]['tax'])) . zen_draw_hidden_field('shipping', $quotes[$i]['id'] . '_' . $quotes[$i]['methods'][$j]['id']); ?></div>
<?php
            }
?>

<?php echo zen_draw_radio_field('shipping', $quotes[$i]['id'] . '_' . $quotes[$i]['methods'][$j]['id'], $checked, 'id="ship-'.$quotes[$i]['id'] . '-' . $quotes[$i]['methods'][$j]['id'].'"'); ?>
<label for="ship-<?php echo $quotes[$i]['id'] . '-' . $quotes[$i]['methods'][$j]['id']; ?>" class="checkboxLabel" ><?php echo $quotes[$i]['methods'][$j]['title']; ?></label>
<!--</div>-->
<br class="clearBoth" />
<?php
            $radio_buttons++;
          }
        }
?>

</fieldset>
<?php
    }
// eof: field set
      }
    }
?>

Using this code change solved the problem with free shipping option box displaying when it shouldn't. However, it introduces another problem. Now FedEx Ground and FedEx Express are gone. The only shipping option showing is USPS even though I have not changed anything else in Admin-Module-Shipping

Thanks!

[BlessIsaacola]

Also wanted to report that if I add enough item so cart is => $99 Free shipping options shows up as expected.

[BlessIsaacola]

Another point of interest, if I enabled UPS shipping and use the file with your updated code, the free shipping option box issue is resolved and USPS and UPS shipping methods are available but NOT fedEx Ground or Express. It appears that your updated code only recognize default shipping methods.

[DrByte]

As far as comparing the whole site, I can do that as a last resort I was hoping I won't have to compare 1,000s of file one at a time.

Comparing 1000 files that should be largely identical takes about 10 seconds using WinMerge, once you've downloaded them. It's not a laborious task. In fact, it's the first place to start in troubleshooting. Surely you've done this while preparing for a site-upgrade before?

If one can concretely rule out 'files', then the next place to look is whether you have all the same modules installed in both sites.

Then compare configuration settings:
Try running this SQL on both databases, take the output and compare (winmerge again) the two resultsets to determine which database config settings are different:

select configuration_key, configuration_value from configuration order by configuration_key;

Save to CSV format and compare the files.