From 17bd8169664deedb7f0d97aecac844ce2c4f93e8 Mon Sep 17 00:00:00 2001
From: Kevin Ford <kevinford@loc.gov>
Date: Fri, 30 Aug 2024 17:01:31 -0400
Subject: [PATCH] Partial fix for #237; other issues may have been solved
 earlier.

---
 test/ConvSpec-1XX,7XX,8XX-names.xspec |  6 +-
 test/marc2bibframe2.xspec             | 12 ----
 xsl/ConvSpec-1XX,7XX,8XX-names.xsl    | 91 ++++++++++++++++++++++++---
 3 files changed, 86 insertions(+), 23 deletions(-)

diff --git a/test/ConvSpec-1XX,7XX,8XX-names.xspec b/test/ConvSpec-1XX,7XX,8XX-names.xspec
index 235fb11c..0a917ad6 100644
--- a/test/ConvSpec-1XX,7XX,8XX-names.xspec
+++ b/test/ConvSpec-1XX,7XX,8XX-names.xspec
@@ -65,17 +65,17 @@
     <x:expect label="$1 becomes Agent URI" test="//bf:Work[1]/bf:contribution[8]/bf:Contribution/bf:agent/bf:Agent/@rdf:about='http://id.loc.gov/rwo/agents/n83170323'"/>
     <x:expect label="...$0 becomes isIdentifiedByAuthority" test="//bf:Work[1]/bf:contribution[8]/bf:Contribution/bf:agent/bf:Agent/madsrdf:isIdentifiedByAuthority/@rdf:resource='http://id.loc.gov/authorities/names/n83170323'"/>
     
-    <x:expect label="Subject $0 becomes isIdentifiedByAuthority" test="//bf:Work[1]/bf:subject[1]/bf:Agent/madsrdf:isIdentifiedByAuthority/@rdf:resource='http://id.loc.gov/authorities/names/n79027239'"/>
+    <x:expect label="Subject $0 becomes isIdentifiedByAuthority" test="//bf:Work[1]/bf:subject[1]/bf:Agent/madsrdf:isIdentifiedByAuthority/madsrdf:PersonalName/@rdf:about='http://id.loc.gov/authorities/names/n79027239'"/>
     <x:expect label="...And creates $1 from ID Authorities URI" test="//bf:Work[1]/bf:subject[1]/bf:Agent/@rdf:about='http://id.loc.gov/rwo/agents/n79027239'"/>
     
     <x:expect label="Subject $1 becomes rdf:about" test="//bf:Work[1]/bf:subject[2]/bf:Agent/@rdf:about='http://id.loc.gov/rwo/agents/n79022244'"/>
-    <x:expect label="...And creates $0 from ID RWO URI" test="//bf:Work[1]/bf:subject[2]/bf:Agent/madsrdf:isIdentifiedByAuthority/@rdf:resource='http://id.loc.gov/authorities/names/n79022244'"/>
+    <x:expect label="...And creates $0 from ID RWO URI" test="//bf:Work[1]/bf:subject[2]/bf:Agent/madsrdf:isIdentifiedByAuthority/madsrdf:PersonalName/@rdf:about='http://id.loc.gov/authorities/names/n79022244'"/>
 
     <x:expect label="Subject $1 becomes Agent URI" test="//bf:Work[1]/bf:subject[3]/bf:Agent/@rdf:about='https://viaf.org/viaf/76349832/'"/>
     <x:expect label="No $0 so no isIdentifiedByAuthority" test="count(//bf:Work[1]/bf:subject[3]/bf:Agent/madsrdf:isIdentifiedByAuthority)=0"/>
     
     <x:expect label="Subject $1 becomes Agent URI" test="//bf:Work[1]/bf:subject[4]/bf:Agent/@rdf:about='http://viaf.org/viaf/92989169'"/>
-    <x:expect label="...$0 becomes isIdentifiedByAuthority" test="//bf:Work[1]/bf:subject[4]/bf:Agent/madsrdf:isIdentifiedByAuthority/@rdf:resource='http://example.org/names/n80005801'"/>
+    <x:expect label="...$0 becomes isIdentifiedByAuthority" test="//bf:Work[1]/bf:subject[4]/bf:Agent/madsrdf:isIdentifiedByAuthority/madsrdf:PersonalName/@rdf:about='http://example.org/names/n80005801'"/>
     
   </x:scenario>
 
diff --git a/test/marc2bibframe2.xspec b/test/marc2bibframe2.xspec
index ae4c44bb..82bdbbbb 100644
--- a/test/marc2bibframe2.xspec
+++ b/test/marc2bibframe2.xspec
@@ -577,9 +577,6 @@
             <bf:subject>
               <bf:Agent rdf:about="http://example.org/12149120#Agent610-22">
                 <rdf:type rdf:resource="http://id.loc.gov/ontologies/bibframe/Organization" />
