Let's get 'er done -- Path to Final Help Required.

[Dan Cole]

Initial post updated again to add those who tested the Modular Checkout modules before this thread was created. 

Tested by: @ArtcoInc @Tsimi @Moxamint

Did I miss anyone? 

If you haven't tested either of these modules yet,  please download them and let us know how it goes.

Dan

[piernas]

@Dan Cole Tested the modular checkout. Placed several orders using different options. - No problems so far. Everything works as expected.

[piernas]

BTW will modular shopping cart get into final? It would be great!

[Dan Cole]

13 minutes ago, piernas said:

@Dan Cole Tested the modular checkout. Placed several orders using different options. - No problems so far. Everything works as expected.

I figured you did. I'll add you to my next update.

Dan

 

[Dan Cole]

13 minutes ago, piernas said:

BTW will modular shopping cart get into final? It would be great!

I'm hopeful and have my fingers crossed.

Dan

[burt]

@piernas

updated module.zip

Also requires change to /admin/includes/functions/general.php
Find the function "tep_cfg_select_option" and change it to:

////
// Alias function for Store configuration values in the Administration Tool
  function tep_cfg_select_option($select_array, $key_value, $key = '', $spacer = '<br>') {
    $string = '';

    for ($i=0, $n=sizeof($select_array); $i<$n; $i++) {
      $name = ((tep_not_null($key)) ? 'configuration[' . $key . ']' : 'configuration_value');

      $string .= $spacer . '<input type="radio" name="' . $name . '" value="' . $select_array[$i] . '"';

      if ($key_value == $select_array[$i]) $string .= ' checked="checked"';

      $string .= ' /> ' . $select_array[$i];
    }

    return $string;
  }

 

[Dan Cole]

2 hours ago, burt said:

Also requires change to /admin/includes/functions/general.php
Find the function "tep_cfg_select_option" and change it to:

@burt Gary @piernas Juanma  I added this to my test site and I couldn't see any problems with the way the spacer is handled.  I assume I shouldn't see any change on the display side of things but I'm curious if my coffee has kicked in.  Oh wait, I haven't had any yet.  Off to get some coffee.  

Dan

[♥raiwa]

3 hours ago, burt said:

@piernas

updated module.zip

Also requires change to /admin/includes/functions/general.php
Find the function "tep_cfg_select_option" and change it to:

Tested and comfirmed. Looks much better.

[burt]

35 minutes ago, Dan Cole said:

@burt Gary @piernas Juanma  I added this to my test site and I couldn't see any problems with the way the spacer is handled.  I assume I shouldn't see any change on the display side of things but I'm curious if my coffee has kicked in.  Oh wait, I haven't had any yet.  Off to get some coffee.  

Dan

Updated Module should change the layout of the white boxes (when configuring a module), and it should tighten the overall table into one (instead of two).

[Dan Cole]

1 hour ago, burt said:

Updated Module should change the layout of the white boxes (when configuring a module), and it should tighten the overall table into one (instead of two).

@burt I guess I really did need that coffee.  I completely missed the fact that you updated the module files.  

It works and looks much better/cleaner.

Dan

[piernas]

@burt just came back from a trip. Will take a look tomorrow!

[piernas]

Tested and looking good :)