Bug #2745

RCP: 0.8.0, Dialog boxes of Sub-corpus and Partition creation are broken when CQP corpus is not ready

Ajouté par Sebastien Jacquot il y a plus de 5 ans. Mis à jour il y a plus d'un an.

Statut:Feedback Début:24/01/2020
Priorité:Normal Echéance:
Assigné à:- % réalisé:

80%

Catégorie:UI Temps passé: -
Version cible:TXM 0.8.4

Description

Dialog boxes of Sub-corpus and Partition creation are broken when CQP corpus is not ready.
Where trying to get some parent corpus properties to fill the widgets, CQP exceptions are raised when we can't create sub-corpus or partition.

Solution 1

  • use the same solution as in the TXMEditor to manage the parents branch computation
    => Use the new modal dialog box class /org.txm.rcp/src/main/java/org/txm/rcp/swt/dialog/AdvancedProgressMonitorDialog.java to ensure the parents are computed and to block the UI during the process

Solution 2

  • move all sub-corpus and partition creation code to some TXMEditor to benefit of the already managed parents branch computation.

Validation tests

WIP SJ:
  • purpose here is to test on large corpora that contains not computed partitions or corpus that the commands to create a partition and sub-corpus when called on a sub-corpus hierarchy works as expected

Demandes liées

lié à Bug #2742: RCP: 0.8.0, TXMEditor parameter toolbar is broken when CQ... Feedback 24/01/2020
lié à Feature #2756: RCP: 0.8.0, Convert Dialog boxes of Sub-corpus and Partit... Feedback 24/01/2020

Révisions associées

Révision 2567
Ajouté par Sebastien Jacquot il y a plus de 5 ans

Fixes: Dialog boxes of Sub-corpus and Partition creation are broken when CQP corpus is not ready (refs #2745)
Progresses on: Editor results computing executed with not computed parents branch or too long process may lead to display wrong data in editor (refs #2682)

Historique

#1 Mis à jour par Sebastien Jacquot il y a plus de 5 ans

  • Description mis à jour (diff)
  • Statut changé de New à Feedback
  • % réalisé changé de 0 à 80

Solution 1 has been implemented. Need new issue and wiki specs for solution 2.

#2 Mis à jour par Sebastien Jacquot il y a plus de 5 ans

New issue for solution 2: #2756.

#3 Mis à jour par Sebastien Jacquot il y a plus d'un an

  • Version cible changé de TXM 0.8.1 à TXM 0.8.4

Formats disponibles : Atom PDF