fix #774 infinite loop in net.sourceforge.phpeclipse.builder.IdentifierIndexManager...
[phpeclipse.git] / net.sourceforge.phpeclipse / src / net / sourceforge / phpdt / ui / actions / GenerateActionGroup.java
index 40aa451..f627deb 100644 (file)
@@ -1,10 +1,10 @@
 /*******************************************************************************
  * Copyright (c) 2002 International Business Machines Corp. and others.
- * All rights reserved. This program and the accompanying materials 
- * are made available under the terms of the Common Public License v0.5 
+ * All rights reserved. This program and the accompanying materials
+ * are made available under the terms of the Common Public License v0.5
  * which accompanies this distribution, and is available at
  * http://www.eclipse.org/legal/cpl-v05.html
- * 
+ *
  * Contributors:
  *     IBM Corporation - initial API and implementation
  ******************************************************************************/
@@ -29,13 +29,11 @@ import org.eclipse.jface.viewers.ISelectionProvider;
 import org.eclipse.jface.viewers.IStructuredSelection;
 import org.eclipse.ui.IActionBars;
 import org.eclipse.ui.IViewPart;
-import org.eclipse.ui.IWorkbenchActionConstants;
 import org.eclipse.ui.IWorkbenchSite;
 import org.eclipse.ui.actions.ActionGroup;
 import org.eclipse.ui.actions.AddBookmarkAction;
+import org.eclipse.ui.ide.IDEActionFactory;
 import org.eclipse.ui.part.Page;
-import org.eclipse.ui.texteditor.ConvertLineDelimitersAction;
-import org.eclipse.ui.texteditor.ITextEditorActionDefinitionIds;
 import org.eclipse.ui.texteditor.IUpdate;
 
 /**
@@ -49,313 +47,324 @@ import org.eclipse.ui.texteditor.IUpdate;
  * @since 2.0
  */
 public class GenerateActionGroup extends ActionGroup {
-  /**
-        * Pop-up menu: id of the source sub menu (value <code>org.eclipse.jdt.ui.source.menu</code>).
+       /**
+        * Pop-up menu: id of the source sub menu (value
+        * <code>net.sourceforge.phpdt.ui.source.menu</code>).
         * 
         * @since 3.0
         */
-       public static final String MENU_ID= "net.sourceforge.pheclipse.ui.source.menu"; //$NON-NLS-1$
-       
-  private PHPEditor fEditor;
-//     private boolean fEditorIsOwner;
+       public static final String MENU_ID = "net.sourceforge.pheclipse.ui.source.menu"; //$NON-NLS-1$
+
+       private PHPEditor fEditor;
+
+       // 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;
 
-       private ConvertLineDelimitersAction fConvertToWindows;
-       private ConvertLineDelimitersAction fConvertToUNIX;
-       private ConvertLineDelimitersAction fConvertToMac;
-       
        /**
-        * 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$              
-               
-               fConvertToWindows= new ConvertLineDelimitersAction(editor, "\r\n"); //$NON-NLS-1$
-               fConvertToWindows.setActionDefinitionId( ITextEditorActionDefinitionIds .CONVERT_LINE_DELIMITERS_TO_WINDOWS);
-               editor.setAction("ConvertLineDelimitersToWindows", fConvertToWindows); //$NON-NLS-1$            
-               
-               fConvertToUNIX= new ConvertLineDelimitersAction(editor, "\n"); //$NON-NLS-1$
-               fConvertToUNIX.setActionDefinitionId( ITextEditorActionDefinitionIds .CONVERT_LINE_DELIMITERS_TO_UNIX);
-               editor.setAction("ConvertLineDelimitersToUNIX", fConvertToUNIX); //$NON-NLS-1$          
-
-               fConvertToMac= new ConvertLineDelimitersAction(editor, "\r"); //$NON-NLS-1$
-               fConvertToMac.setActionDefinitionId( ITextEditorActionDefinitionIds .CONVERT_LINE_DELIMITERS_TO_MAC);
-               editor.setAction("ConvertLineDelimitersToMac", fConvertToMac); //$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 <code>GenerateActionGroup</code>. The group 
-        * requires that the selection provided by the page's selection provider 
-        * is of type <code>org.eclipse.jface.viewers.IStructuredSelection</code>.
+        * Creates a new <code>GenerateActionGroup</code>. The group requires
+        * that the selection provided by the page's selection provider is of type
+        * <code>org.eclipse.jface.viewers.IStructuredSelection</code>.
         * 
-        * @param page the page that owns this action group
+        * @param page
+        *            the page that owns this action group
         */
        public GenerateActionGroup(Page page) {
                this(page.getSite());
        }
 
        /**
-        * Creates a new <code>GenerateActionGroup</code>. The group 
-        * requires that the selection provided by the part's selection provider 
-        * is of type <code>org.eclipse.jface.viewers.IStructuredSelection</code>.
+        * Creates a new <code>GenerateActionGroup</code>. The group requires
+        * that the selection provided by the part's selection provider is of type
+        * <code>org.eclipse.jface.viewers.IStructuredSelection</code>.
         * 
-        * @param part the view part that owns this action group
+        * @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());
-               
+
                // http://dev.eclipse.org/bugs/show_bug.cgi?id=17709
-               fConvertToMac.update();
-               fConvertToUNIX.update();
-               fConvertToWindows.update();
        }
 
-       /* (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);
-               } 
-//             else {
-//                     added= fillViewSubMenu(subMenu);
-//             }
+                       added = fillEditorSubMenu(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(PHPdtActionConstants.CONVERT_LINE_DELIMITERS_TO_WINDOWS, fConvertToWindows);
-               actionBar.setGlobalActionHandler(PHPdtActionConstants.CONVERT_LINE_DELIMITERS_TO_UNIX, fConvertToUNIX);
-               actionBar.setGlobalActionHandler(PHPdtActionConstants.CONVERT_LINE_DELIMITERS_TO_MAC, fConvertToMac);
+               // 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(IWorkbenchActionConstants.BOOKMARK, fAddBookmark);
-                       actionBar.setGlobalActionHandler(IWorkbenchActionConstants.ADD_TASK, fAddTaskAction);
+                       actionBar.setGlobalActionHandler(IDEActionFactory.BOOKMARK.getId(),
+                                       fAddBookmark);
+                       actionBar.setGlobalActionHandler(IDEActionFactory.ADD_TASK.getId(),
+                                       fAddTaskAction);
                }
        }
-       
+
        private int appendToGroup(IMenuManager menu, IAction action) {
                if (action != null && action.isEnabled()) {
                        menu.appendToGroup(fGroupName, action);
                        return 1;
                }
                return 0;
-       }       
+       }
 
        private int addAction(IMenuManager menu, IAction action) {
                if (action != null && action.isEnabled()) {
@@ -363,24 +372,24 @@ public class GenerateActionGroup extends ActionGroup {
                        return 1;
                }
                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;
+       }
 }