-                <rdf:type rdf:resource="http://www.loc.gov/mads/rdf/v1#CorporateName" />
-                <madsrdf:authoritativeLabel>White House (Washington, D.C.)</madsrdf:authoritativeLabel>
-                <madsrdf:isMemberOfMADSScheme rdf:resource="http://id.loc.gov/authorities/subjects" />
                 <rdfs:label>White House (Washington, D.C.)</rdfs:label>
                 <bflc:marcKey>61020$aWhite House (Washington, D.C.)</bflc:marcKey>
                 <bf:source rdf:resource="http://id.loc.gov/authorities/subjects" />
@@ -588,9 +585,6 @@
             <bf:subject>
               <bf:Agent rdf:about="http://example.org/12149120#Agent610-23">
                 <rdf:type rdf:resource="http://id.loc.gov/ontologies/bibframe/Jurisdiction" />
-                <rdf:type rdf:resource="http://www.loc.gov/mads/rdf/v1#CorporateName" />
-                <madsrdf:authoritativeLabel>United States. Executive Office of the President.</madsrdf:authoritativeLabel>
-                <madsrdf:isMemberOfMADSScheme rdf:resource="http://id.loc.gov/authorities/subjects" />
                 <rdfs:label>United States. Executive Office of the President.</rdfs:label>
                 <bflc:marcKey>61010$aUnited States.$bExecutive Office of the President.</bflc:marcKey>
                 <bf:source rdf:resource="http://id.loc.gov/authorities/subjects" />
@@ -599,9 +593,6 @@
             <bf:subject>
               <bf:Agent rdf:about="http://example.org/12149120#Agent610-24">
                 <rdf:type rdf:resource="http://id.loc.gov/ontologies/bibframe/Jurisdiction" />
-                <rdf:type rdf:resource="http://www.loc.gov/mads/rdf/v1#CorporateName" />
-                <madsrdf:authoritativeLabel>United States. Office of the Vice President.</madsrdf:authoritativeLabel>
-                <madsrdf:isMemberOfMADSScheme rdf:resource="http://id.loc.gov/authorities/subjects" />
                 <rdfs:label>United States. Office of the Vice President.</rdfs:label>
                 <bflc:marcKey>61010$aUnited States.$bOffice of the Vice President.</bflc:marcKey>
                 <bf:source rdf:resource="http://id.loc.gov/authorities/subjects" />
@@ -610,9 +601,6 @@
             <bf:subject>
               <bf:Agent rdf:about="http://example.org/12149120#Agent610-25">
                 <rdf:type rdf:resource="http://id.loc.gov/ontologies/bibframe/Jurisdiction" />
-                <rdf:type rdf:resource="http://www.loc.gov/mads/rdf/v1#CorporateName" />
-                <madsrdf:authoritativeLabel>United States. Office of the First Lady.</madsrdf:authoritativeLabel>
-                <madsrdf:isMemberOfMADSScheme rdf:resource="http://id.loc.gov/authorities/subjects" />
                 <rdfs:label>United States. Office of the First Lady.</rdfs:label>
                 <bflc:marcKey>61010$aUnited States.$bOffice of the First Lady.</bflc:marcKey>
                 <bf:source rdf:resource="http://id.loc.gov/authorities/subjects" />
diff --git a/xsl/ConvSpec-1XX,7XX,8XX-names.xsl b/xsl/ConvSpec-1XX,7XX,8XX-names.xsl
index 83d42c67..674be52d 100644
--- a/xsl/ConvSpec-1XX,7XX,8XX-names.xsl
+++ b/xsl/ConvSpec-1XX,7XX,8XX-names.xsl
@@ -645,6 +645,7 @@
             <rdf:type><xsl:attribute name="rdf:resource"><xsl:value-of select="$bflc"/>Uncontrolled</xsl:attribute></rdf:type>
           </xsl:if>
           <xsl:if test="substring($tag,1,1)='6'">
+            <!--
             <xsl:if test="$pMADSClass != ''">
               <rdf:type>
                 <xsl:attribute name="rdf:resource"><xsl:value-of select="concat($madsrdf,$pMADSClass)"/></xsl:attribute>
@@ -665,6 +666,7 @@
                 </madsrdf:isMemberOfMADSScheme>
               </xsl:for-each>
             </xsl:if>
+            -->
             <xsl:if test="not(marc:subfield[@code='t'])">
               <xsl:choose>
                 <xsl:when test="substring($tag,2,2)='11'">
@@ -766,18 +768,91 @@
                             contains(marc:subfield[@code='1'], 'id.loc.gov/rwo/') and 
                             not(marc:subfield[@code='0' or @code='w'][starts-with(text(),'(uri)') or starts-with(text(),'http')])">
                 <madsrdf:isIdentifiedByAuthority>
