Bug - Custom ID display on checkout_confirmation

potteryhouse / stock_by_attribute_1.5.3

Stock by Attribute for Zen Cart 1.5.1

GNU General Public License v2.0

1 stars 0 forks source link

Bug - Custom ID display on checkout_confirmation #9

Closed jeking928 closed 10 years ago

jeking928 commented 10 years ago

If purchasing the same item with different attributes, on the checkout_confirmation page the second item shows both custom ID's, the third shows three, etc.

potteryhouse commented 10 years ago

Can you provide a screen shot of your SBA Stock page, so I can see how the items are listed, maybe an export of the SBA table would help also.

Thanks

jeking928 commented 10 years ago

It's the ZC default store and here's the screen shot. If you need the export, let me know.

potteryhouse commented 10 years ago

I duplicated this issue, and found a similar error on other pages in admin, all have been corrected. Changes are checked into both 1.5.3 and 1.5.4 branches. Please test these, if all is good, I think we can update the Instructions and than release the 1.5.3 update on Zen Cart Plugins.

jeking928 commented 10 years ago

Fixes look good. One suggestion. When viewing the order in the admin, what about adding a separator between the attribute name/value and the customID name/value. For example, my test I see:

Color: Blue Item # blue What about this:
Color: Blue (Item # blue) putting the entire block in parenthesis. Just seems to visually separate them better.

potteryhouse commented 10 years ago

Done, suggestion included in Update4, Thanks.