Permalink
Browse files

Fix issue with SEF URL's when no manfacturer is selected in advanced …

…search. Fixes issue #40
  • Loading branch information...
1 parent 04bf34d commit c95c7e7573be8649082f66259f46ac894e4d4127 @markkevans markkevans committed with haraldpdl Oct 23, 2010
Showing with 2 additions and 2 deletions.
  1. +2 −2 catalog/includes/modules/product_listing.php
@@ -102,7 +102,7 @@
$prod_list_contents .= ' <td>' . $listing['products_model'] . '</td>';
break;
case 'PRODUCT_LIST_NAME':
- if (isset($HTTP_GET_VARS['manufacturers_id'])) {
+ if (isset($HTTP_GET_VARS['manufacturers_id']) && tep_not_null($HTTP_GET_VARS['manufacturers_id'])) {
$prod_list_contents .= ' <td><a href="' . tep_href_link(FILENAME_PRODUCT_INFO, 'manufacturers_id=' . $HTTP_GET_VARS['manufacturers_id'] . '&products_id=' . $listing['products_id']) . '">' . $listing['products_name'] . '</a></td>';
} else {
$prod_list_contents .= ' <td><a href="' . tep_href_link(FILENAME_PRODUCT_INFO, ($cPath ? 'cPath=' . $cPath . '&' : '') . 'products_id=' . $listing['products_id']) . '">' . $listing['products_name'] . '</a></td>';
@@ -125,7 +125,7 @@
$prod_list_contents .= ' <td align="right">' . $listing['products_weight'] . '</td>';
break;
case 'PRODUCT_LIST_IMAGE':
- if (isset($HTTP_GET_VARS['manufacturers_id'])) {
+ if (isset($HTTP_GET_VARS['manufacturers_id']) && tep_not_null($HTTP_GET_VARS['manufacturers_id'])) {
$prod_list_contents .= ' <td align="center"><a href="' . tep_href_link(FILENAME_PRODUCT_INFO, 'manufacturers_id=' . $HTTP_GET_VARS['manufacturers_id'] . '&products_id=' . $listing['products_id']) . '">' . tep_image(DIR_WS_IMAGES . $listing['products_image'], $listing['products_name'], SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT) . '</a></td>';
} else {
$prod_list_contents .= ' <td align="center"><a href="' . tep_href_link(FILENAME_PRODUCT_INFO, ($cPath ? 'cPath=' . $cPath . '&' : '') . 'products_id=' . $listing['products_id']) . '">' . tep_image(DIR_WS_IMAGES . $listing['products_image'], $listing['products_name'], SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT) . '</a></td>';

0 comments on commit c95c7e7

Please sign in to comment.