-                  <xsl:attribute name="rdf:resource">
-                    <xsl:apply-templates mode="generateUriFrom0" select=".">
-                      <xsl:with-param name="pEntity">bf:Agent</xsl:with-param>
-                    </xsl:apply-templates>
-                  </xsl:attribute>
+                  <xsl:choose>
+                    <xsl:when test="substring($tag,1,1)='6' and $pMADSClass != ''">
+                      <xsl:element name="{concat('madsrdf:', $pMADSClass)}">
+                        <xsl:attribute name="rdf:about">
+                          <xsl:apply-templates mode="generateUriFrom0" select=".">
+                            <xsl:with-param name="pEntity">bf:Agent</xsl:with-param>
+                          </xsl:apply-templates>
+                        </xsl:attribute>
+                        <xsl:if test="$pMADSLabel != ''">
+                          <madsrdf:authoritativeLabel>
+                            <xsl:if test="$vXmlLang != ''">
+                              <xsl:attribute name="xml:lang"><xsl:value-of select="$vXmlLang"/></xsl:attribute>
+                            </xsl:if>
+                            <xsl:call-template name="tChopPunct">
+                              <xsl:with-param name="pString" select="$pMADSLabel"/>
+                            </xsl:call-template>
+                          </madsrdf:authoritativeLabel>
+                        </xsl:if>
+                        <!--
+                          <xsl:for-each select="$subjectThesaurus/subjectThesaurus/subject[@ind2=current()/@ind2]/madsscheme">
+                          <madsrdf:isMemberOfMADSScheme>
+                            <xsl:attribute name="rdf:resource"><xsl:value-of select="."/></xsl:attribute>
+                          </madsrdf:isMemberOfMADSScheme>
+                          </xsl:for-each>
+                        -->
+                        <madsrdf:isMemberOfMADSScheme rdf:resource="http://id.loc.gov/authorities/names" />
+                      </xsl:element>
+                    </xsl:when>
+                    <xsl:otherwise>
+                      <madsrdf:isIdentifiedByAuthority>
+                        <xsl:attribute name="rdf:resource">
+                          <xsl:apply-templates mode="generateUriFrom0" select=".">
+                            <xsl:with-param name="pEntity">bf:Agent</xsl:with-param>
+                          </xsl:apply-templates>
+                        </xsl:attribute>
+                      </madsrdf:isIdentifiedByAuthority>
+                    </xsl:otherwise>
+                  </xsl:choose>
                 </madsrdf:isIdentifiedByAuthority>
               </xsl:if>
               <xsl:for-each select="marc:subfield[@code='0' or @code='w'][starts-with(text(),'(uri)') or starts-with(text(),'http')]">
                 <xsl:if test="position() = 1">
-                  <xsl:apply-templates mode="subfield0orw" select=".">
-                    <xsl:with-param name="serialization" select="$serialization"/>
-                  </xsl:apply-templates>
+                  <xsl:choose>
+                    <xsl:when test="substring($tag,1,1)='6' and $pMADSClass != ''">
+                      <madsrdf:isIdentifiedByAuthority>
+                        <xsl:element name="{concat('madsrdf:', $pMADSClass)}">
+                          <xsl:choose>
+                            <xsl:when test="starts-with(text(),'http')">
+                              <xsl:attribute name="rdf:about">
+                                <xsl:value-of select="text()" />
+                              </xsl:attribute>
+                            </xsl:when>
+                            <xsl:when test="starts-with(text(),'(uri)')">
+                              <xsl:attribute name="rdf:about">
+                                <xsl:value-of select="substring-after(text(),'(uri)')" />
+                              </xsl:attribute>
+                            </xsl:when>
+                          </xsl:choose>
+                          <xsl:if test="$pMADSLabel != ''">
+                            <madsrdf:authoritativeLabel>
+                              <xsl:if test="$vXmlLang != ''">
+                                <xsl:attribute name="xml:lang"><xsl:value-of select="$vXmlLang"/></xsl:attribute>
+                              </xsl:if>
+                              <xsl:call-template name="tChopPunct">
+                                <xsl:with-param name="pString" select="$pMADSLabel"/>
+                              </xsl:call-template>
+                            </madsrdf:authoritativeLabel>
+                          </xsl:if>
+                          <!--
+                          <xsl:for-each select="$subjectThesaurus/subjectThesaurus/subject[@ind2=current()/@ind2]/madsscheme">
+                          <madsrdf:isMemberOfMADSScheme>
+                            <xsl:attribute name="rdf:resource"><xsl:value-of select="."/></xsl:attribute>
+                          </madsrdf:isMemberOfMADSScheme>
+                          </xsl:for-each>
+                        -->
+                          <madsrdf:isMemberOfMADSScheme rdf:resource="http://id.loc.gov/authorities/names" />
+                        </xsl:element>
+                      </madsrdf:isIdentifiedByAuthority>
+                    </xsl:when>
+                    <xsl:otherwise>
+                      <xsl:apply-templates mode="subfield0orw" select=".">
+                        <xsl:with-param name="serialization" select="$serialization"/>
+                      </xsl:apply-templates>
+                    </xsl:otherwise>
+                  </xsl:choose>
                 </xsl:if>
               </xsl:for-each>
               <xsl:for-each select="marc:subfield[@code='0' or @code='w']">