fix(JF-1177): check type before casting

Author: welschsn

src/main/java/org/jadice/filetype/Context.java

@@ -93,4 +93,9 @@ public void warning(final Object src, final String message) {
   public Locale getLocale() {
     return locale;
   }
+
+  @Override
+  public String toString() {
+    return "Context{" + "result=" + result + ", locale=" + locale + ", statedExtension='" + statedExtension + '\'' + '}';
+  }
 }

src/main/java/org/jadice/filetype/database/DescriptionAction.java

@@ -28,7 +28,7 @@ public class DescriptionAction extends Action {
   public static class Description implements Serializable {
     private static final long serialVersionUID = 1L;
 
-    private List<Map<String, String>> content = new ArrayList<Map<String, String>>();
+    private final List<Map<String, String>> content = new ArrayList<>();
 
     private final Locale locale;
 
@@ -48,7 +48,7 @@ public void append(String lang, String description, boolean replace) {
       // default desc. ist der indikator für eine neue map,
       // alle lok. sprachen müssen danach in der xml datei definiert werden
       if (content.isEmpty() || lang.equalsIgnoreCase(DEFAULT_LANG)) {
-        content.add(new HashMap<String, String>());
+        content.add(new HashMap<>());
       }
 
       Map<String, String> map = content.get(content.size() - 1);
@@ -94,7 +94,14 @@ public String toString() {
 
   @Override
   public void perform(Context ctx) {
-    Description desc = (Description) ctx.getProperty(KEY);
+    Description desc = null;
+
+    if (ctx.getProperty(KEY) instanceof Description) {
+      desc = (Description) ctx.getProperty(KEY);
+    } else if (ctx.getProperty(KEY) instanceof String) {
+      desc = new Description(ctx.getLocale());
+      desc.append(null, (String) ctx.getProperty(KEY), replace);
+    }
 
     if (desc == null) {
       desc = new Description(ctx.getLocale());

src/main/java/org/jadice/filetype/matchers/TextMatcher.java

@@ -108,7 +108,8 @@ public boolean matches(final Context context) {
       boolean isCSV = false;
       String mimeType = "text/plain";
       final String statedExtension = context.getStatedExtension();
-      if(matches && statedExtension != null && statedExtension.equals("csv")) {
+      LOGGER.debug("stated extension: {}", statedExtension);
+      if (matches && statedExtension != null && statedExtension.equals("csv")) {
         isCSV = true;
         mimeType = "text/csv";
       }
@@ -117,7 +118,7 @@ public boolean matches(final Context context) {
         // Inject charset in MIME type for later usage
         context.setProperty(MimeTypeAction.KEY, mimeType + ";charset=" + bom.charset.name());
         context.info(this, String.format("Determined charset: %s", bom.charset.name()));
-        if(isCSV){
+        if (isCSV) {
           context.setProperty(ExtensionAction.KEY, "csv");
           context.setProperty(DescriptionAction.KEY, "Comma-separated values (CSV)");
         }
@@ -159,10 +160,10 @@ private float countCharacters(final String text, final Context context) {
           || Character.isIdentifierIgnorable(c) //
           || Character.isSpaceChar(c) //
           || Character.isWhitespace(c)) {
-        LOGGER.debug("Recognized char '" + c + "' @ " + i);
+        LOGGER.debug("Recognized char '{}' @ {}", c, i);
         defined++;
       } else if (PUNCTUATION.contains(c)) {
-        LOGGER.debug("Ignoring punctuation char '" + c + "' @ " + i);
+        LOGGER.debug("Ignoring punctuation char '{}' @ {}", c, i);
         ignored++;
       }
     }