* This class may be instantiated; it is not intended to be subclassed. *
- * + * * @since 2.0 */ public class GenerateActionGroup extends ActionGroup { - /** - * Pop-up menu: id of the source sub menu (valuenet.sourceforge.phpdt.ui.source.menu).
- *
+ /**
+ * Pop-up menu: id of the source sub menu (value
+ * net.sourceforge.phpdt.ui.source.menu).
+ *
* @since 3.0
*/
- public static final String MENU_ID= "net.sourceforge.pheclipse.ui.source.menu"; //$NON-NLS-1$
+ public static final String MENU_ID = "net.sourceforge.pheclipse.ui.source.menu"; //$NON-NLS-1$
+
+ private PHPEditor fEditor;
- private PHPEditor fEditor;
-// private boolean fEditorIsOwner;
+ // private boolean fEditorIsOwner;
private IWorkbenchSite fSite;
- private String fGroupName= IContextMenuConstants.GROUP_SOURCE;
+
+ private String fGroupName = IContextMenuConstants.GROUP_SOURCE;
+
private List fRegisteredSelectionListeners;
-// private AddImportOnSelectionAction fAddImport;
-// private OverrideMethodsAction fOverrideMethods;
-// private AddGetterSetterAction fAddGetterSetter;
-// private AddUnimplementedConstructorsAction fAddUnimplementedConstructors;
-// private AddJavaDocStubAction fAddJavaDocStub;
+ // private AddImportOnSelectionAction fAddImport;
+ // private OverrideMethodsAction fOverrideMethods;
+ // private AddGetterSetterAction fAddGetterSetter;
+ // private AddUnimplementedConstructorsAction fAddUnimplementedConstructors;
+ // private AddJavaDocStubAction fAddJavaDocStub;
private AddBookmarkAction fAddBookmark;
+
private AddTaskAction fAddTaskAction;
-// private ExternalizeStringsAction fExternalizeStrings;
-// private FindStringsToExternalizeAction fFindStringsToExternalize;
-// private SurroundWithTryCatchAction fSurroundWithTryCatch;
+
+ // private ExternalizeStringsAction fExternalizeStrings;
+ // private FindStringsToExternalizeAction fFindStringsToExternalize;
+ // private SurroundWithTryCatchAction fSurroundWithTryCatch;
// private OrganizeImportsAction fOrganizeImports;
/**
- * Note: This constructor is for internal use only. Clients should not call this constructor.
+ * Note: This constructor is for internal use only. Clients should not call
+ * this constructor.
*/
public GenerateActionGroup(PHPEditor editor, String groupName) {
- fSite= editor.getSite();
- fEditor= editor;
- fGroupName= groupName;
-
- ISelectionProvider provider= fSite.getSelectionProvider();
- ISelection selection= provider.getSelection();
-
-// fAddImport= new AddImportOnSelectionAction(editor);
-// fAddImport.setActionDefinitionId(IJavaEditorActionDefinitionIds.ADD_IMPORT);
-// fAddImport.update();
-// editor.setAction("AddImport", fAddImport); //$NON-NLS-1$
-
-// fOrganizeImports= new OrganizeImportsAction(editor);
-// fOrganizeImports.setActionDefinitionId(IJavaEditorActionDefinitionIds.ORGANIZE_IMPORTS);
-// fOrganizeImports.editorStateChanged();
-// editor.setAction("OrganizeImports", fOrganizeImports); //$NON-NLS-1$
-
-// fOverrideMethods= new OverrideMethodsAction(editor);
-// fOverrideMethods.setActionDefinitionId(IJavaEditorActionDefinitionIds.OVERRIDE_METHODS);
-// fOverrideMethods.editorStateChanged();
-// editor.setAction("OverrideMethods", fOverrideMethods); //$NON-NLS-1$
-
-// fAddGetterSetter= new AddGetterSetterAction(editor);
-// fAddGetterSetter.setActionDefinitionId(IJavaEditorActionDefinitionIds.CREATE_GETTER_SETTER);
-// fAddGetterSetter.editorStateChanged();
-// editor.setAction("AddGetterSetter", fAddGetterSetter); //$NON-NLS-1$
-
-// fAddUnimplementedConstructors= new AddUnimplementedConstructorsAction(editor);
-// fAddUnimplementedConstructors.setActionDefinitionId(IJavaEditorActionDefinitionIds.ADD_UNIMPLEMENTED_CONTRUCTORS);
-// fAddUnimplementedConstructors.editorStateChanged();
-// editor.setAction("AddUnimplementedConstructors", fAddUnimplementedConstructors); //$NON-NLS-1$
-
-// fAddJavaDocStub= new AddJavaDocStubAction(editor);
-// fAddJavaDocStub.editorStateChanged();
-//
-// fSurroundWithTryCatch= new SurroundWithTryCatchAction(editor);
-// fSurroundWithTryCatch.setActionDefinitionId(IJavaEditorActionDefinitionIds.SURROUND_WITH_TRY_CATCH);
-// fSurroundWithTryCatch.update(selection);
-// provider.addSelectionChangedListener(fSurroundWithTryCatch);
-// editor.setAction("SurroundWithTryCatch", fSurroundWithTryCatch); //$NON-NLS-1$
-//
-// fExternalizeStrings= new ExternalizeStringsAction(editor);
-// fExternalizeStrings.setActionDefinitionId(IJavaEditorActionDefinitionIds.EXTERNALIZE_STRINGS);
-// fExternalizeStrings.editorStateChanged();
-// editor.setAction("ExternalizeStrings", fExternalizeStrings); //$NON-NLS-1$
+ fSite = editor.getSite();
+ fEditor = editor;
+ fGroupName = groupName;
+
+ ISelectionProvider provider = fSite.getSelectionProvider();
+ ISelection selection = provider.getSelection();
+
+ // fAddImport= new AddImportOnSelectionAction(editor);
+ // fAddImport.setActionDefinitionId(IJavaEditorActionDefinitionIds.ADD_IMPORT);
+ // fAddImport.update();
+ // editor.setAction("AddImport", fAddImport); //$NON-NLS-1$
+
+ // fOrganizeImports= new OrganizeImportsAction(editor);
+ // fOrganizeImports.setActionDefinitionId(IJavaEditorActionDefinitionIds.ORGANIZE_IMPORTS);
+ // fOrganizeImports.editorStateChanged();
+ // editor.setAction("OrganizeImports", fOrganizeImports); //$NON-NLS-1$
+
+ // fOverrideMethods= new OverrideMethodsAction(editor);
+ // fOverrideMethods.setActionDefinitionId(IJavaEditorActionDefinitionIds.OVERRIDE_METHODS);
+ // fOverrideMethods.editorStateChanged();
+ // editor.setAction("OverrideMethods", fOverrideMethods); //$NON-NLS-1$
+
+ // fAddGetterSetter= new AddGetterSetterAction(editor);
+ // fAddGetterSetter.setActionDefinitionId(IJavaEditorActionDefinitionIds.CREATE_GETTER_SETTER);
+ // fAddGetterSetter.editorStateChanged();
+ // editor.setAction("AddGetterSetter", fAddGetterSetter); //$NON-NLS-1$
+
+ // fAddUnimplementedConstructors= new
+ // AddUnimplementedConstructorsAction(editor);
+ // fAddUnimplementedConstructors.setActionDefinitionId(IJavaEditorActionDefinitionIds.ADD_UNIMPLEMENTED_CONTRUCTORS);
+ // fAddUnimplementedConstructors.editorStateChanged();
+ // editor.setAction("AddUnimplementedConstructors",
+ // fAddUnimplementedConstructors); //$NON-NLS-1$
+
+ // fAddJavaDocStub= new AddJavaDocStubAction(editor);
+ // fAddJavaDocStub.editorStateChanged();
+ //
+ // fSurroundWithTryCatch= new SurroundWithTryCatchAction(editor);
+ // fSurroundWithTryCatch.setActionDefinitionId(IJavaEditorActionDefinitionIds.SURROUND_WITH_TRY_CATCH);
+ // fSurroundWithTryCatch.update(selection);
+ // provider.addSelectionChangedListener(fSurroundWithTryCatch);
+ // editor.setAction("SurroundWithTryCatch", fSurroundWithTryCatch);
+ // //$NON-NLS-1$
+ //
+ // fExternalizeStrings= new ExternalizeStringsAction(editor);
+ // fExternalizeStrings.setActionDefinitionId(IJavaEditorActionDefinitionIds.EXTERNALIZE_STRINGS);
+ // fExternalizeStrings.editorStateChanged();
+ // editor.setAction("ExternalizeStrings", fExternalizeStrings);
+ // //$NON-NLS-1$
}
/**
- * Creates a new GenerateActionGroup. The group
- * requires that the selection provided by the page's selection provider
- * is of type org.eclipse.jface.viewers.IStructuredSelection.
- *
- * @param page the page that owns this action group
+ * Creates a new GenerateActionGroup. The group requires
+ * that the selection provided by the page's selection provider is of type
+ * org.eclipse.jface.viewers.IStructuredSelection.
+ *
+ * @param page
+ * the page that owns this action group
*/
public GenerateActionGroup(Page page) {
this(page.getSite());
}
/**
- * Creates a new GenerateActionGroup. The group
- * requires that the selection provided by the part's selection provider
- * is of type org.eclipse.jface.viewers.IStructuredSelection.
- *
- * @param part the view part that owns this action group
+ * Creates a new GenerateActionGroup. The group requires
+ * that the selection provided by the part's selection provider is of type
+ * org.eclipse.jface.viewers.IStructuredSelection.
+ *
+ * @param part
+ * the view part that owns this action group
*/
public GenerateActionGroup(IViewPart part) {
this(part.getSite());
}
private GenerateActionGroup(IWorkbenchSite site) {
- fSite= site;
- ISelectionProvider provider= fSite.getSelectionProvider();
- ISelection selection= provider.getSelection();
-
-// fOverrideMethods= new OverrideMethodsAction(site);
-// fAddGetterSetter= new AddGetterSetterAction(site);
-// fAddUnimplementedConstructors= new AddUnimplementedConstructorsAction(site);
-// fAddJavaDocStub= new AddJavaDocStubAction(site);
- fAddBookmark= new AddBookmarkAction(site.getShell());
- fAddTaskAction= new AddTaskAction(site);
-// fExternalizeStrings= new ExternalizeStringsAction(site);
-// fFindStringsToExternalize= new FindStringsToExternalizeAction(site);
-// fOrganizeImports= new OrganizeImportsAction(site);
-//
-// fOverrideMethods.update(selection);
-// fAddGetterSetter.update(selection);
-// fAddUnimplementedConstructors.update(selection);
-// fAddJavaDocStub.update(selection);
-// fExternalizeStrings.update(selection);
-// fFindStringsToExternalize.update(selection);
+ fSite = site;
+ ISelectionProvider provider = fSite.getSelectionProvider();
+ ISelection selection = provider.getSelection();
+
+ // fOverrideMethods= new OverrideMethodsAction(site);
+ // fAddGetterSetter= new AddGetterSetterAction(site);
+ // fAddUnimplementedConstructors= new
+ // AddUnimplementedConstructorsAction(site);
+ // fAddJavaDocStub= new AddJavaDocStubAction(site);
+ fAddBookmark = new AddBookmarkAction(site.getShell());
+ fAddTaskAction = new AddTaskAction(site);
+ // fExternalizeStrings= new ExternalizeStringsAction(site);
+ // fFindStringsToExternalize= new FindStringsToExternalizeAction(site);
+ // fOrganizeImports= new OrganizeImportsAction(site);
+ //
+ // fOverrideMethods.update(selection);
+ // fAddGetterSetter.update(selection);
+ // fAddUnimplementedConstructors.update(selection);
+ // fAddJavaDocStub.update(selection);
+ // fExternalizeStrings.update(selection);
+ // fFindStringsToExternalize.update(selection);
fAddTaskAction.update(selection);
-// fOrganizeImports.update(selection);
+ // fOrganizeImports.update(selection);
if (selection instanceof IStructuredSelection) {
- IStructuredSelection ss= (IStructuredSelection)selection;
+ IStructuredSelection ss = (IStructuredSelection) selection;
fAddBookmark.selectionChanged(ss);
} else {
fAddBookmark.setEnabled(false);
}
-// registerSelectionListener(provider, fOverrideMethods);
-// registerSelectionListener(provider, fAddGetterSetter);
-// registerSelectionListener(provider, fAddUnimplementedConstructors);
-// registerSelectionListener(provider, fAddJavaDocStub);
+ // registerSelectionListener(provider, fOverrideMethods);
+ // registerSelectionListener(provider, fAddGetterSetter);
+ // registerSelectionListener(provider, fAddUnimplementedConstructors);
+ // registerSelectionListener(provider, fAddJavaDocStub);
registerSelectionListener(provider, fAddBookmark);
-// registerSelectionListener(provider, fExternalizeStrings);
-// registerSelectionListener(provider, fFindStringsToExternalize);
-// registerSelectionListener(provider, fOrganizeImports);
+ // registerSelectionListener(provider, fExternalizeStrings);
+ // registerSelectionListener(provider, fFindStringsToExternalize);
+ // registerSelectionListener(provider, fOrganizeImports);
registerSelectionListener(provider, fAddTaskAction);
}
- private void registerSelectionListener(ISelectionProvider provider, ISelectionChangedListener listener) {
+ private void registerSelectionListener(ISelectionProvider provider,
+ ISelectionChangedListener listener) {
if (fRegisteredSelectionListeners == null)
- fRegisteredSelectionListeners= new ArrayList(12);
+ fRegisteredSelectionListeners = new ArrayList(12);
provider.addSelectionChangedListener(listener);
fRegisteredSelectionListeners.add(listener);
}
/*
- * The state of the editor owning this action group has changed.
- * This method does nothing if the group's owner isn't an
- * editor.
+ * The state of the editor owning this action group has changed. This method
+ * does nothing if the group's owner isn't an editor.
*/
/**
- * Note: This method is for internal use only. Clients should not call this method.
+ * Note: This method is for internal use only. Clients should not call this
+ * method.
*/
public void editorStateChanged() {
Assert.isTrue(isEditorOwner());
@@ -209,120 +224,137 @@ public class GenerateActionGroup extends ActionGroup {
// http://dev.eclipse.org/bugs/show_bug.cgi?id=17709
}
- /* (non-Javadoc)
- * Method declared in ActionGroup
+ /*
+ * (non-Javadoc) Method declared in ActionGroup
*/
public void fillActionBars(IActionBars actionBar) {
super.fillActionBars(actionBar);
setGlobalActionHandlers(actionBar);
}
- /* (non-Javadoc)
- * Method declared in ActionGroup
+ /*
+ * (non-Javadoc) Method declared in ActionGroup
*/
-// public void fillContextMenu(IMenuManager menu) {
-// super.fillContextMenu(menu);
-// if (fEditorIsOwner) {
-// IMenuManager subMenu= createEditorSubMenu(menu);
-// if (subMenu != null)
-// menu.appendToGroup(fGroupName, subMenu);
-// } else {
-//// appendToGroup(menu, fOrganizeImports);
-//// appendToGroup(menu, fOverrideMethods);
-//// appendToGroup(menu, fAddGetterSetter);
-//// appendToGroup(menu, fAddUnimplementedConstructors);
-//// appendToGroup(menu, fAddJavaDocStub);
-// appendToGroup(menu, fAddBookmark);
-// }
-// }
- /* (non-Javadoc)
- * Method declared in ActionGroup
+ // public void fillContextMenu(IMenuManager menu) {
+ // super.fillContextMenu(menu);
+ // if (fEditorIsOwner) {
+ // IMenuManager subMenu= createEditorSubMenu(menu);
+ // if (subMenu != null)
+ // menu.appendToGroup(fGroupName, subMenu);
+ // } else {
+ // // appendToGroup(menu, fOrganizeImports);
+ // // appendToGroup(menu, fOverrideMethods);
+ // // appendToGroup(menu, fAddGetterSetter);
+ // // appendToGroup(menu, fAddUnimplementedConstructors);
+ // // appendToGroup(menu, fAddJavaDocStub);
+ // appendToGroup(menu, fAddBookmark);
+ // }
+ // }
+ /*
+ * (non-Javadoc) Method declared in ActionGroup
*/
-// public void fillContextMenu(IMenuManager menu) {
-// super.fillContextMenu(menu);
-// IMenuManager subMenu= null;
-// if (isEditorOwner()) {
-// subMenu= fillEditorSubMenu(menu);
-// } else {
-// // subMenu= createViewSubMenu(menu);
-// }
-// if (subMenu != null)
-// menu.appendToGroup(fGroupName, subMenu);
-// }
+ // public void fillContextMenu(IMenuManager menu) {
+ // super.fillContextMenu(menu);
+ // IMenuManager subMenu= null;
+ // if (isEditorOwner()) {
+ // subMenu= fillEditorSubMenu(menu);
+ // } else {
+ // // subMenu= createViewSubMenu(menu);
+ // }
+ // if (subMenu != null)
+ // menu.appendToGroup(fGroupName, subMenu);
+ // }
public void fillContextMenu(IMenuManager menu) {
super.fillContextMenu(menu);
- String shortCut= null; //$NON-NLS-1$
-// if (fQuickAccessAction != null) {
-// shortCut= fQuickAccessAction.getShortCutString(); //$NON-NLS-1$
-// }
- IMenuManager subMenu= new MenuManager(
- ActionMessages.getString("SourceMenu.label") + (shortCut != null ? "\t" + shortCut : ""), //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
- MENU_ID);
- int added= 0;
+ String shortCut = null; //$NON-NLS-1$
+ // if (fQuickAccessAction != null) {
+ // shortCut= fQuickAccessAction.getShortCutString(); //$NON-NLS-1$
+ // }
+ IMenuManager subMenu = new MenuManager(
+ ActionMessages.getString("SourceMenu.label") + (shortCut != null ? "\t" + shortCut : ""), //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
+ MENU_ID);
+ int added = 0;
if (isEditorOwner()) {
- added= fillEditorSubMenu(subMenu);
+ added = fillEditorSubMenu(subMenu);
}
-// else {
-// added= fillViewSubMenu(subMenu);
-// }
+ // else {
+ // added= fillViewSubMenu(subMenu);
+ // }
if (added > 0)
menu.appendToGroup(fGroupName, subMenu);
}
- private int fillEditorSubMenu(IMenuManager source) {
-// IMenuManager result= new MenuManager(ActionMessages.getString("SourceMenu.label")); //$NON-NLS-1$
- int added= 0;
- added+= addEditorAction(source, "Comment"); //$NON-NLS-1$
- added+= addEditorAction(source, "Uncomment"); //$NON-NLS-1$
- added+= addEditorAction(source, "ToggleComment"); //$NON-NLS-1$
- added+= addEditorAction(source, "AddBlockComment"); //$NON-NLS-1$
- added+= addEditorAction(source, "RemoveBlockComment"); //$NON-NLS-1$
- added+= addEditorAction(source, "Format"); //$NON-NLS-1$
- added+= addEditorAction(source, "Indent"); //$NON-NLS-1$
-// result.add(new Separator());
-// added+= addAction(result, fOrganizeImports);
-// added+= addAction(result, fAddImport);
-// result.add(new Separator());
-// added+= addAction(result, fOverrideMethods);
-// added+= addAction(result, fAddGetterSetter);
-// added+= addAction(result, fAddUnimplementedConstructors);
-// added+= addAction(result, fAddJavaDocStub);
-// added+= addAction(result, fAddBookmark);
-// result.add(new Separator());
-// added+= addAction(result, fSurroundWithTryCatch);
-// added+= addAction(result, fExternalizeStrings);
-// if (added == 0)
-// result= null;
- return added;
- }
- /* (non-Javadoc)
- * Method declared in ActionGroup
+
+ private int fillEditorSubMenu(IMenuManager source) {
+ // IMenuManager result= new
+ // MenuManager(ActionMessages.getString("SourceMenu.label"));
+ // //$NON-NLS-1$
+ int added = 0;
+ added += addEditorAction(source, "Comment"); //$NON-NLS-1$
+ added += addEditorAction(source, "Uncomment"); //$NON-NLS-1$
+ added += addEditorAction(source, "ToggleComment"); //$NON-NLS-1$
+ added += addEditorAction(source, "AddBlockComment"); //$NON-NLS-1$
+ added += addEditorAction(source, "RemoveBlockComment"); //$NON-NLS-1$
+ added += addEditorAction(source, "Format"); //$NON-NLS-1$
+ added += addEditorAction(source, "Indent"); //$NON-NLS-1$
+ // result.add(new Separator());
+ // added+= addAction(result, fOrganizeImports);
+ // added+= addAction(result, fAddImport);
+ // result.add(new Separator());
+ // added+= addAction(result, fOverrideMethods);
+ // added+= addAction(result, fAddGetterSetter);
+ // added+= addAction(result, fAddUnimplementedConstructors);
+ // added+= addAction(result, fAddJavaDocStub);
+ // added+= addAction(result, fAddBookmark);
+ // result.add(new Separator());
+ // added+= addAction(result, fSurroundWithTryCatch);
+ // added+= addAction(result, fExternalizeStrings);
+ // if (added == 0)
+ // result= null;
+ return added;
+ }
+
+ /*
+ * (non-Javadoc) Method declared in ActionGroup
*/
public void dispose() {
if (fRegisteredSelectionListeners != null) {
- ISelectionProvider provider= fSite.getSelectionProvider();
- for (Iterator iter= fRegisteredSelectionListeners.iterator(); iter.hasNext();) {
- ISelectionChangedListener listener= (ISelectionChangedListener) iter.next();
+ ISelectionProvider provider = fSite.getSelectionProvider();
+ for (Iterator iter = fRegisteredSelectionListeners.iterator(); iter
+ .hasNext();) {
+ ISelectionChangedListener listener = (ISelectionChangedListener) iter
+ .next();
provider.removeSelectionChangedListener(listener);
}
}
- fEditor= null;
+ fEditor = null;
super.dispose();
}
private void setGlobalActionHandlers(IActionBars actionBar) {
-// actionBar.setGlobalActionHandler(JdtActionConstants.ADD_IMPORT, fAddImport);
-// actionBar.setGlobalActionHandler(JdtActionConstants.SURROUND_WITH_TRY_CATCH, fSurroundWithTryCatch);
-// actionBar.setGlobalActionHandler(JdtActionConstants.OVERRIDE_METHODS, fOverrideMethods);
-// actionBar.setGlobalActionHandler(JdtActionConstants.GENERATE_GETTER_SETTER, fAddGetterSetter);
-// actionBar.setGlobalActionHandler(JdtActionConstants.ADD_CONSTRUCTOR_FROM_SUPERCLASS, fAddUnimplementedConstructors);
-// actionBar.setGlobalActionHandler(JdtActionConstants.ADD_JAVA_DOC_COMMENT, fAddJavaDocStub);
-// actionBar.setGlobalActionHandler(JdtActionConstants.EXTERNALIZE_STRINGS, fExternalizeStrings);
-// actionBar.setGlobalActionHandler(JdtActionConstants.FIND_STRINGS_TO_EXTERNALIZE, fFindStringsToExternalize);
-// actionBar.setGlobalActionHandler(JdtActionConstants.ORGANIZE_IMPORTS, fOrganizeImports);
+ // actionBar.setGlobalActionHandler(JdtActionConstants.ADD_IMPORT,
+ // fAddImport);
+ // actionBar.setGlobalActionHandler(JdtActionConstants.SURROUND_WITH_TRY_CATCH,
+ // fSurroundWithTryCatch);
+ // actionBar.setGlobalActionHandler(JdtActionConstants.OVERRIDE_METHODS,
+ // fOverrideMethods);
+ // actionBar.setGlobalActionHandler(JdtActionConstants.GENERATE_GETTER_SETTER,
+ // fAddGetterSetter);
+ // actionBar.setGlobalActionHandler(JdtActionConstants.ADD_CONSTRUCTOR_FROM_SUPERCLASS,
+ // fAddUnimplementedConstructors);
+ // actionBar.setGlobalActionHandler(JdtActionConstants.ADD_JAVA_DOC_COMMENT,
+ // fAddJavaDocStub);
+ // actionBar.setGlobalActionHandler(JdtActionConstants.EXTERNALIZE_STRINGS,
+ // fExternalizeStrings);
+ // actionBar.setGlobalActionHandler(JdtActionConstants.FIND_STRINGS_TO_EXTERNALIZE,
+ // fFindStringsToExternalize);
+ // actionBar.setGlobalActionHandler(JdtActionConstants.ORGANIZE_IMPORTS,
+ // fOrganizeImports);
if (!isEditorOwner()) {
// editor provides its own implementation of these actions.
- actionBar.setGlobalActionHandler(IDEActionFactory.BOOKMARK.getId(), fAddBookmark);
- actionBar.setGlobalActionHandler(IDEActionFactory.ADD_TASK.getId(), fAddTaskAction);
+ actionBar.setGlobalActionHandler(IDEActionFactory.BOOKMARK.getId(),
+ fAddBookmark);
+ actionBar.setGlobalActionHandler(IDEActionFactory.ADD_TASK.getId(),
+ fAddTaskAction);
}
}
@@ -342,22 +374,22 @@ public class GenerateActionGroup extends ActionGroup {
return 0;
}
- private int addEditorAction(IMenuManager menu, String actionID) {
- if (fEditor == null)
- return 0;
- IAction action= fEditor.getAction(actionID);
- if (action == null)
- return 0;
- if (action instanceof IUpdate)
- ((IUpdate)action).update();
- if (action.isEnabled()) {
- menu.add(action);
- return 1;
- }
- return 0;
- }
-
- private boolean isEditorOwner() {
- return fEditor != null;
- }
+ private int addEditorAction(IMenuManager menu, String actionID) {
+ if (fEditor == null)
+ return 0;
+ IAction action = fEditor.getAction(actionID);
+ if (action == null)
+ return 0;
+ if (action instanceof IUpdate)
+ ((IUpdate) action).update();
+ if (action.isEnabled()) {
+ menu.add(action);
+ return 1;
+ }
+ return 0;
+ }
+
+ private boolean isEditorOwner() {
+ return fEditor != null;
+ }
}