hello! our new 2.1 site has a problem see it here
[url]http://secondascent.com/sa/index.php[/url]
The manufacturers product filter isnt generating thumbnails. All old products are working fine but any new ones we get a no image for the thumbnail. If you select the big agnes brand in the left side box you will see. all old items from the import work fine this is just new items. The thumbnail is working on the product details page etc and categories page just not when you are at [url]http://secondascent.com/sa/index.php?dispatch=product_features.view&variant_id=88[/url] IE home > big agnes
Anyone have any ideas for us?
Thanks
Scott
I’d try clearing the cache and the /var/compiled/ directories.
Thanks. I got this response from CS team in case anyone else has the problem
Thank you for your request.
This problem was caused by a minor bug in CS-Cart. We have fixed it on your server by making some changes in the “product_features.php” file located in the “controllers/customer” directory.
Please check it and let me know the result.
Thank you.
—
Sincerely yours,
Kate Lazarevskaya
Technical support engineer
Interesting. If you could paste the fix in [ CODE ] blocks it would be of great help to the rest of us (in case we ever need it).
Please post the solution. I’m having the same problem
Please help, same problems here!
Think this info should be shared anyway, since it’s a bug!
hey! sorry they didnt tell me what they changed but here is the file that they said they changed
scott
```php
/***************************************************************************
-
*
- Copyright (c) 2004 Simbirsk Technologies Ltd. All rights reserved. *
-
*
- This is commercial software, only users who have purchased a valid *
- license and accept to the terms of the License Agreement can install *
- and use this program. *
-
*
****************************************************************************
- PLEASE READ THE FULL TEXT OF THE SOFTWARE LICENSE AGREEMENT IN THE *
- “copyright.txt” FILE PROVIDED WITH THIS DISTRIBUTION PACKAGE. *
****************************************************************************/
//
// $Id: product_features.php 10433 2010-08-17 11:08:30Z alexions $
//
if ( !defined(‘AREA’) ) { die(‘Access denied’); }
$_REQUEST[‘variant_id’] = empty($_REQUEST[‘variant_id’]) ? 0 : $_REQUEST[‘variant_id’];
if (empty($action)) {
$action = ‘show_all’;
}
$list = ‘features’;
if (empty($_SESSION[‘excluded_features’])) {
$_SESSION[‘excluded_features’] = array();
}
if (empty($_SESSION[‘excluded_features’])) {
$_SESSION[‘excluded_features’] = array();
}
if ($_SERVER[‘REQUEST_METHOD’] == ‘POST’) {
// Add feature to comparison list
if ($mode == ‘add_feature’) {
if (!empty($_REQUEST[‘add_features’])) {
$_SESSION[‘excluded_features’] = array_diff($_SESSION[‘excluded_features’], $_REQUEST[‘add_features’]);
}
}
return array(CONTROLLER_STATUS_OK);
}
// Add product to comparison list
if ($mode == ‘add_product’) {
if (empty($_SESSION[‘comparison_list’])) {
$_SESSION[‘comparison_list’] = array();
}
$p_id = $_REQUEST[‘product_id’];
if (!in_array($p_id, $_SESSION[‘comparison_list’])) {
array_unshift($_SESSION[‘comparison_list’], $p_id);
$added_products = array();
$added_products[$p_id][‘product_id’] = $p_id;
$added_products[$p_id][‘display_price’] = fn_get_product_price($p_id, 1, $_SESSION[‘auth’]);
$added_products[$p_id][‘amount’] = 1;
$view->assign(‘added_products’, $added_products);
$title = fn_get_lang_var(‘product_added_to_cl’);
} else {
$view->assign(‘empty_text’, fn_get_lang_var(‘product_in_compare_list’));
$title = fn_get_lang_var(‘notice’);
}
$view->assign(‘n_type’, ‘C’);
$msg = $view->display(‘views/products/components/product_notification.tpl’, false);
fn_set_notification(‘C’, $title, $msg);
if (defined(‘AJAX_REQUEST’)) {
$compared_products = array();
$_tmp = db_get_hash_array(“SELECT product_id, product FROM ?:product_descriptions WHERE product_id IN (?n) AND lang_code = ?s”, ‘product_id’, $_SESSION[‘comparison_list’], CART_LANGUAGE);
foreach ($_SESSION[‘comparison_list’] as $p_id) {
$compared_products[] = $_tmp[$p_id];
}
$view->assign(‘compared_products’, $compared_products);
$view->assign(‘new_product’, $p_id);
$view->display(‘blocks/feature_comparison.tpl’);
exit;
}
return array(CONTROLLER_STATUS_REDIRECT);
} elseif ($mode == ‘clear_list’) {
unset($_SESSION[‘comparison_list’]);
unset($_SESSION[‘excluded_features’]);
if (defined(‘AJAX_REQUEST’)) {
$view->assign(‘compared_products’, array());
$view->display(‘blocks/feature_comparison.tpl’);
exit;
}
return array(CONTROLLER_STATUS_REDIRECT);
} elseif ($mode == ‘delete_product’ && !empty($_REQUEST[‘product_id’])) {
$key = array_search ($_REQUEST[‘product_id’], $_SESSION[‘comparison_list’]);
unset($_SESSION[‘comparison_list’][$key]);
return array(CONTROLLER_STATUS_REDIRECT);
} elseif ($mode == ‘delete_feature’) {
$_SESSION[‘excluded_features’][] = $_REQUEST[‘feature_id’];
return array(CONTROLLER_STATUS_REDIRECT);
} elseif ($mode == ‘compare’) {
fn_add_breadcrumb(fn_get_lang_var(‘feature_comparison’));
if (!empty($_SESSION[‘comparison_list’])) {
$view->assign(‘comparison_data’, fn_get_product_data_for_compare($_SESSION[‘comparison_list’], $action));
$view->assign(‘total_products’, count($_SESSION[‘comparison_list’]));
}
$view->assign(‘list’, $list);
$view->assign(‘action’, $action);
if (!empty($_SESSION[‘continue_url’])) {
$view->assign(‘continue_url’, $_SESSION[‘continue_url’]);
}
} elseif ($mode == ‘view_all’) {
parse_str(substr($_REQUEST[‘q’], strpos($_REQUEST[‘q’], ‘?’) + 1), $params);
$params[‘view_all’] = ‘Y’;
$params[‘get_custom’] = ‘Y’;
if (!empty($params[‘category_id’])) {
$parent_ids = explode(‘/’, db_get_field(“SELECT id_path FROM ?:categories WHERE category_id = ?i”, $params[‘category_id’]));
if (!empty($parent_ids)) {
$cats = fn_get_category_name($parent_ids);
foreach($cats as $c_id => $c_name) {
fn_add_breadcrumb($c_name, “categories.view?category_id=$c_id”);
}
}
}
list( , $view_all_filter) = fn_get_filters_products_count($params);
fn_add_breadcrumb(db_get_field(“SELECT filter FROM ?:product_filter_descriptions WHERE filter_id = ?i AND lang_code = ?s”, $params[‘filter_id’], CART_LANGUAGE));
$view->assign(‘params’, $params);
$view->assign(‘view_all_filter’, $view_all_filter);
} elseif ($mode == ‘view’) {
fn_define(‘FILTER_BLOCKING_VARIANT’, true); // this constant means that extended features on this page should be displayed as simple
$variant_data = fn_get_product_feature_variant($_REQUEST[‘variant_id’]);
$view->assign(‘variant_data’, $variant_data);
if (!empty($_REQUEST[‘features_hash’]) || !empty($_REQUEST[‘advanced_filter’])) {
fn_add_breadcrumb($variant_data[‘variant’], “product_features.view?variant_id=$_REQUEST[variant_id]”);
fn_add_filter_ranges_breadcrumbs($_REQUEST, “product_features.view?variant_id=$_REQUEST[variant_id]”);
} else {
fn_add_breadcrumb($variant_data[‘variant’]);
}
// Override meta description/keywords
if (!empty($variant_data[‘meta_description’]) || !empty($variant_data[‘meta_keywords’])) {
$view->assign(‘meta_description’, $variant_data[‘meta_description’]);
$view->assign(‘meta_keywords’, $variant_data[‘meta_keywords’]);
}
// Override page title
if (!empty($variant_data[‘page_title’])) {
$view->assign(‘page_title’, $variant_data[‘page_title’]);
}
fn_define(‘FILTER_CUSTOM_ADVANCED’, true); // this constant means that extended filtering should be stayed on the same page
$params = $_REQUEST;
$params[‘features_hash’] = (!empty($params[‘features_hash’]) ? ($params[‘features_hash’] . ‘.’) : ‘’) . ‘V’ . $params[‘variant_id’];
if (!empty($params[‘advanced_filter’]) && $params[‘advanced_filter’] == ‘Y’) {
fn_add_breadcrumb(fn_get_lang_var(‘advanced_filter’));
list($filters) = fn_get_filters_products_count($params);
$view->assign(‘filter_features’, $filters);
}
// Get products
$params[‘type’] = ‘extended’;
list($products, $search) = fn_get_products($params, Registry::get(‘settings.Appearance.products_per_page’));
if (!empty($products)) {
foreach ($products as $k => $v) {
fn_gather_additional_product_data($products[$k], true, true, true, true, true);
}
}
$selected_layout = fn_get_products_layout($_REQUEST);
$view->assign(‘products’, $products);
$view->assign(‘search’, $search);
$view->assign(‘selected_layout’, $selected_layout);
}
function fn_get_product_data_for_compare($product_ids, $action)
{
$auth = & $_SESSION[‘auth’];
$comparison_data = array(
‘product_features’ => array(0 => array())
);
$tmp = array();
foreach ($product_ids as $product_id) {
$product_data = fn_get_product_data($product_id, $auth, CART_LANGUAGE, ‘’, false, true, false, false);
fn_gather_additional_product_data($product_data, false, false, false, true, false);
if (!empty($product_data[‘product_features’])) {
foreach ($product_data[‘product_features’] as $k => $v) {
if ($v[‘feature_type’] == ‘G’ && empty($v[‘subfeatures’])) {
continue;
}
$_features = ($v[‘feature_type’] == ‘G’) ? $v[‘subfeatures’] : array($k => $v);
$group_id = ($v[‘feature_type’] == ‘G’) ? $k : 0;
$comparison_data[‘feature_groups’][$k] = $v[‘description’];
foreach ($_features as $_k => $_v) {
if (in_array($_k, $_SESSION[‘excluded_features’])) {
if (empty($comparison_data[‘hidden_features’][$_k])) {
$comparison_data[‘hidden_features’][$_k] = $_v[‘description’];
}
continue;
}
if (empty($comparison_data[‘product_features’][$group_id][$_k])) {
$comparison_data[‘product_features’][$group_id][$_k] = $_v[‘description’];
}
}
}
}
$comparison_data[‘products’][] = $product_data;
unset($product_data);
}
if ($action != ‘show_all’ && !empty($comparison_data[‘product_features’])) {
$value = ‘’;
foreach ($comparison_data[‘product_features’] as $group_id => $v) {
foreach ($v as $feature_id => $_v) {
unset($value);
$c = ($action == ‘similar_only’) ? true : false;
foreach ($comparison_data[‘products’] as $product) {
$features = !empty($group_id) ? $product[‘product_features’][$group_id][‘subfeatures’] : $product[‘product_features’];
if (empty($features[$feature_id])) {
$c = !$c;
break;
}
if (!isset($value)) {
$value = fn_get_feature_selected_value($features[$feature_id]);
continue;
} elseif ($value != fn_get_feature_selected_value($features[$feature_id])) {
$c = !$c;
break;
}
}
if ($c == false) {
unset($comparison_data[‘product_features’][$group_id][$feature_id]);
}
}
}
}
return $comparison_data;
}
function fn_get_feature_selected_value($feature)
{
$value = null;
if (strpos(‘SMNE’, $feature[‘feature_type’]) !== false) {
if ($feature[‘feature_type’] == ‘M’) {
foreach ($feature[‘variants’] as $v) {
if ($v[‘selected’]) {
$value[] = $v[‘variant_id’];
}
}
} else {
$value = $feature[‘variant_id’];
}
} elseif (strpos(‘OD’, $feature[‘feature_type’]) !== false) {
$value = $feature[‘value_int’];
} else {
$value = $feature[‘value’];
}
return $value;
}
?> ```
You have smilies in your code 
[quote name=‘simonv74’]You have smilies in your code :)[/quote]
Fixed.