GemsTracker / gemstracker-library

GEneric Medical Survey Tracker, main library
http://gemstracker.org
BSD 3-Clause "New" or "Revised" License
6 stars 2 forks source link

Fix for Check all rounds function #831

Closed rvm-peercode closed 10 months ago

rvm-peercode commented 10 months ago

The 'Check all rounds' function didn't complete on the test hosts, because prevRoundId happened to be null, which caused a backtrace. Since AnyStepEngine already allowed prevRoundId to be null, I allowed this in the other StepEngine classes as well. Also, don't try to assign if relationId is null.

Not sure if this should be a hard error instead.

mddejong commented 10 months ago

Since it is a fact that a round can not have a previous round this is obviously a correct solution.