I am using the sort products contribution on OS Commerce and the
Products columns listings mixed together. Instead of them making the
table using straight SQL & html to make the product layout it uses an
array and it seems to list
once parameter more than once.

Eg 
Products Name
Product Price
Products Model
Product Model (this is the repeat)

Then the next product likewise.

If I reverse the order of the parameters I get....
Products Name
Products Model
Product Price
Product Price

and that is then repeated for each product.

Please suggest code change to rectify. first correct is correct answer.

Thanks


begin code >>>>>>>>>>>>>>>>>>>>>>>
<?php
/*
  $Id: product_listing.php,v 1.44 2003/06/09 22:49:43 hpdl Exp $
*/

  $listing_split = new splitPageResults($listing_sql,
MAX_DISPLAY_SEARCH_RESULTS, 'p.products_id');

  if ( ($listing_split->number_of_rows > 0) && (
(PREV_NEXT_BAR_LOCATION == '1') || (PREV_NEXT_BAR_LOCATION == '3') ) )
{
?>
<table border="0" width="100%" cellspacing="0" cellpadding="2">
  <tr>
    <td class="smallText"><?php echo
$listing_split->display_count(TEXT_DISPLAY_NUMBER_OF_PRODUCTS);
?></td>
    <td class="smallText" align="right"><?php echo TEXT_RESULT_PAGE .
' ' . $listing_split->display_links(MAX_DISPLAY_PAGE_LINKS,
tep_get_all_get_params(array('page', 'info', 'x', 'y'))); ?></td>
  </tr>
</table>
<?php
  }

  $list_box_contents = array();

  if ($listing_split->number_of_rows > 0) {
    $listing_query = tep_db_query($listing_split->sql_query);

    $row = 0;
    $column = 0;
    while ($listing = tep_db_fetch_array($listing_query)) {

      $product_contents = array();

      for ($col=0, $n=sizeof($column_list); $col<$n; $col++) {
        $lc_align = '';

        switch ($column_list[$col]) {
          case 'PRODUCT_LIST_MODEL':
            $lc_align = '';
            $lc_text = '&nbsp;' . $listing['products_model'] . '&nbsp;';
            break;
          case 'PRODUCT_LIST_NAME':
            $lc_align = '';
            if (isset($HTTP_GET_VARS['manufacturers_id'])) {
              $lc_text = '<a href="' .
tep_href_link(FILENAME_PRODUCT_INFO, 'manufacturers_id=' .
$HTTP_GET_VARS['manufacturers_id'] . '&products_id=' .
$listing['products_id']) . '">' . $listing['products_name'] . '</a>';
            } else {
              $lc_text = '&nbsp;<a href="' .
tep_href_link(FILENAME_PRODUCT_INFO, ($cPath ? 'cPath=' . $cPath . '&'
: '') . 'products_id=' . $listing['products_id']) . '">' .
$listing['products_name'] . '</a>&nbsp;';
            }
            break;
          case 'PRODUCT_LIST_MANUFACTURER':
            $lc_align = '';
            $lc_text = '&nbsp;<a href="' .
tep_href_link(FILENAME_DEFAULT, 'manufacturers_id=' .
$listing['manufacturers_id']) . '">' . $listing['manufacturers_name']
. '</a>&nbsp;';
            break;
          case 'PRODUCT_LIST_PRICE':
            $lc_align = 'right';
            if (tep_not_null($listing['specials_new_products_price'])) {
              $lc_text = '&nbsp;<s>' . 
$currencies->display_price($listing['products_price'],
tep_get_tax_rate($listing['products_tax_class_id'])) .
'</s>&nbsp;&nbsp;<span class="productSpecialPrice">' .
$currencies->display_price($listing['specials_new_products_price'],
tep_get_tax_rate($listing['products_tax_class_id'])) .
'</span>&nbsp;';
            } else {
              $lc_text = '&nbsp;' .
$currencies->display_price($listing['products_price'],
tep_get_tax_rate($listing['products_tax_class_id'])) . '&nbsp;';
            }
            break;
          case 'PRODUCT_LIST_QUANTITY':
            $lc_align = 'right';
            $lc_text = '&nbsp;' . $listing['products_quantity'] . '&nbsp;';
            break;
          case 'PRODUCT_LIST_WEIGHT':
            $lc_align = 'right';
            $lc_text = '&nbsp;' . $listing['products_weight'] . '&nbsp;';
            break;
          case 'PRODUCT_LIST_IMAGE':
            $lc_align = 'center';
            if (isset($HTTP_GET_VARS['manufacturers_id'])) {
              $lc_text = '<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>';
            } else {
              $lc_text = '&nbsp;<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>&nbsp;';
            }
            break;
          case 'PRODUCT_LIST_BUY_NOW':
            $lc_align = 'center';
            $lc_text = '<a href="' .
tep_href_link(basename($PHP_SELF),
tep_get_all_get_params(array('action')) .
'action=buy_now&products_id=' . $listing['products_id']) . '">' .
tep_image_button('button_buy_now.gif', IMAGE_BUTTON_BUY_NOW) .
'</a>&nbsp;';
            break;
        }
        $product_contents[] = $lc_text;

      }
      $lc_text = implode('<br>', $product_contents);
      $list_box_contents[$row][$column] = array('align' => 'center',
                                                'params' =>
'class="productListing-data"',
                                                'text'  => $lc_text);
      $column ++;
      if ($column >= 2) {
        $row ++;
        $column = 0;
      }
    }

    new productListingBox($list_box_contents);
  } else {
    $list_box_contents = array();

    $list_box_contents[0] = array('params' => 'class="productListing-odd"');
    $list_box_contents[0][] = array('params' => 'class="productListing-data"',
                                   'text' => TEXT_NO_PRODUCTS);

    new productListingBox($list_box_contents);
  }

  if ( ($listing_split->number_of_rows > 0) &&
((PREV_NEXT_BAR_LOCATION == '2') || (PREV_NEXT_BAR_LOCATION == '3')) )
{
?>
<table border="0" width="100%" cellspacing="0" cellpadding="2">
  <tr>
    <td class="smallText"><?php echo
$listing_split->display_count(TEXT_DISPLAY_NUMBER_OF_PRODUCTS);
?></td>
    <td class="smallText" align="right"><?php echo TEXT_RESULT_PAGE .
' ' . $listing_split->display_links(MAX_DISPLAY_PAGE_LINKS,
tep_get_all_get_params(array('page', 'info', 'x', 'y'))); ?></td>
	<tr><td> <?php echo '<a href="' . tep_href_link(FILENAME_DEFAULT)
.'&cPath='.$cPath. '">' . 'Back to Top'. '</a>'?></td></tr>
  </tr>
</table>
<?php
  }
?>
>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
end code

Live page
http://www.touchadjustclip.com/index.php?cPath=31

You can email OSCommerc support or write in their forum.

You can post this question in their forum.

I have done this several times with no response.