Warning Msg Required

[xtal]

Please add a warning msg when changing the test list or test list cycle for a unit test collection. Without warning one can change the test list (or cycle) thereby loosing the link to historic data.

[Randle Taylor]

Any reason we wouldn't go one step further and just disable the ability to change it at all?

On a related note, overall, we need stronger protections like this in the admin I think. Another centre (who shall remain nameless ;) recently accidentally deleted a TestList and therefore lost all the historical data for it. Luckily it looks like we will be able to recover the data from backup in this case but this kind of thing can be avoided by implementing a soft delete (using a "deleted" flag on the object rather than actually removing it from the DB). At least we should try our best to prevent any TestListInstances & TestInstances from being deleted.

[Randle Taylor]

@xtal is there ever a reason we would want to allow changing the test list for a unit test collection? Seems to me like it would be safer to just not allow editing of the test list/cycle field after a unit test collection is created.

[Randle Taylor]

Resolved by 0f24efbf5ab60a794