Changeset 5636

Timestamp:

09/16/14 07:55:44 (10 years ago)

Author:

Twan Goosen

Message:

Added optional context parameter to set EPIC PID infix
Fixes #632

Location:

VirtualCollectionRegistry/trunk/VirtualCollectionRegistry

Files:

• doc/README.txt (modified) (1 diff)
• src/main/java/eu/clarin/cmdi/virtualcollectionregistry/pid/EPICPersistentIdentifierProvider.java (modified) (3 diffs)
• src/test/java/eu/clarin/cmdi/virtualcollectionregistry/pid/EPICPersistentIdentifierProviderTest.java (modified) (2 diffs)

VirtualCollectionRegistry/trunk/VirtualCollectionRegistry/doc/README.txt

@@ -89 +89 @@
              value="$epic_password"
              override="false"/>
-
+             
+  Add the following parameter to configure a custom PID 'infix'. The example below
+  configures the default behaviour, i.e. '{prefix}/VCR-{id}'.
+   
+  <Parameter name="pid_provider.epic.infix"
+             value="VCR-"/>
+             
 * AUTHENTICATION *
 

VirtualCollectionRegistry/trunk/VirtualCollectionRegistry/src/main/java/eu/clarin/cmdi/virtualcollectionregistry/pid/EPICPersistentIdentifierProvider.java

@@ -38 +38 @@
     private String baseUri;
 
+    @Value("${pid_provider.epic.infix:VCR-}")
+    private String infix;
+
     /**
      *
@@ -54 +57 @@
         final Map<HandleField, String> fieldMap = createPIDFieldMap(vc);
         try {
-            final String requestedPid = String.format("VCR-%d", vc.getId());
+            final String requestedPid = String.format("%s%d", infix, vc.getId());
             final String pid = pidWriter.registerNewPID(configuration, fieldMap, requestedPid);
             return new PersistentIdentifier(vc, PersistentIdentifier.Type.HANDLE, pid);
@@ -89 +92 @@
         this.baseUri = baseUri;
     }
+    
+    protected void setInfix(String infix) {
+        this.infix = infix;
+    }
 
 }

VirtualCollectionRegistry/trunk/VirtualCollectionRegistry/src/test/java/eu/clarin/cmdi/virtualcollectionregistry/pid/EPICPersistentIdentifierProviderTest.java

@@ -31 +31 @@
         instance = new EPICPersistentIdentifierProvider(pidWriter, pidConfig);
         instance.setBaseUri("http://vcr");
+        instance.setInfix("VCR-test-");
     }
 
@@ -55 +56 @@
                                 hasEntry(HandleField.CREATOR, "Joe Unit")
                         ),
-                        equalTo("VCR-123")
+                        equalTo("VCR-test-123")
                 );
-                will(returnValue("9999/VCR-123"));
+                will(returnValue("9999/VCR-test-123"));
             }
         });
 
         PersistentIdentifier result = instance.createIdentifier(vc);
-        assertEquals("9999/VCR-123", result.getIdentifier());
+        assertEquals("9999/VCR-test-123", result.getIdentifier());
         assertEquals(Type.HANDLE, result.getType());
     }