play1 icon indicating copy to clipboard operation
play1 copied to clipboard

What happened to OrderBy support in simplified JPA queries?

Open jvosloo opened this issue 6 years ago • 5 comments

In 1.5.1 at least, the support for OrderBy in simplified JPA queries seems to be missing? I.e. to be able to write simplified queries like this: Person.find("byNameOrderByName");

If I read this right: https://play.lighthouseapp.com/projects/57987/tickets/878-orderby-support-in-simplified-queries it was introduced in 1.3.x already?

Am I missing something?

jvosloo avatar Nov 06 '18 12:11 jvosloo

?

jvosloo avatar Dec 07 '18 01:12 jvosloo

@jvosloo Wow. You found The Lost Change. :)

Support for OrderBy was added in commit https://github.com/playframework/play1/commit/b91659a7 - at that time it was in master branch.

Later, we found that master branch is not up-to-date with some other release branches. Me and @xael-fry decided that master was not in valid state, and renamed it to old_master. So we lost commit b91659a7. :)

@xael-fry It seems that we need to merge 3 commits with commit message [#878] ... from master_old to master.

asolntsev avatar Dec 12 '18 21:12 asolntsev

Hehe... glad I could be helpful (and sorry for making more work for you!) 😄

jvosloo avatar Dec 14 '18 15:12 jvosloo

@xael-fry , @jvosloo

I tried my best in PR #1343 to port those commits to master.

xabolcs avatar Jan 04 '21 22:01 xabolcs

@jvosloo please check recent master!

My PR just landed, thanks to @xael-fry!

xabolcs avatar Apr 02 '22 17:04 xabolcs