Skip to content

add datepicker with indeterminate position to iso19139 schema. fix datepicker. layout fix #9151

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.

Sign up for GitHub

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

Open
davidblasby wants to merge 14 commits into
base: main
Choose a base branch
from
Open

add datepicker with indeterminate position to iso19139 schema. fix datepicker. layout fix #9151

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
be90512
add datepicker with indeterminate position to iso19139 schema. fix d…
david-blasby Jan 29, 2026
29726d8
backout hnap submodule
david-blasby Jan 29, 2026
e544915
force PR to re-evaluate
david-blasby Jan 29, 2026
fa6fb38
remove hnap
david-blasby Jan 29, 2026
8638316
remove unused xslt vars, add label text back in
david-blasby Feb 2, 2026
a243b37
add for <begin|end> and handle gml OLD and gml 3.2
david-blasby Feb 9, 2026
efd8b22
layout changes, hide datapicker components when not needed
david-blasby Feb 18, 2026
fd322b0
make indeterminate position dropdown larger
david-blasby Feb 18, 2026
2ef8342
add better gml-version agnostic matching. Add suggestion from jose f…
david-blasby Mar 27, 2026
05befaf
Update web-ui/src/main/resources/catalog/views/default/less/gn_editor…
josegar74 May 11, 2026
144bace
Update rendering of temporal period for beginPosition / endPosition t…
josegar74 May 11, 2026
2d1af24
Merge branch 'main' into _date_indeterminate
josegar74 May 27, 2026
e753a94
Merge branch 'main' into _date_indeterminate
josegar74 Aug 12, 2026
4ee9328
Update gml:TimePeriod template to preserve original namespace
josegar74 Aug 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions schemas/iso19139/src/main/plugin/iso19139/layout/layout-custom-fields-date.xsl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,7 @@
<xsl:template mode="mode-iso19139"
priority="2000"
match="gmd:CI_Date/gmd:date">

<xsl:param name="schema" select="$schema" required="no"/>
<xsl:param name="labels" select="$labels" required="no"/>
<xsl:param name="listOfValues" select="$iso19139codelists" required="no"/>
Expand Down Expand Up @@ -175,6 +176,8 @@
<xsl:template mode="mode-iso19139"
priority="2000"
match="*[(gco:Date|gco:DateTime) and not(../gmd:dateType)]">


<xsl:param name="schema" select="$schema" required="no"/>
<xsl:param name="labels" select="$labels" required="no"/>
<xsl:param name="overrideLabel" select="''" required="no"/>
Expand Down
260 changes: 256 additions & 4 deletions schemas/iso19139/src/main/plugin/iso19139/layout/layout-custom-fields.xsl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -151,18 +151,270 @@

-->

<xsl:template mode="mode-iso19139"
match="gml320:TimeInstant[gml320:timePosition]|gml:TimeInstant[gml:timePosition]"
priority="50000">

<xsl:variable name="configName" select="concat(local-name(..),'Position')"/>
<!-- 'temporalRangeSection' this is the name of the section (defined elsewhere)
and uses to find the translated tag for the section in the UI -->
<xsl:variable name="translation"
select="$strings/*[name() = $configName]"/>
<xsl:variable name="name"
select="if ($translation != '')
then $translation
else $configName"/>

<xsl:variable name="id" select="concat('_X', gn:element/@ref, '_replace')"/>

<xsl:variable name="isFirst" select="position() = 1"/>
<xsl:variable name="originalNode"
select="gn-fn-metadata:getOriginalNode($metadata, .)"/>
<xsl:variable name="del" select="./@del"/>

<xsl:variable name="refToDelete">
<xsl:call-template name="get-ref-element-to-delete">
<xsl:with-param name="node" select="$originalNode"/>
<xsl:with-param name="delXpath" select="$del"/>
</xsl:call-template>
</xsl:variable>

<xsl:variable name="parent_parent" select="."/>

<!-- look at the actual XML's gml namespace URI
if its GML 32, then that matches most of the implicit
assumptions inside GN. Especially the JS, which assumes that
an element with a "gml:" prefix is GML 3.2.
However, this is a really bad assumption - all the iso19139 sample data
has "gml:" as the old (NOT GML 3.2).
Also, in this XSLT;
"gml:" - version 3.2
"gml320:" - OLD VERSION (!!!!!!!!)
-->
<xsl:variable name="element_ns_uri" select="namespace-uri()"/>
<xsl:variable name="element_ns_prefix" select="prefix-from-QName(node-name(.))"/>
<xsl:variable name="gml_is_32" select="$element_ns_uri = 'http://www.opengis.net/gml/3.2'"/>
<xsl:variable name="timeinstance_id" select="@*[local-name() = 'id']"/>

<xsl:variable name="prefix_to_use">gml</xsl:variable>

<xsl:variable name="prefix_to_use_select">
<xsl:choose>
<xsl:when test="$gml_is_32">gml</xsl:when>
<xsl:otherwise>gml320</xsl:otherwise>
</xsl:choose>
</xsl:variable>


<xsl:variable name="label_node" select="local-name(..)"/>


<xsl:variable name="template">
<template xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:gn="http://www.fao.org/geonetwork" xmlns:gco="http://www.isotc211.org/2005/gco" xmlns:gmd="http://www.isotc211.org/2005/gmd" xmlns:gmx="http://www.isotc211.org/2005/gmx" xmlns:srv="http://www.isotc211.org/2005/srv" xmlns:xlink="http://www.w3.org/1999/xlink">
<values>
<key
xpath="."
use="gn-date-picker"
tooltip="gmd:extent|gmd:EX_TemporalExtent">
<xsl:attribute name="label" select="$label_node"/>

<directiveAttributes
data-assume-gml-ns="true"
>
<xsl:attribute name="data-indeterminate-position" select="concat('eval#',$prefix_to_use_select,':timePosition/@indeterminatePosition')"/>
<xsl:attribute name="data-tag-name" select="concat($prefix_to_use,':timePosition')"/>

</directiveAttributes>
</key>
</values>
<snippet>
<!-- this is more explicitly done, below -->
<gml:TimeInstant>
{{<xsl:copy-of select="local-name(..)"/>}}
</gml:TimeInstant>
</snippet>
</template>
</xsl:variable>

<xsl:variable name="keyValues">
<xsl:call-template name="build-key-value-configuration">
<xsl:with-param name="template" select="$template/template"/>
<xsl:with-param name="currentNode" select="$parent_parent"/>
<xsl:with-param name="readonly" select="'false'"/>
</xsl:call-template>
</xsl:variable>

<xsl:variable name="templateCombinedWithNode" as="node()">
<template>
<xsl:copy-of select="$template/template/values"/>

<xsl:element name="snippet" inherit-namespaces="no">
<xsl:attribute name="data-remove-gn-ns-def">true</xsl:attribute>

<xsl:element name="gml:TimeInstant" namespace="{$element_ns_uri}" inherit-namespaces="no">
<xsl:attribute namespace="{$element_ns_uri}" name="gml:id"><xsl:value-of select="$timeinstance_id"/></xsl:attribute>
{{<xsl:copy-of select="local-name(..)"/>}}
</xsl:element>


</xsl:element>
</template>
</xsl:variable>


<xsl:call-template name="render-element-template-field">
<xsl:with-param name="name" select="$name"/>
<xsl:with-param name="id" select="$id"/>
<xsl:with-param name="isExisting" select="true()"/>
<xsl:with-param name="template" select="$templateCombinedWithNode"/>
<xsl:with-param name="keyValues" select="$keyValues"/>
<xsl:with-param name="refToDelete" select="$refToDelete/gn:element"/>
<xsl:with-param name="isFirst" select="$isFirst"/>
</xsl:call-template>
</xsl:template>

<!---
For temporal extent, we want a more complicated control.
This will allow the <beginPosition indeterminatePosition="now"></beginPosition>
It also sets up the DatePicker so it has the indeterminatePosition drop-down, plus
a calendar date selector, a time selector, a timezone selector, and "mode" selector (i.e. only year, year + month, date & time).

This is basically a replacement for https://github.com/geonetwork/core-geonetwork/blob/5f6571e72e2bd39237394caf1f076a6cc0dc8758/schemas/iso19139/src/main/plugin/iso19139/layout/config-editor.xml#L1471-L1513
See the "template" variable, below.

This is basically a stand-in for the <field name="temporalRangeSection"> and uses the same template to do the controls.

