Do not set property if calculation failed

fix for bug Bug 3232
bioclipse · Aug 21, 2012 · 584222e · 584222e
1 parent e9198f2
commit 584222e
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/...se.cdk.ui.sdfeditor/src/net/bioclipse/cdk/ui/sdfeditor/business/MoleculeTableManager.java b/...se.cdk.ui.sdfeditor/src/net/bioclipse/cdk/ui/sdfeditor/business/MoleculeTableManager.java
@@ -224,8 +224,10 @@ public void calculateProperty( IMoleculesEditorModel model,
             for(IPropertyCalculator<?> calculator:calculators) {
                 calculateProgress.subTask( String.format( "%d/%d", prop++,
                                                           calculators.length) );
-                model.setPropertyFor( i, calculator.getPropertyName(),
-                             calculator.calculate( model.getMoleculeAt( i ) ) );
+                Object property = calculator.calculate(model.getMoleculeAt(i));
+                if(property!= null) {
+                	model.setPropertyFor( i, calculator.getPropertyName(),property);
+                }
                 calculateProgress.worked( 100 );
                 if(progress.isCanceled())
                     throw new OperationCanceledException();