is the code in 
org.jboss.seam.navigation.Pages.convertAndValidateStringValuesInPageContext() 
really correct?

The order in which the conversion and validation of page parameters is done, 
looks to me like a bug. 


  | // 
org.jboss.seam.navigation.Pages.convertAndValidateStringValuesInPageContext()
  | 
  | 
  | 
  | String value = (String) Contexts.getPageContext().get( 
pageParameter.getName() );
  |                if (value!=null)
  |                {
  |                   pageParameter.validateConvertedValue(facesContext, value);
  |                   Object convertedValue = 
pageParameter.convertValueFromString(facesContext, value);
  |                   Contexts.getEventContext().set( pageParameter.getName(), 
convertedValue );
  |                }
  | 


Shouldn't conversion and validation occur in the order as follows?


  | // 
org.jboss.seam.navigation.Pages.convertAndValidateStringValuesInPageContext()
  | 
  | 
  | String value = (String) Contexts.getPageContext().get( 
pageParameter.getName() );
  |                if (value!=null)
  |                {
  |                   Object convertedValue = 
pageParameter.convertValueFromString(facesContext, value);
  |                   pageParameter.validateConvertedValue(facesContext, 
convertedValue);
  |                   Contexts.getEventContext().set( pageParameter.getName(), 
convertedValue );
  |                }
  | 


View the original post : 
http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4109610#4109610

Reply to the post : 
http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4109610
_______________________________________________
jboss-user mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/jboss-user

Reply via email to