Multi Cross Sell mod [Support thread]

lina0962:

Got "WARNING: An Error occurred, please refresh the page and try again." in Admin->Catalog->Multi Cross Sell. Any ideas?

Check Post# 291 and see if that helps?

gilby:

See Post# 338 for the fix

Thank you so much, now works fine!

gilby:

Check Post# 291 and see if that helps?
Hi,

That fix didn't work for me. Well, it worked because I have STRICT_ERROR_REPORTING and was giving me the installation error but after getting rid of that error now it's saying:

1366 Incorrect integer value: '' for column 'configuration_group_id' at row 1
in:
[INSERT INTO zen_configuration_group VALUES ('', 'Cross Sell Settings', 'Set Cross Sell Options', '1', '1')]

It's also giving me a similar error when trying to restore using DrByte's MySql Backup module:

1366 Incorrect integer value: '' for column 'configuration_id' at row 1
in:
[insert into zen_configuration values ('', 'Last Database Restore', 'DB_LAST_RESTORE', 'db_name-20160711061243.sql.gz', 'Last database restore file', '6', '', '', now(), '', '')]

I have this module installed on my store in same server, with same parameters:
Zen Cart Version 1.5.4
Apache Version 2.4.18
PHP Version 5.6.22
MySQL Version 5.5.50-cll

ideasgirl:

1366 Incorrect integer value: '' for column 'configuration_group_id' at row 1 in
[INSERT INTO zen_configuration_group VALUES ('', 'Cross Sell Settings', 'Set Cross Sell Options', '1', '1')]
I have this module installed on my store in same server, with same parameters:
Zen Cart Version 1.5.4
Apache Version 2.4.18
PHP Version 5.6.22
MySQL Version 5.5.50-cll

Your MySQL is apparently configured to disallow the (old) use of an empty string to denote a 0 value. In the past we got away with using '' or 0 or null to trigger an autoincrement field, but newer MySQL versions (depending on their configuration) disallow the string option.

So simply changing that first '' to null should work. Sometimes 0 will still work.

ie:

INSERT INTO zen_configuration_group VALUES (null, 'Cross Sell Settings', 'Set Cross Sell Options', '1', '1')

I don't know exactly what you want me to do here.

This is what I tried:
-Do the upload of the file for multi cross sell to have to button to install module = Same result
-Did the install and did a query at phpadmin as you put it there. It showed up on database but doesn't show up on admin and the installation button gives same error.

-The backup module gives same error when restoring.

I was just pointing out that the SQL you're executing has a syntax error because your MySQL server's configuration is (appropriately) enforcing a modern mode of strictness.

I wasn't telling you to do anything. Just showing you an explanation of what one could do to the code to fix the error.

I'm currently testing to update website to ZC Ver. 1.5.6

However, as I am trying to install Multi Cross Sell. I run into error when I tried to update configuration in admin.
(Configuration -> Cross Sell Settings)

Try to change anythings and I will get into this error in the log...

[03-Jan-2019 19:12:24 UTC] Request URI: /zc156/adm/configuration.php?gID=43&cID=916&action=save, IP address: xxx.xxx.xxx.xxx
#1  trigger_error() called at [/home/user/public_html/zc156/includes/classes/db/mysql/query_factory.php:171]
#2  queryFactory->show_error() called at [/home/user/public_html/zc156/includes/classes/db/mysql/query_factory.php:143]
#3  queryFactory->set_error() called at [/home/user/public_html/zc156/includes/classes/db/mysql/query_factory.php:270]
#4  queryFactory->Execute() called at [/home/user/public_html/zc156/adm/configuration.php:26]
--> PHP Fatal error: 1054:Unknown column 'val_function' in 'field list' :: SELECT val_function FROM configuration WHERE configuration_id = '916' ==> (as called by) /home/user/public_html/zc156/adm/configuration.php on line 26 <== in /home/user/public_html/zc156/includes/classes/db/mysql/query_factory.php on line 171.

So, the SQL is not valid.
I ran in PhpMyAdmin and I got same result...

SQL query: Documentation
SELECT val_function FROM configuration WHERE configuration_id = '916' LIMIT 0, 25
MySQL said: Documentation
#1054 - Unknown column 'val_function' in 'field list'

I check configuration.php on admin file. It seems the problematic line is on this part...

$configuration_value = zen_db_prepare_input($_POST['configuration_value']);
        // See if there are any configuration checks
        $checks = $db->Execute("SELECT val_function FROM " . TABLE_CONFIGURATION . " WHERE configuration_id = '" . (int)$cID . "'");
        if (!$checks->EOF && $checks->fields['val_function'] != NULL) {
           require_once('includes/functions/configuration_checks.php');
           zen_validate_configuration_entry($configuration_value, $checks->fields['val_function']);
}

This part is on ZC 1.5.6 and that code was NOT work like that in ZC 1.5.1 (my current store version).

I do not think "val_function" exist anywhere in database... and it is hard coded there. I wonder is this intended for other functions or not?

