Fix PDF parser swallows error message about duplicate securities (#4383)

Co-authored-by: Alexander Ott <[email protected]>
portfolio-performance · Dec 3, 2024 · d5bbd9c · d5bbd9c
1 parent e773fdc
commit d5bbd9c
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 1 deletion.
diff --git a/...abuchen.portfolio/src/name/abuchen/portfolio/datatransfer/DuplicateSecurityException.java b/...abuchen.portfolio/src/name/abuchen/portfolio/datatransfer/DuplicateSecurityException.java
@@ -0,0 +1,11 @@
+package name.abuchen.portfolio.datatransfer;
+
+public class DuplicateSecurityException extends IllegalArgumentException
+{
+    private static final long serialVersionUID = 1L;
+
+    public DuplicateSecurityException(String s)
+    {
+        super(s);
+    }
+}
diff --git a/name.abuchen.portfolio/src/name/abuchen/portfolio/datatransfer/SecurityCache.java b/name.abuchen.portfolio/src/name/abuchen/portfolio/datatransfer/SecurityCache.java
@@ -86,7 +86,7 @@ public Security lookup(String isin, String tickerSymbol, String wkn, String name
         // message is only returned to the user if the other attributes also did
         // not match
         if (idOfAttributeWithDuplicateSecurities >= 0)
-            throw new IllegalArgumentException(MessageFormat.format(MESSAGES.get(idOfAttributeWithDuplicateSecurities),
+            throw new DuplicateSecurityException(MessageFormat.format(MESSAGES.get(idOfAttributeWithDuplicateSecurities),
                             attributes.get(idOfAttributeWithDuplicateSecurities)));
 
         // second: check the name. But: even if the name matches, we also must

diff --git a/name.abuchen.portfolio/src/name/abuchen/portfolio/datatransfer/pdf/PDFParser.java b/name.abuchen.portfolio/src/name/abuchen/portfolio/datatransfer/pdf/PDFParser.java
@@ -22,6 +22,7 @@
 
 import name.abuchen.portfolio.Messages;
 import name.abuchen.portfolio.datatransfer.DocumentContext;
+import name.abuchen.portfolio.datatransfer.DuplicateSecurityException;
 import name.abuchen.portfolio.datatransfer.Extractor.Item;
 import name.abuchen.portfolio.datatransfer.ImportAction;
 import name.abuchen.portfolio.datatransfer.ImportAction.Context;
@@ -366,6 +367,10 @@ public void parse(String filename, DocumentContext documentContext, String[] lin
                             // if parsing was successful, then return
                             return;
                         }
+                        catch (DuplicateSecurityException e)
+                        {
+                            throw e;
+                        }
                         catch (IllegalArgumentException ignore)
                         {
                             // try next sub-section