Default sort order

[mslabko]

Default product sort order should be:

• category position ASC (only in case if we have fitter by category)
• product id DESC

Default sort order for category:

• category position ASC

Open questions:

• what does "sort by position" mean if you do a product search across all categories?(\Magento\CatalogGraphQl\DataProvider\Product\SearchCriteriaBuilder::addDefaultSortOrder)

AC:

• Implement correct sort behavior for B2C
  • Changes delivered to 2.4-develop (without storefront modules), tests are green (done)
• align sort in CatalogStorefront and unskip tests:
  • \Magento\GraphQl\Catalog\ProductSearchTest::testFilterProductsByDropDownCustomAttribute
  • \Magento\GraphQl\Catalog\ProductSearchTest::testFilterByMultipleProductUrlKeys
  • \Magento\GraphQl\Catalog\ProductSearchTest::testFilterProductsBySingleCategoryId Details: https://github.com/magento/catalog-storefront/issues/69#issuecomment-645716058

Note: for collection correct order is implemented, need to fix for GQL: https://github.com/magento/magento2/commit/9f5d25a32fc611b32e890b4dcb97d72c725cee75

Update We have the same problem for categories. Currently, sort order for categories is not determined and may be different on different MySQL engine version or even platform

Need to add default sort order:

• \Magento\CatalogGraphQl\Model\Resolver\Categories::resolve

[m2-assistant[bot]]

Hi @mslabko. Thank you for your report. To help us process this issue please make sure that you provided sufficient information.

Please, add a comment to assign the issue: @magento I am working on this

---

• Join Magento Community Engineering Slack and ask your questions in #github channel.

[Dnyomo]

@magento I am working on this

[Dnyomo]

Blocking this as we await feedback from Yaroslav Honcharuk on the sort behavior.

[Dnyomo]

The first part of this ticket was completed with a PR into the main Magento 2 repo here https://github.com/magento/magento2/pull/29301. The rest of the ticket it blocked as we will not be testing GraphQL in the store front.