From: <fg...@us...> - 2007-02-26 11:13:53
|
Revision: 308 http://svn.sourceforge.net/openutils/?rev=308&view=rev Author: fgiust Date: 2007-02-26 03:13:54 -0800 (Mon, 26 Feb 2007) Log Message: ----------- fix potential NPE Modified Paths: -------------- trunk/openutils-dbmigration/src/main/java/it/openutils/migration/task/setup/ExcelConfigurationTask.java Modified: trunk/openutils-dbmigration/src/main/java/it/openutils/migration/task/setup/ExcelConfigurationTask.java =================================================================== --- trunk/openutils-dbmigration/src/main/java/it/openutils/migration/task/setup/ExcelConfigurationTask.java 2007-02-26 11:12:57 UTC (rev 307) +++ trunk/openutils-dbmigration/src/main/java/it/openutils/migration/task/setup/ExcelConfigurationTask.java 2007-02-26 11:13:54 UTC (rev 308) @@ -123,15 +123,19 @@ HSSFRow row = sheet.getRow(0); for (short k = 0; k < row.getLastCellNum(); k++) { - String columnName = row.getCell(k).getStringCellValue(); - if (StringUtils.isNotBlank(columnName)) + HSSFCell cell = row.getCell(k); + if (cell != null) { - columns.add(StringUtils.trim(columnName)); + String columnName = cell.getStringCellValue(); + if (StringUtils.isNotBlank(columnName)) + { + columns.add(StringUtils.trim(columnName)); + } + else + { + break; + } } - else - { - break; - } } log.debug("Table: {}, Columns: {}", tableName, columns); This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |