From 1f41c978e406e26d51081aad71da073b863e0b18 Mon Sep 17 00:00:00 2001 From: Teo Sarca Date: Mon, 4 Dec 2017 11:51:39 +0200 Subject: [PATCH] InternalUserSessionData: initialize language in afterPropertiesSet() https://github.com/metasfresh/metasfresh-webui-api/issues/713 --- .../web/session/InternalUserSessionData.java | 39 ++++++++++--------- 1 file changed, 20 insertions(+), 19 deletions(-) diff --git a/src/main/java/de/metas/ui/web/session/InternalUserSessionData.java b/src/main/java/de/metas/ui/web/session/InternalUserSessionData.java index 466b3554e..4b15038c4 100644 --- a/src/main/java/de/metas/ui/web/session/InternalUserSessionData.java +++ b/src/main/java/de/metas/ui/web/session/InternalUserSessionData.java @@ -111,44 +111,45 @@ public InternalUserSessionData() ctx = new Properties(); Env.setContext(ctx, CTXNAME_IsServerContext, false); + UserSession.logger.trace("User session created: {}", this); + } + + @Override + public void afterPropertiesSet() throws Exception + { + // + // Set initial properties + setShowColumnNamesForCaption(defaultShowColumnNamesForCaption); + setAllowDeprecatedRestAPI(defaultAllowDeprecatedRestAPI); + setHttpCacheMaxAge(defaultHttpCacheMaxAge); + setUseHttpAcceptLanguage(defaultUseHttpAcceptLanguage); + + // // Set initial language try { final Language language = findInitialLanguage(); verifyLanguageAndSet(language); } - catch (final Exception e) + catch (final Throwable ex) { - UserSession.logger.warn("Failed setting the language, but moving on", e); + UserSession.logger.warn("Failed setting the language, but moving on", ex); } - - UserSession.logger.trace("User session created: {}", this); } - + private static final Language findInitialLanguage() { final Locale locale = LocaleContextHolder.getLocale(); - if(locale != null) + if (locale != null) { final Language language = Language.findLanguageByLocale(locale); - if(language != null) + if (language != null) { return language; } } - - return Language.getBaseLanguage(); - } - @Override - public void afterPropertiesSet() throws Exception - { - // - // Set initial properties - setShowColumnNamesForCaption(defaultShowColumnNamesForCaption); - setAllowDeprecatedRestAPI(defaultAllowDeprecatedRestAPI); - setHttpCacheMaxAge(defaultHttpCacheMaxAge); - setUseHttpAcceptLanguage(defaultUseHttpAcceptLanguage); + return Language.getBaseLanguage(); } @Override