Can you help me find my mistake?

ebryant77 · November 16, 2010

Please help...I'm going blind!!! I am trying to set up a simple feedback form on a website. The form is at www.hotelguestdirect.com if you need to see that.

This is the error I get...Parse error: syntax error, unexpected ',' in /home/content/h/o/t/hotelguestdir/html/survey.php on line 160

Here is my code

<?php
// Field Mapping
$sFirstName = $_POST["txtFirstName"];
$sLastName = $_POST["txtLastName"];
$sHomeCity = $_POST["txtHomeCity"];
$sHomeState = $_POST["selHomeState"];
$sCountry = $_POST["selCountry"];
$sHotel = $_POST["txtHotel"];
$sCity = $_POST["txtCity"];
$sState = $_POST["selState"];
$sStayLength = $_POST["selStayLength"];
$sHowHelpful = $_POST["txtHowHelpful"];
$sImprovements = $_POST["txtImprovements"];


if (($sHotel == "") || ($sCity == "") || ($sState == "") || ($sStayLength == "") || ($sFirstName == ""))
{
print('<span style="color:#333; font-size:14px;">Please complete the required fields <a style="color:#333" href="index.html" onClick="history.go(-1);">Click here to go back</a></span>');
}
else
{
$sMailFrom = '[email protected]';

$sMailTo = '[email protected]';
$sMailSubject = 'Guest Survey - Key Holder Brochures';

$sMailBody = '<p>Guest Response Survey</p>
<b>Hotel: </b>' . $sName . '<br />
<b>City: </b>' . $sCity . '<br />
<b>State: </b>' . $sState . '<br />
<b>Lenth of Stay </b> ' . $sStayLength . '<br />
<b>How was the brochure helpful: </b>' . $sHowHelpful . '<br />
<b>Input or Improvements: </b>' . $sImprovements . '<br />';

$sMailHeaders = "From: $sMailFrom\n";
$sMailHeaders = "Content-Type: text/html; charset=iso-8859-1\r\n";

if (mail($sMailTo, $sMailSubject, $sMailBody, $sMailHeaders))
{
	print('<span style="color:#333; font-size:14px;"><div align="center"><br /><br />Thank you, your information has been emailed!</div></span>');
}
else
{
	print('<span style="color:#333; font-size:14px;"><div align="center"><br /><br />Could not send e-mail, please contact the webmaster!</div></span>');
}
}
?>

harristweed · November 16, 2010

no errors in this code, post the whole thing!

mikosiko · November 16, 2010

This is the error I get...Parse error: syntax error, unexpected ',' in /home/content/h/o/t/hotelguestdir/html/survey.php on line 160

posted code doesn't have line 160.... something is not adding up

ebryant77 · November 16, 2010

here is all the code from the top!!

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

<head>

<title>Hotel Guest Direct - Guest Survey</title>

<!--

body {

background-color: #313131;

text-align: center;

font-size: 12px;

margin-top: 20px;

margin-right: 0px;

margin-bottom: 0px;

margin-left: 0px;

}

body,td,th {

font-family: Verdana, Geneva, sans-serif;

color: #333;

}

#container {

text-align: center;

height: 100%;

width: 787px;

position: relative;

margin-right: auto;

margin-left: auto;

}

.formarea {

text-align: left;

padding-right: 10px;

padding-left: 10px;

margin: 0px;

}

#top {

background-repeat: no-repeat;

text-align: center;

left: 0px;

top: 0px;

right: 0px;

margin-right: auto;

margin-left: auto;

width: 786px;

height: 104px;

margin-top: 0px;

}

#topbar {

width: 786px;

background-repeat: no-repeat;

text-align: center;

left: 0px;

top: 104px;

right: 0px;

margin-right: auto;

margin-left: auto;

width: 786px;

height: 100px;

margin-top: 0px;

margin-bottom: 0px;

position: absolute;

}

#titlebar {

height: 56px;

width: 786px;

margin-right: auto;

margin-left: auto;

margin-top: 0px;

margin-bottom: -10px;

font-size: 18px;

