spencermjax Posted June 3, 2007 Posted June 3, 2007 I am having trouble with an extra row being added to my products list. I cannot figure out how to get rid of it. I have posted all of the code below from the product_listing.php file. I am sure it is simply changing a lne or two but I cannot for the life of me figure it out. here is an image of the problem: here is the code: <?php /* $Id: product_listing.php,v 1.44 2003/06/09 22:49:59 hpdl Exp $ osCommerce, Open Source E-Commerce Solutions http://www.oscommerce.com Copyright (c) 2003 osCommerce Released under the GNU General Public License */ $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 == '4') ) ) { ?> <table border="0" width="100%" cellspacing="0" cellpadding="2"> <tr> <td class="smallText">QQQ<?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(); for ($col=0, $n=sizeof($column_list); $col<$n; $col++) { switch ($column_list[$col]) { case 'PRODUCT_LIST_MODEL': $lc_text = TABLE_HEADING_MODEL; $lc_align = ''; break; case 'PRODUCT_LIST_NAME': $lc_text = TABLE_HEADING_PRODUCTS; $lc_align = ''; break; case 'PRODUCT_LIST_MANUFACTURER': $lc_text = TABLE_HEADING_MANUFACTURER; $lc_align = ''; break; // EZier New Fields added case 'PRODUCT_LIST_RETAIL_PRICE': $lc_text = TABLE_HEADING_RETAIL_PRICE; $lc_align = 'right'; break; // End EZier New Fields added case 'PRODUCT_LIST_PRICE': $lc_text = TABLE_HEADING_PRICE; $lc_align = 'right'; break; // EZier New Fields added case 'PRODUCT_LIST_SAVE': $lc_text = TABLE_HEADING_SAVE; $lc_align = 'right'; break; // End EZier New Fields added case 'PRODUCT_LIST_QUANTITY': $lc_text = TABLE_HEADING_QUANTITY; $lc_align = 'right'; break; case 'PRODUCT_LIST_WEIGHT': $lc_text = TABLE_HEADING_WEIGHT; $lc_align = 'right'; break; case 'PRODUCT_LIST_IMAGE': $lc_text = TABLE_HEADING_IMAGE; $lc_align = 'center'; break; case 'PRODUCT_LIST_BUY_NOW': $lc_text = TABLE_HEADING_BUY_NOW; $lc_align = 'center'; break; } if ( ($column_list[$col] != 'PRODUCT_LIST_BUY_NOW') && ($column_list[$col] != 'PRODUCT_LIST_IMAGE') && ($column_list[$col] != 'PRODUCT_LIST_SAVE') ) { $lc_text = tep_create_sort_heading($HTTP_GET_VARS['sort'], $col+1, $lc_text); } $list_box_contents[0][] = array('align' => $lc_align, 'params' => 'class="productListing-heading"', 'text' => ' ' . $lc_text . ' '); } if ($listing_split->number_of_rows > 0) { $rows = 0; $column = 0; echo ' <table border=0 cellspacing=0 cellpadding=0> <tr><td class="bg_categories" width=496 height=37> <span class="white_text_10"> '.$breadcrumb->trail(' » ').'</span> </td></tr> <tr><td class=bg2> <table cellspacing=0 cellpadding=0 width=461 align=center> <tr><td height=8 bgcolor=#083080 align=center colspan=3></td></tr> <!-- <tr><td height=15 bgcolor=#ffffff align=center colspan=3></td></tr> --> <tr> '; $listing_query = tep_db_query($listing_split->sql_query); while ($listing = tep_db_fetch_array($listing_query)) { $product_contents = array(); $rows++; if (($rows/2) == floor($rows/2)) { $list_box_contents[] = array('params' => 'class="productListing-even"'); } else { $list_box_contents[] = array('params' => 'class="productListing-odd"'); } $cur_row = sizeof($list_box_contents) -1; for ($col=0, $n=sizeof($column_list); $col<$n; $col++) { $lc_align = ''; switch ($column_list[$col]) { case 'PRODUCT_LIST_MODEL': $lc_align = ''; $lc_text = ' ' . $listing['products_model'] . ' '; break; case 'PRODUCT_LIST_NAME': $lc_align = ''; if (isset($HTTP_GET_VARS['manufacturers_id'])) { $lc_text = '<a class=ml5 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 = '<a class=ml5 href="' . tep_href_link(FILENAME_PRODUCT_INFO, ($cPath ? 'cPath=' . $cPath . '&' : '') . 'products_id=' . $listing['products_id']) . '">' . $listing['products_name'] . '</a>'; } break; case 'PRODUCT_LIST_MANUFACTURER': $lc_align = ''; $lc_text = ' <a href="' . tep_href_link(FILENAME_DEFAULT, 'manufacturers_id=' . $listing['manufacturers_id']) . '">' . $listing['manufacturers_name'] . '</a> '; break; // EZier New Fields added case 'PRODUCT_LIST_RETAIL_PRICE': $lc_align = 'right'; if ((tep_not_null($listing['products_retail_price'])) && ($listing['products_retail_price']) > 0) { $lc_text = '<font color="#000000">Retail: ' . $currencies->display_price($listing['products_retail_price'], tep_get_tax_rate($listing['products_tax_class_id'])); } else { $lc_text = ' '; } break; // End EZier New Fields added case 'PRODUCT_LIST_PRICE': $lc_align = 'right'; if (tep_not_null($listing['specials_new_products_price'])) { $lc_text = '<span style="color:#7F7F7F;font-size:12px"><s>' . $currencies->display_price($listing['products_price'], tep_get_tax_rate($listing['products_tax_class_id'])) . '</s></span> <span class=sc2>' . $currencies->display_price($listing['specials_new_products_price'], tep_get_tax_rate($listing['products_tax_class_id'])) . '</span> '; } else { $lc_text = ' ' . $currencies->display_price($listing['products_price'], tep_get_tax_rate($listing['products_tax_class_id'])) . ' '; } break; // EZier New Fields added case 'PRODUCT_LIST_SAVE': $lc_align = 'right'; if (tep_not_null($listing['specials_new_products_price'])) { $lc_save = round(100 - (( $listing['specials_new_products_price'] / $listing['products_retail_price'] ) * 100 )); $lc_text = '<font color="red"> ' . $lc_save . '% </font>'; } else { if ((tep_not_null($listing['products_retail_price'])) && ($listing['products_retail_price']) > 0) { $lc_save = round(100 - (( $listing['products_price'] / $listing['products_retail_price'] ) * 100 )); $lc_text = '<font color="red"> ' . $lc_save . '% </font>'; } else { $lc_text = ' '; }} break; // End EZier New Fields added case 'PRODUCT_LIST_QUANTITY': $lc_align = 'right'; $lc_text = ' ' . $listing['products_quantity'] . ' '; break; case 'PRODUCT_LIST_WEIGHT': $lc_align = 'right'; $lc_text = ' ' . $listing['products_weight'] . ' '; 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) . '</a>'; } else { $lc_text = ' <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) . '</a> '; } 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> '; break; } $list_box_contents[$cur_row][] = array('align' => $lc_align, 'params' => 'class="productListing-data"', 'text' => $lc_text); $product_contents[] = $lc_text; } $product_query = tep_db_query("select products_description from " . TABLE_PRODUCTS_DESCRIPTION . " where products_id = '" . (int)$listing['products_id'] . "' and language_id = '" . (int)1 . "'"); $product = tep_db_fetch_array($product_query); $new_products['products_description'] = $product['products_description']; echo ' <td width=241 valign=top> <table border="0" cellspacing="0" cellpadding="0" width="200" align="center"> <tr> <td colspan="2" align="left" class="ml5" style="height:48px;"> '.$product_contents[2].'<br /> <br /> </td> </tr> <tr> <td width="99" style="height:95px;" align="center" valign="top" > '.$product_contents[0].' </td> <td valign="top" width="101" > <br /> <span class="cs2" >'.$product_contents[3].'</span><br /> <br /> <font color="#000000"> '.$product_contents[4].'</font> </td> </tr> <tr align="center"> <td width="101" valign="top" style="padding-top:6px; padding-left:47px;padding-bottom:8px; " colspan="2"> <a href="' . tep_href_link(FILENAME_PRODUCT_INFO, 'products_id=' . $listing['products_id']) . '">' . tep_image_button('small_view.gif') . '</a></td> </tr> </table> '; $column ++; if ($column >= 2) { $rows ++; $column = 0; echo ' </tr> <!--************** <tr><td colspan=3 height=2 bgcolor=#ffffff></td></tr> <tr><td colspan=3 bgcolor=#ffffff><img src=images/m35.gif wight=483 height=1></td></tr> <tr><td colspan=3 height=2 bgcolor=#ffffff></td></tr> **************** <tr><td colspan=3 bgcolor=#ffffff><img src=images/m35.gif wight=483 height=18></td></tr> --> <tr><td colspan=3 bgcolor=#ffffff><img src=images/m35.gif wight=483 height=18></td></tr> <tr> '; } else echo '<td width=1></td>'; } echo ' </tr> </table> </td></tr> <tr><td><img src=images/m26.gif width=496 height=22></td></tr> </table> '; //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); echo '<table border=0 cellspacing=0 cellpadding=0> <tr><td class="bg_categories" width=496 height=37> <span class="blue_text_10"> '.$breadcrumb->trail(' » ').'</span> </td></tr> <tr><td class=bg2> <table cellspacing=0 cellpadding=0 width=461 align=center> <tr><td height=8 bgcolor=#ffffff align=center colspan=3><br><br>'; new productListingBox($list_box_contents); echo '<br><img src=images/spacer.gif width=1 height=420></td></tr></table> </td></tr> <tr><td><img src=images/m26.gif width=496 height=22></td></tr> </table> '; } 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 height=20></td> </tr> <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 } ?>
Recommended Posts
Archived
This topic is now archived and is closed to further replies.