Description:

Sorting of ecommerce item reports currently does not work correctly in some case.
Sometimes when trying to sort for e.g. item views, the reports are sorted by label instead.

The reason for that can be found in the way the reports are built. As e.g. item view metrics are only added for those items that actually had any views tracked, not all rows the report contains the same set of columns.

The sorting is then currently broken, if the first row does not contain the column that was requested for sorting. See #21829 (comment) for further details.

This fix was the one with the lowest required effort. It might nevertheless be good to start a discussion at some point around how to ensure that all rows in a report contain the same set of columns, as this could lead to problems elsewhere as well.

fixes #21829

Review

• Functional review done
• Potential edge cases thought about (behavior of the code with strange input, with strange internal state or possible interactions with other Matomo subsystems)
• Usability review done (is anything maybe unclear or think about anything that would cause people to reach out to support)
• Security review done
• Wording review done
• Code review done
• Tests were added if useful/possible
• Reviewed for breaking changes
• Developer changelog updated if needed
• Documentation added if needed
• Existing documentation updated if needed