X-Cart forums

[intel352]

Product Options have the ability to be ordered, but this ordering isn't passed on to Variants. This is a failing in Qualiteam code logic, as Variants are an extension of Product Options, and thus should carry the same ordering value as it's related Option.

I've fixed this problem, attached is a diff which should apply on 4.1.8/4.1.9. I believe a better fix would be to rewrite the query used that establishes the first variants array (I would personally use INNER JOIN between xcart_variants, xcart_variant_items, and xcart_class_options, and then use the xcart_class_options.orderby value to order), but I didn't want to affect the pricing and whatnot that is being pulled into the original $variants array.

Result is that instead of Variants being ordered by name, they'll be ordered by the Order By value of each variant's related Option.

This affects modules/Product_Options/func.php, diff is attached in zip file

To apply the diff (aka patch), you must use the Patch/Upgrade mechanism in X-Cart administration.

Cheers

[carpeperdiem]

Jon,

This patch does not make sense to me, as my variants are in the POS order as defined in product options.

They are NOT alphabetical.

POS determines ordering for me.

Are you certain that a stock cart needs this? I am not so sure...

[intel352]

Thanks for posting Jeremy, after investigation, you're correct. A stock cart does not need this for the frontend, this patch will only help a stock cart in the backend (if you're particular about how your variants are ordered in the variant administration page.

The cart I was working on has a customized frontend, and to verify this was an x-cart issue, I checked administration, the variants there were ordered similarly to the ones in the frontend, but now I remember stock X-Cart orders variants in the frontend based on the order of the options.

Ah well.

BTW, I misspoke in my first post, default variant ordering is by order added to the database, not alphabetical (and not using POS), even in X-Cart variant administration.

Cheers

[carpeperdiem]

default variant ordering is by order added to the database, not alphabetical (and not using POS), even in X-Cart variant administration

That makes sense, Jon. For my store, we were very organized on the way in, and used a spreadsheet and csv import to propagate products -- so our admin section looks quite organized and in perfect order. But in my test store, it's a freaking mess! So I am able to test both ways.

I always suggest a csv import, for products anyway!

[carlisleglass]

I was trying to find a way of displaying the variants in the right order for the customer, but this is even better !!!

Thank You !

[intel352]

Glad it helped you Darren. I'm going to either have to add this into the Variant Charts module install, or change the core coding. I'd rather just add this patch in

Cheers

[benz]

Thanks, I was just starting to wade through the SQL when a last minute search found this thread...brilliant

[UHS]

I found this old thread and was wondering if anyone had anything to add to it? My situation is as follows. If I decide to remove a variant in the product admin, It changes the order by of the remaining variants for that product and I have to re-order all of the variants for that product. Does this mod help with that? This particular cart is 4.1.10

-Jonathon

[alarnold]

Still working for 4.6.X. Don't understand why this is not built into X-cart to begin with!

[alarnold]

The Product Options variables are displayed in the order they were added. The Option Value "OrderBy" is ignored when Option Value displayed. This patch can not be applied, but does have the needed information to correct this on-going issue. Version 7.4.12 changed the way the variables are accessed, but the code can be replaced with the patched code. (starts around line 767)