onsubmit if checkbox is checked 1

[evr72]

Hello,

I have a script that when you click the submit button it opens outlook and sends an email. I have added a checkbox to the form, what I am trying to accomplish is if the checkbox is checked then run the open email function
here is what I have done so far

<input type="submit" onClick="openoutlook()" value="Insert Record" >

<script type=text/javascript>
<-- I added this if statement
if (document.form.rmacomplete.checked == true)


function openoutlook()
{
		
  var L=document.form.CMNumber.value;
  var S=document.form.company.value;
  var B=document.form.email.value;
  var M=document.form.credupdatedby.value;
  var A= <% = Cdbl((rsGuestbook("ID")))%>
  var url = '[URL unfurl="true"]http://mywebsite.com/a1/view.asp?id='[/URL] + escape(A);

  
 self.location="mailto:someemail@mycompany.com?subject=?Record Number  "+A+" Has Been Updated by "+M+" &body= Here is the link to the updated Record   "+url+"   The Reference is "+L ;
}
</script>

but my if statement does not seem to be working, I also tried to add the onclick to the submit

 

[vacunita]

That's not valid syntax. You can't Condition the definition of a function.

Why: because the function needs to exist for it to be called. And even then since your If has no curly braces, it only affects the line immediately after it. Your definition is 2 or 3 lines further down which means it is unaffected.

If that actually worked you'd get an error because your script would be trying to call an undefined function.

You can do 2 things here:

1. Place the condition inside the function so you Condition the actions it takes.

function openoutlook()
{
 
[red]       
if (document.form.rmacomplete.checked == true)
{
[/red]
  var L=document.form.CMNumber.value;
  var S=document.form.company.value;
  var B=document.form.email.value;
  var M=document.form.credupdatedby.value;
  var A= <% = Cdbl((rsGuestbook("ID")))%>
  var url = '[URL unfurl="true"]http://mywebsite.com/a1/view.asp?id='[/URL] + escape(A);

  
 self.location="mailto:someemail@mycompany.com?subject=?Record Number  "+A+" Has Been Updated by "+M+" &body= Here is the link to the updated Record   "+url+"   The Reference is "+L ;
[red]}[/red]

}

Or

2. Condition the calling of the function.

<input type="submit" onClick="[red]if (document.form.rmacomplete.checked == true){[/red]openoutlook();[red]}[/red]" value="Insert Record" >

----------------------------------
Phil AKA Vacunita
----------------------------------
Ignorance is not necessarily Bliss, case in point:
Unknown has caused an Unknown Error on Unknown and must be shutdown to prevent damage to Unknown.

 

[evr72]

ohhh wo

http://wwwww

I spent almost all morning trying to figure this one out!!!
Vacunita!!!!! You rule!!! as always...

Thank you, I opted to go with the

<input type="submit" onClick="if (document.form.rmacomplete.checked == true){openoutlook();}" value="Insert Record" >

I tried to do something similar but was missing the {} brackets
Again, Thank you!!!!

 

[BillyRayPreachersSon]

You can't Condition the definition of a function.

Yes you can, given the function is just another type... e.g

a = 1;
b = 1;
if (a == b) {
   wibble = function() {
      alert('a == b');
   }
} else {
   wibble = function() {
      alert('a != b');
   }
}
wibble();

Dan



Coedit Limited - Delivering standards compliant, accessible web solutions

Dan's Page [blue]@[/blue] Code Couch:

http://www.codecouch.com/dan/

Code Couch Tech Snippets & Info:

http://www.codecouch.com/

 

[vacunita]

O.K. I stand corrected. You can condition the definition of a function.
With that said the rest of my post remains valid.

That is without an alternate definition for the function you get an error when trying to call it.

----------------------------------
Phil AKA Vacunita
----------------------------------
Ignorance is not necessarily Bliss, case in point:
Unknown has caused an Unknown Error on Unknown and must be shutdown to prevent damage to Unknown.

 

[feherke]

Hi

You can condition the definition of a function.
With that said the rest of my post remains valid.

Yes, excepting this :

And even then since your If has no curly braces, it only affects the line immediately after it. Your definition is 2 or 3 lines further down which means it is unaffected.

[tt]if[/tt] affects the next one instruction. That one instruction can be in the same line or after pages of nothing. ( Yes, one instruction. A block is actually one instruction containing others. )

So Dan's code will work with abit of changes like this :

a [teal]=[/teal] [purple]1[/purple][teal];[/teal]
b [teal]=[/teal] [purple]1[/purple][teal];[/teal]

[b]if[/b] [teal]([/teal]a [teal]==[/teal] b[teal])[/teal]

[gray]// empty lines and comments are ignored by the interpreter[/gray]

   wibble [teal]=[/teal] [b]function[/b][teal]()[/teal] [teal]{[/teal]
      [COLOR=darkgoldenrod]alert[/color][teal]([/teal][green][i]'a == b'[/i][/green][teal]);[/teal]
   [teal]}[/teal]

[gray]// empty lines and comments are ignored by the interpreter[/gray]

[b]else[/b]

[gray]// empty lines and comments are ignored by the interpreter[/gray]

   wibble [teal]=[/teal] [b]function[/b][teal]()[/teal] [teal]{[/teal]
      [COLOR=darkgoldenrod]alert[/color][teal]([/teal][green][i]'a != b'[/i][/green][teal]);[/teal]
   [teal]}[/teal]

[COLOR=darkgoldenrod]wibble[/color][teal]();[/teal]

( No, I do not suggest to write unreadable & unmaintainable code like above. )

Feherke.

http://free.rootshell.be/~feherke/