directory-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From elecha...@apache.org
Subject svn commit: r1693071 - in /directory/studio/trunk/plugins: combinededitor/src/main/java/org/apache/directory/studio/combinededitor/editor/ ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/filtereditor/ ldapbrowser.common/...
Date Tue, 28 Jul 2015 11:43:16 GMT
Author: elecharny
Date: Tue Jul 28 11:43:16 2015
New Revision: 1693071

URL: http://svn.apache.org/r1693071
Log:
Fixed some PMD errors : final static variable names must be uppercase

Modified:
    directory/studio/trunk/plugins/combinededitor/src/main/java/org/apache/directory/studio/combinededitor/editor/TableEditorPageActionGroup.java
    directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/filtereditor/FilterCharacterPairMatcher.java
    directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/filtereditor/FilterContentAssistProcessor.java
    directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/widgets/browser/BrowserActionGroup.java
    directory/studio/trunk/plugins/ldapbrowser.ui/src/main/java/org/apache/directory/studio/ldapbrowser/ui/editors/entry/EntryEditorActionGroup.java
    directory/studio/trunk/plugins/ldapbrowser.ui/src/main/java/org/apache/directory/studio/ldapbrowser/ui/views/browser/BrowserViewActionGroup.java

Modified: directory/studio/trunk/plugins/combinededitor/src/main/java/org/apache/directory/studio/combinededitor/editor/TableEditorPageActionGroup.java
URL: http://svn.apache.org/viewvc/directory/studio/trunk/plugins/combinededitor/src/main/java/org/apache/directory/studio/combinededitor/editor/TableEditorPageActionGroup.java?rev=1693071&r1=1693070&r2=1693071&view=diff
==============================================================================
--- directory/studio/trunk/plugins/combinededitor/src/main/java/org/apache/directory/studio/combinededitor/editor/TableEditorPageActionGroup.java
(original)
+++ directory/studio/trunk/plugins/combinededitor/src/main/java/org/apache/directory/studio/combinededitor/editor/TableEditorPageActionGroup.java
Tue Jul 28 11:43:16 2015
@@ -242,7 +242,7 @@ public class TableEditorPageActionGroup
         entryEditorActionMap.put( fetchOperationalAttributesAction, new EntryEditorActionProxy(
viewer,
             new FetchOperationalAttributesAction() ) );
 
