hey I created ADFFACES-238 to keep track of it and we should have issues in jira for almost all commits.
Since you agreed to this issue, I commit the change to the template tomorrow or so On 10/13/06, Arjuna Wijeyekoon <[EMAIL PROTECTED]> wrote:
I think you're right. I could have sworn that we were special-casing the required-validator; I even looked at the code in the old corporate repository, but this bug exists there. --arjuna On 10/13/06, Matthias Wessendorf <[EMAIL PROTECTED]> wrote: > > Hi > > please take a look at MYFACES-1467 which is also trure for > UIXEditableValue.java's validate() method. > > But the spec javadoc for validate() says: > Retrieve the submitted value with getSubmittedValue(). If this returns > null, exit without further processing. (This indicates that no value > was submitted for this component.) > > the patch is basicly doing this instead: > > Object submittedValue = getSubmittedValue(); > if (submittedValue == null && !this.isRequired()) return; > > (it add's the && !this.isRequired()) > > > Why? > See the descr. for the issue, since a man-in-the-middle tool can do > some funny things. I saw David's demo this afternoon in ApacheCon > Hackaton. > > I think the javadoc for jsf 1.1 and 1.2 should be changed... > > What do you think? > > -Matt > -- > Matthias Wessendorf > http://tinyurl.com/fmywh > > further stuff: > blog: http://jroller.com/page/mwessendorf > mail: mwessendorf-at-gmail-dot-com >
-- Matthias Wessendorf http://tinyurl.com/fmywh further stuff: blog: http://jroller.com/page/mwessendorf mail: mwessendorf-at-gmail-dot-com
