TMS-Uni-Stuttgart / Tutor-Management-System

Tool for tracking exam requirements of students at universities (in Germany)
3 stars 6 forks source link

All present button broken #1034

Open nk-coding opened 3 months ago

nk-coding commented 3 months ago

The "all present" button seems to be broken, however investigation required here

Morphenoed commented 3 months ago

:heavy_check_mark: Expected behaviour

After clicking on all present button, the attendances of all students that are undefined are set to present.

⚡ Actual behaviour

Something goes wrong and even though the view is updated, the database entries are not. After page reload there are only 2-4 random students whose attendances were set to present.

:joystick: Steps to reproduce

  1. Click all present
  2. Reload
  3. Aaaand... broken...

💡 My thoughts

The problem does not lie within all present button, but rather in how the async updates are performed. If you set attendaces of some students fast enough and/or with a timing. After reload not every attendance will be updated.