MODLD-939: Update the mapping of 019

Author: pkjacob

NEWS.md

@@ -38,6 +38,7 @@
 - Ensure only one MARC 100/110/111/130 is present in the derived MARC, converting any additional 1XX fields to 7XX [MODLD-934](https://folio-org.atlassian.net/browse/MODLD-934)
 - Create MARC 600 when subject is a HUB [MODLD-538](https://folio-org.atlassian.net/browse/MODLD-538)
 - Handle invalid place codes in MARC 26X [MODLD-949](https://folio-org.atlassian.net/browse/MODLD-949)
+- Update the mapping of 019 to use NAME property instead of LOCAL_ID_VALUE property.
 
 ## 1.0.3 (04-09-2025)
 - Geographic coverage: single "geographicCoverage" edge is created for Work which contains 043 with multiple $a fields [MODLD-694](https://folio-org.atlassian.net/browse/MODLD-694)

src/main/java/org/folio/marc4ld/service/marc2ld/authority/control/AuthorityIdentifierProcessorImpl.java

@@ -60,3 +60,4 @@ private Map<String, List<String>> makeProperties(String data) {
     ));
   }
 }
+

src/main/resources/label.yml

@@ -94,7 +94,7 @@ labelRules:
       - NAME
   - types: ID_LOCAL, IDENTIFIER
     properties:
-      - LOCAL_ID_VALUE
+      - NAME
   - types: ID_STRN, IDENTIFIER
     properties:
       - NAME

src/main/resources/marc4ld.yml

@@ -126,41 +126,10 @@ bibFieldRules:
     - types: ID_LCCN
       include: cancelled_instance_identifier
   '019':
+    - types: ID_LOCAL
+      include: current_instance_identifier
     - types: ID_LOCAL, IDENTIFIER
-      parent: INSTANCE
-      predicate: MAP
-      subfields:
-        a: LOCAL_ID_VALUE
-      marc2ldCondition:
-        fieldsAllOf:
-          a: presented
-      ld2marcCondition:
-        edge:
-          STATUS:
-            anyMatch:
-              - present: false
-              - present: true
-                properties:
-                  LINK: http://id.loc.gov/vocabulary/mstatus/current
-      edges:
-        - include: status_current
-    - types: ID_LOCAL, IDENTIFIER
-      parent: INSTANCE
-      predicate: MAP
-      subfields:
-        z: LOCAL_ID_VALUE
-      marc2ldCondition:
-        fieldsAllOf:
-          z: presented
-      ld2marcCondition:
-        edge:
-          STATUS:
-            allMatch:
-              - present: true
-                properties:
-                  LINK: http://id.loc.gov/vocabulary/mstatus/cancinv
-      edges:
-        - include: status_cancelled
+      include: cancelled_instance_identifier
   '020':
     - types: ID_ISBN
       include: current_instance_identifier
@@ -1876,7 +1845,7 @@ bibSharedRules:
             predicate: MAP
             multiply: true
             subfields:
-              o: LOCAL_ID_VALUE
+              o: NAME
           - types: ID_STRN, IDENTIFIER
             parent: INSTANCE
             predicate: MAP

src/test/java/org/folio/marc4ld/mapper/MarcBib2LdMapperIT.java

@@ -55,7 +55,6 @@
 import static org.folio.ld.dictionary.PropertyDictionary.LABEL;
 import static org.folio.ld.dictionary.PropertyDictionary.LANGUAGE_NOTE;
 import static org.folio.ld.dictionary.PropertyDictionary.LINK;
-import static org.folio.ld.dictionary.PropertyDictionary.LOCAL_ID_VALUE;
 import static org.folio.ld.dictionary.PropertyDictionary.LOCATION_OF_ORIGINALS_DUPLICATES;
 import static org.folio.ld.dictionary.PropertyDictionary.LOCATION_OF_OTHER_ARCHIVAL_MATERIAL;
 import static org.folio.ld.dictionary.PropertyDictionary.MAIN_TITLE;
@@ -397,9 +396,9 @@ private void validateLocalId(ResourceEdge edge, String number, String status) {
     assertThat(edge.getTarget().getLabel()).isEqualTo(number);
     assertThat(edge.getTarget().getTypes()).containsOnly(ID_LOCAL, IDENTIFIER);
     assertThat(edge.getTarget().getDoc()).hasSize(1);
-    assertThat(edge.getTarget().getDoc().has(LOCAL_ID_VALUE.getValue())).isTrue();
-    assertThat(edge.getTarget().getDoc().get(LOCAL_ID_VALUE.getValue())).hasSize(1);
-    assertThat(edge.getTarget().getDoc().get(LOCAL_ID_VALUE.getValue()).get(0).asText()).isEqualTo(number);
+    assertThat(edge.getTarget().getDoc().has(NAME.getValue())).isTrue();
+    assertThat(edge.getTarget().getDoc().get(NAME.getValue())).hasSize(1);
+    assertThat(edge.getTarget().getDoc().get(NAME.getValue()).get(0).asText()).isEqualTo(number);
     assertThat(edge.getTarget().getOutgoingEdges()).isNotEmpty();
     var edgeIterator = edge.getTarget().getOutgoingEdges().iterator();
     validateIdStatus(edgeIterator.next(), status);

src/test/java/org/folio/marc4ld/mapper/test/MonographTestUtil.java

@@ -76,7 +76,6 @@
 import static org.folio.ld.dictionary.PropertyDictionary.LABEL;
 import static org.folio.ld.dictionary.PropertyDictionary.LINK;
 import static org.folio.ld.dictionary.PropertyDictionary.LINKAGE;
-import static org.folio.ld.dictionary.PropertyDictionary.LOCAL_ID_VALUE;
 import static org.folio.ld.dictionary.PropertyDictionary.LOCATION_OF_EVENT;
 import static org.folio.ld.dictionary.PropertyDictionary.LOCATION_OF_ORIGINALS_DUPLICATES;
 import static org.folio.ld.dictionary.PropertyDictionary.LOCATION_OF_OTHER_ARCHIVAL_MATERIAL;
@@ -330,15 +329,15 @@ public static Resource getSampleInstanceResource() {
 
     var localId = createResource(
       Map.of(
-        LOCAL_ID_VALUE, List.of("19861509")
+        NAME, List.of("19861509")
       ),
       Set.of(IDENTIFIER, ID_LOCAL),
       Map.of(STATUS, List.of(status("current")))
     ).setLabel("19861509");
 
     var localIdCancelled = createResource(
       Map.of(
-        LOCAL_ID_VALUE, List.of("09151986")
+        NAME, List.of("09151986")
       ),
       Set.of(IDENTIFIER, ID_LOCAL),
       Map.of(STATUS, List.of(status("canceled or invalid")))

src/test/java/org/folio/marc4ld/test/helper/LinkingEntriesTestHelper.java

@@ -157,7 +157,7 @@ public static void validateLocalId(Resource resource, String... expectedLocalIds
       validateEdge(resourceEdges.get(index), MAP,
         List.of(IDENTIFIER, ID_LOCAL),
         Map.of(
-          "http://bibfra.me/vocab/library/localId", List.of(expectedLocalId)
+          "http://bibfra.me/vocab/lite/name", List.of(expectedLocalId)
         ), expectedLocalId);
       index++;
     }