Added patch from #1437426: error in assign template
[phpeclipse.git] / net.sourceforge.phpeclipse / src / net / sourceforge / phpdt / ui / actions / GenerateActionGroup.java
index 36797b8..e5f593a 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,33 +29,37 @@ 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;
 
 /**
  * Action group that adds the source and generate actions to a part's context
  * menu and installs handlers for the corresponding global menu actions.
- * 
+ *
  * <p>
  * This class may be instantiated; it is not intended to be subclassed.
  * </p>
- * 
+ *
  * @since 2.0
  */
 public class GenerateActionGroup extends ActionGroup {
-       
+  /**
+        * 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;
        private IWorkbenchSite fSite;
        private String fGroupName= IContextMenuConstants.GROUP_SOURCE;
        private List fRegisteredSelectionListeners;
-       
+
 //     private AddImportOnSelectionAction fAddImport;
 //     private OverrideMethodsAction fOverrideMethods;
 //     private AddGetterSetterAction fAddGetterSetter;
@@ -66,13 +70,9 @@ public class GenerateActionGroup extends ActionGroup {
 //     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.
         */
@@ -80,7 +80,7 @@ public class GenerateActionGroup extends ActionGroup {
                fSite= editor.getSite();
                fEditor= editor;
                fGroupName= groupName;
-                               
+
                ISelectionProvider provider= fSite.getSelectionProvider();
                ISelection selection= provider.getSelection();
 
@@ -88,7 +88,7 @@ public class GenerateActionGroup extends ActionGroup {
 //             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();
@@ -98,49 +98,38 @@ public class GenerateActionGroup extends ActionGroup {
 //             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$          
-               
+//             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$          
-//             
+//             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$          
+//             editor.setAction("ExternalizeStrings", fExternalizeStrings); //$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$            
        }
-       
+
        /**
-        * Creates a new <code>GenerateActionGroup</code>. The group 
-        * requires that the selection provided by the page's selection provider 
+        * 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
         */
        public GenerateActionGroup(Page page) {
@@ -148,21 +137,21 @@ public class GenerateActionGroup extends ActionGroup {
        }
 
        /**
-        * Creates a new <code>GenerateActionGroup</code>. The group 
-        * requires that the selection provided by the part's selection provider 
+        * 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
         */
        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);
@@ -172,10 +161,10 @@ public class GenerateActionGroup extends ActionGroup {
 //             fExternalizeStrings= new ExternalizeStringsAction(site);
 //             fFindStringsToExternalize= new FindStringsToExternalizeAction(site);
 //             fOrganizeImports= new OrganizeImportsAction(site);
-//             
+//
 //             fOverrideMethods.update(selection);
 //             fAddGetterSetter.update(selection);
-//             fAddUnimplementedConstructors.update(selection);        
+//             fAddUnimplementedConstructors.update(selection);
 //             fAddJavaDocStub.update(selection);
 //             fExternalizeStrings.update(selection);
 //             fFindStringsToExternalize.update(selection);
@@ -187,7 +176,7 @@ public class GenerateActionGroup extends ActionGroup {
                } else {
                        fAddBookmark.setEnabled(false);
                }
-               
+
 //             registerSelectionListener(provider, fOverrideMethods);
 //             registerSelectionListener(provider, fAddGetterSetter);
 //             registerSelectionListener(provider, fAddUnimplementedConstructors);
@@ -198,16 +187,16 @@ public class GenerateActionGroup extends ActionGroup {
 //             registerSelectionListener(provider, fOrganizeImports);
                registerSelectionListener(provider, fAddTaskAction);
        }
-       
+
        private void registerSelectionListener(ISelectionProvider provider, ISelectionChangedListener listener) {
                if (fRegisteredSelectionListeners == null)
                        fRegisteredSelectionListeners= new ArrayList(12);
                provider.addSelectionChangedListener(listener);
                fRegisteredSelectionListeners.add(listener);
        }
-       
+
        /*
-        * The state of the editor owning this action group has changed. 
+        * The state of the editor owning this action group has changed.
         * This method does nothing if the group's owner isn't an
         * editor.
         */
@@ -216,11 +205,8 @@ public class GenerateActionGroup extends ActionGroup {
         */
        public void editorStateChanged() {
                Assert.isTrue(isEditorOwner());
-               
+
                // http://dev.eclipse.org/bugs/show_bug.cgi?id=17709
-               fConvertToMac.update();
-               fConvertToUNIX.update();
-               fConvertToWindows.update();
        }
 
        /* (non-Javadoc)
@@ -230,7 +216,7 @@ public class GenerateActionGroup extends ActionGroup {
                super.fillActionBars(actionBar);
                setGlobalActionHandlers(actionBar);
        }
-       
+
        /* (non-Javadoc)
         * Method declared in ActionGroup
         */
@@ -252,24 +238,46 @@ public class GenerateActionGroup extends ActionGroup {
        /* (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;
+               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()) {
-                       subMenu= createEditorSubMenu(menu);
-               } else {
-       //              subMenu= createViewSubMenu(menu);
+                       added= fillEditorSubMenu(subMenu);
                }
-               if (subMenu != null)
+//             else {
+//                     added= fillViewSubMenu(subMenu);
+//             }
+               if (added > 0)
                        menu.appendToGroup(fGroupName, subMenu);
        }
-       
-  private IMenuManager createEditorSubMenu(IMenuManager mainMenu) {
-    IMenuManager result= new MenuManager(ActionMessages.getString("SourceMenu.label")); //$NON-NLS-1$
+  private int fillEditorSubMenu(IMenuManager source) {
+//    IMenuManager result= new MenuManager(ActionMessages.getString("SourceMenu.label")); //$NON-NLS-1$
     int added= 0;
-    added+= addEditorAction(result, "Comment"); //$NON-NLS-1$
-    added+= addEditorAction(result, "Uncomment"); //$NON-NLS-1$
-               added+= addEditorAction(result, "Format"); //$NON-NLS-1$
+    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);
@@ -279,12 +287,12 @@ public class GenerateActionGroup extends ActionGroup {
 //    added+= addAction(result, fAddUnimplementedConstructors);
 //    added+= addAction(result, fAddJavaDocStub);
 //    added+= addAction(result, fAddBookmark);
-//    result.add(new Separator());             
+//    result.add(new Separator());
 //    added+= addAction(result, fSurroundWithTryCatch);
 //    added+= addAction(result, fExternalizeStrings);
-    if (added == 0)
-      result= null;
-    return result;
+//    if (added == 0)
+//      result= null;
+    return added;
   }
        /* (non-Javadoc)
         * Method declared in ActionGroup
@@ -300,7 +308,7 @@ public class GenerateActionGroup extends ActionGroup {
     fEditor= null;
                super.dispose();
        }
-       
+
        private void setGlobalActionHandlers(IActionBars actionBar) {
 //             actionBar.setGlobalActionHandler(JdtActionConstants.ADD_IMPORT, fAddImport);
 //             actionBar.setGlobalActionHandler(JdtActionConstants.SURROUND_WITH_TRY_CATCH, fSurroundWithTryCatch);
@@ -311,23 +319,20 @@ public class GenerateActionGroup extends ActionGroup {
 //             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);
                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()) {
@@ -335,8 +340,8 @@ public class GenerateActionGroup extends ActionGroup {
                        return 1;
                }
                return 0;
-       }       
-       
+       }
+
   private int addEditorAction(IMenuManager menu, String actionID) {
     if (fEditor == null)
       return 0;
@@ -351,8 +356,8 @@ public class GenerateActionGroup extends ActionGroup {
     }
     return 0;
   }
-  
+
   private boolean isEditorOwner() {
     return fEditor != null;
-  }    
+  }
 }