ZCA Bootstrap Template 1.0 (BETA)

[badarac]

I've installed the template from github on 1.5.5f . Everything works ok so far with the zca bootstrap template installed. I installed the clone template plugin and created a copy of the bootstrap template. I'm getting an error when the product listing page splits the pages. It works fine on the bootstrap template but not the clone. I went through the custom template folders to make sure that all the contents had been copied. They look identical. I deleted the bootstrap and clone folders and started from scratch with the same results. Here is the debug file:

 PHP Fatal error:  Class 'zca_splitPageResults' not found in /includes/modules/shaft/product_listing.php on line 32 

The zca folder is in includes/classes. Any ideas?

[badarac]

I found the problem. In includes/functions/zca_bootstrap_functions.php there is a test for the template bootstrap being loaded.

function zca_bootstrap_active()
{
    return (isset($GLOBALS['template_dir']) && $GLOBALS['template_dir'] == 'bootstrap');
} 

I edited that file and put the name of the template directory and it works fine. Not sure what the best solution is to this one. For now I'll just continue this way.

[lat9]

I found the problem. In includes/functions/zca_bootstrap_functions.php there is a test for the template bootstrap being loaded.

function zca_bootstrap_active()
{
    return (isset($GLOBALS['template_dir']) && $GLOBALS['template_dir'] == 'bootstrap');
} 

I edited that file and put the name of the template directory and it works fine. Not sure what the best solution is to this one. For now I'll just continue this way.

Darn it, I didn't consider template-cloning. That function is intended to "direct" the template's observer-class as to whether or not it's to apply the template modifications to the various price and HTML elements.

I'll need to ponder this to determine the "best" solution.

[lat9]

GitHub issue created to identify this condition.

[badarac]

A class was removed hiddenField. I found this on the shipping estimator popup where it displayed an additional field under the state pulldown. I added it back to the css .hiddenField {display:none;}. That class is used in a number of places.

[badarac]

A formatting issue on the checkout_shipping page. If the UPS or USPS modules are used for shipping and multiple shipping options are presented, the names are shown on the left and the prices are floated to the right.

[rbarbour]

A class was removed hiddenField. I found this on the shipping estimator popup where it displayed an additional field under the state pulldown. I added it back to the css .hiddenField {display:none;}. That class is used in a number of places.

I have been working on this, converting to us BS classes that is......

[barco57]

Darn it, I didn't consider template-cloning. That function is intended to "direct" the template's observer-class as to whether or not it's to apply the template modifications to the various price and HTML elements.

I'll need to ponder this to determine the "best" solution.

how about an extra entry of some sort in the template_info.php with comments that say "if this is entry is changed or deleted the template will fail to function correctly. Added to allow cloning of the template and/or changing the name of the template."

[lat9]

how about an extra entry of some sort in the template_info.php with comments that say "if this is entry is changed or deleted the template will fail to function correctly. Added to allow cloning of the template and/or changing the name of the template."

Hmm, that might be the plan. I was considering a similar update in a template-specific language file, but that might work too!

Thanks for the suggestion, @barco57!

[rbarbour]

A formatting issue on the checkout_shipping page. If the UPS or USPS modules are used for shipping and multiple shipping options are presented, the names are shown on the left and the prices are floated to the right.

A fix was provided for this in the /includes/templates/bootstrap/templates/tpl_checkout_shipping_default.php file.

This has been updated in GITHUB.

Nice Catch @badarac