Tabbed Products ''LITE'' version - Delimiter based

scubes13:

So it is actually being output. I am checking further CSS to see if it is being hidden for some reason (although a select all does display it).

Sounds like white font on white background or something

I have not downloaded this contribution yet. Looking through the thread I wanted to know if you can have a tab that will display the reviews of the product? It appears this would be a hard coded tab like "Customers Also Purchased" tab?

Thanks

It was discussed as one of the original ideas but it will be added to a later release, so not yet.

Anyone yet adapt the Tabbed Lite 2.7a version to support Cross Sell images or Ask A Question button. Being able to place Cross Sell (We Also Recommend) into a tab would be especially cool. I have attempted but so far no success.

Below is the necessary code that inserted into tpl_product_info_display.php

Cross Sell images:

<!--bof xsell module-->
<?php
      require($template->get_template_dir('tpl_modules_xsell_products.php', DIR_WS_TEMPLATE, $current_page_base,'templates'). '/' . 'tpl_modules_xsell_products.php');
?>
<!--eof xsell module-->

Ask A Question button:

<!--bof Ask A Question button -->
<?php
  if ($flag_show_product_info_ask_question == 1) { ?>
<div id="productAskQuestionLink" class="buttonRow forward"><?php echo ($flag_show_product_info_ask_question == 1 ? '<a href="' . zen_href_link(FILENAME_ASK_QUESTION, 'products_id=' . $_GET['products_id']) . '">' . zen_image_button(BUTTON_IMAGE_ASK_QUESTION, BUTTON_ASK_QUESTION_ALT) . '</a>' : ''); ?></div>
<?php
  }
?>
<!--eof Ask A Question button -->

Woody

I just install V2.7, but got error
Parse error: parse error, unexpected $end in C:\wamp\www\includes\templates\ilync\templates\tpl_product_info_display.php on line 680

any idea?

Thanks
ERIC

ericpeng:

I just install V2.7, but got error
Parse error: parse error, unexpected $end in \templates\tpl_product_info_display.php on line 680
Updated version "2.7a" has been released but it does not look to be yet available in Zen Cart downloads area.

To obtain patch see qhome post #88 . Download and replace tpl_product_info_display.php (or you easily make the two minor edits yourself). Then report back.