-        entryEditorActionMap.put( propertyDialogAction, new EntryEditorActionProxy( viewer,
+        entryEditorActionMap.put( PROPERTY_DIALOG_ACTION, new EntryEditorActionProxy( viewer,
             new EditorPagePropertiesAction( entryEditor ) ) );
     }
 
@@ -275,10 +275,10 @@ public class TableEditorPageActionGroup
     public void fillToolBar( IToolBarManager toolBarManager )
     {
         toolBarManager.add( new Separator() );
-        toolBarManager.add( entryEditorActionMap.get( newValueAction ) );
+        toolBarManager.add( entryEditorActionMap.get( NEW_VALUE_ACTION ) );
         toolBarManager.add( entryEditorActionMap.get( newAttributeAction ) );
         toolBarManager.add( new Separator() );
-        toolBarManager.add( entryEditorActionMap.get( deleteAction ) );
+        toolBarManager.add( entryEditorActionMap.get( DELETE_ACTION ) );
         toolBarManager.add( entryEditorActionMap.get( deleteAllValuesAction ) );
         toolBarManager.add( new Separator() );
         toolBarManager.add( entryEditorActionMap.get( refreshAttributesAction ) );
@@ -320,7 +320,7 @@ public class TableEditorPageActionGroup
     {
         // new
         menuManager.add( entryEditorActionMap.get( newAttributeAction ) );
-        menuManager.add( entryEditorActionMap.get( newValueAction ) );
+        menuManager.add( entryEditorActionMap.get( NEW_VALUE_ACTION ) );
         menuManager.add( entryEditorActionMap.get( newSearchAction ) );
         menuManager.add( entryEditorActionMap.get( newBatchOperationAction ) );
         menuManager.add( new Separator() );
@@ -344,10 +344,10 @@ public class TableEditorPageActionGroup
         menuManager.add( new Separator() );
 
         // copy, paste, delete
-        menuManager.add( entryEditorActionMap.get( copyAction ) );
-        menuManager.add( entryEditorActionMap.get( pasteAction ) );
-        menuManager.add( entryEditorActionMap.get( deleteAction ) );
-        menuManager.add( entryEditorActionMap.get( selectAllAction ) );
+        menuManager.add( entryEditorActionMap.get( COPY_ACTION ) );
+        menuManager.add( entryEditorActionMap.get( PASTE_ACTION ) );
+        menuManager.add( entryEditorActionMap.get( DELETE_ACTION ) );
+        menuManager.add( entryEditorActionMap.get( SELECT_ALL_ACTION ) );
         MenuManager advancedMenuManager = new MenuManager( Messages.getString( "TableEditorPageActionGroup.Advanced"
) ); //$NON-NLS-1$
         advancedMenuManager.add( entryEditorActionMap.get( copyDnAction ) );
         advancedMenuManager.add( entryEditorActionMap.get( copyUrlAction ) );
@@ -388,7 +388,7 @@ public class TableEditorPageActionGroup
         menuManager.add( new Separator( IWorkbenchActionConstants.MB_ADDITIONS ) );
 
         // properties
-        menuManager.add( entryEditorActionMap.get( propertyDialogAction ) );
+        menuManager.add( entryEditorActionMap.get( PROPERTY_DIALOG_ACTION ) );
     }
 
 

Modified: directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/filtereditor/FilterCharacterPairMatcher.java
URL: http://svn.apache.org/viewvc/directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/filtereditor/FilterCharacterPairMatcher.java?rev=1693071&r1=1693070&r2=1693071&view=diff
==============================================================================
--- directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/filtereditor/FilterCharacterPairMatcher.java
(original)
+++ directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/filtereditor/FilterCharacterPairMatcher.java
Tue Jul 28 11:43:16 2015
@@ -55,7 +55,7 @@ public class FilterCharacterPairMatcher
     public FilterCharacterPairMatcher( ISourceViewer sourceViewer, LdapFilterParser parser
)
     {
         this.parser = parser;
-        this.clear();
+        clear();
     }
 
 

Modified: directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/filtereditor/FilterContentAssistProcessor.java
URL: http://svn.apache.org/viewvc/directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/filtereditor/FilterContentAssistProcessor.java?rev=1693071&r1=1693070&r2=1693071&view=diff
==============================================================================
--- directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/filtereditor/FilterContentAssistProcessor.java
(original)
+++ directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/filtereditor/FilterContentAssistProcessor.java
Tue Jul 28 11:43:16 2015
@@ -69,7 +69,7 @@ public class FilterContentAssistProcesso
     IContentProposalProvider
 {
 
-    private static final Comparator<String> nameAndOidComparator = new Comparator<String>()
+    private static final Comparator<String> NAME_AND_OID_COMPARATOR = new Comparator<String>()
     {
         public int compare( String s1, String s2 )
         {
@@ -168,10 +168,10 @@ public class FilterContentAssistProcesso
     {
         this.schema = schema;
 
-        possibleAttributeTypes = new TreeMap<String, AttributeType>( nameAndOidComparator
);
+        possibleAttributeTypes = new TreeMap<String, AttributeType>( NAME_AND_OID_COMPARATOR
);
         possibleFilterTypes = new LinkedHashMap<String, String>();
-        possibleObjectClasses = new TreeMap<String, ObjectClass>( nameAndOidComparator
);
-        possibleMatchingRules = new TreeMap<String, MatchingRule>( nameAndOidComparator
);
+        possibleObjectClasses = new TreeMap<String, ObjectClass>( NAME_AND_OID_COMPARATOR
);
+        possibleMatchingRules = new TreeMap<String, MatchingRule>( NAME_AND_OID_COMPARATOR
);
 
         if ( schema != null )
         {

Modified: directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/widgets/browser/BrowserActionGroup.java
URL: http://svn.apache.org/viewvc/directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/widgets/browser/BrowserActionGroup.java?rev=1693071&r1=1693070&r2=1693071&view=diff
==============================================================================
--- directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/widgets/browser/BrowserActionGroup.java
(original)
+++ directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/widgets/browser/BrowserActionGroup.java
Tue Jul 28 11:43:16 2015
@@ -65,22 +65,22 @@ public class BrowserActionGroup implemen
     protected CollapseAllAction collapseAllAction;
 
     /** The Constant upAction. */
-    protected static final String upAction = "upAction"; //$NON-NLS-1$
+    protected static final String UP_ACTION = "upAction"; //$NON-NLS-1$
 
     /** The Constant refreshAction. */
-    protected static final String refreshAction = "refreshAction"; //$NON-NLS-1$
+    protected static final String REFRESH_ACTION = "refreshAction"; //$NON-NLS-1$
 
     /** The Constant filterChildrenAction. */
-    protected static final String filterChildrenAction = "filterChildrenAction"; //$NON-NLS-1$
+    protected static final String FILTER_CHILDREN_ACTION = "filterChildrenAction"; //$NON-NLS-1$
 
     /** The Constant openQuickSearchAction. */
-    protected static final String openQuickSearchAction = "openQuickSearch"; //$NON-NLS-1$
+    protected static final String OPEN_QUICK_SEARCH_ACTION = "openQuickSearch"; //$NON-NLS-1$
 
     /** The Constant unfilterChildrenAction. */
-    protected static final String unfilterChildrenAction = "unfilterChildrenAction"; //$NON-NLS-1$
+    protected static final String UNFILTER_CHILDREN_ACTION = "unfilterChildrenAction"; //$NON-NLS-1$
 
     /** The Constant propertyDialogAction. */
-    protected static final String propertyDialogAction = "propertyDialogAction"; //$NON-NLS-1$
+    protected static final String PROPERTY_DIALOG_ACTION = "propertyDialogAction"; //$NON-NLS-1$
 
     /** The browser action map. */
     protected Map<String, BrowserViewActionProxy> browserActionMap;
@@ -108,14 +108,14 @@ public class BrowserActionGroup implemen
         showQuickSearchAction = new ShowQuickSearchAction( mainWidget.getQuickSearchWidget()
);
         collapseAllAction = new CollapseAllAction( viewer );
 
-        browserActionMap.put( openQuickSearchAction, new BrowserViewActionProxy( viewer,
new OpenQuickSearchAction(
+        browserActionMap.put( OPEN_QUICK_SEARCH_ACTION, new BrowserViewActionProxy( viewer,
new OpenQuickSearchAction(
             mainWidget ) ) );
-        browserActionMap.put( upAction, new BrowserViewActionProxy( viewer, new UpAction(
viewer ) ) );
-        browserActionMap.put( refreshAction, new BrowserViewActionProxy( viewer, new RefreshAction()
) );
-        browserActionMap.put( filterChildrenAction, new BrowserViewActionProxy( viewer, new
FilterChildrenAction() ) );
+        browserActionMap.put( UP_ACTION, new BrowserViewActionProxy( viewer, new UpAction(
viewer ) ) );
+        browserActionMap.put( REFRESH_ACTION, new BrowserViewActionProxy( viewer, new RefreshAction()
) );
+        browserActionMap.put( FILTER_CHILDREN_ACTION, new BrowserViewActionProxy( viewer,
new FilterChildrenAction() ) );
         browserActionMap
-            .put( unfilterChildrenAction, new BrowserViewActionProxy( viewer, new UnfilterChildrenAction()
) );
-        browserActionMap.put( propertyDialogAction, new BrowserViewActionProxy( viewer, new
PropertiesAction() ) );
+            .put( UNFILTER_CHILDREN_ACTION, new BrowserViewActionProxy( viewer, new UnfilterChildrenAction()
) );
+        browserActionMap.put( PROPERTY_DIALOG_ACTION, new BrowserViewActionProxy( viewer,
new PropertiesAction() ) );
     }
 
 
@@ -164,9 +164,9 @@ public class BrowserActionGroup implemen
      */
     public void fillToolBar( IToolBarManager toolBarManager )
     {
-        toolBarManager.add( browserActionMap.get( upAction ) );
+        toolBarManager.add( browserActionMap.get( UP_ACTION ) );
         toolBarManager.add( new Separator() );
-        toolBarManager.add( browserActionMap.get( refreshAction ) );
+        toolBarManager.add( browserActionMap.get( REFRESH_ACTION ) );
         toolBarManager.add( new Separator() );
         toolBarManager.add( collapseAllAction );
         toolBarManager.update( true );
@@ -208,20 +208,20 @@ public class BrowserActionGroup implemen
     public void menuAboutToShow( IMenuManager menuManager )
     {
         // up
-        menuManager.add( browserActionMap.get( upAction ) );
+        menuManager.add( browserActionMap.get( UP_ACTION ) );
         menuManager.add( new Separator() );
 
         // filter
-        menuManager.add( browserActionMap.get( filterChildrenAction ) );
-        if ( ( browserActionMap.get( unfilterChildrenAction ) ).isEnabled() )
+        menuManager.add( browserActionMap.get( FILTER_CHILDREN_ACTION ) );
+        if ( ( browserActionMap.get( UNFILTER_CHILDREN_ACTION ) ).isEnabled() )
         {
-            menuManager.add( browserActionMap.get( unfilterChildrenAction ) );
+            menuManager.add( browserActionMap.get( UNFILTER_CHILDREN_ACTION ) );
         }
-        menuManager.add( browserActionMap.get( openQuickSearchAction ) );
+        menuManager.add( browserActionMap.get( OPEN_QUICK_SEARCH_ACTION ) );
         menuManager.add( new Separator() );
 
         // refresh
-        menuManager.add( browserActionMap.get( refreshAction ) );
+        menuManager.add( browserActionMap.get( REFRESH_ACTION ) );
         menuManager.add( new Separator() );
 
         // additions
@@ -229,7 +229,7 @@ public class BrowserActionGroup implemen
         menuManager.add( new Separator() );
 
         // properties
-        menuManager.add( browserActionMap.get( propertyDialogAction ) );
+        menuManager.add( browserActionMap.get( PROPERTY_DIALOG_ACTION ) );
     }
 
 
@@ -241,26 +241,26 @@ public class BrowserActionGroup implemen
         if ( actionBars != null )
         {
             actionBars.setGlobalActionHandler( ActionFactory.REFRESH.getId(), ( IAction )
browserActionMap
-                .get( refreshAction ) );
+                .get( REFRESH_ACTION ) );
             actionBars.setGlobalActionHandler( ActionFactory.PROPERTIES.getId(), ( IAction
) browserActionMap
-                .get( propertyDialogAction ) );
+                .get( PROPERTY_DIALOG_ACTION ) );
             actionBars.setGlobalActionHandler( ActionFactory.FIND.getId(), showQuickSearchAction
); // IWorkbenchActionDefinitionIds.FIND_REPLACE
             actionBars.updateActionBars();
         }
         else
         {
-            IAction pda = browserActionMap.get( propertyDialogAction );
+            IAction pda = browserActionMap.get( PROPERTY_DIALOG_ACTION );
             pda.setActionDefinitionId( BrowserCommonConstants.CMD_PROPERTIES );
             ActionUtils.activateActionHandler( pda );
 
-            IAction ra = browserActionMap.get( refreshAction );
+            IAction ra = browserActionMap.get( REFRESH_ACTION );
             ActionUtils.activateActionHandler( ra );
 
             showQuickSearchAction.setActionDefinitionId( BrowserCommonConstants.CMD_FIND
);
             ActionUtils.activateActionHandler( showQuickSearchAction );
         }
 
-        IAction ua = browserActionMap.get( upAction );
+        IAction ua = browserActionMap.get( UP_ACTION );
         ActionUtils.activateActionHandler( ua );
     }
 
@@ -278,14 +278,14 @@ public class BrowserActionGroup implemen
         }
         else
         {
-            IAction ra = browserActionMap.get( refreshAction );
+            IAction ra = browserActionMap.get( REFRESH_ACTION );
             ActionUtils.deactivateActionHandler( ra );
 
-            IAction pda = browserActionMap.get( propertyDialogAction );
+            IAction pda = browserActionMap.get( PROPERTY_DIALOG_ACTION );
             ActionUtils.deactivateActionHandler( pda );
         }
 
-        IAction ua = browserActionMap.get( upAction );
+        IAction ua = browserActionMap.get( UP_ACTION );
         ActionUtils.deactivateActionHandler( ua );
     }
 

Modified: directory/studio/trunk/plugins/ldapbrowser.ui/src/main/java/org/apache/directory/studio/ldapbrowser/ui/editors/entry/EntryEditorActionGroup.java
URL: http://svn.apache.org/viewvc/directory/studio/trunk/plugins/ldapbrowser.ui/src/main/java/org/apache/directory/studio/ldapbrowser/ui/editors/entry/EntryEditorActionGroup.java?rev=1693071&r1=1693070&r2=1693071&view=diff
==============================================================================
--- directory/studio/trunk/plugins/ldapbrowser.ui/src/main/java/org/apache/directory/studio/ldapbrowser/ui/editors/entry/EntryEditorActionGroup.java
(original)
+++ directory/studio/trunk/plugins/ldapbrowser.ui/src/main/java/org/apache/directory/studio/ldapbrowser/ui/editors/entry/EntryEditorActionGroup.java
Tue Jul 28 11:43:16 2015
@@ -239,7 +239,7 @@ public class EntryEditorActionGroup exte
         entryEditorActionMap.put( fetchOperationalAttributesAction, new EntryEditorActionProxy(
viewer,
             new FetchOperationalAttributesAction() ) );
 
-        entryEditorActionMap.put( propertyDialogAction, new EntryEditorActionProxy( viewer,
+        entryEditorActionMap.put( PROPERTY_DIALOG_ACTION, new EntryEditorActionProxy( viewer,
             new EntryEditorPropertiesAction( entryEditor ) ) );
     }
 
@@ -273,10 +273,10 @@ public class EntryEditorActionGroup exte
     public void fillToolBar( IToolBarManager toolBarManager )
     {
         toolBarManager.add( new Separator() );
-        toolBarManager.add( entryEditorActionMap.get( newValueAction ) );
+        toolBarManager.add( entryEditorActionMap.get( NEW_VALUE_ACTION ) );
         toolBarManager.add( entryEditorActionMap.get( newAttributeAction ) );
         toolBarManager.add( new Separator() );
-        toolBarManager.add( entryEditorActionMap.get( deleteAction ) );
+        toolBarManager.add( entryEditorActionMap.get( DELETE_ACTION ) );
         toolBarManager.add( entryEditorActionMap.get( deleteAllValuesAction ) );
         toolBarManager.add( new Separator() );
         toolBarManager.add( entryEditorActionMap.get( refreshAttributesAction ) );
@@ -321,7 +321,7 @@ public class EntryEditorActionGroup exte
     {
         // new
         menuManager.add( entryEditorActionMap.get( newAttributeAction ) );
-        menuManager.add( entryEditorActionMap.get( newValueAction ) );
+        menuManager.add( entryEditorActionMap.get( NEW_VALUE_ACTION ) );
         menuManager.add( entryEditorActionMap.get( newSearchAction ) );
         menuManager.add( entryEditorActionMap.get( newBatchOperationAction ) );
         menuManager.add( new Separator() );
@@ -341,10 +341,10 @@ public class EntryEditorActionGroup exte
         menuManager.add( new Separator() );
 
         // copy, paste, delete
-        menuManager.add( entryEditorActionMap.get( copyAction ) );
-        menuManager.add( entryEditorActionMap.get( pasteAction ) );
-        menuManager.add( entryEditorActionMap.get( deleteAction ) );
-        menuManager.add( entryEditorActionMap.get( selectAllAction ) );
+        menuManager.add( entryEditorActionMap.get( COPY_ACTION ) );
+        menuManager.add( entryEditorActionMap.get( PASTE_ACTION ) );
+        menuManager.add( entryEditorActionMap.get( DELETE_ACTION ) );
+        menuManager.add( entryEditorActionMap.get( SELECT_ALL_ACTION ) );
         MenuManager advancedMenuManager = new MenuManager( Messages.getString( "EntryEditorActionGroup.Advanced"
) ); //$NON-NLS-1$
         advancedMenuManager.add( entryEditorActionMap.get( copyDnAction ) );
         advancedMenuManager.add( entryEditorActionMap.get( copyUrlAction ) );
@@ -385,7 +385,7 @@ public class EntryEditorActionGroup exte
         menuManager.add( new Separator( IWorkbenchActionConstants.MB_ADDITIONS ) );
 
         // properties
-        menuManager.add( entryEditorActionMap.get( propertyDialogAction ) );
+        menuManager.add( entryEditorActionMap.get( PROPERTY_DIALOG_ACTION ) );
     }
 
 

Modified: directory/studio/trunk/plugins/ldapbrowser.ui/src/main/java/org/apache/directory/studio/ldapbrowser/ui/views/browser/BrowserViewActionGroup.java
URL: http://svn.apache.org/viewvc/directory/studio/trunk/plugins/ldapbrowser.ui/src/main/java/org/apache/directory/studio/ldapbrowser/ui/views/browser/BrowserViewActionGroup.java?rev=1693071&r1=1693070&r2=1693071&view=diff
==============================================================================
--- directory/studio/trunk/plugins/ldapbrowser.ui/src/main/java/org/apache/directory/studio/ldapbrowser/ui/views/browser/BrowserViewActionGroup.java
(original)
+++ directory/studio/trunk/plugins/ldapbrowser.ui/src/main/java/org/apache/directory/studio/ldapbrowser/ui/views/browser/BrowserViewActionGroup.java
Tue Jul 28 11:43:16 2015
@@ -301,9 +301,9 @@ public class BrowserViewActionGroup exte
      */
     public void fillToolBar( IToolBarManager toolBarManager )
     {
-        toolBarManager.add( browserActionMap.get( upAction ) );
+        toolBarManager.add( browserActionMap.get( UP_ACTION ) );
         toolBarManager.add( new Separator() );
-        toolBarManager.add( browserActionMap.get( refreshAction ) );
+        toolBarManager.add( browserActionMap.get( REFRESH_ACTION ) );
         toolBarManager.add( new Separator() );
         toolBarManager.add( collapseAllAction );
         toolBarManager.add( linkWithEditorAction );
@@ -355,7 +355,7 @@ public class BrowserViewActionGroup exte
         leid.setImageDescriptor( leid.getAction().getImageDescriptor() );
         menuManager.add( leid );
         menuManager.add( browserActionMap.get( gotoDnAction ) );
-        menuManager.add( browserActionMap.get( upAction ) );
+        menuManager.add( browserActionMap.get( UP_ACTION ) );
         menuManager.add( new Separator() );
 
         // copy/paste/...
@@ -384,12 +384,12 @@ public class BrowserViewActionGroup exte
         menuManager.add( new Separator() );
 
         // filter, batch
-        menuManager.add( browserActionMap.get( filterChildrenAction ) );
-        if ( ( browserActionMap.get( unfilterChildrenAction ) ).isEnabled() )
+        menuManager.add( browserActionMap.get( FILTER_CHILDREN_ACTION ) );
+        if ( ( browserActionMap.get( UNFILTER_CHILDREN_ACTION ) ).isEnabled() )
         {
-            menuManager.add( browserActionMap.get( unfilterChildrenAction ) );
+            menuManager.add( browserActionMap.get( UNFILTER_CHILDREN_ACTION ) );
         }
-        menuManager.add( browserActionMap.get( openQuickSearchAction ) );
+        menuManager.add( browserActionMap.get( OPEN_QUICK_SEARCH_ACTION ) );
         menuManager.add( new Separator() );
 
         // import/export
@@ -410,7 +410,7 @@ public class BrowserViewActionGroup exte
         menuManager.add( new Separator() );
 
         // refresh
-        menuManager.add( browserActionMap.get( refreshAction ) );
+        menuManager.add( browserActionMap.get( REFRESH_ACTION ) );
         MenuManager fetchMenuManager = new MenuManager( Messages.getString( "BrowserViewActionGroup.Fetch"
) ); //$NON-NLS-1$
         if ( browserActionMap.get( fetchOperationalAttributesAction ).isEnabled() )
         {
@@ -436,7 +436,7 @@ public class BrowserViewActionGroup exte
         menuManager.add( new Separator() );
 
         // properties
-        menuManager.add( browserActionMap.get( propertyDialogAction ) );
+        menuManager.add( browserActionMap.get( PROPERTY_DIALOG_ACTION ) );
     }
 
 



Mime
View raw message