Stealth1

No more support for this?

No more support for this?

Let me know what is wrong please so we can get this fixed.
 
Thanks :)

No more support for this?

?????

That only seems to have made things worse. Now it is only doing the original popupwindow and not the lightbox anymore
 
I have the following code above if (!tep_db_num_rows($product_info_query)) {

// BOF: Additional Images: Added: , p.products_image_med, p.products_image_pop, p.products_image_description $product_info_query = tep_db_query("select p.products_id, p.products_model, p.products_image, p.products_image_med, p.products_image_pop, p.products_image_description, p.products_price, p.products_tax_class_id, pd.products_name from " . TABLE_PRODUCTS . " p, " . TABLE_PRODUCTS_DESCRIPTION . " pd where p.products_id = '" . (int)$HTTP_GET_VARS['products_id'] . "' and p.products_status = '1' and p.products_id = pd.products_id and pd.language_id = '" . (int)$languages_id . "'"); // EOF: Additional Images
 
I have the following above the </head> and I also have a setup without it but both are behaving the same

<?php if (ENABLE_LIGHTBOX == 'true') { ?> <!-- BOF: Lightbox Contribution --> <script type="text/javascript" src="lightbox/prototype.js"></script> <script type="text/javascript" src="lightbox/scriptaculous.js?load=effects"></script> <link rel="stylesheet" href="lightbox/lightbox.css" type="text/css" media="screen"> <script type="text/javascript" src="lightbox/builder.js"></script> <script type="text/javascript" src="lightbox/lightbox.js"></script> <!-- EOF: Lightbox Contribution --> <?php } ?> <?php // EOF: Additional Images ?>
 
 
And I have this code just below the line <?php if (tep_not_null($product_info['products_image'])) { ?>

<?php // BOF: Additional Images: ?> <script language="javascript"><!-- document.write('<?php echo '<a href="javascript:popupWindow(\\\'' . tep_href_link(FILENAME_POPUP_ADD_IMAGE, 'pID=' . $product_info['products_id']) . '\\\')">' . tep_image(DIR_WS_IMAGES . (!empty($product_info['products_image_med'])?$product_info['products_image_med']:(!empty($product_info['products_image_pop'])?$product_info['products_image_pop']:(!empty($product_info['products_image'])?$product_info['products_image']:''))), addslashes($product_info['products_name']), (ADDIMAGES_RESTRICT_PARENT=='true'?DISPLAY_IMAGE_WIDTH:''), (ADDIMAGES_RESTRICT_PARENT=='true'?DISPLAY_IMAGE_HEIGHT:''), 'hspace="5" vspace="5"') . '<br>' . (!empty($product_info['products_image_description'])?$product_info['products_image_description']:TEXT_CLICK_TO_ENLARGE) . '</a>'; ?>'); //--></script> <noscript> <?php echo '<a href="' . tep_href_link(DIR_WS_IMAGES . (!empty($product_info['products_image_pop'])?$product_info['products_image_pop']:$product_info['products_image'])) . '" target="_blank">' . tep_image(DIR_WS_IMAGES . (!empty($product_info['products_image_med'])?$product_info['products_image_med']:(!empty($product_info['products_image_pop'])?$product_info['products_image_pop']:(!empty($product_info['products_image'])?$product_info['products_image']:''))), $product_info['products_name'], (ADDIMAGES_RESTRICT_PARENT=='true'?DISPLAY_IMAGE_WIDTH:''), (ADDIMAGES_RESTRICT_PARENT=='true'?DISPLAY_IMAGE_HEIGHT:''), 'hspace="5" vspace="5"') . '<br>' . (!empty($product_info['products_image_description'])?$product_info['products_image_description']:TEXT_CLICK_TO_ENLARGE) . '</a>'; ?> </noscript> <?php /* OLD CODE <script language="javascript"><!-- document.write('<?php echo '<a href="javascript:popupWindow(\\\'' . tep_href_link(FILENAME_POPUP_IMAGE, 'pID=' . $product_info['products_id']) . '\\\')">' . tep_image(DIR_WS_IMAGES . $product_info['products_image'], addslashes($product_info['products_name']), SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT, 'hspace="5" vspace="5"') . '<br>' . TEXT_CLICK_TO_ENLARGE . '</a>'; ?>'); //--></script> <noscript> <?php echo '<a href="' . tep_href_link(DIR_WS_IMAGES . $product_info['products_image']) . '" target="_blank">' . tep_image(DIR_WS_IMAGES . $product_info['products_image'], $product_info['products_name'], SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT, 'hspace="5" vspace="5"') . '<br>' . TEXT_CLICK_TO_ENLARGE . '</a>'; ?> </noscript> */ // EOF: Additional Images: ?>

it's going to be the exact same changes that were as to the products_reviews.php. If you compare the products_reviews.php package file with the stock RC2a file, the changes should be fairly obvious. couple of additions to the fields list in the query, and replace the popup code.

simply because you don't get a reply doesn't mean support has ceased. Support doesn't necessarily constitute writing new functionality for each person that asks. This forum is all volunteer. I will add this to the suggestions for when I have further time to work on the project. In the meantime, feel free to investigate the solution yourself, and post your suggestion.
 
you're really not suppose to bump your post. forum rules suggest offering additional information if you wish to revisit a post.

That only seems to have made things worse. Now it is only doing the original popupwindow and not the lightbox anymore
 
I have the following code above if (!tep_db_num_rows($product_info_query)) {

// BOF: Additional Images: Added: , p.products_image_med, p.products_image_pop, p.products_image_description $product_info_query = tep_db_query("select p.products_id, p.products_model, p.products_image, p.products_image_med, p.products_image_pop, p.products_image_description, p.products_price, p.products_tax_class_id, pd.products_name from " . TABLE_PRODUCTS . " p, " . TABLE_PRODUCTS_DESCRIPTION . " pd where p.products_id = '" . (int)$HTTP_GET_VARS['products_id'] . "' and p.products_status = '1' and p.products_id = pd.products_id and pd.language_id = '" . (int)$languages_id . "'"); // EOF: Additional Images
 
I have the following above the </head> and I also have a setup without it but both are behaving the same

<?php if (ENABLE_LIGHTBOX == 'true') { ?> <!-- BOF: Lightbox Contribution --> <script type="text/javascript" src="lightbox/prototype.js"></script> <script type="text/javascript" src="lightbox/scriptaculous.js?load=effects"></script> <link rel="stylesheet" href="lightbox/lightbox.css" type="text/css" media="screen"> <script type="text/javascript" src="lightbox/builder.js"></script> <script type="text/javascript" src="lightbox/lightbox.js"></script> <!-- EOF: Lightbox Contribution --> <?php } ?> <?php // EOF: Additional Images ?>
 
 
And I have this code just below the line <?php if (tep_not_null($product_info['products_image'])) { ?>

<?php // BOF: Additional Images: ?> <script language="javascript"><!-- document.write('<?php echo '<a href="javascript:popupWindow(\\\'' . tep_href_link(FILENAME_POPUP_ADD_IMAGE, 'pID=' . $product_info['products_id']) . '\\\')">' . tep_image(DIR_WS_IMAGES . (!empty($product_info['products_image_med'])?$product_info['products_image_med']:(!empty($product_info['products_image_pop'])?$product_info['products_image_pop']:(!empty($product_info['products_image'])?$product_info['products_image']:''))), addslashes($product_info['products_name']), (ADDIMAGES_RESTRICT_PARENT=='true'?DISPLAY_IMAGE_WIDTH:''), (ADDIMAGES_RESTRICT_PARENT=='true'?DISPLAY_IMAGE_HEIGHT:''), 'hspace="5" vspace="5"') . '<br>' . (!empty($product_info['products_image_description'])?$product_info['products_image_description']:TEXT_CLICK_TO_ENLARGE) . '</a>'; ?>'); //--></script> <noscript> <?php echo '<a href="' . tep_href_link(DIR_WS_IMAGES . (!empty($product_info['products_image_pop'])?$product_info['products_image_pop']:$product_info['products_image'])) . '" target="_blank">' . tep_image(DIR_WS_IMAGES . (!empty($product_info['products_image_med'])?$product_info['products_image_med']:(!empty($product_info['products_image_pop'])?$product_info['products_image_pop']:(!empty($product_info['products_image'])?$product_info['products_image']:''))), $product_info['products_name'], (ADDIMAGES_RESTRICT_PARENT=='true'?DISPLAY_IMAGE_WIDTH:''), (ADDIMAGES_RESTRICT_PARENT=='true'?DISPLAY_IMAGE_HEIGHT:''), 'hspace="5" vspace="5"') . '<br>' . (!empty($product_info['products_image_description'])?$product_info['products_image_description']:TEXT_CLICK_TO_ENLARGE) . '</a>'; ?> </noscript> <?php /* OLD CODE <script language="javascript"><!-- document.write('<?php echo '<a href="javascript:popupWindow(\\\'' . tep_href_link(FILENAME_POPUP_IMAGE, 'pID=' . $product_info['products_id']) . '\\\')">' . tep_image(DIR_WS_IMAGES . $product_info['products_image'], addslashes($product_info['products_name']), SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT, 'hspace="5" vspace="5"') . '<br>' . TEXT_CLICK_TO_ENLARGE . '</a>'; ?>'); //--></script> <noscript> <?php echo '<a href="' . tep_href_link(DIR_WS_IMAGES . $product_info['products_image']) . '" target="_blank">' . tep_image(DIR_WS_IMAGES . $product_info['products_image'], $product_info['products_name'], SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT, 'hspace="5" vspace="5"') . '<br>' . TEXT_CLICK_TO_ENLARGE . '</a>'; ?> </noscript> */ // EOF: Additional Images: ?>

No more support for this?

Let me know what is wrong please so we can get this fixed.
 
Thanks :)

?????

Let me know what is wrong please so we can get this fixed.
 
Thanks :)

Let me know what is wrong please so we can get this fixed.
 
Thanks :)

?????

That only seems to have made things worse. Now it is only doing the original popupwindow and not the lightbox anymore
 
I have the following code above if (!tep_db_num_rows($product_info_query)) {

// BOF: Additional Images: Added: , p.products_image_med, p.products_image_pop, p.products_image_description $product_info_query = tep_db_query("select p.products_id, p.products_model, p.products_image, p.products_image_med, p.products_image_pop, p.products_image_description, p.products_price, p.products_tax_class_id, pd.products_name from " . TABLE_PRODUCTS . " p, " . TABLE_PRODUCTS_DESCRIPTION . " pd where p.products_id = '" . (int)$HTTP_GET_VARS['products_id'] . "' and p.products_status = '1' and p.products_id = pd.products_id and pd.language_id = '" . (int)$languages_id . "'"); // EOF: Additional Images
 
I have the following above the </head> and I also have a setup without it but both are behaving the same

<?php if (ENABLE_LIGHTBOX == 'true') { ?> <!-- BOF: Lightbox Contribution --> <script type="text/javascript" src="lightbox/prototype.js"></script> <script type="text/javascript" src="lightbox/scriptaculous.js?load=effects"></script> <link rel="stylesheet" href="lightbox/lightbox.css" type="text/css" media="screen"> <script type="text/javascript" src="lightbox/builder.js"></script> <script type="text/javascript" src="lightbox/lightbox.js"></script> <!-- EOF: Lightbox Contribution --> <?php } ?> <?php // EOF: Additional Images ?>
 
 
And I have this code just below the line <?php if (tep_not_null($product_info['products_image'])) { ?>

<?php // BOF: Additional Images: ?> <script language="javascript"><!-- document.write('<?php echo '<a href="javascript:popupWindow(\\\'' . tep_href_link(FILENAME_POPUP_ADD_IMAGE, 'pID=' . $product_info['products_id']) . '\\\')">' . tep_image(DIR_WS_IMAGES . (!empty($product_info['products_image_med'])?$product_info['products_image_med']:(!empty($product_info['products_image_pop'])?$product_info['products_image_pop']:(!empty($product_info['products_image'])?$product_info['products_image']:''))), addslashes($product_info['products_name']), (ADDIMAGES_RESTRICT_PARENT=='true'?DISPLAY_IMAGE_WIDTH:''), (ADDIMAGES_RESTRICT_PARENT=='true'?DISPLAY_IMAGE_HEIGHT:''), 'hspace="5" vspace="5"') . '<br>' . (!empty($product_info['products_image_description'])?$product_info['products_image_description']:TEXT_CLICK_TO_ENLARGE) . '</a>'; ?>'); //--></script> <noscript> <?php echo '<a href="' . tep_href_link(DIR_WS_IMAGES . (!empty($product_info['products_image_pop'])?$product_info['products_image_pop']:$product_info['products_image'])) . '" target="_blank">' . tep_image(DIR_WS_IMAGES . (!empty($product_info['products_image_med'])?$product_info['products_image_med']:(!empty($product_info['products_image_pop'])?$product_info['products_image_pop']:(!empty($product_info['products_image'])?$product_info['products_image']:''))), $product_info['products_name'], (ADDIMAGES_RESTRICT_PARENT=='true'?DISPLAY_IMAGE_WIDTH:''), (ADDIMAGES_RESTRICT_PARENT=='true'?DISPLAY_IMAGE_HEIGHT:''), 'hspace="5" vspace="5"') . '<br>' . (!empty($product_info['products_image_description'])?$product_info['products_image_description']:TEXT_CLICK_TO_ENLARGE) . '</a>'; ?> </noscript> <?php /* OLD CODE <script language="javascript"><!-- document.write('<?php echo '<a href="javascript:popupWindow(\\\'' . tep_href_link(FILENAME_POPUP_IMAGE, 'pID=' . $product_info['products_id']) . '\\\')">' . tep_image(DIR_WS_IMAGES . $product_info['products_image'], addslashes($product_info['products_name']), SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT, 'hspace="5" vspace="5"') . '<br>' . TEXT_CLICK_TO_ENLARGE . '</a>'; ?>'); //--></script> <noscript> <?php echo '<a href="' . tep_href_link(DIR_WS_IMAGES . $product_info['products_image']) . '" target="_blank">' . tep_image(DIR_WS_IMAGES . $product_info['products_image'], $product_info['products_name'], SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT, 'hspace="5" vspace="5"') . '<br>' . TEXT_CLICK_TO_ENLARGE . '</a>'; ?> </noscript> */ // EOF: Additional Images: ?>

?????

That only seems to have made things worse. Now it is only doing the original popupwindow and not the lightbox anymore
 
I have the following code above if (!tep_db_num_rows($product_info_query)) {

// BOF: Additional Images: Added: , p.products_image_med, p.products_image_pop, p.products_image_description $product_info_query = tep_db_query("select p.products_id, p.products_model, p.products_image, p.products_image_med, p.products_image_pop, p.products_image_description, p.products_price, p.products_tax_class_id, pd.products_name from " . TABLE_PRODUCTS . " p, " . TABLE_PRODUCTS_DESCRIPTION . " pd where p.products_id = '" . (int)$HTTP_GET_VARS['products_id'] . "' and p.products_status = '1' and p.products_id = pd.products_id and pd.language_id = '" . (int)$languages_id . "'"); // EOF: Additional Images
 
I have the following above the </head> and I also have a setup without it but both are behaving the same

<?php if (ENABLE_LIGHTBOX == 'true') { ?> <!-- BOF: Lightbox Contribution --> <script type="text/javascript" src="lightbox/prototype.js"></script> <script type="text/javascript" src="lightbox/scriptaculous.js?load=effects"></script> <link rel="stylesheet" href="lightbox/lightbox.css" type="text/css" media="screen"> <script type="text/javascript" src="lightbox/builder.js"></script> <script type="text/javascript" src="lightbox/lightbox.js"></script> <!-- EOF: Lightbox Contribution --> <?php } ?> <?php // EOF: Additional Images ?>
 
 
And I have this code just below the line <?php if (tep_not_null($product_info['products_image'])) { ?>

<?php // BOF: Additional Images: ?> <script language="javascript"><!-- document.write('<?php echo '<a href="javascript:popupWindow(\\\'' . tep_href_link(FILENAME_POPUP_ADD_IMAGE, 'pID=' . $product_info['products_id']) . '\\\')">' . tep_image(DIR_WS_IMAGES . (!empty($product_info['products_image_med'])?$product_info['products_image_med']:(!empty($product_info['products_image_pop'])?$product_info['products_image_pop']:(!empty($product_info['products_image'])?$product_info['products_image']:''))), addslashes($product_info['products_name']), (ADDIMAGES_RESTRICT_PARENT=='true'?DISPLAY_IMAGE_WIDTH:''), (ADDIMAGES_RESTRICT_PARENT=='true'?DISPLAY_IMAGE_HEIGHT:''), 'hspace="5" vspace="5"') . '<br>' . (!empty($product_info['products_image_description'])?$product_info['products_image_description']:TEXT_CLICK_TO_ENLARGE) . '</a>'; ?>'); //--></script> <noscript> <?php echo '<a href="' . tep_href_link(DIR_WS_IMAGES . (!empty($product_info['products_image_pop'])?$product_info['products_image_pop']:$product_info['products_image'])) . '" target="_blank">' . tep_image(DIR_WS_IMAGES . (!empty($product_info['products_image_med'])?$product_info['products_image_med']:(!empty($product_info['products_image_pop'])?$product_info['products_image_pop']:(!empty($product_info['products_image'])?$product_info['products_image']:''))), $product_info['products_name'], (ADDIMAGES_RESTRICT_PARENT=='true'?DISPLAY_IMAGE_WIDTH:''), (ADDIMAGES_RESTRICT_PARENT=='true'?DISPLAY_IMAGE_HEIGHT:''), 'hspace="5" vspace="5"') . '<br>' . (!empty($product_info['products_image_description'])?$product_info['products_image_description']:TEXT_CLICK_TO_ENLARGE) . '</a>'; ?> </noscript> <?php /* OLD CODE <script language="javascript"><!-- document.write('<?php echo '<a href="javascript:popupWindow(\\\'' . tep_href_link(FILENAME_POPUP_IMAGE, 'pID=' . $product_info['products_id']) . '\\\')">' . tep_image(DIR_WS_IMAGES . $product_info['products_image'], addslashes($product_info['products_name']), SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT, 'hspace="5" vspace="5"') . '<br>' . TEXT_CLICK_TO_ENLARGE . '</a>'; ?>'); //--></script> <noscript> <?php echo '<a href="' . tep_href_link(DIR_WS_IMAGES . $product_info['products_image']) . '" target="_blank">' . tep_image(DIR_WS_IMAGES . $product_info['products_image'], $product_info['products_name'], SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT, 'hspace="5" vspace="5"') . '<br>' . TEXT_CLICK_TO_ENLARGE . '</a>'; ?> </noscript> */ // EOF: Additional Images: ?>

Found your error in the instructions, if done as decribed in your instructions the code ends up like below.
 

eval('$keys .= ' . $value['set_function'] . "'" . $value['value'] . "', '" . $key . "');"); // STS V4.6 drop start } else { if($key == 'MODULE_STS_TEMPLATE_FOLDER'){ $dir_name = array(); if ($handle = opendir('../'.MODULE_STS_TEMPLATES_FOLDER.'/')) { while (false !== ($file = readdir($handle))) { $pos = strpos($file, "."); if($pos === false){ $dir_name[] = array("id" => $file, "text" => $file); $i++; } } closedir($handle); } $keys .= tep_draw_pull_down_menu('configuration[MODULE_STS_TEMPLATE_FOLDER]', $dir_name, $value['value']); }else{ $keys .= tep_draw_input_field('configuration[' . $key . ']', $value['value']); } } // STS V4.6 drop end $keys .= tep_draw_input_field('configuration[' . $key . ']', $value['value']); } $keys .= '<br><br>'; } $keys = substr($keys, 0, strrpos($keys, '<br><br>'));
 
This means that the following line is shown twice
 

$keys .= tep_draw_input_field('configuration[' . $key . ']', $value['value']); }
 
Another case of incorrect instructions...
 

What you proved is a clean install of 4.6 works on a stock OSC store, it didn't show that upgrading from 4.5.8 to 4.6 as described in the instructions works which is what the problem at hand is.
 
It is one change in one place, not hard to do and I know it was done correctly.

Oh I and I just re-uploaded the 4.5.8 STS folder and removed the code added to admin/modules.php for 4.6 and my site is back working...

STS Module was disabled before the update, I uploaded the STS folder to the correct directories overwriting the previous 4.5.8 files.
 
I then did the following as per your instructions:
 
Find:

switch ($set) {
 
Add After

// START STS 4.1 case 'sts': $module_type = 'sts'; $module_directory = DIR_FS_CATALOG_MODULES . 'sts/'; $module_key = 'MODULE_STS_INSTALLED'; define('HEADING_TITLE', HEADING_TITLE_MODULES_STS); break; // END STS 4.1
 
Find:

eval('$keys .= ' . $value['set_function'] . "'" . $value['value'] . "', '" . $key . "');"); } else {
 
Replace With:

eval('$keys .= ' . $value['set_function'] . "'" . $value['value'] . "', '" . $key . "');"); // STS V4.6 drop start } else { if($key == 'MODULE_STS_TEMPLATE_FOLDER'){ $dir_name = array(); if ($handle = opendir('../'.MODULE_STS_TEMPLATES_FOLDER.'/')) { while (false !== ($file = readdir($handle))) { $pos = strpos($file, "."); if($pos === false){ $dir_name[] = array("id" => $file, "text" => $file); $i++; } } closedir($handle); } $keys .= tep_draw_pull_down_menu('configuration[MODULE_STS_TEMPLATE_FOLDER]', $dir_name, $value['value']); }else{ $keys .= tep_draw_input_field('configuration[' . $key . ']', $value['value']); } } // STS V4.6 drop end
 
Everything was fine before I did this upgrade so something in the code has screwed it up... Running the installation on a stock OSC is not proving anything, install 4.5.8 and then do the upgrade as per your instructions and let me know then.

Hi Chris
 
KissMT uses "shortwords" to intelligently build a correct page title. However .. because it is automated there will always be instances where you do not want it to display in the manner dictated by the system .. at this time there is no way to change this on a "one title" basis but I'll consider it for future modifications.

The whole point of category and manufacturers descriptions is that it gives these pages content. Without such content these pages are irrelevant to a search engine and won't have any "weight" in the SERPS at all.

Disable it! why? it's perfect, as long as you build the description correctly.
 
You dictate the "on page text" including the keywords ( making sure the first 90 characters are keyword rich ). The H1 page title, meta title, description and keywords match exactly making a powerful page.