search

DSpace / DSpace

(Official) The DSpace digital asset management system that powers your Institutional Repository
https://wiki.lyrasis.org/display/DSDOC7x/
BSD 3-Clause "New" or "Revised" License
853 stars 1.29k forks source link

I18Util.getEPersonLocale: consider reducing log level from error to info #9585

Closed saschaszott closed 1 month ago

saschaszott commented 1 month ago

Description

I18Util.getEPersonLocale generates log messages of level error if the given eperson does not have a default language:

https://github.com/DSpace/DSpace/blob/1517e8cd0fe59645827164099060b1d3e80b9f9f/dspace-api/src/main/java/org/dspace/core/I18nUtil.java#L97

Since the method has valid defaults (falls back to default locale) the log level should be changed to info.

tdonohue commented 1 month ago

Agreed, this doesn't seem like an error. Needs volunteer.

saschaszott commented 1 month ago

Agreed, this doesn't seem like an error. Needs volunteer.

PR is available: https://github.com/DSpace/DSpace/pull/9588