Cuando estamos instalando oscommerce y queremos mejorarlo con una plantilla o codigo propio puede que no encontremos que al dar click en el logo o en el nombre de una subcategoria , se produce el siguiente error:
1054 – Unknown column ‘p.products_id’ in ‘on clause’
select count(p.products_id) as total from products_description pd, products p left join manufacturers m on p.manufacturers_id = m.manufacturers_id, products_to_categories p2c left join specials s on p.products_id = s.products_id where p.products_status = ‘1’ and p.products_id = p2c.products_id and pd.products_id = p2c.products_id and pd.language_id = ‘3’ and p2c.categories_id = ’19’
[TEP STOP]
Segun la web oficial de oscommer es un bug que aparentemente ya esta corregido, y que sucede al correr una version de oscommerce preparada para php 4 en php5 , ahora bien, si bien oscommerce es un excelente paquete para el ecommerce facil de instalar y un poco menos facil de rediseñar su aspecto y colocarle lo que llamamos contribuciones o modulos adicionales, su codigo fuente esta un poco desfasado en el tiempo, aun existiendo una version propuesta oscommerce 3, se sigue viendo codigo de llamadas a la base de datos en archivos que son de la presentacion, no tiene un modelo MVC definido, asi mismo ocurre en WordPress, que siendo un excelente paquete para publicar contenidos web, no presenta un MVC en su estructura, entre otras cosas, aun asi, existen soluciones para las cosas y aqui dejo el codigo para solucionar el error en cuestion de este articulo:
En el archivo index.php
Ubicar estas lineas de codigo:
// show the products of a specified manufacturer
if (isset($HTTP_GET_VARS[‘manufacturers_id’])) {
if (isset($HTTP_GET_VARS[‘filter_id’]) && tep_not_null($HTTP_GET_VARS[‘filter_id’])) {
// We are asked to show only a specific category
$listing_sql = «select » . $select_column_list . » p.products_id, p.manufacturers_id, p.products_price, p.products_tax_class_id, IF(s.status, s.specials_new_products_price, NULL) as specials_new_products_price, IF(s.status, s.specials_new_products_price, p.products_price) as final_price from » . TABLE_PRODUCTS . » p, » . TABLE_PRODUCTS_DESCRIPTION . » pd, » . TABLE_MANUFACTURERS . » m, » . TABLE_PRODUCTS_TO_CATEGORIES . » p2c left join » . TABLE_SPECIALS . » s on p.products_id = s.products_id where p.products_status = ‘1’ and p.manufacturers_id = m.manufacturers_id and m.manufacturers_id = ‘» . (int)$HTTP_GET_VARS[‘manufacturers_id’] . «‘ and p.products_id = p2c.products_id and pd.products_id = p2c.products_id and pd.language_id = ‘» . (int)$languages_id . «‘ and p2c.categories_id = ‘» . (int)$HTTP_GET_VARS[‘filter_id’] . «‘»;
} else {
// We show them all
$listing_sql = «select » . $select_column_list . » p.products_id, p.manufacturers_id, p.products_price, p.products_tax_class_id, IF(s.status, s.specials_new_products_price, NULL) as specials_new_products_price, IF(s.status, s.specials_new_products_price, p.products_price) as final_price from » . TABLE_PRODUCTS . » p, » . TABLE_PRODUCTS_DESCRIPTION . » pd, » . TABLE_MANUFACTURERS . » m left join » . TABLE_SPECIALS . » s on p.products_id = s.products_id where p.products_status = ‘1’ and pd.products_id = p.products_id and pd.language_id = ‘» . (int)$languages_id . «‘ and p.manufacturers_id = m.manufacturers_id and m.manufacturers_id = ‘» . (int)$HTTP_GET_VARS[‘manufacturers_id’] . «‘»;
}
} else {
// show the products in a given categorie
if (isset($HTTP_GET_VARS[‘filter_id’]) && tep_not_null($HTTP_GET_VARS[‘filter_id’])) {
// We are asked to show only specific catgeory
$listing_sql = «select » . $select_column_list . » p.products_id, p.manufacturers_id, p.products_price, p.products_tax_class_id, IF(s.status, s.specials_new_products_price, NULL) as specials_new_products_price, IF(s.status, s.specials_new_products_price, p.products_price) as final_price from » . TABLE_PRODUCTS . » p, » . TABLE_PRODUCTS_DESCRIPTION . » pd, » . TABLE_MANUFACTURERS . » m, » . TABLE_PRODUCTS_TO_CATEGORIES . » p2c left join » . TABLE_SPECIALS . » s on p.products_id = s.products_id where p.products_status = ‘1’ and p.manufacturers_id = m.manufacturers_id and m.manufacturers_id = ‘» . (int)$HTTP_GET_VARS[‘filter_id’] . «‘ and p.products_id = p2c.products_id and pd.products_id = p2c.products_id and pd.language_id = ‘» . (int)$languages_id . «‘ and p2c.categories_id = ‘» . (int)$current_category_id . «‘»;
} else {
// We show them all
$listing_sql = «select » . $select_column_list . » p.products_id, p.manufacturers_id, p.products_price, p.products_tax_class_id, IF(s.status, s.specials_new_products_price, NULL) as specials_new_products_price, IF(s.status, s.specials_new_products_price, p.products_price) as final_price from » . TABLE_PRODUCTS_DESCRIPTION . » pd, » . TABLE_PRODUCTS . » p left join » . TABLE_MANUFACTURERS . » m on p.manufacturers_id = m.manufacturers_id, » . TABLE_PRODUCTS_TO_CATEGORIES . » p2c left join » . TABLE_SPECIALS . » s on p.products_id = s.products_id where p.products_status = ‘1’ and p.products_id = p2c.products_id and pd.products_id = p2c.products_id and pd.language_id = ‘» . (int)$languages_id . «‘ and p2c.categories_id = ‘» . (int)$current_category_id . «‘»;
}
}
————————————-
y reemplazarlo con estas lineas
// show the products of a specified manufacturer
if (isset($HTTP_GET_VARS[‘manufacturers_id’])) {
if (isset($HTTP_GET_VARS[‘filter_id’]) && tep_not_null($HTTP_GET_VARS[‘filter_id’])) {
// We are asked to show only a specific category
$listing_sql = «select » . $select_column_list . » p.products_id, p.manufacturers_id, p.products_price, p.products_tax_class_id, IF(s.status, s.specials_new_products_price, NULL) as specials_new_products_price, IF(s.status, s.specials_new_products_price, p.products_price) as final_price from (» . TABLE_PRODUCTS . » p, » . TABLE_PRODUCTS_DESCRIPTION . » pd, » . TABLE_MANUFACTURERS . » m, » . TABLE_PRODUCTS_TO_CATEGORIES . » p2c ) left join » . TABLE_SPECIALS . » s on p.products_id = s.products_id where p.products_status = ‘1’ and p.manufacturers_id = m.manufacturers_id and m.manufacturers_id = ‘» . (int)$HTTP_GET_VARS[‘manufacturers_id’] . «‘ and p.products_id = p2c.products_id and pd.products_id = p2c.products_id and pd.language_id = ‘» . (int)$languages_id . «‘ and p2c.categories_id = ‘» . (int)$HTTP_GET_VARS[‘filter_id’] . «‘»;
} else {
// We show them all
$listing_sql = «select » . $select_column_list . » p.products_id, p.manufacturers_id, p.products_price, p.products_tax_class_id, IF(s.status, s.specials_new_products_price, NULL) as specials_new_products_price, IF(s.status, s.specials_new_products_price, p.products_price) as final_price from (» . TABLE_PRODUCTS . » p, » . TABLE_PRODUCTS_DESCRIPTION . » pd, » . TABLE_MANUFACTURERS . » m) left join » . TABLE_SPECIALS . » s on p.products_id = s.products_id where p.products_status = ‘1’ and pd.products_id = p.products_id and pd.language_id = ‘» . (int)$languages_id . «‘ and p.manufacturers_id = m.manufacturers_id and m.manufacturers_id = ‘» . (int)$HTTP_GET_VARS[‘manufacturers_id’] . «‘»;
}
} else {
// show the products in a given categorie
if (isset($HTTP_GET_VARS[‘filter_id’]) && tep_not_null($HTTP_GET_VARS[‘filter_id’])) {
// We are asked to show only specific catgeory
$listing_sql = «select » . $select_column_list . » p.products_id, p.manufacturers_id, p.products_price, p.products_tax_class_id, IF(s.status, s.specials_new_products_price, NULL) as specials_new_products_price, IF(s.status, s.specials_new_products_price, p.products_price) as final_price from (» . TABLE_PRODUCTS . » p, » . TABLE_PRODUCTS_DESCRIPTION . » pd, » . TABLE_MANUFACTURERS . » m, » . TABLE_PRODUCTS_TO_CATEGORIES . » p2c) left join » . TABLE_SPECIALS . » s on p.products_id = s.products_id where p.products_status = ‘1’ and p.manufacturers_id = m.manufacturers_id and m.manufacturers_id = ‘» . (int)$HTTP_GET_VARS[‘filter_id’] . «‘ and p.products_id = p2c.products_id and pd.products_id = p2c.products_id and pd.language_id = ‘» . (int)$languages_id . «‘ and p2c.categories_id = ‘» . (int)$current_category_id . «‘»;
} else {
// We show them all
$listing_sql = «select » . $select_column_list . » p.products_id, p.manufacturers_id, p.products_price, p.products_tax_class_id, IF(s.status, s.specials_new_products_price, NULL) as specials_new_products_price, IF(s.status, s.specials_new_products_price, p.products_price) as final_price from ((» . TABLE_PRODUCTS_DESCRIPTION . » pd, » . TABLE_PRODUCTS . » p) left join » . TABLE_MANUFACTURERS . » m on p.manufacturers_id = m.manufacturers_id, » . TABLE_PRODUCTS_TO_CATEGORIES . » p2c) left join » . TABLE_SPECIALS . » s on p.products_id = s.products_id where p.products_status = ‘1’ and p.products_id = p2c.products_id and pd.products_id = p2c.products_id and pd.language_id = ‘» . (int)$languages_id . «‘ and p2c.categories_id = ‘» . (int)$current_category_id . «‘»;
}
}
——————————————————————-
en el archivo advanced_search_result.php
encontrar estas lineas
$from_str = «from » . TABLE_PRODUCTS . » p left join » . TABLE_MANUFACTURERS . » m using(manufacturers_id), » . TABLE_PRODUCTS_DESCRIPTION . » pd left join » . TABLE_SPECIALS . » s on p.products_id = s.products_id, » . TABLE_CATEGORIES . » c, » . TABLE_PRODUCTS_TO_CATEGORIES . » p2c»;
—————————————-
y reemplazarlas con esto
$from_str = «from ((» . TABLE_PRODUCTS . » p) left join » . TABLE_MANUFACTURERS . » m using(manufacturers_id), » . TABLE_PRODUCTS_DESCRIPTION . » pd) left join » . TABLE_SPECIALS . » s on p.products_id = s.products_id, » . TABLE_CATEGORIES . » c, » . TABLE_PRODUCTS_TO_CATEGORIES . » p2c»;