Jump to content

jhkoning

New Members
  • Posts

    3
  • Joined

  • Last visited

jhkoning's Achievements

Newbie

Newbie (1/14)

0

Reputation

  1. Thanks for the feedback jlhaslip, I knew the security was lacking, I was going to deal with that after I got the thing actually working correctly. I will (as you say) Google it. Cheers Jas
  2. Apologies, here we go; > </pre> <table width="100%" border="0" cellspacing="0" cellpadding="0">* Name: (This name will appear on your invoice).Telephone:Cellphone:* E-mail:* Delivery Address:Postcode:</table> <br><table width="100%" border="0" cellspacing="0" cellpadding="0">Regional Series Size:600mm wide x 1000mm high $490900mm wide x 1500mm high $690</table> <br><table width="100%" border="0" cellspacing="0" cellpadding="0">North IslandAucklandHawkes Bay/Bay of Plenty/GisborneNorthlandWellingtonTaranaki/Manawatu/WanganuiWaikato South IslandCanterburyTasman/Nelson/MarlboroughWest CoastOtagoCentral OtagoSouthland New Zealand </table> <br><table width="100%" border="0" cellspacing="0" cellpadding="0">Personalised Size:600mm wide x 1000mm high $5901000mm wide x 1500mm high $790Please note down approximately 16 names (i.e. place names, roads, streets, mountains, lakes, beaches, rivers etc.) Try to vary the length of the words to create more interest. We choose the 12 names that fit best in the design so please underline the important ones that you don?t want missed out of the design & we will endeavour to fit them in. Special Instructions</table> <br
  3. <?php if (isset($_POST['submit'])) { $to = "someone@somewhere.com"; $subject = "Website Order"; $name_field = $_POST['name']; $phone_field = $_POST['phone']; $mobile_field = $_POST['mobile']; $email_field = $_POST['email']; $address = $_POST['address']; $postcode_field = $_POST['postcode']; foreach ($_POST['regionalSeries'] as $value) { $radio_msg .= "Regional Series: $value"; } if (isset($_POST[?personalised?])) { foreach ($_POST['personalised'] as $value) { $radio_msg2 .= "Regional Series: $value\n"; } } foreach ($_POST['locations'] as $value) { $check_msg .= "Locations: $value\n"; } //foreach ($_POST['personalised'] as $value) { // $radio_msg2 .= "Personalised: $value"; // } $description = $_POST['description']; $instructions = $_POST['instructions']; $body = "\nFrom: $name_field\nTelephone: $phone_field\nCellphone: $mobile_field\nE-Mail: $email_field\n\nDelivery Address: $address\nPostcode: $postcode_field\n\n$radio_msg\n$check_msg\n\n$radio_msg2\nPersonalised: $description\nSpecial Instructions: $instructions"; //echo "Data has been submitted to $to!"; header("Location: thank-you.html"); mail($to, $subject, $body); } else { echo "blarg!"; } ?> What I'm trying to do here is (well, probably obvious) - just not to me. Any advice on how to make this better? Error correction. Hmmm... Security? What's that? Basically, it works (well, the $radio_msg2 doesn't display in the email) so I guess it doesn't really. I'm just looking for some pointers, to improve the way the form data is handled. If this isn't the right place to post such a thing - I apologise in advance. Thanks for looking. Jas
×
×
  • Create New...