From 6b429231111defbf9baecc7fb49776af43cad6d9 Mon Sep 17 00:00:00 2001 From: Dancovich Date: Wed, 10 Jul 2013 11:20:42 -0300 Subject: [PATCH] Removidos TODO's e externalizadas strings para arquivo bundle. --- impl/extension/jsf/src/main/java/br/gov/frameworkdemoiselle/internal/implementation/SecurityObserver.java | 27 +++++++++------------------ impl/extension/jsf/src/main/resources/demoiselle-jsf-bundle.properties | 5 ++++- 2 files changed, 13 insertions(+), 19 deletions(-) diff --git a/impl/extension/jsf/src/main/java/br/gov/frameworkdemoiselle/internal/implementation/SecurityObserver.java b/impl/extension/jsf/src/main/java/br/gov/frameworkdemoiselle/internal/implementation/SecurityObserver.java index 995e484..c7ccd48 100644 --- a/impl/extension/jsf/src/main/java/br/gov/frameworkdemoiselle/internal/implementation/SecurityObserver.java +++ b/impl/extension/jsf/src/main/java/br/gov/frameworkdemoiselle/internal/implementation/SecurityObserver.java @@ -48,6 +48,7 @@ import javax.servlet.http.HttpSession; import org.slf4j.Logger; +import br.gov.frameworkdemoiselle.annotation.Name; import br.gov.frameworkdemoiselle.configuration.ConfigurationException; import br.gov.frameworkdemoiselle.internal.configuration.JsfSecurityConfig; import br.gov.frameworkdemoiselle.security.AfterLoginSuccessful; @@ -55,6 +56,7 @@ import br.gov.frameworkdemoiselle.security.AfterLogoutSuccessful; import br.gov.frameworkdemoiselle.util.Beans; import br.gov.frameworkdemoiselle.util.PageNotFoundException; import br.gov.frameworkdemoiselle.util.Redirector; +import br.gov.frameworkdemoiselle.util.ResourceBundle; @SessionScoped public class SecurityObserver implements Serializable { @@ -69,6 +71,10 @@ public class SecurityObserver implements Serializable { @Inject private Logger logger; + + @Inject + @Name("demoiselle-jsf-bundle") + private ResourceBundle bundle; public SecurityObserver() { clear(); @@ -107,12 +113,7 @@ public class SecurityObserver implements Serializable { Redirector.redirect(getConfig().getLoginPage()); } catch (PageNotFoundException cause) { - // TODO Colocar a mensagem no bundle - throw new ConfigurationException( - "A tela de login \"" - + cause.getViewId() - + "\" não foi encontrada. Caso o seu projeto possua outra, defina no arquivo de configuração a chave \"" - + "frameworkdemoiselle.security.login.page" + "\"", cause); + throw new ConfigurationException( bundle.getString("login-page-not-found",cause.getViewId()) , cause); } } @@ -130,12 +131,7 @@ public class SecurityObserver implements Serializable { } catch (PageNotFoundException cause) { if (redirectedFromConfig) { - // TODO Colocar a mensagem no bundle - throw new ConfigurationException( - "A tela \"" - + cause.getViewId() - + "\" que é invocada após o logon não foi encontrada. Caso o seu projeto possua outra, defina no arquivo de configuração a chave \"" - + "frameworkdemoiselle.security.redirect.after.login" + "\"", cause); + throw new ConfigurationException( bundle.getString("after-login-page-not-found",cause.getViewId()) , cause); } else { throw cause; } @@ -152,12 +148,7 @@ public class SecurityObserver implements Serializable { } } catch (PageNotFoundException cause) { - // TODO Colocar a mensagem no bundle - throw new ConfigurationException( - "A tela \"" - + cause.getViewId() - + "\" que é invocada após o logout não foi encontrada. Caso o seu projeto possua outra, defina no arquivo de configuração a chave \"" - + "frameworkdemoiselle.security.redirect.after.logout" + "\"", cause); + throw new ConfigurationException( bundle.getString("after-logout-page-not-found",cause.getViewId()) , cause); } finally { try { diff --git a/impl/extension/jsf/src/main/resources/demoiselle-jsf-bundle.properties b/impl/extension/jsf/src/main/resources/demoiselle-jsf-bundle.properties index 52a1346..ae1a582 100644 --- a/impl/extension/jsf/src/main/resources/demoiselle-jsf-bundle.properties +++ b/impl/extension/jsf/src/main/resources/demoiselle-jsf-bundle.properties @@ -33,4 +33,7 @@ # ou escreva para a Fundação do Software Livre (FSF) Inc., # 51 Franklin St, Fifth Floor, Boston, MA 02111-1301, USA. -id-converter-not-found=Voc\u00EA precisa criar um FacesConverter para a classe "{0}". \ No newline at end of file +id-converter-not-found=Voc\u00EA precisa criar um FacesConverter para a classe "{0}". +login-page-not-found=A tela de login "{0}" n\u00E3o foi encontrada. Caso essa n\u00E3o seja a p\u00E1gina correta, defina a p\u00E1gina no arquivo de configura\u00E7\u00E3o usando a chave "frameworkdemoiselle.security.login.page" +after-login-page-not-found=A tela "{0}" acessada ap\u00F3s o login n\u00E3o foi encontrada. Caso essa n\u00E3o seja a p\u00E1gina correta, defina a p\u00E1gina no arquivo de configura\u00E7\u00E3o usando a chave "frameworkdemoiselle.security.redirect.after.login" +after-logout-page-not-found=A tela "{0}" acessada ap\u00F3s o logout n\u00E3o foi encontrada. Caso essa n\u00E3o seja a p\u00E1gina correta, defina a p\u00E1gina no arquivo de configura\u00E7\u00E3o usando a chave "frameworkdemoiselle.security.redirect.after.logout" \ No newline at end of file -- libgit2 0.21.2