Page 1 of 1

Footer not showing after making code changes

Posted: Wed Feb 20, 2008 9:07 am
by ChrisM
Hi, I am very new to PHP and am having some difficulties with some script as part of a contact page.
The page concerned can be found here. http://ufo-design.co.uk/dev2/sendemail.php

The problem I am having is the footer of my website is not showing due to the page getting broken up
somewhere. If I view source I see the following but then nothing after. I have looked hard and cannot find what is causing it.
If anyone has any advice I would be most grateful. I think the problem is somewhere between line 90 and 130.

Thanks in advance.

Code: Select all

<div class="side_content2"> 
      <span class="headings">Possibilities</span><br> We can design anything from a three page information site to a full ecommerce/shopping cart website suitable for small or large business. <p><span class="headings">No Templates</span><br> We do not use templates, every site is unique in layout and design. It is important to us to provide our clients with a web site they are proud of that looks unique and functions well. </p>    </div>
    <p><img src="images/options.gif" alt="Options" width="154" height="49" /></p>
  <!-- end #sidebar2 --></div>
    <div class="mainContent_ok"><br>
 
    <!-- Reminder: Add the link for the 'next page' (at the bottom) -->
    <!-- Reminder: Change 'YourEmail' to Your real email -->
  
Please use the back button and fill in all fields
Use back button on browser! !
Code of page sendmail.php is as follows.

Code: Select all

 
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
<title>Website Design - East Sussex - UFO Website Design Company</title>
<!--[if IE 5]>
<style type="text/css"> 
/* place css box model fixes for IE 5* in this conditional comment */
.thrColFixHdr #sidebar1 { width: 180px; }
.thrColFixHdr #sidebar2 { width: 190px; }
</style>
<![endif]-->
<!--[if IE]>
<style type="text/css"> 
/* place css fixes for all versions of IE in this conditional comment */
.thrColFixHdr #sidebar2, .thrColFixHdr #sidebar1 { padding-top: 30px; }
.thrColFixHdr #mainContent { zoom: 1; }
/* the above proprietary zoom property gives IE the hasLayout it needs to avoid several bugs */
</style>
<![endif]-->
<link href="css/ufo.css" rel="stylesheet" type="text/css" />
<meta name="keywords" content="Website design,web site design ,east sussex,west sussex,graphic design,web developers,development,internet,Hailsham, Eastbourne,Heathfield,Hastings,Tonbridge, Flash Design,banner advert design">
 
<style type="text/css">
<!--
.headings {
    color: #E67E1A;
    font-weight: bold;
}
.side1_content_img {
    padding-left: 5px;
    padding-bottom: 5px;
}
a:link {
    color: #0066CC;
    text-decoration: none;
}
a:visited {
    text-decoration: none;
    color: #0066CC;
}
a:hover {
    text-decoration: underline;
    color: #CC6600;
}
a:active {
    text-decoration: none;
    color: #CC6600;
}
a {
    font-size: 9pt;
}
-->
</style>
</head>
 
<body class="thrColFixHdr">
 
<div id="container">
  <div id="header">
    <div class="image_head"><a href="index.php"><img src="rotate.php" alt="" border="0" /></a></div>
   
  <!-- end #header --></div>
  <div id="sidebar1">
    <div class="side1_content">
      <?php include("left_content.html"); ?>
    </div>
 
  <!-- end #sidebar1 --></div>
  <div id="sidebar2">
    <div class="side2_menu">
      <?php include("menu.html"); ?>
    </div>
    <div class="side_content2"> 
      <?php include("news.php"); ?>
    </div>
    <p><img src="images/options.gif" alt="Options" width="154" height="49" /></p>
  <!-- end #sidebar2 --></div>
    <div class="mainContent_ok"><br>
<?php
 
$ip = $_POST['ip'];
$httpref = $_POST['httpref'];
$httpagent = $_POST['httpagent'];
$visitor = $_POST['visitor'];
$visitormail = $_POST['visitormail'];
$notes = $_POST['notes'];
$attn = $_POST['attn'];
 
