Download as
Browse Code »

Commit 8dea402d38052267a8ac41a1581d6d72bf2e13cc

Authored by Emerson Oliveira
1 parent cc801e45
Exists in master

IN PROGRESS - issue FWK-35: Mecanismo para messageContext chamar 

getFacesContext().validationFailed() 
https://demoiselle.atlassian.net/browse/FWK-35
Depreciação do método add(Message message, Object... params) e ajuste
deste para ignorar arrays de parametro sem elementos
Showing 2 changed files with 16 additions and 9 deletions   Show diff stats
impl/core/src/main/java/br/gov/frameworkdemoiselle/internal/implementation/MessageContextImpl.java
  Diff comments   View file @8dea402
... ... @@ -66,17 +66,24 @@ public class MessageContextImpl implements Serializable, MessageContext {
66 66 private transient Logger logger;
67 67  
68 68 @Override
  69 + @Deprecated
69 70 public void add(final Message message, Object... params) {
70   - Message aux;
71   -
72   - if (params != null) {
73   - aux = new DefaultMessage(message.getText(), message.getSeverity(), params);
  71 +
  72 + getLogger().debug(getBundle().getString("adding-message-to-context", message.toString()));
  73 + if (params == null || params.length == 0) {
  74 + getAppender().append(message);
74 75 } else {
75   - aux = message;
  76 + getLogger().warn("Atualmente, ao chamar o método add do MessageContext passando um objeto"
  77 + + " do tipo Message e mais parâmetros, será recriando um objeto" +"\n"
  78 + + " Message, na implementação DefaultMessage para que os parâmetros sejam utilizados."
  79 + + " Note que isso poderá trazer problemas para sua aplicação, caso" +"\n"
  80 + + " a implementação de Message utilizada não seja a DefaultMessage. Para evitar esse tipo de problema"
  81 + + " e garantir compatibilidade com versões futuras, recomendamos que" +"\n"
  82 + + " o objeto message seja criado com os parâmetros, e que para o método add apenas seja passado"
  83 + + " esse objeto como parâmetro.");
  84 +
  85 + getAppender().append(new DefaultMessage(message.getText(), message.getSeverity(), params));
76 86 }
77   -
78   - getLogger().debug(getBundle().getString("adding-message-to-context", message.toString()));
79   - getAppender().append(aux);
80 87 }
81 88  
82 89 private MessageAppender getAppender() {
... ...
impl/core/src/main/java/br/gov/frameworkdemoiselle/message/MessageContext.java
  Diff comments   View file @8dea402
... ... @@ -55,7 +55,7 @@ public interface MessageContext {
55 55 *
56 56 * @param message
57 57 */
58   -
  58 + @Deprecated
59 59 void add(Message message, Object... params);
60 60  
61 61 void add(String text, Object... params);
... ...