Use beginTrustListImport/finishTrustListImport in deleteWithoutCommit(Identity) instead of manually calling computeAllScoresWithoutCommit
This will be used to make sure that restoreIdentity does not trigger double score recomputation.
Call finishTrustListImport AFTER deleteWithoutCommit to make sure that the deletion of the identity does not trigger a second computeAllScoresWithoutCommit.
- Move updateLastInsertDate() to OwnIdentity.restoreEdition(): It
doesn't concern the actual trust graph so its misplaced in
WebOfTrust.restoreEdition():
* This fixes a missing identity.storeWithoutCommit() after the
updateLastInsertDate().
* Also, this gets rid of unneeded duplicate calling of
updateLastInsertDate()
- Move TODO with regards to insert date scheduling to
OwnIdentity.needsInsert(). It was quite misplaced in
WebOfTrust.restoreEdition(). Also, improve that TODO in depth of
explanation.
Add initial implementations of unit tests for restoreIdentity(). Not
finished yet in any way!