Cross Sell Advanced [Support Thread]

[SavaTom]

Nevermind, i just noticed the requirements.....this is an upgrade

"REQUIREMENTS: It REQUIRES cross sell to be installed."

[DivaVocals]

Nevermind, i just noticed the requirements.....this is an upgrade

"REQUIREMENTS: It REQUIRES cross sell to be installed."

No it doesn't.. This is an UPGRADE of the original Cross Sell module.. The description is incorrect.. it SHOULD read:

Based on the original Cross Sell module! It is easy to use, has a graphical interface to quickly find products to cross sell and is very light weight. 6 cross sell products can set specifically set for each of your Zen Cart product pages.


Advanced Cross Sell expands the original Cross Sell for even more ease of use by allowing you to setup 6 cross sells using the product model numbers.

[DivaVocals]

Nevermind, i just noticed the requirements.....this is an upgrade

"REQUIREMENTS: It REQUIRES cross sell to be installed."

Module description has been updated to reflect the current codebase:

A new and improved version of the original Cross Sell module!

What is Cross Selling??
=================
Cross selling is a suggestive selling method where the shopowner offers customers the opportunity to purchase items related to the product they are viewing.

Out the box, Zen Cart includes a "Customers who bought this product also purchased.." centerbox so that whenever a customer purchases an item, it looks and finds other customers who also purchased the same item and selects other products they purchased.

But what if the shopowner wanted to select specific related items to offer their customers? Advanced Cross Sell adds this exact kind of functionality.

Advanced Cross Sell has a graphical interface to quickly find products for cross selling. Set up to six (6) optional products per product for each Zen Cart product page. Additionally this module has an advanced cross sell user interface which allows the shopowner/administrator to setup all six (6) cross sells on the same screen using products numbers.

How Does Advanced Cross Sell Work??
=============================
The shop sells Walkmans.. If the shopowner/administrator adds cross sell products to the Walkman product via the Advanced Cross Sell interfaces, the product page will now includes a new centerbox.. "may we also recommend batteries, case cd's etc."

You ONLY need to install this module..

[numinix]

If you were previously using a different version of this plugin, the auto installer doesn't remove the old database entries. The logic could be improved so that duplicate DB entry errors are not a problem.

[Ajeh]

Maybe I am not reading the code right in the file:
YOUR_ADMIN\includes\init_includes\init_xsell_config.php

but if there wasn't an old version or a previous version to clean up, wouldn't this end up deleting the configuration_keys for:
PRODUCTS_OPTIONS_TYPE_SELECT
UPLOAD_PREFIX
TEXT_PREFIX

I have not tried to install this, but I was peeking at the code and it made me start to wonder about doom and gloom ...

[DivaVocals]

If you were previously using a different version of this plugin, the auto installer doesn't remove the old database entries. The logic could be improved so that duplicate DB entry errors are not a problem.

If you are seeing this then perhaps you are right.. I'm not seeing the problem for no other reason except that I'm not really a true code monkey.. so perhaps a HINT on what the issue is would be most appreciated..

Maybe I am not reading the code right in the file:
YOUR_ADMIN\includes\init_includes\init_xsell_config.php

but if there wasn't an old version or a previous version to clean up, wouldn't this end up deleting the configuration_keys for:
PRODUCTS_OPTIONS_TYPE_SELECT
UPLOAD_PREFIX
TEXT_PREFIX

I have not tried to install this, but I was peeking at the code and it made me start to wonder about doom and gloom ...

I'm not sure how this would happen since the delete statements target ONLY the Cross Sell configuration entries and groups.. But again, I'm not seeing the problem for no other reason except that I'm not really a true code monkey.. a hint as to where the issue lies would be appreciated..

[Ajeh]

In that file, you check for the configuration_group_id in the configuration_group table then Delete for that value in the configuration table ...

        /* Find configuation group ID of Previous Version of Cross Sell */
        $sql = "SELECT configuration_group_id FROM ".TABLE_CONFIGURATION_GROUP." WHERE configuration_group_title='".$xsell_old_menu_title."' LIMIT 1";
        $result = $db->Execute($sql);
        $xsell_old_configuration_id = $result->fields['configuration_group_id'];

        /* Remove Previous Version of Cross Sell from the configuration group table */
        $sql = "DELETE FROM ".TABLE_CONFIGURATION_GROUP." WHERE configuration_group_id ='".$xsell_old_configuration_id."'";
        $db->Execute($sql);

        /* Remove Previous Version of Cross Sell items from the configuration table */
        $sql = "DELETE FROM ".TABLE_CONFIGURATION." WHERE configuration_group_id ='".$xsell_old_configuration_id."'";
        $db->Execute($sql);

But if nothing is found on the configuration_group, the value is probably going to be 0 for $xsell_old_configuration_id so you first delete from the configuration_group table where the configuration_group_id is 0 which should probably not cause any harm but is not a good idea ...

The problem is you then delete in the configuration table for that configuration_group_id and the 0 is the configuration_group_id for these 3 values in the configuration table ...

So, rather than deleting for what you mean to delete for, you delete important configuration_keys for Zen Cart ...

This thread better explains the issue:
http://www.zen-cart.com/showthread.p...ag-not-working

and post #9 is a nice write up by swguy explaining the specifics of the issue with a possible better approach to this ...

[Ajeh]

NOTE: you really need to read through that file to find any other similar issues ...

[DivaVocals]

NOTE: you really need to read through that file to find any other similar issues ...

Well if I run the query

SELECT configuration_group_id FROM zen_configuration_group WHERE configuration_group_title = 'Deluxe Cross Sell' LIMIT 1;

I get zero results. So wouldn't the delete statement be deleting nothing and NOT the 0 configuration group???

[SavaTom]

Cant login to my admin after uploading the files (yes I followed instructions and renamed admin folder correctly and renamed the your template folders correctly, etc, etc. Below is the error log file. I have omitted my domain from the path.

white screen of death...

WARNING: An Error occurred, please refresh the page and try again.

error log...

1062uplicate entry 'MIN_DISPLAY_XSELL' for key 2 :: INSERT INTO configuration (configuration_id, configuration_title, configuration_key, configuration_value, configuration_description, configuration_group_id, sort_order, last_modified, date_added, use_function, set_function) VALUES (NULL, 'Display Cross-Sell Products Minimum', 'MIN_DISPLAY_XSELL', '1', 'This is the minimum number of configured Cross-Sell products required in order to cause the Cross Sell information to be displayed.<br />Default: 1', '76', 20, NULL, now(), NULL, NULL) in /var/www/vhosts/1/124646/webspace/httpdocs/myprivatedomain/includes/classes/db/mysql/query_factory.php on line 120