Don't invoke FormSupport.addValidation() unless you have client-side validation for your validator. The name of the method, and documentation, could be better.
On Sat, Nov 15, 2008 at 6:03 PM, Geoff Callender <[EMAIL PROTECTED]> wrote: > I haven't created client-side validation for this translator, but that > hasn't previously been a problem. How do I stop it being a problem now? > > On 16/11/2008, at 12:59 PM, Howard Lewis Ship wrote: > >> Looks like user error to me ... I only see the base set of JavaScript >> files being loaded; where's you @IncludeJavaScriptLibrary annotation? >> You can't expect the code to exist if its never been loaded! >> >> On Sat, Nov 15, 2008 at 4:45 PM, Geoff Callender >> <[EMAIL PROTECTED]> wrote: >>> >>> That's fixed the email validator. But what must I do to fix my custom >>> translator? >>> You can see the problem in action here: >>> >>> >>> >>> http://jumpstart.doublenegative.com.au:8180/jumpstart/examples/input/creatingtranslators1 >>> >>> or if that site is not up try here: >>> >>> >>> >>> http://jumpstart.doublenegative.com.au:8080/jumpstart/examples/input/creatingtranslators1 >>> >>> Cheers, >>> Geoff >>> >>> On 16/11/2008, at 8:09 AM, Howard Lewis Ship wrote: >>> >>>> OK, I've updated the maven artifacts, the site, and the downloads to >>>> incude the TAP5-359 fix (the client-side warning about email >>>> validation). >>>> >>>> On Sat, Nov 15, 2008 at 9:56 AM, Howard Lewis Ship <[EMAIL PROTECTED]> >>>> wrote: >>>>> >>>>> Ok, I'm producing a new 5.0.16 build now. >>>>> >>>>> >>>>> On Sat, Nov 15, 2008 at 9:12 AM, Howard Lewis Ship <[EMAIL PROTECTED]> >>>>> wrote: >>>>>> >>>>>> You are correct, looking at the code, there's an errant call to setup >>>>>> client-side validation, even though there is no client-side validator. >>>>>> >>>>>> Since the release is not yet public, I'll correct that and build a new >>>>>> 5.0.16. >>>>>> >>>>>> >>>>>> On Thu, Nov 13, 2008 at 5:49 PM, Geoff Callender <[EMAIL PROTECTED]> >>>>>> wrote: >>>>>>> >>>>>>> -1 (non-binding) >>>>>>> I must change my vote until someone confirms there is not a >>>>>>> validators >>>>>>> problem. >>>>>>> >>>>>>> Can someone please try an input field with a required, email >>>>>>> validator >>>>>>> and >>>>>>> client-side validation turned on and confirm whether they, too, get a >>>>>>> red >>>>>>> error overlaying the top of the page saying something like "Function >>>>>>> Tapestry.Validator.email() does not exist for field >>>>>>> 'myEmailAddress'." >>>>>>> >>>>>>> It's so distracting and clearly unsuitable for production. >>>>>>> >>>>>>> Regards, >>>>>>> Geoff >>>>>>> >>>>>>> >>>>>>> >>>>>>> Geoff Callender-3 wrote: >>>>>>>> >>>>>>>> +0 (non-binding) >>>>>>>> >>>>>>>> It's passed all of my tests but I'm seeing red messages at the top >>>>>>>> of >>>>>>>> some screens, eg. >>>>>>>> >>>>>>>> Function Tapestry.Validator.yesno() does not exist for field >>>>>>>> 'newToTapestry'. >>>>>>>> >>>>>>>> I'm seeing it on the following page after upgrade to 5.0.16: >>>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> http://jumpstart.doublenegative.com.au:8080/jumpstart/examples/input/creatingtranslators1 >>>>>>>> >>>>>>>> Does this just require a simple code change at my end? >>>>>>>> >>>>>>>> Regards, >>>>>>>> Geoff >>>>>>>> >>>>>>>> On 11/11/2008, at 7:46 PM, Ted Steen wrote: >>>>>>>> >>>>>>>>> Ted Steen: +1 (non-binding) >>>>>>>>> >>>>>>>>> 2008/11/11 David Solis <[EMAIL PROTECTED]>: >>>>>>>>>> >>>>>>>>>> David Solis: +1 >>>>>>>>>> >>>>>>>>>> On Mon, Nov 10, 2008 at 6:55 PM, Howard Lewis Ship >>>>>>>>>> <[EMAIL PROTECTED]> wrote: >>>>>>>>>>> >>>>>>>>>>> I've created and uploaded a release of Tapestry 5.0.16, ready to >>>>>>>>>>> be voted upon. >>>>>>>>>>> >>>>>>>>>>> It's been two months since 5.0.15, and I've fixed over 80 bugs. >>>>>>>>>>> I >>>>>>>>>>> think Tapestry 5.0.16 is of release candidate quality and it >>>>>>>>>>> should be >>>>>>>>>>> made publically available to verify its readiness to be the GA >>>>>>>>>>> release >>>>>>>>>>> as well. >>>>>>>>>>> >>>>>>>>>>> The files are uploaded to: >>>>>>>>>>> >>>>>>>>>>> http://people.apache.org/~hlship/tapestry-releases/ >>>>>>>>>>> >>>>>>>>>>> and a Maven repository: >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> http://people.apache.org/~hlship/tapestry-ibiblio-rsynch-repository/ >>>>>>>>>>> >>>>>>>>>>> Please examine these files to determine if a new preview release, >>>>>>>>>>> 5.0.16, is ready, and of release candidate quality. >>>>>>>>>>> >>>>>>>>>>> I've also created a 5.0.16 tag in Subversion: >>>>>>>>>>> >>>>>>>>>>> http://svn.apache.org/viewvc/tapestry/tapestry5/tags/releases/ >>>>>>>>>>> 5.0.16/ >>>>>>>>>>> >>>>>>>>>>> On a successful vote, I'll move the files from these directories >>>>>>>>>>> to >>>>>>>>>>> the proper distribution directories. >>>>>>>>>>> >>>>>>>>>>> Vote will run for three days; on success I'll move the voted >>>>>>>>>>> artifacts >>>>>>>>>>> into place and send out appropriate notifications. >>>>>>>>>>> >>>>>>>>>>> -- >>>>>>>>>>> Howard M. Lewis Ship >>>>>>>>>>> >>>>>>>>>>> Creator Apache Tapestry and Apache HiveMind >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> --------------------------------------------------------------------- >>>>>>>>>>> To unsubscribe, e-mail: [EMAIL PROTECTED] >>>>>>>>>>> For additional commands, e-mail: [EMAIL PROTECTED] >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> --------------------------------------------------------------------- >>>>>>>>>> To unsubscribe, e-mail: [EMAIL PROTECTED] >>>>>>>>>> For additional commands, e-mail: [EMAIL PROTECTED] >>>>>>>>>> >>>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>> --------------------------------------------------------------------- >>>>>>>>> To unsubscribe, e-mail: [EMAIL PROTECTED] >>>>>>>>> For additional commands, e-mail: [EMAIL PROTECTED] >>>>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> --------------------------------------------------------------------- >>>>>>>> To unsubscribe, e-mail: [EMAIL PROTECTED] >>>>>>>> For additional commands, e-mail: [EMAIL PROTECTED] >>>>>>>> >>>>>>>> >>>>>>>> >>>>>>> >>>>>>> -- >>>>>>> View this message in context: >>>>>>> >>>>>>> http://n2.nabble.com/-VOTE--Release-5.0.16----Release-Candidate-tp1483148p1496710.html >>>>>>> Sent from the Tapestry Developers mailing list archive at Nabble.com. >>>>>>> >>>>>>> >>>>>>> --------------------------------------------------------------------- >>>>>>> To unsubscribe, e-mail: [EMAIL PROTECTED] >>>>>>> For additional commands, e-mail: [EMAIL PROTECTED] >>>>>>> >>>>>>> >>>>>> >>>>>> >>>>>> >>>>>> -- >>>>>> Howard M. Lewis Ship >>>>>> >>>>>> Creator Apache Tapestry and Apache HiveMind >>>>>> >>>>> >>>>> >>>>> >>>>> -- >>>>> Howard M. Lewis Ship >>>>> >>>>> Creator Apache Tapestry and Apache HiveMind >>>>> >>>> >>>> >>>> >>>> -- >>>> Howard M. Lewis Ship >>>> >>>> Creator Apache Tapestry and Apache HiveMind >>>> >>>> --------------------------------------------------------------------- >>>> To unsubscribe, e-mail: [EMAIL PROTECTED] >>>> For additional commands, e-mail: [EMAIL PROTECTED] >>>> >>> >>> >>> --------------------------------------------------------------------- >>> To unsubscribe, e-mail: [EMAIL PROTECTED] >>> For additional commands, e-mail: [EMAIL PROTECTED] >>> >>> >> >> >> >> -- >> Howard M. Lewis Ship >> >> Creator Apache Tapestry and Apache HiveMind >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [EMAIL PROTECTED] >> For additional commands, e-mail: [EMAIL PROTECTED] >> > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > -- Howard M. Lewis Ship Creator Apache Tapestry and Apache HiveMind --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
