refactor out the "generic properties map" that is only used by a single property (the signature note)

2 files changed, 6 insertions, 14 deletions

diff --git a/pdf-over-signer/src/main/java/at/asit/pdfover/signer/pdfas/PdfAs4SignatureParameter.java b/pdf-over-signer/src/main/java/at/asit/pdfover/signer/pdfas/PdfAs4SignatureParameter.java
index c52625dd..c4999014 100644
--- a/pdf-over-signer/src/main/java/at/asit/pdfover/signer/pdfas/PdfAs4SignatureParameter.java
+++ b/pdf-over-signer/src/main/java/at/asit/pdfover/signer/pdfas/PdfAs4SignatureParameter.java
@@ -75,18 +75,19 @@ public class PdfAs4SignatureParameter {
 	/** The signature device */
 	public BKUs signatureDevice;
 
+    /** The siganture note, if any */
+    public String signatureNote = null;
+
 	/** Whether so look for placeholder signatures or not. */
 	public boolean searchForPlaceholderSignatures = false;
 
-    private HashMap<String, String> genericProperties = new HashMap<String, String>();
-
     public Profile signatureProfile = Profile.getDefaultProfile();
 
     public Image getPlaceholder() {
         String sigProfile = getPdfAsSignatureProfileId();
 
         String sigEmblem = (this.emblem == null ? null : this.emblem.getCachedFileName());
-        String sigNote = getProperty("SIG_NOTE");
+        String sigNote = this.signatureNote;
 
         try {
             X509Certificate cert = new X509Certificate(PdfAs4SignatureParameter.class.getResourceAsStream("/example.cer"));
@@ -133,15 +134,6 @@ public class PdfAs4SignatureParameter {
         }
     }
 
-    // TODO review this
-    public void setProperty(String key, String value) {
-        this.genericProperties.put(key, value);
-    }
-
-    public String getProperty(String key) {
-        return this.genericProperties.get(key);
-    }
-
     /**
      * Gets the Signature Position String for PDF-AS
      *
@@ -195,7 +187,7 @@ public class PdfAs4SignatureParameter {
                 break;
         }
 
-        if (getProperty("SIG_NOTE") != null)
+        if (this.signatureNote)
             profileId += "_NOTE";
 
         if (this.enablePDFACompat)
diff --git a/pdf-over-signer/src/main/java/at/asit/pdfover/signer/pdfas/PdfAs4Signer.java b/pdf-over-signer/src/main/java/at/asit/pdfover/signer/pdfas/PdfAs4Signer.java
index 7a67a35a..51e36037 100644
--- a/pdf-over-signer/src/main/java/at/asit/pdfover/signer/pdfas/PdfAs4Signer.java
+++ b/pdf-over-signer/src/main/java/at/asit/pdfover/signer/pdfas/PdfAs4Signer.java
@@ -47,7 +47,7 @@ public class PdfAs4Signer {
 
 		String sigProfile = parameter.getPdfAsSignatureProfileId();
 		String sigEmblem = (parameter.emblem == null ? null : parameter.emblem.getCachedFileName());
-		String sigNote = parameter.getProperty("SIG_NOTE");
+		String sigNote = parameter.signatureNote;
 		String sigPos = null;
 		if (parameter.signaturePosition != null) {
 			sigPos = parameter.getPdfAsSignaturePosition();