Skip to content

[FINNA-4765] Add tests for LIDO-EDM XslTransformation#198

Open
mshroom wants to merge 6 commits into
NatLibFi:devfrom
mshroom:finna-4765
Open

[FINNA-4765] Add tests for LIDO-EDM XslTransformation#198
mshroom wants to merge 6 commits into
NatLibFi:devfrom
mshroom:finna-4765

Conversation

@mshroom

@mshroom mshroom commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Tests for transforming LIDO records with image, video, text and 3D resources to valid EDM records

@mshroom mshroom requested a review from EreMaijala June 3, 2026 12:53

@EreMaijala EreMaijala left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Melkeinpä sääli piilottaa näitä testien alle...

Kun en muuta keksinyt, niin lisäsin muutaman ehdotuksen sisennysten viilaamiseksi. :)

Mahdollisia jatkotehtäviä:

  • Voisiko vertailujen ympärille lisätä välilyönnit kaikkialla? Eli ...lang='en'])!=''"> => ...lang='en']) != ''">?

  • Toistoa voisi vähentää ottamalla arvoja muuttujiin. Esim.

      <!-- dc:language -->
      <xsl:choose>
        <xsl:when test="//lido:classification[@lido:type='language']/lido:term[normalize-space(.)!='']">
          <dc:language>
            <xsl:call-template name="validatetextlang">
              <xsl:with-param name="lang" select="//lido:classification[@lido:type='language']/lido:term[normalize-space(.)!='']"/>
            </xsl:call-template>
          </dc:language>
        </xsl:when>

voisi olla:

      <!-- dc:language -->
      <xsl:variable name="f_language" select="//lido:classification[@lido:type='language']/lido:term[normalize-space(.) !='' ]"/>
      <xsl:choose>
        <xsl:when test="$f_language">
          <dc:language>
            <xsl:call-template name="validatetextlang">
              <xsl:with-param name="lang" select="$f_language"/>
            </xsl:call-template>
          </dc:language>
        </xsl:when>

(laitoin tohon esimerkkiin f_-prefiksin, ettei sotkeudu mahdollisiin muihin muuttujiin)

Comment thread tests/fixtures/Base/config/xsltransformationtest/transformations/lido2edm.xsl Outdated
Comment thread tests/fixtures/Base/config/xsltransformationtest/transformations/lido2edm.xsl Outdated
Comment thread tests/fixtures/Base/config/xsltransformationtest/transformations/lido2edm.xsl Outdated
<xsl:value-of select="normalize-space(//lido:descriptiveMetadata/lido:objectRelationWrap/lido:relatedWorksWrap/lido:relatedWorkSet/lido:relatedWork/lido:object/lido:objectWebResource)"/>
</xsl:when>
<xsl:otherwise>
<xsl:value-of select="concat('http://finna.fi', '/Record/', $museum, '.', $recordID, '?lng=en-gb')"/>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ehkä 'https://finna.fi' voisi olla parametri, jolla on oletusarvo (vaikka objectWebResourceBaseURL, jonka voisi vaikka asettaa tyhjäksi, jos ei halua tätä toimintoa)? Silloin tämä ei olisi välttämättä ihan niin Finna-spesifinen. (kommentin Finna.fi pitäisi tietty myös korvata)

mshroom and others added 3 commits June 4, 2026 12:19
…ns/lido2edm.xsl

Co-authored-by: Ere Maijala <ere.maijala@helsinki.fi>
…ns/lido2edm.xsl

Co-authored-by: Ere Maijala <ere.maijala@helsinki.fi>
…ns/lido2edm.xsl

Co-authored-by: Ere Maijala <ere.maijala@helsinki.fi>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants