Skip to content
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

Geonetwork 4.2.7 #260

Merged
merged 954 commits into from
Jan 26, 2024
Merged

Geonetwork 4.2.7 #260

merged 954 commits into from
Jan 26, 2024

Conversation

f-necas
Copy link
Collaborator

@f-necas f-necas commented Nov 16, 2023

josegar74 and others added 30 commits May 30, 2023 09:50
…iguration of the online resources. Fixes #7103
…r information.

This allows to display a sort by a field in ascending and descending order, for example the metadata title
* Fix selection / filter Only my records

* Fix selection / filter Only my records: use setting to enable Only my records in the editor application, not in the search application
…h the case where no port is set in the settings manager
… that we can see the real error in the logs. (#7119)
* Allow multiple request matchers to decide if require CSRF token
* Fix a problem with the Keycloak configuration that was making the server
to require CSRF tokens for GN URLs that shouldn't be protected by CSRF.
To do this, now the GeonetworkCsrfSecurityRequestMatcher allows to be configured
with additional RequestMatchers, for example the KeycloakCsrfRequestMatcher.
If Keycloak security is enabled, then the KeycloakCsrfRequestMatcher is added to the
original GeoNetwork patterns listed in `config-security-core.xml`
The link for 'Installing from source code' is broken.  The label of Maintainer Guide is also incorrect and should be Installation Guide.
…ustom status (#7104)

* Core cleanup translation.

* Workflow / Remove magic numbers in label and avoid issue when using
custom status.

The status workflow index field contains 2 numbers concatenated - status of the record + status of the draft.
If using custom status, the concatenation of the number can match other status number eg. 111 can be 1+11 or 11+1 so we can't always get a label from that value.
Use status name instead of ids and rely on db translation as much as
possible (there is still room for improvements when custom status are
used).

Co-authored-by: Joachim Nielandt <[email protected]>

* Workflow / Add configuration for the list of status for which to start a
draft.

Co-authored-by: Joachim Nielandt <[email protected]>

* Workflow / Better build label.

Co-authored-by: Joachim Nielandt <[email protected]>

* Workflow / Record view / Use db translations.

* Workflow / Label improvement.

---------

Co-authored-by: Joachim Nielandt <[email protected]>
…ved (#7074)

When a metadata is published / un-published it can be configured a mail notification.

When the metadata workflow is enabled, published metadata can be edited and re-approved. 
This change request expands the email notifications in this case, to send an email to let you know 
that a published metadata has been re-approved, with a new version published.

Related to #6457

---------

Co-authored-by: Juan Luis Rodríguez <[email protected]>
…nd labels (#7076)

* UI / Language picker / Display all entries on focus. Search by code and english.

* Prettier.
* Map / Layer manager / Add support for multilingual layer title.

Metadata may be multilingual and provide multilingual layer name in the
online source description field. GetCapabilities document do not provide
multilingual support, so this adds the possibility to translate layer
name in maps.

A multilingual description in a link:

```xml
<gmd:onLine>
  <gmd:CI_OnlineResource>
    <gmd:linkage>
      <gmd:URL>https://data.apps.fao.org/map/gsrv/gsrv1/geonetwork/ows?SERVICE=WMS&amp;</gmd:URL>
    </gmd:linkage>
    <gmd:protocol>
      <gmx:MimeFileType type="">OGC:WMS</gmx:MimeFileType>
    </gmd:protocol>
    <gmd:name xsi:type="gmd:PT_FreeText_PropertyType">
      <gco:CharacterString>geonetwork:basins_296</gco:CharacterString>
      <gmd:PT_FreeText>
        <gmd:textGroup>
          <gmd:LocalisedCharacterString locale="#EN">geonetwork:basins_296</gmd:LocalisedCharacterString>
        </gmd:textGroup>
        <gmd:textGroup>
          <gmd:LocalisedCharacterString locale="#FR" />
        </gmd:textGroup>
      </gmd:PT_FreeText>
    </gmd:name>
    <gmd:description xsi:type="gmd:PT_FreeText_PropertyType">
      <gco:CharacterString>Hydrological basins in Africa</gco:CharacterString>
      <gmd:PT_FreeText>
        <gmd:textGroup>
          <gmd:LocalisedCharacterString locale="#EN">Hydrological basins in Africa</gmd:LocalisedCharacterString>
        </gmd:textGroup>
        <gmd:textGroup>
          <gmd:LocalisedCharacterString locale="#FR">Bassins versants en Afrique</gmd:LocalisedCharacterString>
        </gmd:textGroup>
      </gmd:PT_FreeText>
    </gmd:description>
  </gmd:CI_OnlineResource>
</gmd:onLine>
```

The description is retrieved from the metadata on load from the metadata
UUID which is attached to the layer (and saved in the OWS context
extension of the layer) using the service URL and the layer name.

This also allows to override layer name when a group of layer (eg.
`a,b,c`) is used in the online source name.

* Map / When loading a map, switch to layer tool panel.

* Map / Layer manager / Add support for multilingual layer title / Layer search.

* Map / Layer manager / Add support for multilingual layer title / Layer filter.

* Remove openTool after map load.

Configuration done in geonetwork/core-geonetwork#7126
* Map / Background layer / Fix active layer on load.

Test:
* Open the map
* Reload page
* Active background is not enabled.

This was related to the fact that the search map is also affecting bgLayers.

Only the main map viewer is defining the bgLayers which is used for the base switcher directive.
In case of the context has no active background, use the first one (and log a warning).
Reset default map :
* may fail if no context defined. Add a fallback to the default one (and log a warning).
* tried to use the map layer config if no context defined as it was ignored (but not working - require more work or removal of those unused ? settings)

* Map / Avoid JSON parse error on context having layer with filter. Clear
queue on context load.

Context layer can contain a JSON extension which could be formatted and
XML formatting may add extra space which needs to be clean up for proper
JSON string:

```xml
       <ows-context:Layer name="surval_parametre_point" group="/Human Activities" hidden="false" opacity="1"
                           showLayerInfo="true">
            <ows:Title>Surval données par paramètre (point)</ows:Title>
            <ows-context:Server service="urn:ogc:serviceType:WMS" version="1.3.0">
                <ows-context:OnlineResource xlink:href="https://sextant.ifremer.fr/services/wms/environnement_marin"/>
            </ows-context:Server>
            <ows-context:Extension>{"uuid":"cf5048f6-5bbf-4e44-ba74-e6f429af51ea","label":"Surval données par paramètre
                (point)","filters":{"qParams":{"ft_LIEU_LIBELLE_s":{"type":"terms","values":{"001-P-003 - Zuydcoote
                filière":true}}}},"wfsUrl":"https://sextant.ifremer.fr/services/wfs/environnement_marin"}
            </ows-context:Extension>
        </ows-context:Layer>
```

Clear map queue on load - may have side effect when loading differents
maps quickly.

Avoid JS error if server layer is not defined (which should not be the
case unless custom context with new layer types was created).

* Update OwsContextDirective.js

* Map / Add option to configure default active tool and default active tool after map load.

* Update OwsContextService.js
@fvanderbiest
Copy link
Member

Don't know if related, but metadata template called "Template for feature catalogue" cannot be edited

@f-necas
Copy link
Collaborator Author

f-necas commented Dec 27, 2023

Don't know if related, but metadata template called "Template for feature catalogue" cannot be edited

I could edit the title. What was the error message ?

@fvanderbiest
Copy link
Member

Probably a local error on MEL integration then.

@landryb
Copy link
Member

landryb commented Jan 11, 2024

passing by and stumbled upon #272, upstreaming stalled in geonetwork/core-geonetwork#5826.. should i rebase the PR on top of your branch (since it's the future..) or recreate/cherrypick 1ec55be ? @pmauduit ?

@f-necas
Copy link
Collaborator Author

f-necas commented Jan 12, 2024

@landryb maybe pushing it directly in this PR with updating the doc in geonetwork-migrations ?

Or you can PR it on gn-4.2.x and it should create conflict here I'll resolve when merging it. Both suits me !

@landryb
Copy link
Member

landryb commented Jan 15, 2024

#277, #278 and #279 should/could be added/pushed to that georchestra-gn4.2.x-rebase branch.. or the branch should be rebased on those once merged ?

@f-necas
Copy link
Collaborator Author

f-necas commented Jan 15, 2024

I think I'll rebase before merging in order to update the documentation. I edit the description of this PR to remember to add your PRs.

By the way, I implemented a backport bot on geochestra and geor/gn, see #276.

@landryb
Copy link
Member

landryb commented Jan 15, 2024

By the way, I implemented a backport bot on geochestra and geor/gn, see #276.

i've seen it and i'll gladly try it once the PRs are merged :)

Copy link
Member

@landryb landryb left a comment

Choose a reason for hiding this comment

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

im no GN expert, but i fully support getting this beast upgraded :)

@f-necas f-necas force-pushed the georchestra-gn4.2.x branch from 4eb9b94 to 5dc598b Compare January 24, 2024 09:48
@f-necas f-necas force-pushed the georchestra-gn4.2.x-rebase branch 4 times, most recently from 7dd1867 to a15df19 Compare January 26, 2024 08:32
f-necas and others added 5 commits January 26, 2024 10:03
logging configuration file should be taken from georchestra datadir
fix: add missing import
dont ship an extra version of groovy jar (fixes #267)
Placeholder for ogc api features added as external resource
@f-necas f-necas force-pushed the georchestra-gn4.2.x-rebase branch from 8af78f2 to 8765a1d Compare January 26, 2024 09:07
@f-necas f-necas merged commit 8419cfa into georchestra-gn4.2.x Jan 26, 2024
1 check passed
@f-necas f-necas deleted the georchestra-gn4.2.x-rebase branch January 26, 2024 09:35
@landryb
Copy link
Member

landryb commented Jan 26, 2024

WOO! :)

@fgravin
Copy link
Member

fgravin commented Jan 26, 2024

👏

4.2.8 babes !!!

@f-necas
Copy link
Collaborator Author

f-necas commented Jan 26, 2024

👏

4.2.8 babes !!!

One thing at a time. 😄

@landryb landryb mentioned this pull request Jun 26, 2024
4 tasks
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.