padding-top: 10px;

background-repeat: no-repeat;

padding-bottom: -10px;

}

#contentbox {

background-repeat: repeat-y;

height: 475px;

width: 786px;

margin-right: auto;

margin-left: auto;

background-color: #DCDCDC;

margin-top: -10px;

margin-bottom: -10px;

position: absolute;

top: 197px;

}

.header {

font-size: 18px;

font-weight: bold;

font-style: normal;

height: 10px;

}

#dividerbar {

width: 786px;

height: 7px;

text-align: center;

margin-top: 0px;

margin-right: auto;

margin-bottom: 0px;

margin-left: auto;

position: absolute;

top: 180px;

}

#bottom {

height: 75px;

width: 786px;

margin-top: -10px;

margin-right: auto;

margin-bottom: 15px;

margin-left: auto;

top: 662px;

position: absolute;

}

#footer {

font-size: 10px;

color: #CCC;

position: absolute;

top: 750px;

margin-right: auto;

margin-left: auto;

text-align: center;

width: 786px;

}

-->

</style></head>

<body>

<div id="titlebar">Guest Survey</div>

<?php

// Field Mapping

$sFirstName = $_POST["txtFirstName"];

$sLastName = $_POST["txtLastName"];

$sHomeCity = $_POST["txtHomeCity"];

$sHomeState = $_POST["selHomeState"];

$sCountry = $_POST["selCountry"];

$sHotel = $_POST["txtHotel"];

$sCity = $_POST["txtCity"];

$sState = $_POST["selState"];

$sStayLength = $_POST["selStayLength"];

$sHowHelpful = $_POST["txtHowHelpful"];

$sImprovements = $_POST["txtImprovements"];

if (($sHotel == "") || ($sCity == "") || ($sState == "") || ($sStayLength == "") || ($sFirstName == ""))

{

print('<span style="color:#333; font-size:14px;">Please complete the required fields <a style="color:#333" href="index.html" onClick="history.go(-1);">Click here to go back</a></span>');

}

else