Your issue is likely with:
if ($bCustomerAlsoPurchased === false) {

Should be :
if ($bCustomersAlsoPurchased === false) {

Note the "s" needs added to the end of Customer

If this does not repair and you reeive an error report identifying a line number, open the file in a text editor that shows line numbers, and identify and report back with the code on that line, and also just before and just after that line (provide a brief code snippet).

Woody

Woodymon:

Updated version "2.7a" has been released but it does not look to be yet available in Zen Cart downloads area.

To obtain patch see qhome post #88 . Download and replace tpl_product_info_display.php (or you easily make the two minor edits yourself). Then report back.

Your issue is likely with:
if ($bCustomerAlsoPurchased === false) {

Should be :
if ($bCustomersAlsoPurchased === false) {

Note the "s" needs added to the end of Customer

Woody

Thanks for your reply. I tried the bugs on #88 but still the same problem.

The 2.7a is availiable.But I got the same problem.
Parse error: parse error, unexpected $end in C:\wamp\www\includes\templates\ilync\templates\tpl_product_info_display.php on line 681

ericpeng:

The 2.7a is availiable.But I got the same problem.
Parse error: parse error, unexpected $end in C:\wamp\www\includes\templates\ilync\templates\tpl_product_info_display.php on line 681

I found the problem is in Line#134
change <? to <?php

ericpeng:

I found the problem is in Line#134
change <? to <?php
Good catch. Time for v.2.7b ;)

Just downloaded v.2.7a. Besides the two reported diffs between v2.7a and v2.7, I note one other diff which was not previosuly reported. Looks insignificant, but just checking.

2.7 line 476 shows:
echo '</div>'; //ditch-tab-pane-wrap

2.7a line 476 shows:
echo '</div id="duh">'; //ditch-tab-pane-wrap

• Woody

BWAHAHA! that was my way of finding out which div was which lol! When I was trying to find the problem with the table ending, I was trying to see how many times it looped through so I put that little identifier on there :P

I'll remove it and fix that <? tag for the next version.

I'll also take a look at cross sell. It should be pretty easy to make it a tab and Ill try to document how I do it.

CrossSell added to tabs in 10 steps :)

DEMO: http://www.unbannable.com/zen/index.php?main_page=product_info&products_id=1

Attached a zip.. inside has 10 steps to adding the CrossSell Contrib as a tab. These 10 steps can also be a template to follow for adding any others.

Also included the completed tpl file for those who don't want to take the time adding those steps by themselves.

(Also fixed the <? and duh bugs mentioned above)

Thanks guys!

Is this a bug?
In admin panel - product description :

Line1

Line2

The 'Extra Specs' tab won't work.(left hand side bottom - error on page.)
But if I add <!--#AdditionalImages#--> to the bottom, then it works fine.

This is a great contribution, it makes the website looks much much better.
I just think if the tab could be apeared in admin panel(WYSIWYG), it'll be perfect.

Thank you
ERIC

ericpeng:

Is this a bug?
In admin panel - product description :

<!--%Product Description%-->

Line1

<!--%Extra Specs%-->

Line2

The 'Extra Specs' tab won't work.(left hand side bottom - error on page.)
But if I add <!--#AdditionalImages#--> to the bottom, then it works fine.

Hmm Im trying this on my demo site and its working fine with and without the <!--#AdditionalImages#--> tag. can you post or PM the full description you are using that doesn't work.

ericpeng:

This is a great contribution, it makes the website looks much much better.
I just think if the tab could be apeared in admin panel(WYSIWYG), it'll be perfect.

It originally was. There is a TabbedProducts nonlite version that creates a whole new product type. But that requires a database field to be added for each tab you want. And it was too hard to customize.. thats why I switched to the this "tag" method so it could be done on a per-product level.

ECKK.. i see it now. I removed the text before the tabs and i see the problem you are having now.. crraaaaappp

Ok ill take a look.. you can blame woodymon for needing all these extra features (LOL j/k) :)

[QUOTE=qhome]Hmm Im trying this on my demo site and its working fine with and without the <!--#AdditionalImages#--> tag. can you post or PM the full description you are using that doesn't work.

I tried again and same problem.

Detailed Product Description
512M Memory

This is another tab named test

That's all I have in products description.

ericpeng:

Is this a bug?
In admin panel - product description :

<!--%Product Description%-->

Line1

<!--%Extra Specs%-->

Line2

The 'Extra Specs' tab won't work.(left hand side bottom - error on page.)
But if I add <!--#AdditionalImages#--> to the bottom, then it works fine.

FIXED:

FIND (TWICE!!) :

if (($chkTabStart && $chkTabEnd) || ($bAdditionalImages || $bProductDetails || $bCustomersAlsoPurchased)) {

REPLACE WITH (TWICE!!) :

if (($chkTabStart >= 0 && $chkTabEnd >= 0) || ($bAdditionalImages || $bProductDetails || $bCustomersAlsoPurchased) {

I was using absolute string value check.. but if the tags weren't used, it sets the value of the chktabstart and end to -1 which is infact a real value...

Thanks eric :)

qhome:

FIXED:

FIND (TWICE!!) :

if (($chkTabStart && $chkTabEnd) || ($bAdditionalImages || $bProductDetails || $bCustomersAlsoPurchased)) {

> 
> **REPLACE WITH (TWICE!!) :**
> ```
if (($chkTabStart >= 0 && $chkTabEnd >= 0) || ($bAdditionalImages || $bProductDetails || $bCustomersAlsoPurchased) {

I was using absolute string value check.. but if the tags weren't used, it sets the value of the chktabstart and end to -1 which is infact a real value...

Thanks eric :)

I changed on Line# 261 and Line# 415 and it works.
Thank you so much.
I think change the line #133 to <br class="clearBoth" /> will be a good idea(in case of big prodcut image).

Thansks again
ERIC

FYI - don't know if you've noticed this yet (haven't read the whole thread) but the first tab fits nicely in a small browser width but going to the second tab makes it jump off the right side of the screen.