Author: max.andersen(a)jboss.com Date: 2008-02-07 09:01:12 -0500 (Thu, 07 Feb 2008) New Revision: 6162 Modified: branches/jbosstools-2.0.x/hibernatetools/plugins/org.hibernate.eclipse.console/src/org/hibernate/eclipse/console/views/ConfigurationsViewActionGroup.java Log: JBIDE-1752 broken null check Modified: branches/jbosstools-2.0.x/hibernatetools/plugins/org.hibernate.eclipse.console/src/org/hibernate/eclipse/console/views/ConfigurationsViewActionGroup.java =================================================================== --- branches/jbosstools-2.0.x/hibernatetools/plugins/org.hibernate.eclipse.console/src/org/hibernate/eclipse/console/views/ConfigurationsViewActionGroup.java 2008-02-07 13:50:45 UTC (rev 6161) +++ branches/jbosstools-2.0.x/hibernatetools/plugins/org.hibernate.eclipse.console/src/org/hibernate/eclipse/console/views/ConfigurationsViewActionGroup.java 2008-02-07 14:01:12 UTC (rev 6162) @@ -138,8 +138,8 @@ menu.add(schemaExportAction); } menu.add(new Separator() ); - if (first instanceof PersistentClass - || first.getClass() == Property.class){ + // TODO: shouldn't these and maybe the others not be defined via menu extension points ? + if (first != null && (first instanceof PersistentClass || first.getClass() == Property.class)) { menu.add(openSourceAction); menu.add(openMappingAction); }