Better error handling. Related to bug: 3231

bioclipse · Jun 20, 2012 · f94862a · f94862a
1 parent 02ff8a2
commit f94862a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/...t.bioclipse.align.kalign.ws/src/net/bioclipse/align/kalign/ws/business/KalignManager.java b/...t.bioclipse.align.kalign.ws/src/net/bioclipse/align/kalign/ws/business/KalignManager.java
@@ -267,7 +267,8 @@ private List<? extends ISequence> align(List<? extends ISequence>
             }
 
         } catch ( Exception e ) {
-            throw new BioclipseException("Error in KAlign WS: ", e);
+            throw new BioclipseException(
+                          "Error in KAlign WS: " + e.getMessage(), e );
         }
 
     }

diff --git a/...clipse.align.kalign.ws/src/net/bioclipse/align/kalign/ws/handlers/KalignPopupHandler.java b/...clipse.align.kalign.ws/src/net/bioclipse/align/kalign/ws/handlers/KalignPopupHandler.java
@@ -20,6 +20,7 @@
 import net.bioclipse.core.business.BioclipseException;
 import net.bioclipse.core.domain.IProtein;
 import net.bioclipse.core.domain.ISequence;
+import net.bioclipse.core.util.LogUtils;
 import net.bioclipse.ui.business.IUIManager;
 
 import org.apache.log4j.Logger;
@@ -107,7 +108,7 @@ public Object execute( ExecutionEvent event ) throws ExecutionException {
         try {
             alignedProteins = kalign.alignProteins(proteins);
         } catch (BioclipseException e) {
-            // TODO: Better error handling
+            LogUtils.handleException( e, logger, "net.bioclipse.align.kalign.ws" );
             return null;
         }
         List<ISequence> alignedSequences = new ArrayList<ISequence>();