-
-
Notifications
You must be signed in to change notification settings - Fork 477
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Standard / ISO19115-3 / Avoid indexing error on multilingual feature catalogue #8013
Conversation
…catalogue On the long run, we need to add multilingual support here.
"code" :"<xsl:value-of select="util:escapeForJson(gfc:FC_FeatureType/gfc:code/*/text())"/>", | ||
"isAbstract" :"<xsl:value-of select="gfc:FC_FeatureType/gfc:isAbstract/*/text()"/>", | ||
"aliases" : "<xsl:value-of select="util:escapeForJson(gfc:FC_FeatureType/gfc:aliases/*/text())"/>" | ||
"definition" :"<xsl:value-of select="util:escapeForJson(gfc:FC_FeatureType/gfc:definition/gco:CharacterString/text())"/>", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should not be (gco:CharacterString|gcx:Anchor)
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've never seen Anchor
used in here. If putting a link to a document for the feature type, it would go in definitionReference
I suppose
<gfc:featureType>
<gfc:FC_FeatureType>
<gfc:typeName/>
<gfc:definition gco:nilReason="missing" xsi:type="lan:PT_FreeText_PropertyType">
<gco:CharacterString/>
</gfc:definition>
<gfc:isAbstract>
<gco:Boolean>true</gco:Boolean>
</gfc:isAbstract>
<gfc:aliases/>
<gfc:definitionReference>
<gfc:FC_DefinitionReference>
<gfc:definitionSource>
<gfc:FC_DefinitionSource>
<gfc:source>
<cit:CI_Citation>
<cit:title gco:nilReason="missing" xsi:type="lan:PT_FreeText_PropertyType">
<gco:CharacterString/>
<lan:PT_FreeText>
<lan:textGroup>
<lan:LocalisedCharacterString locale="#FR"/>
</lan:textGroup>
<lan:textGroup>
<lan:LocalisedCharacterString locale="#EN">Data model published by the Sandre</lan:LocalisedCharacterString>
</lan:textGroup>
</lan:PT_FreeText>
</cit:title>
<cit:onlineResource>
<cit:CI_OnlineResource>
<cit:linkage xsi:type="lan:PT_FreeText_PropertyType">
<gco:CharacterString/>
<lan:PT_FreeText>
<lan:textGroup>
<lan:LocalisedCharacterString locale="#FR">https://www.sandre.eaufrance.fr/definition/eth/latest</lan:LocalisedCharacterString>
</lan:textGroup>
"aliases" : "<xsl:value-of select="util:escapeForJson(gfc:FC_FeatureType/gfc:aliases/*/text())"/>" | ||
"definition" :"<xsl:value-of select="util:escapeForJson(gfc:FC_FeatureType/gfc:definition/gco:CharacterString/text())"/>", | ||
"code" :"<xsl:value-of select="util:escapeForJson(gfc:FC_FeatureType/gfc:code/(gco:CharacterString|gcx:Anchor)/text())"/>", | ||
"isAbstract" :"<xsl:value-of select="gfc:FC_FeatureType/gfc:isAbstract/gco:CharacterString/text()"/>", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should not be (gco:CharacterString|gcx:Anchor)
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In fact this should be gco:Boolean
. Fixed.
"definition" :"<xsl:value-of select="util:escapeForJson(gfc:FC_FeatureType/gfc:definition/gco:CharacterString/text())"/>", | ||
"code" :"<xsl:value-of select="util:escapeForJson(gfc:FC_FeatureType/gfc:code/(gco:CharacterString|gcx:Anchor)/text())"/>", | ||
"isAbstract" :"<xsl:value-of select="gfc:FC_FeatureType/gfc:isAbstract/gco:CharacterString/text()"/>", | ||
"aliases" : "<xsl:value-of select="util:escapeForJson(gfc:FC_FeatureType/gfc:aliases/gco:CharacterString/text())"/>" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should not be (gco:CharacterString|gcx:Anchor)
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same, never seen usage of Anchor in here.
Quality Gate passedIssues Measures |
To test, create a record with feature catalogue and add more than one language.
On the long run, we need to add multilingual support here.
Checklist
main
branch, backports managed with labelREADME.md
filespom.xml
dependency management. Update build documentation with intended library use and library tutorials or documentation