Community
Participate
Working Groups
The Review Navigator tool bar remains disabled after cancelling the creation of a review. the work around is to click in other groups is available however this may not be trivial depending on the review structure. The same occurs if for any reason there is an abnormal exit of a dialog, Is it necessary to disable the toolbar at all ? if the dialogs are modal disabling the toolbar may not be necessary.
The dialog are not all modal, which is why it is necessary to do this. I noticed this problem too. We do re-enable the toolbar when dialogs are cancelled, however for some reason this is not done in the view. The easy workaround is to click anywhere and the toolbbar will be re-enabled. This should be investigated.
Review Navigator View ToolBar is now enabled when canceling dialogs. I also fixed a bug where a reviewItem was added even if the anomaly addition was cancelled
Testing on LINUX environment When there is an exception in the review property view, it remove the sensitivity of the menu in the R4E preferences and in the R4E navigator view. Kill the R4E property and R4E navigator view, re-open the R4E Navigator view and the menu icon remains insensitive. There is no way to make them sensitive again. The only way to recover the menu icon is by killing Eclipse and restarting it.
I see it also in Windows environment, once it happens, it does not matter where you click the tool bar remains disabled and the following exception is triggered. org.eclipse.core.commands.NotEnabledException: Trying to execute the disabled command org.eclipse.mylyn.reviews.r4e.ui.commands.openElement at org.eclipse.core.commands.Command.executeWithChecks(Command.java:469) at org.eclipse.ui.internal.handlers.HandlerService.executeCommand(HandlerService.java:178) at org.eclipse.ui.internal.handlers.SlaveHandlerService.executeCommand(SlaveHandlerService.java:247) at org.eclipse.mylyn.reviews.r4e.ui.internal.navigator.ReviewNavigatorActionGroup.openElementCommand(ReviewNavigatorActionGroup.java:659) at org.eclipse.mylyn.reviews.r4e.ui.internal.navigator.ReviewNavigatorView$2.doubleClick(ReviewNavigatorView.java:450) at org.eclipse.jface.viewers.StructuredViewer$1.run(StructuredViewer.java:845)
Removed the code that was at the root of the problems, which was obsolete and unnecessary. That should fix it.
*** Bug 358592 has been marked as a duplicate of this bug. ***
It seems that removing the code causes the R4E properties view not to be updated properly when the changes are coming from a command triggered in the Navigator view. More fixes are needed here
I changed the way the Properties view is updated. THis is much better now than it was. I also got rid of many potential bugs in the process. It should finally work fine now
