@kymation Jim
A couple of issues I have encountered while testing this add-on on OS 2.3.4 on my localhost machine, with SSL disabled.
When you are on the product_info.php page, and try to add a product to the shopping list with SSL disabled, you are redirected to https:// which doesn't exist if SSL is off.
I had to change 'SSL' to 'NONSSL' to make it work
function dialog_add_to_list( $self_filename, $products_id, $languages_id=1, $currencies ) {
$dialog = ' <div id="add_new_dialog" title="' . SUBTITLE_PRODUCT_EXISTING_LIST . '" class="hide">' . PHP_EOL;
$dialog .= ' ' . tep_draw_form( 'list_add_product', tep_href_link( $self_filename, tep_get_all_get_params( array( 'action' ) ) . '&action=list_add_product', 'NONSSL' ), 'post', 'class="shopping_list"' ) . PHP_EOL;
Another issue:
When you are on the 'shopping_list.php' page, and click the 'try to add/change shopping list', trying to create a new list, nothing happens.
action=list_change_product is called but no change, should it be action=list_new ?
Another issue:
shopping_list_id is ambiguous
function id_in_shopping_list( $customers_id, $shopping_list_id, $shopping_list_products_id ) {
$shopping_list_query_raw = "select sl.shopping_list_id from " . TABLE_SHOPPING_LIST . " sl join " . TABLE_SHOPPING_LIST_PRODUCTS . " slp on (sl.shopping_list_id = slp.shopping_list_id) where sl.customers_id = '" . (int)$customers_id . "' and sl.shopping_list_id = '" . (int)$shopping_list_id . "' and slp.shopping_list_products_id = '" . (int)$shopping_list_products_id . "'";
Thank you for your time.