Discussion :

[MystEre]

Bonjour,

J'utilise Eclipse 3.8.1 pour coder une appli perso. J'ai ajouté dans le menu éditer de mon appli les actions undo et redo dans ma classe ActionBarAdvisor en n'oubliant pas de faire

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
undoAction = ActionFactory.UNDO.create(window);
register(undoAction);
redoAction = ActionFactory.REDO.create(window);
register(redoAction);

jusqu'ici pas de soucis je vois bien le menu éditer et les actions grisées.

Puis à partir d'une de mes vues (qui hérite de CommonNavigator) j'exécute une action d'un popup menu définie comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
   <extension
         point="org.eclipse.ui.popupMenus">
      <objectContribution
            adaptable="false"
            id="test.personnesElementPopup"
            objectClass="test.ui.resources.PersonnesElement">
         <action
               class="test.ui.actions.personne.NouvellePersonneAction"
               enablesFor="1"
               id="test.newPersonAction"
               label="%NouvellePersonne.name"
               menubarPath="navigatorGroup.new">
         </action>
      </objectContribution>
   </extension>

qui s'exécute ainsi :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
		IUndoableOperation aOperation = new NouvellePersonneOperation("Ajout personne", window, selection);
		try {
			IWorkbenchOperationSupport aWorkbenchOperationSupport = window.getWorkbench().getOperationSupport();
			aOperation.addContext(aWorkbenchOperationSupport.getUndoContext());
			aWorkbenchOperationSupport.getOperationHistory().execute(aOperation, null, null);
		}
		catch (ExecutionException e) {
		}

j'ai bien ajouté dans ma vue ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
	public void createPartControl(Composite aParent) {
		super.createPartControl(aParent);
		IWorkbenchOperationSupport aWorkbenchOperationSupport = getSite().getWorkbenchWindow().getWorkbench().getOperationSupport();
		UndoActionHandler aUndoAction = new UndoActionHandler(getSite(), aWorkbenchOperationSupport.getUndoContext());
		RedoActionHandler aRedoAction = new RedoActionHandler(getSite(), aWorkbenchOperationSupport.getUndoContext());
		IActionBars actionBars = getViewSite().getActionBars();
		actionBars.setGlobalActionHandler(ActionFactory.UNDO.getId(), aUndoAction);
		actionBars.setGlobalActionHandler(ActionFactory.REDO.getId(), aRedoAction);
	}

mais les actions undo/redo restent grisées !
Pourtant en debuggant je passe bien après l'exécution de mon action dans
OperationHistoryActionHandler.update et UndoActionHandler.shouldBeEnabled renvoie true, le texte du undo est bien positionné avec la valeur donnée par mon action mais rien ne se rafraichit dans le menu.

Je n'arrive pas à comprendre ce que je rate...
Merci d'avance pour votre aide.

MystEre

[Gueritarish]

Salut,

J'imagine que ta classe NouvellPersonneOperation hérite d'AbstractOperation ou, au moins, implémente l'interface IUndoableOperation. Est-ce que tu as vérifié que les méthodes canUndo() et canRedo() renvoie bien true?
J'imagine que oui, mais c'est toujours utile à vérifier !
Sinon, tout m'a l'air bon. Tu peux éventuellement rajouter un NullProgressMonitor, dans ton exécution...

Voilà, à+
Gueritarish

[MystEre]

Bonjour et merci pour ta réponse,

Oui mon opération hérite bien de AbstractOperation et du coup canUndo et canRedo renvoyent bien true.
Merci pour le conseil du NullProgressMonitor.

J'ai mis ce problème de côté pour l'instant et avancent sur d'autres trucs, si je finis par trouver la solution je reviendrais l'indiquer, en attendant si vous avez d'autres idées n'hésitez pas.

MystEre