Description:

Various reports contain goal metrics. Currently the order of the goals is a matter of how the database decides to return the goals in.

This PR adds a simply sorting by id, so the order will always be the same and independent from the database.

This will also help to fix certain tests failing for TiDb

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