Revert isShownAt mapping changes for MWDL

dpla · Dec 19, 2023 · e3ddee0 · e3ddee0
1 parent c2dccf6
commit e3ddee0
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 10 deletions.
diff --git a/src/main/scala/dpla/ingestion3/mappers/providers/MwdlMapping.scala b/src/main/scala/dpla/ingestion3/mappers/providers/MwdlMapping.scala
@@ -17,6 +17,9 @@ class MwdlMapping extends XmlMapping with XmlExtractor {
 
   override val enforceDuplicateIds: Boolean = false
 
+  private val baseIsShownAt = "https://utah-primoprod.hosted.exlibrisgroup.com/primo-explore/fulldisplay?docid="
+  private val suffixIsShownAt = "&context=L&vid=MWDL"
+
   val formatBlockList: Set[String] =
     DigitalSurrogateBlockList.termList ++
       FormatTypeValuesBlockList.termList
@@ -114,12 +117,12 @@ class MwdlMapping extends XmlMapping with XmlExtractor {
       .flatMap(extractStrings)
       .map(URI)
 
-  override def isShownAt(data: Document[NodeSeq]): ZeroToMany[EdmWebResource] = {
-  // PrimoNMBib > record > display > identifier.
-    (data \\ "display" \ "identifier")
+  override def isShownAt(data: Document[NodeSeq]): ZeroToMany[EdmWebResource] =
+  // baseIsShownAt + control\recordid
+    (data \\ "control" \ "recordid")
       .flatMap(extractStrings)
+      .map(baseIsShownAt + _.trim + suffixIsShownAt)
       .map(stringOnlyWebResource)
-  }
 
   override def iiifManifest(data: Document[NodeSeq]): ZeroToMany[URI] =
     // links > lln02

diff --git a/src/test/scala/dpla/ingestion3/mappers/providers/MwdlMappingTest.scala b/src/test/scala/dpla/ingestion3/mappers/providers/MwdlMappingTest.scala
@@ -72,14 +72,14 @@ class MwdlMappingTest extends FlatSpec with BeforeAndAfter {
   it should "extract the correct isShownAt" in {
     val xml: NodeSeq = <PrimoNMBib xmlns="http://www.exlibrisgroup.com/xsd/primo/primo_nm_bib">
         <record>
-            <display>
-                <identifier>
-                  http://libarchive.slcc.edu/islandora/object/works%3A598
-                </identifier>
-              </display>
+            <control>
+                <recordid>
+                  digcoll_slc_27works_598
+                </recordid>
+              </control>
           </record>
       </PrimoNMBib>
-    val expected = Seq(uriOnlyWebResource(URI("http://libarchive.slcc.edu/islandora/object/works%3A598")))
+    val expected = Seq(uriOnlyWebResource(URI("https://utah-primoprod.hosted.exlibrisgroup.com/primo-explore/fulldisplay?docid=digcoll_slc_27works_598&context=L&vid=MWDL")))
     assert(extractor.isShownAt(Document(xml)) === expected)
   }
   it should "extract the correct preview" in {