HotcakesCommerce / hotcakes-commerce-core

The core of the e-commerce part of the overall solution. This is an ecommerce shopping cart solution built on top of the DNN (DotNetNuke) CMS. Anyone can do commerce online now!
https://mmmcommerce.com
MIT License
69 stars 55 forks source link

SI: Product Type Properties not sorted properly #209

Closed WillStrohl closed 4 years ago

WillStrohl commented 5 years ago

Describe the bug

Despite the product type properties appearing to be sorted correctly in their edit view, the sort is not correct when viewing them in the drilldown category view, nor when editing a product that may have this feature including.

Software Versions

To Reproduce

Steps to reproduce the behavior:

  1. Clean install
  2. Add sample products
  3. Create a multiple choice product type property that contains a list of values (e.g., the past 10 years in yyyy format)
  4. Save the product type property into a product type and apply it to a product.
  5. View the product in the store and it will likely look correct.
  6. Add new product type property and even sort it in the same view.

Expected behavior

The product type properties appear in the product edit and category viewer drill down views in the saved sort order.

Actual behavior

The product type properties are only in the correct sort order when viewing them on their own edit view.

Screenshots

image

image

image

WillStrohl commented 4 years ago

Example queries to find and potentially update the sort manually until this is resolved.

In DNN:

SELECT * FROM {databaseOwner}[{objectQualifier}hcc_ProductProperty] ORDER BY [PropertyName];
SELECT * FROM {databaseOwner}[{objectQualifier}hcc_ProductPropertyChoice] WHERE [PropertyId] = ;

In SSMS:

SELECT * FROM [dbo].[hcc_ProductProperty] ORDER BY [PropertyName];
SELECT * FROM [dbo].[hcc_ProductPropertyChoice] WHERE [PropertyId] = ;
WillStrohl commented 4 years ago

We just finished cooking up a fresh serving of Hotcakes for you. Enjoy! ;)

https://dnncommunity.org/blogs/Post/5689/Hotcakes-Commerce-03-03-00-Release-Marks-a-Major-Milestone