if(eregi("to:",$visitormail) || eregi("cc:",$visitormail)) {
die ("NOT VALID ");
}
 
if (eregi('http:', $notes)) {
die ("Do not use any URL in any text fields! </br> Click the back button on your browser to continue! ");
}
if(!$visitormail == "" && (!strstr($visitormail,"@") || !strstr($visitormail,".")))
{
echo "Please use the back button - Enter valid e-mail address\n";
$badinput = "Form was NOT submitted\n";
echo $badinput;
die ("Use back button on browser! ! ");
}
 
if(empty($visitor) || empty($visitormail) || empty($notes )) {
echo "Please use the back button and fill in all fields\n";
die ("Use back button on browser! ! ");
}
 
$todayis = date("l, F j, Y, g:i a") ;
 
$attn = $attn ;
$subject = $attn;
 
$notes = stripcslashes($notes);
 
$message = " $todayis [EST] \n
Attention: $attn \n
Message: $notes \n
From: $visitor ($visitormail)\n
Additional Info : IP = $ip \n
Browser Info: $httpagent \n
Referral : $httpref \n
";
 
$from = "From: $visitormail\r\n";
 
 
mail("*********@ufo-design.co.uk", $subject, $message, $from);
 
?>
  <span class="headings">Thank you for your message</span>
    <p align="left"> Date: 
      <?php echo $todayis ?>
      <br>
      <br />
      Thank You : 
      <?php echo $visitor ?>
      ( 
      <?php echo $visitormail ?>
      ) <br>
      <br>
      Your message for the attention of: 
      <?php echo $attn ?>
      has been received we will contact you soon.<br /><br />
     <a href="index.php"> Next Page </a> </p>
    <!-- end #mainContent -->
    
  </div>
  <!-- This clearing element should immediately follow the #mainContent div in order to force the #container div to contain all child floats --><br class="clearfloat" />
  <div id="footer">
    <?php include("bottomnav.html"); ?>
    <br>
    <br>
    
    <!-- end #footer -->
    <br>
    <br>
  </div>
   
<!-- end #container --></div><br /><br />
</body>
</html>
 

Re: Hello Everyone

Posted: Wed Feb 20, 2008 9:19 am
by Zoxive
Its doing exactly as your code says its doing.

See lines 105-108.

Code: Select all

if(empty($visitor) || empty($visitormail) || empty($notes )) {
echo "Please use the back button and fill in all fields\n";
die ("Use back button on browser! ! "); // DIE Stops Execution
}

Re: Hello Everyone

Posted: Wed Feb 20, 2008 9:25 am
by ChrisM
Ahh ok. Perhaps I am jumping too much in the deep end. :oops:
I will try and code it so it does not stop execution.

What would you suggest? Is there a quick fix to this code?

Re: Hello Everyone

Posted: Wed Feb 20, 2008 10:25 am
by liljester
try processing your user input at the beginning of your script, that way you have more flexibility over page behavior based on user input. you can generate errors at the top of the page, and still be able to repopulate any data that was submitted in the fileds previously.

Re: Hello Everyone

Posted: Wed Feb 20, 2008 3:23 pm
by pickle
[url=http://forums.devnetwork.net/viewtopic.php?t=30037]Forum Rules[/url] Section 1.1 wrote:2. Use descriptive subjects when you start a new thread. Vague titles such as "Help!", "Why?" are misleading and keep you from receiving an answer to your question.

Re: Hello Everyone

Posted: Fri Feb 22, 2008 4:16 am
by ChrisM
Thanks for the tip on the forum rules. Just thought I would introduce myself. :banghead:

Re: Hello Everyone

Posted: Fri Feb 22, 2008 11:48 am
by RobertGonzalez
ChrisM wrote:Thanks for the tip on the forum rules. Just thought I would introduce myself. :banghead:
No need to beat yourself up about it.

To introduce yourself, check out our introductions thread.