{

$sMailFrom = '[email protected]';

$sMailTo = '[email protected]';

$sMailSubject = 'Guest Survey - Key Holder Brochures';

$sMailBody = '<p>Guest Response Survey</p>

<b>Hotel: </b>' . $sName . '<br />

<b>City: </b>' . $sCity . '<br />

<b>State: </b>' . $sState . '<br />

<b>Lenth of Stay </b> ' . $sStayLength . '<br />

<b>How was the brochure helpful: </b>' . $sHowHelpful . '<br />

<b>Input or Improvements: </b>' . $sImprovements . '<br />';

$sMailHeaders = "From: $sMailFrom\n";

$sMailHeaders = "Content-Type: text/html; charset=iso-8859-1\r\n";

if (mail($sMailTo, $sMailSubject, $sMailBody, $sMailHeaders))

{

print('<span style="color:#333; font-size:14px;"><div align="center"><br /><br />Thank you, your information has been emailed!</div></span>');

}

else

{

print('<span style="color:#333; font-size:14px;"><div align="center"><br /><br />Could not send e-mail, please contact the webmaster!</div></span>');

}

?>

</div>

</div>

</body>

</html>

ebryant77 · November 16, 2010

Here is the html for the form, just in case that helps.

<p>We appreciate your input. Please fill out the form completely. Fields marked with (*) are required.</p>

First Name*

Last Name

<br />

Where are you from?*<br />

<option selected="selected">State</option>

<option value="al" >Alabama</option>

<option value="ak" >Alaska</option>

<option value="ar" >Arkansas</option>

<option value="az" >Arizona</option>

<option value="ca" >California</option>

<option value="co" >Colorado</option>

<option value="ct" >Connecticut</option>

<option value="de" >Delaware</option>

<option value="dc" >District of Columbia</option>

<option value="fl" >Florida</option>

<option value="ga" >Georgia</option>

<option value="hi" >Hawaii</option>

<option value="id" >Idaho</option>

<option value="il" >Illinois</option>

<option value="in" >Indiana</option>

<option value="ks" >Kansas</option>

<option value="ky" >Kentucky</option>

<option value="la" >Louisiana</option>

<option value="ma" >Massachusetts</option>

<option value="md" >Maryland</option>

<option value="me" >Maine</option>

<option value="mi" >Michigan</option>

<option value="mn" >Minnesota</option>

<option value="mo" >Missouri</option>

<option value="ms" >Mississippi</option>

<option value="mt" >Montana</option>

<option value="nc" >North Carolina</option>

<option value="nd" >North Dakota</option>

<option value="ne" >Nebraska</option>

<option value="nh" >New Hampshire</option>

<option value="nj" >New Jersey</option>

<option value="nm" >New Mexico</option>

<option value="nv" >Nevada</option>

<option value="ok" >Oklahoma</option>

<option value="or" >Oregon</option>

<option value="ot" >Outside of US</option>

<option value="pa" >Pennsylvania</option>

<option value="ri" >Rhode Island</option>

<option value="sc" >South Carolina</option>

<option value="sd" >South Dakota</option>

<option value="tn" >Tennessee</option>

<option value="tx" >Texas</option>

<option value="va" >Virginia</option>

<option value="vt" >Vermont</option>

<option value="wa" >Washington</option>

<option value="wi" >Wisconsin</option>

<option value="wv" >West Virginia</option>

<option value="wy" >Wyoming</option>

</select>

<option value="AF" selected="selected">Country</option>

<option value="AF">Afghanistan</option>

<option value="AX">Ã…Land Islands</option>

<option value="AL">Albania</option>

<option value="DZ">Algeria</option>

<option value="AS">American Samoa</option>

<option value="AD">Andorra</option>

<option value="AO">Angola</option>

<option value="AI">Anguilla</option>

<option value="AQ">Antarctica</option>

<option value="AG">Antigua And Barbuda</option>

<option value="AR">Argentina</option>

<option value="AM">Armenia</option>

<option value="AW">Aruba</option>

<option value="AU">Australia</option>

<option value="AT">Austria</option>

<option value="AZ">Azerbaijan</option>

<option value="BS">Bahamas</option>

<option value="BH">Bahrain</option>

<option value="BD">Bangladesh</option>

<option value="BB">Barbados</option>

<option value="BY">Belarus</option>

<option value="BE">Belgium</option>

<option value="BZ">Belize</option>

<option value="BJ">Benin</option>

<option value="BM">Bermuda</option>

<option value="BT">Bhutan</option>

<option value="BO">Bolivia</option>

<option value="BA">Bosnia And Herzegovina</option>

<option value="BW">Botswana</option>

<option value="BV">Bouvet Island</option>

<option value="BR">Brazil</option>

<option value="IO">British Indian Ocean Territory</option>

<option value="BN">Brunei Darussalam</option>

<option value="BG">Bulgaria</option>

<option value="BF">Burkina Faso</option>

<option value="BI">Burundi</option>

<option value="KH">Cambodia</option>

<option value="CM">Cameroon</option>

<option value="CA">Canada</option>

<option value="CV">Cape Verde</option>

<option value="KY">Cayman Islands</option>

<option value="CF">Central African Republic</option>

<option value="CL">Chile</option>

<option value="CN">China</option>

<option value="CX">Christmas Island</option>

<option value="CC">Cocos (Keeling) Islands</option>

<option value="CO">Colombia</option>

<option value="KM">Comoros</option>

<option value="CG">Congo</option>

<option value="CD">Congo, The Democratic Republic Of The</option>

<option value="CK">Cook Islands</option>

<option value="CR">Costa Rica</option>

<option value="CI">Cote D'Ivoire</option>

<option value="HR">Croatia</option>

<option value="CY">Cyprus</option>

<option value="CZ">Czech Republic</option>

<option value="DK">Denmark</option>

<option value="DJ">Djibouti</option>

<option value="DM">Dominica</option>

<option value="DO">Dominican Republic</option>

<option value="EC">Ecuador</option>

<option value="EG">Egypt</option>

<option value="SV">El Salvador</option>

<option value="GQ">Equatorial Guinea</option>

<option value="ER">Eritrea</option>

<option value="EE">Estonia</option>

<option value="ET">Ethiopia</option>

<option value="FK">Falkland Islands (Malvinas)</option>

<option value="FO">Faroe Islands</option>

<option value="FI">Finland</option>

<option value="FR">France</option>

<option value="GF">French Guiana</option>

<option value="PF">French Polynesia</option>

<option value="TF">French Southern Territories</option>

<option value="GA">Gabon</option>

<option value="GM">Gambia</option>

<option value="GE">Georgia</option>

<option value="DE">Germany</option>

<option value="GH">Ghana</option>

<option value="GI">Gibraltar</option>

<option value="GR">Greece</option>

<option value="GL">Greenland</option>

<option value="GD">Grenada</option>

<option value="GP">Guadeloupe</option>

<option value="GT">Guatemala</option>

<option value=" Gg">Guernsey</option>

<option value="GN">Guinea</option>

<option value="GW">Guinea-Bissau</option>

<option value="GY">Guyana</option>

<option value="HT">Haiti</option>

<option value="HM">Heard Island And Mcdonald Islands</option>

<option value="VA">Holy See (Vatican City State)</option>

<option value="HN">Honduras</option>

<option value="HU">Hungary</option>

<option value="IS">Iceland</option>

<option value="IN">India</option>

<option value="ID">Indonesia</option>

<option value="IR">Iran, Islamic Republic Of</option>

<option value="IE">Ireland</option>

<option value="IL">Israel</option>

<option value="IT">Italy</option>

<option value="JM">Jamaica</option>

<option value="JP">Japan</option>

<option value="JE">Jersey</option>

<option value="JO">Jordan</option>

<option value="KZ">Kazakhstan</option>

<option value="KE">Kenya</option>

<option value="KI">Kiribati</option>

<option value="KP">Korea, Democratic People'S Republic Of</option>

<option value="KR">Korea, Republic Of</option>

<option value="KW">Kuwait</option>

<option value="KG">Kyrgyzstan</option>

<option value="LA">Lao People'S Democratic Republic</option>

<option value="LV">Latvia</option>

<option value="LB">Lebanon</option>

<option value="LS">Lesotho</option>

<option value="LR">Liberia</option>

<option value="LY">Libyan Arab Jamahiriya</option>

<option value="LI">Liechtenstein</option>

<option value="LT">Lithuania</option>

<option value="LU">Luxembourg</option>

<option value="MO">Macao</option>

<option value="MK">Macedonia, The Former Yugoslav Republic Of</option>

<option value="MG">Madagascar</option>

<option value="MW">Malawi</option>

<option value="MY">Malaysia</option>

<option value="MV">Maldives</option>

<option value="MT">Malta</option>

<option value="MH">Marshall Islands</option>

<option value="MQ">Martinique</option>

<option value="MR">Mauritania</option>

<option value="MU">Mauritius</option>

<option value="YT">Mayotte</option>

<option value="MX">Mexico</option>

<option value="FM">Micronesia, Federated States Of</option>

<option value="MD">Moldova, Republic Of</option>

<option value="MC">Monaco</option>

<option value="MN">Mongolia</option>

<option value="MS">Montserrat</option>

<option value="MA">Morocco</option>

<option value="MZ">Mozambique</option>

<option value="MM">Myanmar</option>

<option value="NA">Namibia</option>

<option value="NR">Nauru</option>

<option value="NP">Nepal</option>

<option value="NL">Netherlands</option>

<option value="AN">Netherlands Antilles</option>

<option value="NC">New Caledonia</option>

<option value="NZ">New Zealand</option>

<option value="NI">Nicaragua</option>

<option value="NE">Niger</option>

<option value="NG">Nigeria</option>

<option value="NF">Norfolk Island</option>

<option value="MP">Northern Mariana Islands</option>

<option value="NO">Norway</option>

<option value="PK">Pakistan</option>

<option value="PW">Palau</option>

<option value="PS">Palestinian Territory, Occupied</option>

<option value="PA">Panama</option>

<option value="PG">Papua New Guinea</option>

<option value="PY">Paraguay</option>

<option value="PH">Philippines</option>

<option value="PN">Pitcairn</option>

<option value="PL">Poland</option>

<option value="PT">Portugal</option>

<option value="PR">Puerto Rico</option>

<option value="QA">Qatar</option>

<option value="RE">Reunion</option>

<option value="RO">Romania</option>

<option value="RU">Russian Federation</option>

<option value="RW">Rwanda</option>

<option value="SH">Saint Helena</option>

<option value="KN">Saint Kitts And Nevis</option>

<option value="LC">Saint Lucia</option>

<option value="PM">Saint Pierre And Miquelon</option>

<option value="VC">Saint Vincent And The Grenadines</option>

<option value="WS">Samoa</option>

<option value="SM">San Marino</option>

<option value="ST">Sao Tome And Principe</option>

<option value="SA">Saudi Arabia</option>

<option value="SN">Senegal</option>

<option value="CS">Serbia And Montenegro</option>

<option value="SC">Seychelles</option>

<option value="SL">Sierra Leone</option>

<option value="SG">Singapore</option>

<option value="SK">Slovakia</option>

<option value="SI">Slovenia</option>

<option value="SB">Solomon Islands</option>

<option value="SO">Somalia</option>

<option value="ZA">South Africa</option>

<option value="GS">South Georgia And The South Sandwich Islands</option>

<option value="ES">Spain</option>

<option value="LK">Sri Lanka</option>

<option value="SD">Sudan</option>

<option value="SR">Suriname</option>

<option value="SJ">Svalbard And Jan Mayen</option>

<option value="SZ">Swaziland</option>

<option value="SE">Sweden</option>

<option value="CH">Switzerland</option>

<option value="SY">Syrian Arab Republic</option>

<option value="TW">Taiwan, Province Of China</option>

<option value="TJ">Tajikistan</option>

<option value="TZ">Tanzania, United Republic Of</option>

<option value="TH">Thailand</option>

<option value="TL">Timor-Leste</option>

<option value="TK">Tokelau</option>

<option value="TO">Tonga</option>

<option value="TT">Trinidad And Tobago</option>

<option value="TN">Tunisia</option>

<option value="TR">Turkey</option>

<option value="TM">Turkmenistan</option>

<option value="TC">Turks And Caicos Islands</option>

<option value="TV">Tuvalu</option>

<option value="UG">Uganda</option>

<option value="UA">Ukraine</option>

<option value="AE">United Arab Emirates</option>

<option value="GB">United Kingdom</option>

<option value="US">United States</option>

<option value="UM">United States Minor Outlying Islands</option>

<option value="UY">Uruguay</option>

<option value="UZ">Uzbekistan</option>

<option value="VU">Vanuatu</option>

<option value="VE">Venezuela</option>

<option value="VG">Virgin Islands, British</option>

<option value="VI">Virgin Islands, U.S.</option>

<option value="WF">Wallis And Futuna</option>

<option value="EH">Western Sahara</option>

<option value="YE">Yemen</option>

<option value="ZM">Zambia</option>

<option value="ZW">Zimbabwe</option>

</select>

<p>Where did you stay? *<br />

<option selected="selected">Select State</option>

<option value="al" >Alabama</option>

<option value="ak" >Alaska</option>

<option value="ar" >Arkansas</option>

<option value="az" >Arizona</option>

<option value="ca" >California</option>

<option value="co" >Colorado</option>

<option value="ct" >Connecticut</option>

<option value="de" >Delaware</option>

<option value="dc" >District of Columbia</option>

<option value="fl" >Florida</option>

<option value="ga" >Georgia</option>

<option value="hi" >Hawaii</option>

<option value="id" >Idaho</option>

<option value="il" >Illinois</option>

<option value="in" >Indiana</option>

<option value="ks" >Kansas</option>

<option value="ky" >Kentucky</option>

<option value="la" >Louisiana</option>

<option value="ma" >Massachusetts</option>

<option value="md" >Maryland</option>

<option value="me" >Maine</option>

<option value="mi" >Michigan</option>

<option value="mn" >Minnesota</option>

<option value="mo" >Missouri</option>

<option value="ms" >Mississippi</option>

<option value="mt" >Montana</option>

<option value="nc" >North Carolina</option>

<option value="nd" >North Dakota</option>

<option value="ne" >Nebraska</option>

<option value="nh" >New Hampshire</option>

<option value="nj" >New Jersey</option>

<option value="nm" >New Mexico</option>

<option value="nv" >Nevada</option>

<option value="ok" >Oklahoma</option>

<option value="or" >Oregon</option>

<option value="ot" >Outside of US</option>

<option value="pa" >Pennsylvania</option>

<option value="ri" >Rhode Island</option>

<option value="sc" >South Carolina</option>

<option value="sd" >South Dakota</option>

<option value="tn" >Tennessee</option>

<option value="tx" >Texas</option>

<option value="va" >Virginia</option>

<option value="vt" >Vermont</option>

<option value="wa" >Washington</option>

<option value="wi" >Wisconsin</option>

<option value="wv" >West Virginia</option>

<option value="wy" >Wyoming</option>

</select>

</p>

<option>Select length of stay</option>

<option value="1 night">1 night</option>

<option value="2 nights">2 nights</option>

<option value="3 nights">3 nights</option>

<option value="4 nights">4 nights</option>

<option value="5 nights">5 nights</option>

<option value="6 nights">6 nights</option>

<option value="7 nights">7 nights</option>

<option value="8 nights">8 nights</option>

<option value="9 nights">9 nights</option>

<option value="10 nights">10 nights</option>

<option value="11 nights">11 nights</option>

<option value="12 nights">12 nights</option>

<option value="13 nights">13 nights</option>

<option value="14 + nights">14 or more nights</option>

</select>

</p>

<p>We hope you found the key holder brochure helpful during your stay. Please give us your feedback. Did you find a great restaraunt? Access a suggested service? Use the maps to get around? <br />

</p>

<p>We are always working to improve the key holder brochures. Please let us know if there was anything we could have done better.<br />

</p>

Thank you for your input!

<wff:validation_errors/>

</form>

mikosiko · November 16, 2010

is that your complete form code ?...

harristweed · November 16, 2010

works for me!

http://richardswebs.com/form.html

jcbones · November 16, 2010

I get no parse errors on that code.

doni49 · November 16, 2010

This is the error I get...Parse error: syntax error, unexpected ',' in /home/content/h/o/t/hotelguestdir/html/survey.php on line 160

The ONLY file we need to see is the file mentioned in this error message.

ebryant77 · November 16, 2010

any ideas what else could be going on? I tried uploading the files again, clearing my browser history...still getting the same error.

doni49 · November 16, 2010

Is the code you posted above the entire contents of the one mentioned in the error message?

doni49 · November 16, 2010

Also--I just loaded your page in my browser and didn't get any errors. Does the error happen after submission?

EDIT: I just went ahead and tried submitting the form. I get the same error message.

ebryant77 · November 16, 2010

The php code posted is the entire file that is returning the error message. The error happens on submission

mikosiko · November 16, 2010

You have problems with the names of your fields... this should be the right ones

// Field Mapping
$sFirstName = $_POST['FirstName'];
$sLastName = $_POST['LastName'];
$sHomeCity = $_POST['HomeCity'];
$sHomeState = $_POST['HomeState'];
$sCountry = $_POST['selCountry'];
$sHotel = $_POST['txtHotel'];
$sCity = $_POST['txtCity'];
$sState = $_POST['selState'];
$sStayLength = $_POST['StayLength'];
$sHowHelpful = $_POST['HowHelpful'];
$sImprovements = $_POST['Improvements'];

check them and try

ebryant77 · November 16, 2010

still getting the same error. I guess I'll start over. grrr....thanks for the suggestions!

Sign In

Can you help me find my mistake?

Recommended Posts

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Archived

Important Information