[seam-commits] Seam SVN: r9260 - in trunk/src/excel/org/jboss/seam/excel: jxl and 1 other directory.

[seam-commits at lists.jboss.org]

Author: danielc.roth
Date: 2008-10-11 11:57:58 -0400 (Sat, 11 Oct 2008)
New Revision: 9260

Modified:
   trunk/src/excel/org/jboss/seam/excel/exporter/ExcelExporter.java
   trunk/src/excel/org/jboss/seam/excel/jxl/JXLFactory.java
Log:
minor

Modified: trunk/src/excel/org/jboss/seam/excel/exporter/ExcelExporter.java
===================================================================
--- trunk/src/excel/org/jboss/seam/excel/exporter/ExcelExporter.java	2008-10-11 05:38:57 UTC (rev 9259)
+++ trunk/src/excel/org/jboss/seam/excel/exporter/ExcelExporter.java	2008-10-11 15:57:58 UTC (rev 9260)
@@ -31,8 +31,6 @@
 import org.jboss.seam.excel.ui.UIColumn;
 import org.jboss.seam.excel.ui.UIWorkbook;
 import org.jboss.seam.excel.ui.UIWorksheet;
-import org.jboss.seam.log.Log;
-import org.jboss.seam.log.Logging;
 import org.jboss.seam.navigation.Pages;
 
 /**
@@ -47,14 +45,11 @@
 @BypassInterceptors
 public class ExcelExporter
 {
-   private static Log log = Logging.getLog(ExcelExporter.class);
-   
    // The excel workbook implementation
    private ExcelWorkbook excelWorkbook = null;
-   
+
    // A map of known column widths
    private Map<Integer, Integer> columnWidths = new HashMap<Integer, Integer>();
-   
 
    /**
     * Helper method to call the exporter and use the default excel workbook
@@ -77,7 +72,6 @@
    @SuppressWarnings("unchecked")
    public void export(String dataTableId, String type)
    {
-      // TODO: support "type" ?
       excelWorkbook = ExcelFactory.instance().getExcelWorkbook(type);
 
       // Gets the datatable
@@ -126,9 +120,10 @@
       redirectExport();
 
    }
-   
+
    /**
     * Parses column widths from a worksheet tag
+    * 
     * @param worksheet The worksheet to get the style from
     * @return The map of column number -> column width
     */
@@ -202,7 +197,8 @@
       }
 
       Integer columnWidth = columnWidths.get(columnIndex);
-      if (columnWidth != null) {
+      if (columnWidth != null)
+      {
          UIColumn uiColumn = new UIColumn();
          uiColumn.setStyle(CSSNames.COLUMN_WIDTH + ":" + columnWidth);
          excelWorkbook.applyColumnSettings(uiColumn);

Modified: trunk/src/excel/org/jboss/seam/excel/jxl/JXLFactory.java
===================================================================
--- trunk/src/excel/org/jboss/seam/excel/jxl/JXLFactory.java	2008-10-11 05:38:57 UTC (rev 9259)
+++ trunk/src/excel/org/jboss/seam/excel/jxl/JXLFactory.java	2008-10-11 15:57:58 UTC (rev 9260)
@@ -404,7 +404,7 @@
    /**
     * Creates a JExcelAPI date display format
     * 
-    * @param The format mask
+    * @param mask The format mask
     * @return The display format
     * @see <a
     *      href="http://jexcelapi.sourceforge.net/resources/javadocs/2_6/docs/jxl/write/DateFormat.html">DateFormat</a>
@@ -487,7 +487,7 @@
    /**
     * Creates a JExcelAPI representation of a vertical alignment
     * 
-    * @param color The requested alignment
+    * @param verticalAlignment The requested alignment
     * @return The alignment representation
     * @see <a
     *      href="http://jexcelapi.sourceforge.net/resources/javadocs/2_6/docs/jxl/format/VerticalAlignment.html">VerticalAlignment</a>