DMC/scipro
7
0
Fork 0
Code Issues30 Pull Requests4 Actions Projects1 Releases Wiki Activity

Fix intermittent sorting of criterion to actual order #18

Merged
tozh4728 merged 2 commits from 3495-grading-report-template-random-order into develop 2024-11-14 12:39:42 +01:00
Conversation 2 Commits 2 Files Changed 1 +3 -2
niat8586 commented 2024-11-13 12:03:44 +01:00
Owner

When you tried to edit a grading template in "Project management" -> "Grading Templates" the criterion could be shown in an order not reflecting the sortOrder of the criterion.
The reason was that we only got the arbitrary order depending on the insertion order in the database.

Changed the Collection to a List, it is now an ArrayList. Also added @OrderBy("sortOrder ASC") to actually sort by the order of the criterion thus ensuring visual order is intact.

When you tried to edit a grading template in "Project management" -> "Grading Templates" the criterion could be shown in an order not reflecting the sortOrder of the criterion. The reason was that we only got the arbitrary order depending on the insertion order in the database. Changed the Collection to a List, it is now an ArrayList. Also added @OrderBy("sortOrder ASC") to actually sort by the order of the criterion thus ensuring visual order is intact.
niat8586 added 1 commit 2024-11-13 12:03:45 +01:00 
When you tried to edit a grading template in  "Project management" -> "Grading Templates" the criterion could be shown in an order not reflecting the sortOrder of the criterion.
The reason was that we only got the arbitrary order depending on the insertion order in the database.

Changed the Collection to a List, it is now an ArrayList. Also added @OrderBy("sortOrder ASC") to actually sort by the order of the criterion thus ensuring visual order is intact.
tozh4728 requested review from tozh4728 2024-11-14 10:30:27 +01:00
tozh4728 approved these changes 2024-11-14 10:30:33 +01:00
Dismissed
tozh4728 added 1 commit 2024-11-14 12:39:27 +01:00
tozh4728 approved these changes 2024-11-14 12:39:34 +01:00
tozh4728 merged commit d89417ceea into develop 2024-11-14 12:39:42 +01:00
tozh4728 deleted branch 3495-grading-report-template-random-order 2024-11-14 12:39:42 +01:00
Sign in to join this conversation.
No Reviewers
tozh4728
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: DMC/scipro#18
No description provided.
Delete Branch "3495-grading-report-template-random-order"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?