[seam-commits] Seam SVN: r8630 - trunk/src/excel/org/jboss/seam/excel/jxl.

[seam-commits at lists.jboss.org]

Author: danielc.roth
Date: 2008-08-08 04:12:12 -0400 (Fri, 08 Aug 2008)
New Revision: 8630

Modified:
   trunk/src/excel/org/jboss/seam/excel/jxl/JXLExcelFactory.java
   trunk/src/excel/org/jboss/seam/excel/jxl/JXLTemplates.java
Log:
JBSEAM-3232 JBSEAM-3235 and changed so that we don't swallow exceptions

Modified: trunk/src/excel/org/jboss/seam/excel/jxl/JXLExcelFactory.java
===================================================================
--- trunk/src/excel/org/jboss/seam/excel/jxl/JXLExcelFactory.java	2008-08-08 06:08:31 UTC (rev 8629)
+++ trunk/src/excel/org/jboss/seam/excel/jxl/JXLExcelFactory.java	2008-08-08 08:12:12 UTC (rev 8630)
@@ -203,7 +203,7 @@
       catch (Exception e)
       {
          String message = Interpolator.instance().interpolate("Page orientation {0} not supported, try {1}", orientation, getValidConstants(JXL_PAGE_ORIENTATION_CLASS_NAME));
-         throw new ExcelWorkbookException(message);
+         throw new ExcelWorkbookException(message, e);
       }
    }
 
@@ -226,7 +226,7 @@
       catch (Exception e)
       {
          String message = Interpolator.instance().interpolate("Page size {0} not supported, try {1}", paperSize, getValidConstants(JXL_PAPER_SIZE_CLASS_NAME));
-         throw new ExcelWorkbookException(message);
+         throw new ExcelWorkbookException(message, e);
       }
    }
 
@@ -743,7 +743,7 @@
       catch (Exception e)
       {
          String message = Interpolator.instance().interpolate("Verical alignment {0} not supported, try {1}", verticalAlignment, getValidConstants(JXL_VERTICAL_ALIGNMENT_CLASS_NAME));
-         throw new ExcelWorkbookException(message);
+         throw new ExcelWorkbookException(message, e);
       }
    }
 
@@ -769,7 +769,7 @@
       catch (Exception e)
       {
          String message = Interpolator.instance().interpolate("Orientation {0} not supported, try {1}", orientation, getValidConstants(JXL_ORIENTATION_CLASS_NAME));
-         throw new ExcelWorkbookException(message);
+         throw new ExcelWorkbookException(message, e);
       }
    }
 
@@ -795,7 +795,7 @@
       catch (Exception e)
       {
          String message = Interpolator.instance().interpolate("Alignment {0} not supported, try {1}", alignment, getValidConstants(JXL_ALIGNMENT_CLASS_NAME));
-         throw new ExcelWorkbookException(message);
+         throw new ExcelWorkbookException(message, e);
       }
    }
 
@@ -821,7 +821,7 @@
       catch (Exception e)
       {
          String message = Interpolator.instance().interpolate("Pattern {0} not supported, try {1}", pattern, getValidConstants(JXL_PATTERN_CLASS_NAME));
-         throw new ExcelWorkbookException(message);
+         throw new ExcelWorkbookException(message, e);
       }
    }
 
@@ -847,7 +847,7 @@
       catch (Exception e)
       {
          String message = Interpolator.instance().interpolate("Border line style {0} not supported, try {1}", borderLineStyle, getValidConstants(JXL_BORDER_LINE_STYLE_CLASS_NAME));
-         throw new ExcelWorkbookException(message);
+         throw new ExcelWorkbookException(message, e);
       }
    }
 
@@ -873,7 +873,7 @@
       catch (Exception e)
       {
          String message = Interpolator.instance().interpolate("Border {0} not supported, try {1}", border, getValidConstants(JXL_BORDER_CLASS_NAME));
-         throw new ExcelWorkbookException(message);
+         throw new ExcelWorkbookException(message, e);
       }
    }
 
@@ -899,7 +899,7 @@
       catch (Exception e)
       {
          String message = Interpolator.instance().interpolate("Underline style {0} not supported, try {1}", underlineStyle, getValidConstants(JXL_UNDERLINE_STYLE_CLASS_NAME));
-         throw new ExcelWorkbookException(message);
+         throw new ExcelWorkbookException(message, e);
       }
    }
 
@@ -925,7 +925,7 @@
       catch (Exception e)
       {
          String message = Interpolator.instance().interpolate("Script style {0} not supported, try {1}", scriptStyle, getValidConstants(JXL_SCRIPT_STYLE_CLASS_NAME));
-         throw new ExcelWorkbookException(message);
+         throw new ExcelWorkbookException(message, e);
       }
    }
 
@@ -957,7 +957,7 @@
       catch (Exception e)
       {
          String message = Interpolator.instance().interpolate("Color {0} not supported, try {1}", color, getValidConstants(JXL_COLOR_CLASS_NAME));
-         throw new ExcelWorkbookException(message);
+         throw new ExcelWorkbookException(message, e);
       }
    }
 

Modified: trunk/src/excel/org/jboss/seam/excel/jxl/JXLTemplates.java
===================================================================
--- trunk/src/excel/org/jboss/seam/excel/jxl/JXLTemplates.java	2008-08-08 06:08:31 UTC (rev 8629)
+++ trunk/src/excel/org/jboss/seam/excel/jxl/JXLTemplates.java	2008-08-08 08:12:12 UTC (rev 8630)
@@ -207,7 +207,7 @@
             if (cellTemplate == null)
             {
                String validNames = getValidTemplateNames(cellTemplates.keySet());
-               log.warn(Interpolator.instance().interpolate("Could not find cell template {0}, try {1}", templateName, validNames));
+               log.trace(Interpolator.instance().interpolate("Could not find cell template {0}, try {1}", templateName, validNames));
             }
             else
             {