I think you need to have the submitForm() method return false if there's an error. Otherwise probably it won't get to popping up the alert before doing the submission.
Spike -------------------------------------------- Stephen Milligan Code poet for hire http://www.spike.org.uk Do you cfeclipse? http://cfeclipse.tigris.org >-----Original Message----- >From: >[EMAIL PROTECTED] >[mailto:[EMAIL PROTECTED] o.uk] On Behalf Of Paul Swingewood >Sent: Thursday, August 12, 2004 8:21 AM >To: [EMAIL PROTECTED] >Subject: Re: [ cf-dev ] js drives me mad > >Still doesn't work :( > > >>From: Andy Allan <[EMAIL PROTECTED]> >>Reply-To: <[EMAIL PROTECTED]> >>To: [EMAIL PROTECTED] >>Subject: Re: [ cf-dev ] js drives me mad >>Date: Thu, 12 Aug 2004 17:17:53 +0100 >> >>should be function submitForm() ... remember Javascript is >case sensitive. >> >>Andy >> >>Quoting Paul Swingewood <[EMAIL PROTECTED]>: >> >> > Now this is as simple as it gets... Yeah right ... >> > >> > <script language="JavaScript"> >> > <!-- >> > function submitform() { >> > if (document.addsubctgry.subcatpic.value == "") { >> > alert("You must enter a picture for this subcategory."); >> > } >> > // --> >> > </script> >> > >> > . >> > . >> > . >> > . >> > . >> > <cfform name="addsubctgry" action="adsbctac.cfm" >>onsubmit="submitForm();" >> > enctype="multipart/form-data"> >> > . >> > . >> > . >> > <input type="file" size="35" name="subcatpic"> >> > >> > Why doesn't this validate! - Aghhh! >> > >> > Regards - Paul >> > >> > >> > >> > -- >> > These lists are syncronised with the CFDeveloper forum at >> > http://forum.cfdeveloper.co.uk/ >> > Archive: http://www.mail-archive.com/dev%40lists.cfdeveloper.co.uk/ >> > >> > CFDeveloper Sponsors and contributors:- >> > *Hosting and support provided by CFMXhosting.co.uk* :: *ActivePDF >>provided by >> > activepdf.com* >> > *Forums provided by fusetalk.com* :: *ProWorkFlow provided by >> > proworkflow.com* >> > *Tutorials provided by helmguru.com* :: *Lists hosted by >> > gradwell.com* >> > >> > To unsubscribe, e-mail: [EMAIL PROTECTED] >> > >> >> >>-- >>[EMAIL PROTECTED] >>www.creative-restraint.co.uk >> >>-- >>These lists are syncronised with the CFDeveloper forum at >>http://forum.cfdeveloper.co.uk/ >>Archive: http://www.mail-archive.com/dev%40lists.cfdeveloper.co.uk/ >> >>CFDeveloper Sponsors and contributors:- >>*Hosting and support provided by CFMXhosting.co.uk* :: >*ActivePDF provided >>by activepdf.com* >> *Forums provided by fusetalk.com* :: *ProWorkFlow provided by >>proworkflow.com* >> *Tutorials provided by helmguru.com* :: *Lists hosted by >>gradwell.com* >> >>To unsubscribe, e-mail: [EMAIL PROTECTED] >> > > > >-- >These lists are syncronised with the CFDeveloper forum at >http://forum.cfdeveloper.co.uk/ >Archive: http://www.mail-archive.com/dev%40lists.cfdeveloper.co.uk/ > >CFDeveloper Sponsors and contributors:- >*Hosting and support provided by CFMXhosting.co.uk* :: >*ActivePDF provided by activepdf.com* > *Forums provided by fusetalk.com* :: *ProWorkFlow >provided by proworkflow.com* > *Tutorials provided by helmguru.com* :: *Lists >hosted by gradwell.com* > >To unsubscribe, e-mail: [EMAIL PROTECTED] > -- These lists are syncronised with the CFDeveloper forum at http://forum.cfdeveloper.co.uk/ Archive: http://www.mail-archive.com/dev%40lists.cfdeveloper.co.uk/ CFDeveloper Sponsors and contributors:- *Hosting and support provided by CFMXhosting.co.uk* :: *ActivePDF provided by activepdf.com* *Forums provided by fusetalk.com* :: *ProWorkFlow provided by proworkflow.com* *Tutorials provided by helmguru.com* :: *Lists hosted by gradwell.com* To unsubscribe, e-mail: [EMAIL PROTECTED]
