C212: A transaction should be used when storing the data Patient + Tumours + Sources

[infotel4iarc]

There is no explicit handling of the database transaction. The default configuration is “auto-commit”: each statement is in a separate transaction. http://db.apache.org/derby/docs/10.15/devguide/cdevconcepts19173.html This can be a problem when several database updates should be handled as an atomic operation. For example, a successful first update followed by a failed second update would let the database in an inconsistent state.

[fbinfotel]

The branch is https://github.com/infotel4iarc/CanReg5/tree/feature/C212

Here are the point that were done :

• Adding a transaction in the saveAllAction method. A transaction will ensure that all the data from patient, tumor, source are be recorded at the same time. If an exception occured, no record will be saved
• Adding a pop in case of exception in the method saveAllAction

Will ne merged to the dev branch after the completion of the C211 task

[fbinfotel]

The branch C212 has been merged to the dev branch