* 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).
+ *
+ * @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 +72,13 @@ 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 +86,7 @@ public class GenerateActionGroup extends ActionGroup {
fSite= editor.getSite();
fEditor= editor;
fGroupName= groupName;
-
+
ISelectionProvider provider= fSite.getSelectionProvider();
ISelection selection= provider.getSelection();
@@ -88,7 +94,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 +104,49 @@ 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$
-
+// 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$
-
+ 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("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$
+ editor.setAction("ConvertLineDelimitersToMac", fConvertToMac); //$NON-NLS-1$
}
-
+
/**
- * Creates a new GenerateActionGroup. The group
- * requires that the selection provided by the page's selection provider
+ * 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) {
@@ -148,21 +154,21 @@ public class GenerateActionGroup extends ActionGroup {
}
/**
- * Creates a new GenerateActionGroup. The group
- * requires that the selection provided by the part's selection provider
+ * 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);
@@ -172,10 +178,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 +193,7 @@ public class GenerateActionGroup extends ActionGroup {
} else {
fAddBookmark.setEnabled(false);
}
-
+
// registerSelectionListener(provider, fOverrideMethods);
// registerSelectionListener(provider, fAddGetterSetter);
// registerSelectionListener(provider, fAddUnimplementedConstructors);
@@ -198,16 +204,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,7 +222,7 @@ 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();
@@ -230,7 +236,7 @@ public class GenerateActionGroup extends ActionGroup {
super.fillActionBars(actionBar);
setGlobalActionHandlers(actionBar);
}
-
+
/* (non-Javadoc)
* Method declared in ActionGroup
*/
@@ -252,24 +258,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 +307,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 +328,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);
@@ -316,18 +344,18 @@ public class GenerateActionGroup extends ActionGroup {
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 +363,8 @@ public class GenerateActionGroup extends ActionGroup {
return 1;
}
return 0;
- }
-
+ }
+
private int addEditorAction(IMenuManager menu, String actionID) {
if (fEditor == null)
return 0;
@@ -351,8 +379,8 @@ public class GenerateActionGroup extends ActionGroup {
}
return 0;
}
-
+
private boolean isEditorOwner() {
return fEditor != null;
- }
+ }
}