[ 
https://issues.jboss.org/browse/RF-12028?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12680439#comment-12680439
 ] 

Jean ANDRE edited comment on RF-12028 at 3/29/12 4:07 PM:
----------------------------------------------------------

Well,
     - for a div -> silent, ignore, no message

     - for a div with duplicate attribute ->
       Error 500: javax.servlet.ServletException: Error Parsing 
/pages/client/infoClient.xhtml:
       Error Traced[line: 30] L'attribut "class" a déjà été indiqué pour 
l'élément "div".

     - for a h:message -> silent, ignore, no message
       Error 500: javax.servlet.ServletException: Error Parsing 
/pages/client/infoClient.xhtml:
       Error Traced[line: 49] L'attribut "globalOnly" a déjà été indiqué pour 
l'élément "h:messages"

     - for h:panelGrid -> silent, ignore, no message


     - for h:panelGrid with duplicate attribute ->
       Error 500: javax.servlet.ServletException: Error Parsing 
/pages/client/infoClient.xhtml:
       Error Traced[line: 57] L'attribut "columns" a déjà été indiqué pour 
l'élément "h:panelGrid". 

To summarize:
 1) an invalid attribute is ignored, server does not stop and no message 
appears at the console
 2) an duplicate valid attribute trigger an HTTP error 500. Line number and 
attribute name and tag are clearly identified.
 3) Note also, from our experience, MyFaces implementation is more strict and 
reports more errors at the console level.

From my side, I prefer to have a warning message at the console/log for these 
trivial errors, it helps to maintain a clean code since even the IDE does not 
detect such error.



 
                
      was (Author: jorelia64):
    Well,
     - for a div -> silent, ignore, no message

     - for a div with duplicate attribute ->
       Error 500: javax.servlet.ServletException: Error Parsing 
/pages/client/infoClient.xhtml:
       Error Traced[line: 30] L'attribut "class" a déjà été indiqué pour 
l'élément "div".

     - for a h:message -> silent, ignore, no message
       Error 500: javax.servlet.ServletException: Error Parsing 
/pages/client/infoClient.xhtml:
       Error Traced[line: 49] L'attribut "globalOnly" a déjà été indiqué pour 
l'élément "h:messages"

     - for h:panelGrid -> silent, ignore, no message


     - for h:panelGrid with duplicate attribute ->
       Error 500: javax.servlet.ServletException: Error Parsing 
/pages/client/infoClient.xhtml:
       Error Traced[line: 57] L'attribut "columns" a déjà été indiqué pour 
l'élément "h:panelGrid". 

To summarize:
 1) an invalid attribute is ignore, server does not stop and no message appears 
on the console
 2) an duplicate valid attribute trigger an HTTP error 500. Line number and 
attribute name and tag are clearly identified.
 3) Note also, from our experience, MyFaces implementation is more strict and 
reports more errors at the console level.

From my side, I prefer to have a warning message at the console/log for these 
trivial errors, it helps to maintain a clean code since even the IDE does not 
detect such error.



 
                  
> a4j:outputPanel - Error message enhancement
> -------------------------------------------
>
>                 Key: RF-12028
>                 URL: https://issues.jboss.org/browse/RF-12028
>             Project: RichFaces
>          Issue Type: Feature Request
>      Security Level: Public(Everyone can see) 
>         Environment: JSF / Mojarra 2.1.6 - cssparser-0.9.6 - guava-11.0.1 - 
> hibernate-validator-4.2.0.Final - springframework 3.0.6.RELEASE - 
> richfaces--4.2.0.Final - sac-1.3 - FireFox 10 - WAS 8.0.0.1 / Windows 7
>            Reporter: Jean ANDRE
>              Labels: waiting_on_user
>
> When we set an non existing attribute on a4j:outputPanel, for eg. "class" 
> because we have converted a div to a a4j:outputPanel, the error message 
> displayed at the user interface does not inform the developer about the 
> faulty component. The message we got is only :
> > Error 500: javax.servlet.ServletException: Setter not found for property 
> > class 
> This error message is not very helpful to debug.... ;)
> {code}
> <a4j:outputPanel id="product"
>              layout="block"
>               class="dark-box"
>            rendered="#{productBean.expanded eq true}">
> {code}
> We should have, as usual, the line of the XHTML file or the component name, 
> the line number, well something more accurate that really help to go faster.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.jboss.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

_______________________________________________
richfaces-issues mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/richfaces-issues

Reply via email to