NOTE: we are using [//local-name()='XYZ'] for the `gml:` namespace to make it work with both original gml and gml 3.2.
-->

<xsl:template mode="mode-iso19139"
match="gml:TimePeriod[.//gml:beginPosition]|gml320:TimePeriod[.//gml320:beginPosition]"
priority="30000">

<xsl:variable name="xpath" select="gn-fn-metadata:getXPath(.)"/>
<xsl:variable name="isoType" select="if (../@gco:isoType) then ../@gco:isoType else ''"/>


<xsl:variable name="configName" select="'temporalRangeSection'"/>
<!-- 'temporalRangeSection' this is the name of the section (defined elsewhere)
and uses to find the translated tag for the section in the UI -->
<xsl:variable name="translation"
select="$strings/*[name() = $configName]"/>
<xsl:variable name="name"
select="if ($translation != '')
then $translation
else $configName"/>

<xsl:variable name="id" select="concat('_X', gn:element/@ref, '_replace')"/>

<xsl:variable name="element_ns_uri" select="namespace-uri()"/>
<xsl:variable name="gml_is_32" select="$element_ns_uri = 'http://www.opengis.net/gml/3.2'"/>
<xsl:variable name="timeperiod_id" select="@*[local-name() = 'id']"/>

<xsl:variable name="prefix_to_use">gml</xsl:variable>

<xsl:variable name="prefix_to_use_select">
<xsl:choose>
<xsl:when test="$gml_is_32">gml</xsl:when>
<xsl:otherwise>gml320</xsl:otherwise>
</xsl:choose>
</xsl:variable>

<xsl:variable name="template">
<template xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:gn="http://www.fao.org/geonetwork" xmlns:gco="http://www.isotc211.org/2005/gco" xmlns:gmd="http://www.isotc211.org/2005/gmd" xmlns:gmx="http://www.isotc211.org/2005/gmx" xmlns:srv="http://www.isotc211.org/2005/srv" xmlns:gml="http://www.opengis.net/gml/3.2" xmlns:xlink="http://www.w3.org/1999/xlink">
<values>
<!-- -Need a * for gml:TimePeriodTypeCHOICE_ELEMENT2 added by editor enumerated tree
but this will make the XSLT formatter fails. Using // to access the element in both case. -->
<key label="beginPosition"
xpath=".//*[local-name()='beginPosition']"
use="gn-date-picker"
tooltip="gmd:extent|gmd:EX_TemporalExtent">
<directiveAttributes
data-assume-gml-ns="true"
>
<xsl:attribute name="data-indeterminate-position" select="concat('eval#.//',$prefix_to_use_select,':beginPosition/@indeterminatePosition')"/>
<xsl:attribute name="data-tag-name" select="concat($prefix_to_use,':beginPosition')"/>
</directiveAttributes>
</key>
<key label="endPosition"
xpath=".//*[local-name()='endPosition']"
use="gn-date-picker"
tooltip="gmd:extent|gmd:EX_TemporalExtent">
<directiveAttributes
data-assume-gml-ns="true"
>
<xsl:attribute name="data-indeterminate-position" select="concat('eval#.//',$prefix_to_use_select,':endPosition/@indeterminatePosition')"/>
<xsl:attribute name="data-tag-name" select="concat($prefix_to_use,':endPosition')"/>
</directiveAttributes>
</key>
</values>
<snippet>
<!-- Bind the wrapper to the record's actual gml namespace (old GML or GML 3.2)
rather than hardcoding GML 3.2, so editing begin/end position does not
silently change the namespace of an existing element. -->
<xsl:element name="gml:TimePeriod" namespace="{$element_ns_uri}" inherit-namespaces="no">
<xsl:attribute namespace="{$element_ns_uri}" name="gml:id"><xsl:value-of select="$timeperiod_id"/></xsl:attribute>
{{beginPosition}}
{{endPosition}}
</xsl:element>
</snippet>
</template>
</xsl:variable>
<xsl:variable name="isFirst" select="position() = 1"/>
<xsl:variable name="originalNode"
select="gn-fn-metadata:getOriginalNode($metadata, .)"/>
<xsl:variable name="del" select="./@del"/>

<xsl:variable name="refToDelete">
<xsl:call-template name="get-ref-element-to-delete">
<xsl:with-param name="node" select="$originalNode"/>
<xsl:with-param name="delXpath" select="$del"/>
</xsl:call-template>
</xsl:variable>

<xsl:variable name="parent_parent" select="."/>

<xsl:variable name="keyValues">
<xsl:call-template name="build-key-value-configuration">
<xsl:with-param name="template" select="$template/template"/>
<xsl:with-param name="currentNode" select="$parent_parent"/>
<xsl:with-param name="readonly" select="'false'"/>
</xsl:call-template>
</xsl:variable>


<xsl:variable name="templateCombinedWithNode" as="node()">
<template>
<xsl:copy-of select="$template/template/values"/>
<snippet>
<xsl:apply-templates mode="gn-merge" select="$template/template/snippet/*|$editorConfig/editor/snippets/list[@name = $template/template/snippets/@name]/snippet/*">
<xsl:with-param name="node-to-merge" select="$parent_parent"/>
</xsl:apply-templates>
</snippet>
</template>
</xsl:variable>

<xsl:call-template name="render-element-template-field">
<xsl:with-param name="id" select="$id"/>
<xsl:with-param name="isExisting" select="true()"/>
<xsl:with-param name="template" select="$templateCombinedWithNode"/>
<xsl:with-param name="keyValues" select="$keyValues"/>
<xsl:with-param name="refToDelete" select="$refToDelete/gn:element"/>
<xsl:with-param name="isFirst" select="$isFirst"/>
</xsl:call-template>

</xsl:template>


<!-- ===================================================================== -->
<!-- gml:TimePeriod (format = %Y-%m-%dThh:mm:ss) -->
<!-- ===================================================================== -->

<xsl:template mode="mode-iso19139"
match="gml:beginPosition|gml:endPosition|gml:timePosition|
gml320:beginPosition|gml320:endPosition|gml320:timePosition"
match="gml:beginPosition|gml:endPosition|
gml320:beginPosition|gml320:endPosition"
priority="200">


<xsl:variable name="xpath" select="gn-fn-metadata:getXPath(.)"/>
<xsl:variable name="value" select="normalize-space(text())"/>
<xsl:variable name="isoType" select="if (../@gco:isoType) then ../@gco:isoType else ''"/>
Expand All @@ -181,14 +433,14 @@
</xsl:if>
</xsl:variable>


<xsl:call-template name="render-element">
<xsl:with-param name="label"
select="$labelConfig"/>
<xsl:with-param name="name" select="gn:element/@ref"/>
<xsl:with-param name="value" select="text()"/>
<xsl:with-param name="cls" select="local-name()"/>
<xsl:with-param name="xpath" select="$xpath"/>

<!--
Default field type is Date.

Expand Down
3 changes: 3 additions & 0 deletions schemas/iso19139/src/main/plugin/iso19139/update-fixed-info.xsl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -756,6 +756,9 @@
<xsl:template match="gmd:dataQualityInfo[count(*) = 0]"/>
<xsl:template match="gmd:transferOptions[gmd:MD_DigitalTransferOptions/count(*) = 0]"/>

<!-- Delete empty temporal elements -->
<xsl:template match="gmd:temporalElement[count(*) = 0]"/>

<!-- ================================================================= -->
<!-- copy everything else as is -->

Expand Down
23 changes: 18 additions & 5 deletions web-ui/src/main/resources/catalog/components/edit/datepicker/DatePickerDirective.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -159,6 +159,7 @@
elementRef: "@",
id: "@",
tagName: "@",
assumeGmlNs: "@",
indeterminatePosition: "@",
required: "@",
hideTime: "@",
Expand Down Expand Up @@ -358,14 +359,26 @@
scope.dateTime = "";
}

