-
Notifications
You must be signed in to change notification settings - Fork 0
Guarantee findAllChildrenUuids results order #809
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.
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
Merged
Meklo
merged 30 commits into
main
from
marcellinh/fix-findAllChildrenUuids-result-order
May 12, 2026
Merged
Changes from all commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
2c851e5
preliminary notes
Mathieu-Deharbe 37304cb
use modifications_order for composites
Mathieu-Deharbe e6f5caa
keep order for composites saved to gridexplore
Mathieu-Deharbe 135e1b1
correct drag to composites from root
Mathieu-Deharbe 08894ee
autosort in many places
Mathieu-Deharbe 35a6dca
cleans pom
Mathieu-Deharbe f1aa801
Merge branch 'main' into use-modifications_order-for-coposites
Mathieu-Deharbe 7db3bb9
@OrderBy("modificationsOrder asc")
Mathieu-Deharbe db73cee
don't remove orphans because modification entity may be passed from c…
Mathieu-Deharbe 135d7c1
migrationCompositesOrder.sql
Mathieu-Deharbe d8a1ad6
Merge branch 'main' into use-modifications_order-for-coposites
Mathieu-Deharbe 172488c
rename comment
Mathieu-Deharbe 4437ed0
Merge branch 'main' into use-modifications_order-for-coposites
Mathieu-Deharbe cbdbd5c
updates order when replacing a composite
Mathieu-Deharbe ddf31ce
Merge branch 'main' into use-modifications_order-for-coposites
Mathieu-Deharbe 0b83cc2
remove primary key and prevent migration from running on h2 TUs
Mathieu-Deharbe 4d7dae2
Guarantee findAllChildrenUuids results order
64ee136
Merge branch 'main' into use-modifications_order-for-coposites
Meklo 4a98784
Merge remote-tracking branch 'origin/use-modifications_order-for-copo…
999cfd8
Fix regression
9a7db12
Simplify query
9d11392
COMPOSITE_MODIFICATION_SUB_MODIFICATIONSPK to composite_modification_…
Mathieu-Deharbe c009a47
Merge remote-tracking branch 'origin/use-modifications_order-for-copo…
5a6607f
replace PrimaryKey
Mathieu-Deharbe 2d0cf95
Merge branch 'use-modifications_order-for-coposites' into marcellinh/…
Mathieu-Deharbe d85c88d
Merge branch 'main' into use-modifications_order-for-coposites
Meklo 82f76da
Merge branch 'main' into use-modifications_order-for-coposites
Mathieu-Deharbe b8669b1
Merge remote-tracking branch 'origin/use-modifications_order-for-copo…
7e5e4d0
Merge main
fd55a13
PR suggestions
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would add a small comment like :
// returns the composite uuid and its children and subchildren's uuidsbecause the request is a bit complicated and it is not very logical that compositeUuid is returned given the function name (
findAllChildrenUuids).