Contact Form Issues

[whitebulb]

I was hoping you could help... I'm trying to get this form working on my site. It says "Contact form submitted! We will be in touch soon." But no one will be in touch soon as it never comes through! I've replaced my email address with [e-mail]

Many thanks for you time...

index.html

	<form action="#" id="ContactForm">
                                        <input type="hidden" name="owner_email" id="owner_email" value="[e-mail]" />
                        				<input type="hidden" name="serverProcessorType" id="serverProcessorType" value="php" />
                        				<input type="hidden" name="smtpMailServer" id="smtpMailServer" value="localhost" />
                        				<input type="hidden" name="stripHTML" id="stripHTML" value="true" />
                                    	<div class="success"> Contact form submitted!<br>
										<strong>We will be in touch soon.</strong></div>
										<fieldset>
											<div class="wrapper">
												<div class="col left">
													<div class="wrapper">
														<label class="name">
															<input type="text" value="Name:" class="input">
															<span class="error">*This is not a valid name.</span> <span class="empty">*This field is required.</span> 
														</label>
													</div>
												</div>
												<div class="col right">
													<div class="wrapper">
														<label class="email">
															<input type="text" value="E-mail:" class="input">
															<span class="error">*This is not a valid email address.</span> <span class="empty">*This field is required.</span> 
														</label>
													</div>
												</div>
											</div>
											<div class="wrapper">
												<label class="message">
													<textarea rows="1" cols="1">Message:</textarea>
													<span class="error">*The message is too short.</span> <span class="empty">*This field is required.</span> 
												</label>
											</div>
											<div class="btns"><a href="#" class="button1" data-type="submit"><strong></strong><span>Submit</span></a><a href="#" class="button1" data-type="reset"><strong></strong><span>Clear</span></a></div>
										</fieldset>
									</form>

forms.js (have tried changing mailHandlerURL:'bin/MailHandler.php', to mailHandlerURL:'mail/MailHandler.php',)

