You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by sc...@apache.org on 2005/08/30 17:20:02 UTC

svn commit: r264794 - /myfaces/api/trunk/src/java/javax/faces/component/_ComponentUtils.java

Author: schof
Date: Tue Aug 30 08:19:58 2005
New Revision: 264794

URL: http://svn.apache.org/viewcvs?rev=264794&view=rev
Log:
removed comments after checking spec; set severity of message to error (in case a validator outside of MyFaces forgets to do this.)

Modified:
    myfaces/api/trunk/src/java/javax/faces/component/_ComponentUtils.java

Modified: myfaces/api/trunk/src/java/javax/faces/component/_ComponentUtils.java
URL: http://svn.apache.org/viewcvs/myfaces/api/trunk/src/java/javax/faces/component/_ComponentUtils.java?rev=264794&r1=264793&r2=264794&view=diff
==============================================================================
--- myfaces/api/trunk/src/java/javax/faces/component/_ComponentUtils.java (original)
+++ myfaces/api/trunk/src/java/javax/faces/component/_ComponentUtils.java Tue Aug 30 08:19:58 2005
@@ -138,13 +138,9 @@
                 FacesMessage facesMessage = e.getFacesMessage();
                 if (facesMessage != null)
                 {
+                    facesMessage.setSeverity(FacesMessage.SEVERITY_ERROR);
                     context.addMessage(input.getClientId(context), facesMessage);
                 }
-                else
-                {
-                    //TODO: specification? add a general message?
-                }
-                //TODO: specification? should we abort validation immediately
             }
         }
 
@@ -165,13 +161,9 @@
                     FacesMessage facesMessage = ((ValidatorException)cause).getFacesMessage();
                     if (facesMessage != null)
                     {
+                        facesMessage.setSeverity(FacesMessage.SEVERITY_ERROR);
                         context.addMessage(input.getClientId(context), facesMessage);
                     }
-                    else
-                    {
-                        //TODO: specification? add a general message?
-                    }
-                    //TODO: specification? should we abort validation immediately
                 }
                 else
                 {