You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by sk...@apache.org on 2007/10/05 12:32:35 UTC

svn commit: r582196 - /myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/jsf/JsfConversationMessager.java

Author: skitching
Date: Fri Oct  5 03:32:34 2007
New Revision: 582196

URL: http://svn.apache.org/viewvc?rev=582196&view=rev
Log:
Break up long lines of code. No logic changes.

Modified:
    myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/jsf/JsfConversationMessager.java

Modified: myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/jsf/JsfConversationMessager.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/jsf/JsfConversationMessager.java?rev=582196&r1=582195&r2=582196&view=diff
==============================================================================
--- myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/jsf/JsfConversationMessager.java (original)
+++ myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/jsf/JsfConversationMessager.java Fri Oct  5 03:32:34 2007
@@ -19,12 +19,13 @@
 
 package org.apache.myfaces.orchestra.conversation.jsf;
 
-import org.apache.myfaces.orchestra.conversation.ConversationMessager;
+import java.io.PrintWriter;
+import java.io.StringWriter;
 
 import javax.faces.application.FacesMessage;
 import javax.faces.context.FacesContext;
-import java.io.PrintWriter;
-import java.io.StringWriter;
+
+import org.apache.myfaces.orchestra.conversation.ConversationMessager;
 
 /**
  * Inform the user about some anomalies by displaying the problems as standard JSF messages.
@@ -39,15 +40,18 @@
 {
 	public void setConversationException(Throwable t)
 	{
-		FacesContext.getCurrentInstance().addMessage(null, new FacesMessage(FacesMessage.SEVERITY_FATAL, t.getLocalizedMessage(), getThrowableText(t)));
+		String msgSummary = t.getLocalizedMessage();
+		String msgDetail = getThrowableText(t);
+		FacesMessage msg = new FacesMessage(FacesMessage.SEVERITY_FATAL, msgSummary, msgDetail);
+		FacesContext.getCurrentInstance().addMessage(null, msg); 
 	}
 
 	public void setConversationNotActive(String name)
 	{
-		String message = "Conversation not active";
-		String messageDtl = "Conversation not active. Please start over. (Conversation Name:" + name + ")";
-
-		FacesContext.getCurrentInstance().addMessage(null, new FacesMessage(FacesMessage.SEVERITY_FATAL, message, messageDtl));
+		String msgSummary = "Conversation not active";
+		String msgDetail = "Conversation not active. Please start over. (Conversation Name:" + name + ")";
+		FacesMessage msg = new FacesMessage(FacesMessage.SEVERITY_FATAL, msgSummary, msgDetail);
+		FacesContext.getCurrentInstance().addMessage(null, msg);
 	}
 
 	protected String getThrowableText(Throwable t)