Bug #2742
RCP: 0.8.0, TXMEditor parameter toolbar is broken when CQP corpus is not ready
| 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
TXMEditor parameters toolbar is broken when a CQP corpus is not ready -> CQP exception are raised.
E.g. calling Cooccurrence on a not computed Sub-corpus throws a CQP exception when the Cooccurrence editor is opened and tries to load the available corpus properties.
Solution 1¶
When opening a TXMEditor always pre-compute the parent before. Add parent.compute(false) in- a) TXMEditor.openEditor()
- b) TXMEditor.this.createPartControl
a) can manage the parent.compute in a job
b) always works but might cause freezes at any moment. -> maybe we can use a busyWhile job to correctly freezes the interface
---> Use the b) solution and validate the freeze is ok
Solution 2¶
- protect all TXMResult getters methods with a this.compute(false)
Validation tests¶
WIP SJ:
- purpose here is to test on large corporas that contain not computed partitions or corpus that calling a command or reopening a result that are descendant to the not computed objects works as expected
Demandes liées
Révisions associées
Historique
#1 Mis à jour par Matthieu Decorde il y a plus de 5 ans
- Description mis à jour (diff)
#2 Mis à jour par Matthieu Decorde il y a plus de 5 ans
- Description mis à jour (diff)
#3 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 b) has been implemented. A new modal dialog box class (/org.txm.rcp/src/main/java/org/txm/rcp/swt/dialog/AdvancedProgressMonitorDialog.java) has been developed.
This class runs the parents branch computing in TXMEditor.createPartControl(). The progress bar dialog opens (with a delay of 1 second) and blocks the UI until the process is done so the Editor is not shown before it's fully ready to be used.
#4 Mis à jour par Sebastien Jacquot il y a plus de 5 ans
- Description mis à jour (diff)
Need new issue and wiki specs for solution 2.
#5 Mis à jour par Sebastien Jacquot il y a plus de 5 ans
- Description mis à jour (diff)
#6 Mis à jour par Sebastien Jacquot il y a plus de 5 ans
Since when TXMEditor.createPartControl() is called the editor is already opened, it leads to some Eclipse Workbench errors (shown in Error Log view) and forces us to close back the editor if the process is user canceled.
So the pre-computing of the parents branch has been moved to TXMEditor.openEditor().
#7 Mis à jour par Sebastien Jacquot il y a plus d'un an
- Version cible changé de TXM 0.8.1 à TXM 0.8.4