always set safe sheet name

MetadataConsulting · Oct 13, 2015 · cc44233 · cc44233
1 parent 5bf85bc
commit cc44233
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/...builder-poi/src/main/groovy/org/modelcatalogue/builder/spreadsheet/poi/PoiWorkbook.groovy b/...builder-poi/src/main/groovy/org/modelcatalogue/builder/spreadsheet/poi/PoiWorkbook.groovy
@@ -2,6 +2,7 @@ package org.modelcatalogue.builder.spreadsheet.poi
 
 import groovy.transform.stc.ClosureParams
 import groovy.transform.stc.FromString
+import org.apache.poi.ss.util.WorkbookUtil
 import org.apache.poi.xssf.usermodel.XSSFCell
 import org.apache.poi.xssf.usermodel.XSSFSheet
 import org.apache.poi.xssf.usermodel.XSSFWorkbook
@@ -22,7 +23,7 @@ class PoiWorkbook implements Workbook {
 
     @Override
     void sheet(String name, @DelegatesTo(Sheet.class) @ClosureParams(value=FromString.class, options = "org.modelcatalogue.builder.spreadsheet.api.Sheet") Closure sheetDefinition) {
-        XSSFSheet xssfSheet = workbook.getSheet(name) ?: workbook.createSheet(name)
+        XSSFSheet xssfSheet = workbook.getSheet(WorkbookUtil.createSafeSheetName(name)) ?: workbook.createSheet(WorkbookUtil.createSafeSheetName(name))
 
         PoiSheet sheet = new PoiSheet(this, xssfSheet)
         sheet.with sheetDefinition