Multi Cross Sell mod [Support thread]

[niccol]

@stizzed
sorry missed your question. those sql statements in my post are the ones that I use to get full ( I think) functionality. What exactly don't you have?

@The Mech
A bit more info would help. Zen Cart version?

In general, this is not my mod. Gilby is the man. I jsut posted what I did to get it to play with Zen 1.5.

The only other thing that I may not have answered because this was originally posted for a specific question was that you may need to ensure that BOX_CATOLOG_MULTI_XSELL_PRODUCTS is actually defined somewhere. I can't remember if I had to define that in a new language file or not, to be honest. If I get time I will re-install on a clean server and have a look at it. But it would be best if a Zen 1.5 compatible version was released, and I am kind of busy with other mods...

[The_Mech]

Thanks Niccol,
zen-cart-v150-full-release-12302011

When running your patch for 1.5:

SELECT (@sort_order:=MAX(sort_order)+1) FROM admin_pages WHERE menu_key='configuration';
SELECT (@configuration_group_id:=configuration_group_id) FROM configuration_group WHERE configuration_group_title= 'Cross Sell Settings' LIMIT 1;
INSERT INTO admin_pages (page_key, language_key, main_page, page_params, menu_key, display_on_menu, sort_order) VALUES ('configMultiXsell', 'BOX_CATOLOG_MULTI_XSELL_PRODUCTS', 'FILENAME_CONFIGURATION', CONCAT('gID=', @configuration_group_id), 'configuration', 'Y', @sort_order);

I get this:

Error

SQL query:

INSERT INTO admin_pages( page_key, language_key, main_page, page_params, menu_key, display_on_menu, sort_order )
VALUES (
'configMultiXsell', 'BOX_CATOLOG_MULTI_XSELL_PRODUCTS', 'FILENAME_CONFIGURATION', CONCAT( 'gID=', @configuration_group_id ) , 'configuration', 'Y', @sort_order
);

MySQL said: Documentation
#1048 - Column 'page_params' cannot be null

[The_Mech]

I tried changing the CONCAT bit to 'gID=31' where 31 was the next gID. It entered without error at that point, but nothing happened.

[niccol]

basically that means that you have not got the entry in the configuration_group table there already. that should have been created by the original install of the mod.

have you followed the instructions correctly. For instance :

using that you install the mod with the install button

[The_Mech]

basically that means that you have not got the entry in the configuration_group table there already. that should have been created by the original install of the mod.

have you followed the instructions correctly. For instance :

using that you install the mod with the install button

Yeah, and after installing, I ran your SQL.

[The_Mech]

I've found that the first SQL works without a table prefix but the second SQL only works with...

[gilby]

The above sql's may work for you with the existing 139 code. If so good!

My testing has shown that some functions relied on by this mod have changed in 1.5.0 and may result in blank admin pages on some sites.

------------------
Just an update as to where I am with this.

- Have sorted out the install and uninstall, that now works fine.
- Fixed a number of compatibility problems.

Working on making it work WITH 1.5.0 and not just work IN 1.5.0
eg: replacing some of the gets with posts.

At this stage it looks like being around 2 to 3 weeks before it should be ready.

[niccol]

@The_Mech
Yeah, that may well be it. I hope you get it to work. To be honest I slightly regret having even posted this as it is much better to wait for Gilby to do his update. I have it running fine on 1.5 but then I have tweaked the code anyway.

And as Gilby actively supports this mod I sort of feel that one should have the courtesy of waiting for that update. Gilby, I apologise for having got tangled up in it all. I absolutely understand how much work goes into it all and probably should have kept my mouth shut. It was just one of those cases when I had just done something myself and someone asked how to do it. Apologies.

[gilby]

@The_Mech
Yeah, that may well be it. I hope you get it to work. To be honest I slightly regret having even posted this as it is much better to wait for Gilby to do his update. I have it running fine on 1.5 but then I have tweaked the code anyway.

And as Gilby actively supports this mod I sort of feel that one should have the courtesy of waiting for that update. Gilby, I apologise for having got tangled up in it all. I absolutely understand how much work goes into it all and probably should have kept my mouth shut. It was just one of those cases when I had just done something myself and someone asked how to do it. Apologies.

No need to apologise and if it seemed I was having a go at you, I wasn't. I was just mentioning that there had been some problems that I had seen in some setups with the existing code.

As well I have been busy with my real job and haven't needed to get started on this until recently. If you or anyone want to improve or update it, go ahead I won't be miffed.

As it is, I certainly didn't write this from scratch. I looked at what was available and took what I thought was the best from other cross sells (especially yellow1912's version) and made changes that suited what I wanted it to do at the time.

Others have taken this and adapted it to their own use.
Take a look at ultimate_zc's version. He needed it to work a different way and has done it his way.

I rarely find a mod that works exactly the way I want it and often adapt it to what I need at the time. At the time I adapted yellow1912's cross sell I thought that this might also be useful to others as well, that why I submitted it.
I don't see this as "mine, keep off" it was just an evolution from what was available at the time. It is still useful to me so I will still work on it, but that doesn't mean others can't also.

[niccol]

@gilby
I completely understand.
I kind of just wanted people to know that I was just 'getting it to work' in 1.5 rather than having looked at the code in any deep way, which I haven't. The sql statements are very much a 'hack' and I haven't looked at the code closely enough to make any other observations other than that.
As I say I am kind of busy with other mods and other projects. So, I also completely appreciate your position with this mod. I was trying to find a way of saying ' wait for an official update' without seeming to pressure you to produce one because I know that it takes work and that no-one is ultimately responsible for a mod at all.