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/j... @@ -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/j...