Community
Participate
Working Groups
Build Identifier: I20110524-1000 In the preferences window under Plug-in Development -> API Errors/Warnings. Under API Use tab there is a link "install them now" that is supposed to open the install software screen. It currently does nothing. The install software screen can still be accessed by the help menu. (If the link is greyed out, change the "ignore" option to the top right to warning or error.) Reproducible: Always Steps to Reproduce: 1. Go to the preferences window (Window -> Preferences). 2. Plug-in Development -> API Errors/Warnings. Select the API Use tab. 3. Make sure the options on the right are all warning or error. 4. Click on "install them now".
Following the same instructions in the corresponding 3.7 build, the link works correctly.
This is most likely a problem in 4.1, not PDE.
(In reply to comment #2) > This is most likely a problem in 4.1, not PDE. This is working fine in 3.7 builds, moving to Platform UI for comment.
Looks like PDE is executing the command directly instead of using the IHandlerService. Thread [main] (Suspended (breakpoint at line 1803 in Workbench$MakeHandlersGo)) Workbench$MakeHandlersGo.execute(ExecutionEvent) line: 1803 Command.executeWithChecks(ExecutionEvent) line: 476 ApiErrorsWarningsConfigurationBlock$4.widgetSelected(SelectionEvent)
(In reply to comment #4) > Looks like PDE is executing the command directly instead of using the > IHandlerService. > > Thread [main] (Suspended (breakpoint at line 1803 in Workbench$MakeHandlersGo)) > Workbench$MakeHandlersGo.execute(ExecutionEvent) line: 1803 > Command.executeWithChecks(ExecutionEvent) line: 476 > ApiErrorsWarningsConfigurationBlock$4.widgetSelected(SelectionEvent) Sounds like something we should change.
(In reply to comment #5) > (In reply to comment #4) > > Looks like PDE is executing the command directly instead of using the > > IHandlerService. > > > > Thread [main] (Suspended (breakpoint at line 1803 in Workbench$MakeHandlersGo)) > > Workbench$MakeHandlersGo.execute(ExecutionEvent) line: 1803 > > Command.executeWithChecks(ExecutionEvent) line: 476 > > ApiErrorsWarningsConfigurationBlock$4.widgetSelected(SelectionEvent) > > Sounds like something we should change. We could but on the other hand this is what one does if one wants to invoke the current handler for the command at hand. And its official API which I think is also (indirectly) used by ContributedAction.
(In reply to comment #6) > (In reply to comment #5) > > (In reply to comment #4) > > > Looks like PDE is executing the command directly instead of using the > > > IHandlerService. > > > > > > Thread [main] (Suspended (breakpoint at line 1803 in Workbench$MakeHandlersGo)) > > > Workbench$MakeHandlersGo.execute(ExecutionEvent) line: 1803 > > > Command.executeWithChecks(ExecutionEvent) line: 476 > > > ApiErrorsWarningsConfigurationBlock$4.widgetSelected(SelectionEvent) > > > > Sounds like something we should change. > > We could but on the other hand this is what one does if one wants to invoke the > current handler for the command at hand. And its official API which I think is > also (indirectly) used by ContributedAction. Bug# 348692 raised to handle ContributedAction. Moving this to bug PDE.
(In reply to comment #7) > (In reply to comment #6) > > (In reply to comment #5) > > > (In reply to comment #4) > > > > Looks like PDE is executing the command directly instead of using the > > > > IHandlerService. > > > > > > > > Thread [main] (Suspended (breakpoint at line 1803 in Workbench$MakeHandlersGo)) > > > > Workbench$MakeHandlersGo.execute(ExecutionEvent) line: 1803 > > > > Command.executeWithChecks(ExecutionEvent) line: 476 > > > > ApiErrorsWarningsConfigurationBlock$4.widgetSelected(SelectionEvent) > > > > > > Sounds like something we should change. > > > > We could but on the other hand this is what one does if one wants to invoke the > > current handler for the command at hand. And its official API which I think is > > also (indirectly) used by ContributedAction. > > > Bug# 348692 raised to handle ContributedAction. Moving this to bug PDE. This is not a PDE bug. The API which is used should work and there might be other clients relying on that. This should be fixed in the compatibility layer.
(In reply to comment #8) > > > > Bug# 348692 raised to handle ContributedAction. Moving this to bug PDE. > > This is not a PDE bug. The API which is used should work and there might be > other clients relying on that. This should be fixed in the compatibility layer. It's API in the way Actions are API ... it's not recommended, but it is sometimes necessary in 3.x. ContributedAction was written that way because it was necessary. IHandlerService won't do the correct thing. I agree with Dani that it must be fixed, see bug 328971 I would still recommend PDE use IHandlerService to execute their command, unless they have a need similar to ContributedAction. PW
> I would still recommend PDE use IHandlerService to execute their command, > unless they have a need similar to ContributedAction. > > PW Sounds fair. Filed bug 348711 to track that.
This now works in 4.1 I20110609-2200 PW *** This bug has been marked as a duplicate of bug 328971 ***
Thank you for the bug report, David.
