Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

@OrderBy on a @OneToMany property is not used when lazy loading #154

Closed
rbygrave opened this issue Jun 25, 2014 · 2 comments
Closed

@OrderBy on a @OneToMany property is not used when lazy loading #154

rbygrave opened this issue Jun 25, 2014 · 2 comments
Assignees
@rbygrave
Labels
bug
Milestone
4.1.0

Comments

@rbygrave
Copy link
Member

e.g. On the Order bean:

@OneToMany(cascade=CascadeType.ALL, mappedBy="order")
@OrderBy("id asc, orderQty asc, cretime desc")
List<OrderDetail> details;

The order by is being ignored when the details are lazy loaded
@rbygrave rbygrave added this to the 4.0.5 milestone Jun 25, 2014
@rbygrave rbygrave added the bug label Jun 25, 2014
@rbygrave rbygrave self-assigned this Jun 25, 2014
@rbygrave
Copy link
Member Author

This was likely introduced in v3.x (with the refactoring on the lazy loading on many properties).

rbygrave added a commit that referenced this issue Jun 25, 2014
@rbygrave
Fix for #154 - @orderby on a @onetomany property is not used when laz…
1109d41 
…y loading
@rbygrave
Copy link
Member Author

Fixed in head/master.

@rbygrave rbygrave mentioned this issue Jul 2, 2014
Closed
rbygrave added a commit that referenced this issue May 7, 2015
@rbygrave
Merge pull request #272 from rlamarche/develop-3.3
c766793 
Backport fix for #154 - @orderby on a @onetomany property is not used when lazy loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rbygrave rbygrave

Labels
bug
Projects
None yet
Milestone
4.1.0
Development

No branches or pull requests
1 participant
@rbygrave