//if we don't know what this namespace is (findNamespaceUri returns undefined),
//then we don't add it!
var elementNSDef = "";
var namespaceURI = gnSchemaManagerService.findNamespaceUri(
namespace,
gnCurrentEdit.schema
);
if (namespaceURI !== undefined) {
elementNSDef = " xmlns:" + namespace + '="' + namespaceURI + '"';
}
Comment on lines +365 to +371

@josegar74 josegar74 Feb 24, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Would not be more efficient to put this code in the else of:

              //if data-assume-gml-ns is set, then don't put in the NS declaration
              if (scope.assumeGmlNs === "true" && namespace === "gml") {
                elementNSDef = "";
              }

@davidblasby davidblasby Feb 24, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I don't think efficiency is important here.

@josegar74 josegar74 Feb 24, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

In that case is ok.


//if data-assume-gml-ns is set, then don't put in the NS declaration
if (scope.assumeGmlNs === "true" && namespace === "gml") {
elementNSDef = "";
}

scope.xmlSnippet =
"<" +
tag +
" xmlns:" +
namespace +
'="' +
gnSchemaManagerService.findNamespaceUri(namespace, gnCurrentEdit.schema) +
'"' +
elementNSDef +
attribute +
">" +
scope.dateTime +
Expand Down
Loading
Loading