From 4bf36d6752a49c7ef0e1735589e878668a176826 Mon Sep 17 00:00:00 2001
From: Piotr Gawron <piotr.gawron@uni.lu>
Date: Tue, 28 Feb 2017 18:00:10 +0100
Subject: [PATCH] error reporting improved when color parsing fails
---
.../appointment/parse/PrcSubjectsParser.java | 33 +++++++++++--------
1 file changed, 19 insertions(+), 14 deletions(-)
diff --git a/appointment-import/src/main/java/smash/appointment/parse/PrcSubjectsParser.java b/appointment-import/src/main/java/smash/appointment/parse/PrcSubjectsParser.java
index 5f869b4e..e05389d3 100644
--- a/appointment-import/src/main/java/smash/appointment/parse/PrcSubjectsParser.java
+++ b/appointment-import/src/main/java/smash/appointment/parse/PrcSubjectsParser.java
@@ -56,7 +56,7 @@ public class PrcSubjectsParser extends SubjectParser {
@Override
protected String parseAddDate(Row row) {
- return getDate(row.getCell(14),Calendar.getInstance());
+ return getDate(row.getCell(14), Calendar.getInstance());
}
@Override
@@ -154,19 +154,24 @@ public class PrcSubjectsParser extends SubjectParser {
@Override
protected boolean parseResigned(Row row) {
- IndexedColors color = IndexedColors.fromInt(row.getSheet().getWorkbook().getFontAt(row.getCell(0).getCellStyle().getFontIndex()).getColor());
- switch (color) {
- case RED:
- return true;
- case BLUE:
- return false;
- case BLACK:
- return false;
- case GREEN:
- return false;
- default:
- logger.debug("Unknown font color: " + color);
- return false;
+ try {
+ IndexedColors color = IndexedColors.fromInt(row.getSheet().getWorkbook().getFontAt(row.getCell(0).getCellStyle().getFontIndex()).getColor());
+ switch (color) {
+ case RED:
+ return true;
+ case BLUE:
+ return false;
+ case BLACK:
+ return false;
+ case GREEN:
+ return false;
+ default:
+ logger.debug("Unknown font color: " + color);
+ return false;
+ }
+ } catch (Exception e) {
+ logger.error("Problem with parsing color for subject: " + this.toString(), e);
+ return false;
}
}
}
--
GitLab