DATAJPA-1750 - Fixes order clause creation if function with parameter…

[simonparadies]

• You have read the Spring Data contribution guidelines.
• There is a ticket in the bug tracker for the project in our JIRA.
• You use the code formatters provided here and have them applied to your changes. Don’t submit any formatting related changes.
• You submit test cases (unit or integration tests) that back your changes.
• You added yourself as author in the headers of the classes you touched. Amend the date range in the Apache license header if needed. For new types, add the license header (copy from another file and set the current year only).

[schauder]

Thanks for looking into this. The problem, not with your PR, but with the current situation is, that the current approach of parsing the queries is not maintainable and need some serious work.
This rework should solve the underlying problem.

I'd like to keep the PR open though, in order to eventually merge at least the tests, in order to verify everything works as intended.

[simonparadies]

I'd like to keep the PR open though, in order to eventually merge at least the tests, in order to verify everything works as intended.
Since this PR solves the issue at hand, why not merge it as-is now?
The problem, not with your PR, but with the current situation is, that the current approach of parsing the queries is not maintainable and need some serious work. This rework should solve the underlying problem.
Imo that is a different and independent problem which should not delay this fix. I'd like to get away from ugly and imperformant work-arounds asap. I am afraid that a major rework will take its time. Merging the PR now does not impede a rework in any way. Thanks!

[gregturn]

Backported to 2.7.x and 2.6.x.

Thanks @simonparadies!