When type="both", inputDate does not accept null value ------------------------------------------------------
Key: TOMAHAWK-759 URL: http://issues.apache.org/jira/browse/TOMAHAWK-759 Project: MyFaces Tomahawk Issue Type: Bug Components: Date Affects Versions: 1.1.5-SNAPSHOT Environment: JDK 1.5.0.07, Tomcat 5.5.17 or Tomcat 5.0.28 Reporter: Abhilash Kumar When the value of the type attribute of the inputDate is "both", it is not accepting null values even if required is not true. This can be easily reproduced using the Date example of examples webapp of SNAPAHOT-1.1.5. 1. Take the date example - (/date.jsf) 2. Enter null value for the third input component ( Where both date and time is to be entered ) and click on update. This will give a validation error saying that the entered date is not valid. I have debugged the code and found out that the actual exception is a NumberFormatException from HtmlInputDate.parse(). This happens because the code bypasses the "isSubmitValid()" function where the submitted values are checked for null. The "isTimeSubmitted()" function returns true in the case of type=both. For type==both, only hours and minutes are to be checked (as in the case of short_time) and there is no need of checking the seconds value (which will have value = 0 in the case of type = both). So the fix can be removal of type.equals("both") from the if condition in the second line of the function. Class -> org.apache.myfaces.custom.date.HtmlInputDate private boolean isTimeSubmitted(boolean usesAmpm, String type) { boolean isTimeSubmitted = ! (StringUtils.isEmpty(getHours()) && StringUtils.isEmpty(getMinutes())); if(type.equals("time") || type.equals("full") || type.equals("both")) isTimeSubmitted = isTimeSubmitted || ! StringUtils.isEmpty(getSeconds()); if(usesAmpm) isTimeSubmitted = isTimeSubmitted || isAmpmSubmitted(); return isTimeSubmitted; } -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira