Resolve "Changing the date of a slotted lab does not move the slots"
What does this mr do?
If a slotted lab is edited, it's slots won't update since they are represented by a Slot
which has a fixed start and end. This needs to be switched to an offset based on the start of the lab. `
Actions taken to fix bug
Added a new offsetSequence
number to denote the position of a slot within a slotted lab. The reason we did not get rid of the slot from the timeslot completely, was that it did not work with QueryDSL.
Screenshots
Note that this MR does not support a change in the length of the lab whilst the offset changes. This will be addressed in another MR.
Does this MR meet the acceptance criteria?
-
I have added a changelog entry to reflect the significant changes I made and the bug I fixed. -
A test was created to test the bug. -
I have updated the documentation accordingly. -
I adhere to the style guide.
Merge request reports
Activity
added priomed statusdoing typeimprovement labels
assigned to @hpage
added 35 commits
-
75e428ec...13aacbd0 - 32 commits from branch
development
- e8bea3e2 - update javadoc
- 47c13439 - Merge branch 'development' of gitlab.ewi.tudelft.nl:eip/labrador/queue into...
- 860b386e - Added migration for additional column
Toggle commit list-
75e428ec...13aacbd0 - 32 commits from branch
- Resolved by Henry Page
- Resolved by Henry Page
I see you have opted for adding a sequence number to the time-slots. This does indeed resolve some tracking issues we had by not knowing the index of a time-slot, but it also opens the door for a few new issues:
By keeping track of both slot times and slot indexes, either could be forgotten during updates in new code. If a desync occurs, we'll probably open Queue up for a whole new set of Database issues
.I was hoping to only use sequencing numbers and calculate the slot on-demand. Perhaps this could be as easy as making the
Slot
class calculate its length without storing and retrieving this information from the database. This way we need to make minimal changes while also removing the need for storing the slot times for every slot.
added 1 commit
- 86596853 - new offset based implementation, QueryDSL broken
added 1 commit
- 894fd451 - Revert "new offset based implementation, QueryDSL broken"