(function($){
	$.fn.extend({
		forms:function(opt){
			if(opt===undefined)
				opt={}
			this.each(function(){
				var th=$(this),
					data=th.data('forms'),
					_={
						errorCl:'error',
						emptyCl:'empty',
						invalidCl:'invalid',
						successCl:'success',
						successShow:'4000',
						mailHandlerURL:'bin/MailHandler.php',
						ownerEmail:'[e-mail]',
						stripHTML:'true',
						smtpMailServer:'localhost',
						targets:'input,textarea',
						controls:'a[data-type=reset],a[data-type=submit]',
						validate:true,
						rx:{
							".name":{rx:/^[a-zA-Z'][a-zA-Z-' ]+[a-zA-Z']?$/,target:'input'},
							".state":{rx:/^[a-zA-Z'][a-zA-Z-' ]+[a-zA-Z']?$/,target:'input'},
							".email":{rx:/^(("[\w-\s]+")|([\w-]+(?:\.[\w-]+)*)|("[\w-\s]+")([\w-]+(?:\.[\w-]+)*))(@((?:[\w-]+\.)*\w[\w-]{0,66})\.([a-z]{2,6}(?:\.[a-z]{2})?)$)|(@\[?((25[0-5]\.|2[0-4][0-9]\.|1[0-9]{2}\.|[0-9]{1,2}\.))((25[0-5]|2[0-4][0-9]|1[0-9]{2}|[0-9]{1,2})\.){2}(25[0-5]|2[0-4][0-9]|1[0-9]{2}|[0-9]{1,2})\]?$)/i,target:'input'},
							".phone":{rx:/^\+?(\d[\d\-\+\(\) ]{5,}\d$)/,target:'input'},
							".fax":{rx:/^\+?(\d[\d\-\+\(\) ]{5,}\d$)/,target:'input'},
							".message":{rx:/.{20}/,target:'textarea'}
						},
						preFu:function(){
							_.labels.each(function(){
								var label=$(this),
									inp=$(_.targets,this),
									defVal=inp.attr('value'),
									trueVal=(function(){
												var tmp=inp.is('input')?(tmp=label.html().match(/value=['"](.+?)['"].+/),!!tmp&&!!tmp[1]&&tmp[1]):inp.html()
												return tmp
											})()

							trueVal!=defVal
								&&inp.val(defVal=trueVal||defVal)
							label.data({defVal:defVal})								
							inp
								.bind('focus',function(){
									inp.val()==defVal
										&&(inp.val(''),_.hideEmptyFu(label),label.removeClass(_.invalidCl))
								})
								.bind('blur',function(){
									!inp.val()
										?inp.val(defVal)										
										:(_.isValid(label)
											?_.showErrorFu(label)
											:_.hideErrorFu(label)),
										(_.isEmpty(label)
											?_.showEmptyFu(label)
											:_.hideEmptyFu(label))
								})
								.bind('keyup',function(){
									label.hasClass(_.invalidCl)
										&&_.isValid(label)
											?_.showErrorFu(label)
											:_.hideErrorFu(label)
								})
							label.find('.'+_.errorCl+',.'+_.emptyCl).css({display:'block'}).hide()
						})
						_.success=$('.'+_.successCl,_.form).hide()
					},
					isValid:function(el){
						var ret=true,
							empt=_.isEmpty(el)
						if(empt)
							ret=false,
							el.addClass(_.invalidCl)
						else
							$.each(_.rx,function(k,d){
								if(el.is(k))
									d.rx.test(el.find(d.target).val())
										?(el.removeClass(_.invalidCl),ret=false)
										:el.addClass(_.invalidCl)
							})
						return ret
					},
					isEmpty:function(el){
						var tmp
						return (tmp=el.find(_.targets).val())==''||tmp==el.data('defVal')
					},
					validateFu:function(){							
						_.labels.each(function(){
							var th=$(this)								
							_.isEmpty(th)
								?_.showEmptyFu(th)
								:_.hideEmptyFu(th)
							_.isValid(th)
								?_.showErrorFu(th)
								:_.hideErrorFu(th)
						})
					},
					submitFu:function(){
						_.validateFu()
						if(!_.form.has('.'+_.invalidCl).length)
							$.ajax({
								type: "POST",
								url:_.mailHandlerURL,
								data:{
									name:$('.name input',_.form).val()||'nope',
									email:$('.email input',_.form).val()||'nope',
									phone:$('.phone input',_.form).val()||'nope',
									fax:$('.fax input',_.form).val()||'nope',
									state:$('.state input',_.form).val()||'nope',
									message:$('.message textarea',_.form).val()||'nope',
									ownerEmail:_.ownerEmail,
									stripHTML:_.stripHTML
								},
								success: function(){
									_.showFu()
								}
							})			
					},
					showFu:function(){
						_.success.slideDown(function(){
							setTimeout(function(){
								_.success.slideUp()
								_.form.trigger('reset')
							},_.successShow)
						})
					},
					controlsFu:function(){
						$(_.controls,_.form).each(function(){
							var th=$(this)
							th
								.bind('click',function(){
									_.form.trigger(th.data('type'))
									return false
								})
						})
					},
					showErrorFu:function(label){
						label.find('.'+_.errorCl).slideDown()
					},
					hideErrorFu:function(label){
						label.find('.'+_.errorCl).slideUp()
					},
					showEmptyFu:function(label){
						label.find('.'+_.emptyCl).slideDown()
						_.hideErrorFu(label)
					},
					hideEmptyFu:function(label){
						label.find('.'+_.emptyCl).slideUp()
					},
					init:function(){
						_.form=this
						_.labels=$('label',_.form)

						_.preFu()

						_.controlsFu()

						_.form
							.bind('submit',function(){
								if(_.validate)
									_.submitFu()
								else
									_.form[0].submit()
								return false
							})
							.bind('reset',function(){
								_.labels.removeClass(_.invalidCl)									
								_.labels.each(function(){
									var th=$(this)
									_.hideErrorFu(th)
									_.hideEmptyFu(th)
								})
							})
						_.form.trigger('reset')
					}
				}
			if(!data)
				(typeof opt=='object'?$.extend(_,opt):_).init.call(th),
				th.data({cScroll:_}),
				data=_
			else
				_=typeof opt=='object'?$.extend(data,opt):data
		})
		return this
	}
})
})(jQuery)
$(window).load(function() {	
	$('#ContactForm').forms({
		ownerEmail:'#'
	})
})

MOD EDIT: [noparse]

[/noparse] bbcode tags changed to [noparse][code][/noparse] tags; JavaScript != PHP. ;)

Continued below...

[bradgrafelman]

whitebulb;10995219 wrote:

forms.js (have tried changing mailHandlerURL:'bin/MailHandler.php', to mailHandlerURL:'mail/MailHandler.php',)

Why would you do that? Which one is the correct path to this 'MailHandler.php' script, and why would you bother trying to use the other (presumably invalid) path?

Also, what does this MailHandler.php file look like?

[whitebulb]

MailHandler.php

<?php
	$owner_email = $_POST["owner_email"];
	$headers = 'From:' . $_POST["email"];
	$subject = 'A message from your site visitor ' . $_POST["name"];
	$messageBody = "";

$messageBody .= '<p>Visitor: ' . $_POST["name"] . '</p>' . "\n";
$messageBody .= '<br>' . "\n";
$messageBody .= '<p>Email Address: ' . $_POST['email'] . '</p>' . "\n";
$messageBody .= '<br>' . "\n";
if($_POST['state']!='nope'){		
	$messageBody .= '<p>State: ' . $_POST['state'] . '</p>' . "\n";
	$messageBody .= '<br>' . "\n";
}
if($_POST['phone']!='nope'){		
	$messageBody .= '<p>Phone Number: ' . $_POST['phone'] . '</p>' . "\n";
	$messageBody .= '<br>' . "\n";
}	
if($_POST['fax']!='nope'){		
	$messageBody .= '<p>Fax Number: ' . $_POST['fax'] . '</p>' . "\n";
	$messageBody .= '<br>' . "\n";
}
$messageBody .= '<p>Message: ' . $_POST['message'] . '</p>' . "\n";

if($_POST["stripHTML"] == 'true'){





$messageBody = strip_tags($messageBody);





$messageBody = myStripSlashes($messageBody);



}

function myStripSlashes($data) {
    if (get_magic_quotes_gpc()) return stripslashes($data);
    return $data;
}

try{
	if(!mail($owner_email, $subject, $messageBody, $headers)){
		throw new Exception('mail failed');
	}else{
		echo 'mail sent';
	}
}catch(Exception $e){
	echo $e->getMessage() ."\n";
}
?>

MailHandler.ashx

<%@ WebHandler Language="C#" Class="Handler" Debug="true" %>

using System;
using System.Web;
using System.Net.Mail;
using System.Text.RegularExpressions;

public class Handler : IHttpHandler {
	public void ProcessRequest (HttpContext context) {
		SmtpClient mailClient = new SmtpClient(context.Request.Form.Get("smtpMailServer"));
		string owner_email = context.Request.Form.Get("owner_email");
		string subject = "A message from your site visitor " + context.Request.Form.Get("name");
		string email = context.Request.Form.Get("email");
		string messageBody = "";

	messageBody += "<p>Visitor: " + context.Request.Form.Get("name") + "</p>\n";
	messageBody += "<br>\n";
	messageBody += "<p>Email Address: " + context.Request.Form.Get("email") + "</p>\n";
	messageBody += "<br>\n";
	messageBody += "<p>Phone Number: " + context.Request.Form.Get("phone") + "</p>\n";
	messageBody += "<br>\n";
	messageBody += "<p>Message: " + context.Request.Form.Get("message") + "</p>\n";


	MailMessage message = new MailMessage();

	try{
		message.From = new MailAddress(email.ToString());
	}catch (FormatException e) {
		context.Response.Write(e.Message);
	}

	message.To.Add(owner_email);
	message.Subject = subject;
	if(context.Request.Form.Get("stripHTML") == "true"){
		message.IsBodyHtml = false;
        messageBody = Regex.Replace(messageBody, "<.*?>", string.Empty);
	}else{
	  	message.IsBodyHtml = true;
	}
	message.Body = messageBody;

	try{
		mailClient.Send(message);
	}catch (SmtpException e) {
		context.Response.Write("mail failed");
	}
	context.Response.Write("mail sent");
}

public bool IsReusable {
	get	{
		return false;
	}
}
}

[whitebulb]

bradgrafelman;10995226 wrote:

Why would you do that? Which one is the correct path to this 'MailHandler.php' script, and why would you bother trying to use the other (presumably invalid) path?

Also, what does this MailHandler.php file look like?

Hi Bradgrafelman, thanks for your reply... I couldn't post the rest due to character limitation and as I've just signed up, I was unable to reply to the thread.

I searched for a similar issue on this site and the advice was to try it in "mail/" rather than "bin/" - well no such luck. It's now back in "bin/" and "mail/" has been deleted...

Many thanks,