-
Notifications
You must be signed in to change notification settings - Fork 21
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
Attendance of notes are directly edited #797
Comments
In connection to this we should also think about replacing the "NoteDetailsComponent" or basing it on the EntityDetailsComponent that is usually used |
Is this issue still in the app? I can't seem to replicate the bug |
For me I can still experience it.
However, this is a very minor problem because the changes are not written to the database. |
🎉 This issue has been resolved in version 2.60.0-master.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
🎉 This issue has been resolved in version 2.60.0-master.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
🎉 This issue has been resolved in version 3.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Describe the bug
When editing the attendance information of a note in the notes manager, the changes remain even when the editing is canceled.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
When clicking cancel, all changes in the
NoteDetailsComponent
should be undone.Additional context
The problem occurs because no deep copy of the note is passed to the
NoteDetailsComponent
it is possible to addnote.childrenAttendance = new Map(this.childrenAttendance);
in thecopy()
function of the note entity, but this will only generate a shallow copy of the map. If existingEventAttendance
objects are edited like in the scenario above, the bug remains. Either a deep copy of the map has to be generated or theNoteDetailsComponent
should not directly edit the input note.The text was updated successfully, but these errors were encountered: