[jboss-user] [JBoss Seam] - Page Parameter validation - Bug in org.jboss.seam.navigation

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#... Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&a...