it is the @Parameter validate in RadioGroup that is required not @Validate
as that is never checked in case @Parameter validate is not given in a
RadioGroup ... Please read my answer again :)

It seems to be bug!!

regards
Taha

On Wed, Apr 13, 2011 at 3:40 PM, Adam Zimowski <zimowsk...@gmail.com> wrote:

> Thanks Taha, but I did supply @Validate("required") annotation and
> validation did not happen.
>
> Adam
>
> On Tue, Apr 12, 2011 at 8:15 PM, Taha Hafeez <tawus.tapes...@gmail.com>
> wrote:
> > Hi
> >
> > I compared the code from RadioGroup with that of Select and found the
> > following difference
> >
> > In Select if parameter 'validate' is not given a default is chosen and so
> it
> > is never null. Validation is performed
> > in processSubmission() method by the line
> >
> > fieldValidationSupport.validate(selectedValue, resources, validate);
> >
> >
> > In RadioGroup however, there is no defaultValidate(){} and so validate
> can
> > be null. Now the validation here is done by the line
> >
> > if (validate != null)
> >   fieldValidationSupport.validate(rawValue, resources, validate);
> >
> > so when the validate parameter is not supplied, validation does not
> happen.
> >
> > regards
> > Taha
> >
> >
> > On Wed, Apr 13, 2011 at 2:19 AM, Adam Zimowski <zimowsk...@gmail.com>
> wrote:
> >
> >> I have a bean where @Validate("required") does not work on a
> >> radiogroup, but <t:radiogroup validate="required" ...   does. Other
> >> @Validate annotated fields (TextField's) work. My companyType is
> >> purposely set to null by default, as I do not want any radio
> >> pre-selected, thus it shall be required.
> >>
> >> Is @Validate in the context I have defined supposed to work on the
> >> radio group as well?
> >>
> >> /**
> >>  * @author Adam Zimowski
> >>  */
> >> public class RegisterUiBean {
> >>
> >>        @Validate("required")
> >>        private String email;
> >>
> >>        @Validate("required")
> >>        private String emailRetype;
> >>
> >>        @Validate("required")
> >>        private String password;
> >>
> >>        @Validate("required")
> >>        private String passwordRetype;
> >>
> >>        @Validate("required")
> >>        private CompanyType companyType;
> >>
> >>
> >>        public RegisterUiBean() {
> >>        }
> >>
> >>        public String getEmail() {
> >>                return email;
> >>        }
> >>
> >>
> >>        public void setEmail(String aEmail) {
> >>                email = aEmail;
> >>        }
> >>
> >>
> >>        public String getEmailRetype() {
> >>                return emailRetype;
> >>        }
> >>
> >>
> >>        public void setEmailRetype(String aEmailRetype) {
> >>                emailRetype = aEmailRetype;
> >>        }
> >>
> >>
> >>        public String getPassword() {
> >>                return password;
> >>        }
> >>
> >>
> >>        public void setPassword(String aPassword) {
> >>                password = aPassword;
> >>        }
> >>
> >>
> >>        public String getPasswordRetype() {
> >>                return passwordRetype;
> >>        }
> >>
> >>
> >>        public void setPasswordRetype(String aPasswordRetype) {
> >>                passwordRetype = aPasswordRetype;
> >>        }
> >>
> >>
> >>        public CompanyType getCompanyType() {
> >>                return companyType;
> >>        }
> >>
> >>
> >>        public void setCompanyType(CompanyType aCompanyType) {
> >>                companyType = aCompanyType;
> >>        }
> >>
> >>        public CompanyType getCorporation() {
> >>                return CompanyType.Corporation;
> >>        }
> >>
> >>        public CompanyType getFederalGov() {
> >>                return CompanyType.FederalGovernment;
> >>        }
> >>
> >>        public CompanyType getStateGov() {
> >>                return CompanyType.StateGovernment;
> >>        }
> >>
> >>        public CompanyType getIndividual() {
> >>                return CompanyType.Individual;
> >>        }
> >> }
> >>
> >>  <div class="kk-hdr">Registration Information</div>
> >>  <div class="kk-row">
> >>  <div class="kk-label"><t:label for="r_email1"/> :</div>
> >>  <div class="kk-field"><t:textfield t:id="r_email1"
> >> value="registration.email"/></div>
> >>  <t:error class="literal:kk-error" for="r_email1"/>
> >>  </div>
> >>  <div class="kk-row">
> >>  <div class="kk-label"><t:label for="r_email2"/> :</div>
> >>  <div class="kk-field"><t:textfield t:id="r_email2"
> >> value="registration.emailRetype"/></div>
> >>  <t:error class="literal:kk-error" for="r_email2"/>
> >>  </div>
> >>  <div class="kk-row">
> >>  <div class="kk-label"><t:label for="r_pass1"/> :</div>
> >>  <div class="kk-field"><t:textfield t:id="r_pass1"
> >> value="registration.password"/></div>
> >>  <t:error class="literal:kk-error" for="r_pass1"/>
> >>  </div>
> >>  <div class="kk-row">
> >>  <div class="kk-label"><t:label for="r_pass2"/> :</div>
> >>  <div class="kk-field"><t:textfield t:id="r_pass2"
> >> value="registration.passwordRetype"/></div>
> >>  <t:error class="literal:kk-error" for="r_pass2"/>
> >>  </div>
> >>  <div class="kk-row">
> >>  <div class="kk-label"><t:label for="r_type"/> :</div>
> >>  <div class="kk-field">
> >>  <t:radiogroup t:id="r_type" value="registration.companyType"
> >> validate="required">
> >>        <t:radio t:id="corporation" value="registration.corporation"/>
> >>        <t:label for="corporation"/>
> >>        <t:radio t:id="federalGovernment"
> value="registration.federalGov"/>
> >>        <t:label for="federalGovernment"/>
> >>        <t:radio t:id="stateGovernment" value="registration.stateGov"/>
> >>        <t:label for="stateGovernment"/>
> >>        <t:radio t:id="individual" value="registration.individual"/>
> >>        <t:label for="individual"/>
> >>  </t:radiogroup>
> >>  </div>
> >>
> >> Adam
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
> >> For additional commands, e-mail: users-h...@tapestry.apache.org
> >>
> >>
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
> For additional commands, e-mail: users-h...@tapestry.apache.org
>
>

Reply via email to