First let me say that this is a great mod. It will come in extremely useful. While installing and testing i have noticed four things about this mod... I'm working with Preview Release 2.2-MS1
1) If i don't have a product model entered for the item, i can't add it to my wishlist. However, the wishlist doesnt seem to make any real use of the product model. So i changed the following in "/includes/application_top.php"
case 'add_wishlist' : if (ereg('^[0-9]+$', $HTTP_POST_VARS['products_id'])) {
if ($HTTP_POST_VARS['products_model']) {
to
case 'add_wishlist' : if (ereg('^[0-9]+$', $HTTP_POST_VARS['products_id'])) {
if ($HTTP_POST_VARS['products_name']) {
2) If i remove a product from my catalog entirely, it is still left in the wishlist database.
I added the following to function tep_remove_product() which is loated in "<admin_folder>/includes/functions/general.php"
tep_db_query("delete from " . TABLE_WISHLIST . " where products_id = '" . tep_db_input($product_id) . "'");
directly below
tep_db_query("delete from " . TABLE_SPECIALS . " where products_id = '" . tep_db_input($product_id) . "'");
tep_db_query("delete from " . TABLE_PRODUCTS . " where products_id = '" . tep_db_input($product_id) . "'");
tep_db_query("delete from " . TABLE_PRODUCTS_TO_CATEGORIES . " where products_id = '" . tep_db_input($product_id) . "'");
tep_db_query("delete from " . TABLE_PRODUCTS_DESCRIPTION . " where products_id = '" . tep_db_input($product_id) . "'");
tep_db_query("delete from " . TABLE_PRODUCTS_ATTRIBUTES . " where products_id = '" . tep_db_input($product_id) . "'");
tep_db_query("delete from " . TABLE_CUSTOMERS_BASKET . " where products_id = '" . tep_db_input($product_id) . "'");
tep_db_query("delete from " . TABLE_CUSTOMERS_BASKET_ATTRIBUTES . " where products_id = '" . tep_db_input($product_id) . "'");
and this to "<admin_folder>/includes/application_top.php"
define('TABLE_WISHLIST', 'customers_wishlist');
3) This is just meant to be helpfull and not nit-picky... The directions for modifying column_right.php indicate the following:
//added for wishlist
if (tep_session_is_registered('customer_id')) include(DIR_WS_BOXES . 'wishlist.php')
It is missing a semi-colon at the end of the line..
4) I think this was already discussed earlier in the trhead.. but for the code that gets added to product_info.php "$product_info_values" needs to be "$product_info"