Author: Grid.Qian Date: 2009-06-15 05:02:07 -0400 (Mon, 15 Jun 2009) New Revision: 15942 Modified: trunk/bpel/plugins/org.eclipse.bpel.ui/src/org/eclipse/bpel/ui/properties/CorrSetImplSection.java Log: JBIDE-4482: java.lang.NullPointerException at org.eclipse.jface.viewers.ColumnViewer$2.getCellEditor(ColumnViewer.java:239) when click the propertyid of a correlation Modified: trunk/bpel/plugins/org.eclipse.bpel.ui/src/org/eclipse/bpel/ui/properties/CorrSetImplSection.java =================================================================== --- trunk/bpel/plugins/org.eclipse.bpel.ui/src/org/eclipse/bpel/ui/properties/CorrSetImplSection.java 2009-06-15 02:35:37 UTC (rev 15941) +++ trunk/bpel/plugins/org.eclipse.bpel.ui/src/org/eclipse/bpel/ui/properties/CorrSetImplSection.java 2009-06-15 09:02:07 UTC (rev 15942) @@ -100,7 +100,11 @@ // Update the UnusedPropertyFilter with the proper contents. propertyFilter.setCandidates(((CorrelationSet)getInput()).getProperties(), Collections.singletonList(element)); - return true; + + // change true to false by Grid.Qian + // because the cell editor is null, if the column can be modified, + // when the system will active the cell editor, will get a null exception + return false; } public Object getValue(Object element, String property) { return element; @@ -148,8 +152,8 @@ @Override protected void addAllAdapters() { super.addAllAdapters(); - List corrList = ((CorrelationSet)getInput()).getProperties(); - for (Iterator it = corrList.iterator(); it.hasNext(); ) { + List<Property> corrList = ((CorrelationSet)getInput()).getProperties(); + for (Iterator<Property> it = corrList.iterator(); it.hasNext(); ) { Property property = (Property)it.next(); fAdapters[1].addToObject(property); }