Bug #2819

0.8.0, when deleting a corpus the KR is not deleted

Added by Matthieu Decorde over 3 years ago. Updated over 3 years ago.

Status:New Start date:05/14/2020
Priority:Normal Due date:
Assignee:- % Done:

80%

Category:Annotation Spent time: -
Target version:TXM 0.8.1

Description

When a corpus is removed, the associated knowledge repository (KR) is not removed. If the corpus is re-imported, the old KR is used instead of a fresh new empty KR.

The Corpus<->KR relation life cycle is the following:
  • a corpus is imported and a fresh empty KR is available
  • a corpus is re-imported and a reused empty KR is available
  • a corpus is updated and its KR is preserved

Solution

Catch the CQP "Corpus delete" event and remove the corpus KR from the KRs database.

In both TXM 0.8.0 and TXM 0.8.1beta, the KR is removed but changes are only visible after restarting TXM.

The corpus update process raises a "CQP corpus delete" event -> the KRs database must check if the corpus is being updated, if true the KR must not be deleted.

Associated revisions

Revision 2864
Added by Matthieu Decorde over 3 years ago

fix kr line decorators
fix kr removal when a corpus is deleted refs #2819

Revision 2914
Added by Matthieu Decorde over 3 years ago

fix KR history refs #2819
add SaveAnnotations command refs #2860

History

#1 Updated by Matthieu Decorde over 3 years ago

  • Description updated (diff)

#2 Updated by Matthieu Decorde over 3 years ago

  • Description updated (diff)

#3 Updated by Matthieu Decorde over 3 years ago

  • Description updated (diff)

#4 Updated by Matthieu Decorde over 3 years ago

  • % Done changed from 0 to 80

fixed by r2867

#5 Updated by Serge Heiden over 3 years ago

  • % Done changed from 80 to 40

Hypothesis: the Corpus Update deletes the KR.

#6 Updated by Matthieu Decorde over 3 years ago

  • Description updated (diff)
  • % Done changed from 40 to 80

#7 Updated by Serge Heiden over 3 years ago

  • Description updated (diff)

#8 Updated by Serge Heiden over 3 years ago

  • Description updated (diff)

Also available in: Atom PDF