Fix duplicate sort results

[Bassel17]

What does it do?

In query builder when we query for data and we have joins, we run qb.distinct which should remove duplicate data from the query but after some testing it doesn't seem to affect queries with multiple joins and duplicate data was queried when trying nested sorting. So I opted to use qb.groupBy instead when we have multiple joins and sorting

Why is it needed?

When doing nested sorting, duplicate data is being returned

How to test it?

run a strapi application and create content types with a relation, try to sort the parent content type based on a relation field there should be no duplicates, you can follow the same method in the issue I will reference below.

Related issue(s)/PR(s)

fixes #11892

[codecov]

Codecov Report

Merging #13387 (5b796e1) into master (2ef3556) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

❗ Current head 5b796e1 differs from pull request most recent head 86fe91e. Consider uploading reports for the commit 86fe91e to get more accurate results

@@            Coverage Diff             @@
##           master   #13387      +/-   ##
==========================================
- Coverage   48.22%   48.20%   -0.02%     
==========================================
  Files         239      239              
  Lines        8834     8837       +3     
  Branches     1992     1993       +1     
==========================================
  Hits         4260     4260              
- Misses       3766     3768       +2     
- Partials      808      809       +1     

Flag	Coverage Δ
front	?
unit	48.20% <0.00%> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
packages/core/database/lib/query/query-builder.js	2.08% <0.00%> (-0.04%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2ef3556...86fe91e. Read the comment docs.

[Convly]

Looks like it's breaking some e2e tests in packages/core/content-manager/server/tests/api/basic-relations.test.e2e.js

[alexandrebodin]

Hey based on our lastest discussion on the topic this isn't solving the problem at hand :) I'll close for now :)