-PanZC2020

Found out it is the bug for 1.5.6, which is now has a hotfix.

https://www.zen-cart.com/showthread.php?224809-Done-Admin-gt-Configuration-Unknown-column-val_function-in-field-list&p=1352546#post1352546

So that's a new field for 1.5.6. Good to know!

Otherwise, this one seems to work on admin side. Have to test it out. (Beside it's outdated looks on settings...)

Hello everyone, I love this mod and have been using it for long time,

I have one concern if anybody can advise me :

I noticed the multi cross sale products' links are missing the "cPath" variable, and reads like

https://domain.com/index.php?main_page=product_info&products_id=1

it suppose to be like:

https://domain.com/index.php?main_page=product_info&cPath=2&products_id=1

As the search bot may take this as duplicate content.

Is there any way to include the cPath variable in the final link? Is there any file I can modify to reach the goal?

sjbglx1:

Hello everyone, I love this mod and have been using it for long time,

I have one concern if anybody can advise me :

I noticed the multi cross sale products' links are missing the "cPath" variable, and reads like

https://domain.com/index.php?main_page=product_info&products_id=1

it suppose to be like:

https://domain.com/index.php?main_page=product_info&cPath=2&products_id=1

As the search bot may take this as duplicate content.

Is there any way to include the cPath variable in the final link? Is there any file I can modify to reach the goal?
In a standard zen-cart the "canonical link" removes the cpath as with linked products there will be several ways to get to the same product.
This would create "duplicate product" warnings as it is the same product, just different ways to get there.
Zen-cart normally "removes the cPath from all "canonical links" so that these "duplicate products" will be seen to actually be the same products as they will all have the same canonical link.

Multi Cross Sell goes straight to the "normally" correct canonical link as this mod has no way of knowing what multiple cPath you want to use to get there.

I notice your website does include the cPath in the canonical link. This is NOT usual Zen-cart behaviour (it is optional however) So your website will already be creating duplicate products for linked items.

Hi,

I am running V1.5.5e and have been using Multi Cross sell for some time now. I was not paying attention to my logs and now have seen that it is and has been throwing an error for some time now. I am not sure why as all is working correctly, and am wondering if someone knows what this error is and how to fix it. Here is the error message in the Debug log:

PHP Warning: constant(): Couldn't find constant TEXT_MXSELL1_PRODUCTS in /home/public_html/shop/includes/templates/**MyTemplate/templates/tpl_modules_multi_xsell_products.php on line 30

Thanks in advance for any help you can offer,

Jeff

NWCE:

Hi,

I am running V1.5.5e and have been using Multi Cross sell for some time now. I was not paying attention to my logs and now have seen that it is and has been throwing an error for some time now. I am not sure why as all is working correctly, and am wondering if someone knows what this error is and how to fix it. Here is the error message in the Debug log:

PHP Warning: constant(): Couldn't find constant TEXT_MXSELL1_PRODUCTS in /home/public_html/shop/includes/templates/**MyTemplate/templates/tpl_modules_multi_xsell_products.php on line 30

Thanks in advance for any help you can offer,

Jeff
Have you got the correct language defines in
includes/languages/english/extra_definitions/**MyTemplate/multi_xsell_box_defines.php

or are you using more than one language?
What version php?

Hi Gilby,

Thanks for your reply. I only have English as a language and my PHP version is 7.1.9 I hope that this helps to eliminate some potential ideas......do you happen to have any other things I could check?

I see that this module you have done shows compatibility of v1.5.0, as I have Multi Cross Sell v 1.03. Would this be causing any issues??

Thanks again,

Jeff

NWCE:

Hi Gilby,

Thanks for your reply. I only have English as a language and my PHP version is 7.1.9 I hope that this helps to eliminate some potential ideas......do you happen to have any other things I could check?

I see that this module you have done shows compatibility of v1.5.0, as I have Multi Cross Sell v 1.03. Would this be causing any issues??

Thanks again,

JeffThere will some php things as you upgrade php
But...
You didn't say if you the required language file?
Have you got the correct language defines in
includes/languages/english/extra_definitions/**MyTemplate/multi_xsell_box_defines.php

Hi Gilby,

Sorry that I did not answer your question earlier on the language file, but I see that I have not placed that file in the correct folder......I have it in includes/languages/english/My Template.

Thank you for deciphering my obvious error. Now that I have this in the correct folder, I am assuming that this should stop the error as now it should be able to find it. I will let you know if I experience any further errors.

Thanks again for helping....it is much appreciated.

Jeff

Hello, after updating to 1.5.6c the module is still working.
Products all show correctly as before (1000s of them)

However, when viewing existing cross sells, the first item is not in the list, always.
(Example shown)
It is showing in the row of comma separated ids, but not in the other area, so I can't delete or change sort order.
Any ideas why the first isn't showing?
First number is main item, other 3 are linked items.
But further down the image there are only 2 linked showing.