473,396 Members | 2,147 Online
Bytes | Software Development & Data Engineering Community
Post Job

Home Posts Topics Members FAQ

Join Bytes to post your question to a community of 473,396 software developers and data experts.

Parse Error

I have a pretty limited knowledge of PHP.

My scenario is:

I want one form to have two (2) submit buttons. I want one button to submit an email to a specific address then redirect to a page, and I want the second one to submit to the same email, then redirect to a different page. I've downloaded some code, so I can't take credit for it, but I'm trying to use my basic knowledge to create and IF and ELSE IF. Right now, I'm just trying to get the code to work at all, so I have the form simply submitting the form data to an email address based on the button that was clicked.

I get the following parse error:

Parse error: syntax error, unexpected T_ELSE in /home/salamone/public_html/sendeail.php on line 65

Code:

Expand|Select|Wrap|Line Numbers
  1. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
  2.  
  3. <html>
  4. <head>
  5. <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
  6. <title>Sendemail Script</title>
  7. </head>
  8. <body>
  9.  
  10. <?php
  11.  
  12.  
  13.  
  14.  
  15.  
  16. $ip = $_POST['ip'];
  17. $httpref = $_POST['httpref'];
  18. $httpagent = $_POST['httpagent'];
  19. $visitor = $_POST['visitor'];
  20. $visitormail = $_POST['visitormail'];
  21. $notes = $_POST['notes'];
  22. $attn = $_POST['attn'];
  23.  
  24.  
  25. if (eregi('http:', $notes)) {
  26. die ("Do NOT try that! ! ");
  27. }
  28. if(!$visitormail == "" && (!strstr($visitormail,"@") || !strstr($visitormail,".")))
  29. {
  30. echo "<h2>Use Back - Enter valid e-mail</h2>\n";
  31. $badinput = "<h2>Feedback was NOT submitted</h2>\n";
  32. echo $badinput;
  33. die ("Go back! ! ");
  34. }
  35.  
  36. if(empty($visitor) || empty($visitormail) || empty($notes )) {
  37. echo "<h2>Use Back - fill in all fields</h2>\n";
  38. die ("Use back! ! ");
  39. }
  40.  
  41. if(isset($_POST["sendmail"]) && trim($_POST["sendmail"])!=='') {
  42.  
  43. $todayis = date("l, F j, Y, g:i a") ;
  44.  
  45. $attn = $attn ;
  46. $subject = $attn;
  47.  
  48. $notes = stripcslashes($notes);
  49.  
  50. $message = " $todayis [EST] \n
  51. Attention: $attn \n
  52. Message: $notes \n
  53. From: $visitor ($visitormail)\n
  54. Additional Info : IP = $ip \n
  55. Browser Info: $httpagent \n
  56. Referral : $httpref \n
  57. ";
  58.  
  59. $from = "From: $visitormail\r\n";
  60.  
  61.  
  62. mail("test1@myemail.com", $subject, $message, $from)
  63.  
  64.  
  65. else if(isset($_POST["sendmail2"]) && trim($_POST["sendmail2"])!=='') {
  66. $todayis = date("l, F j, Y, g:i a") ;
  67.  
  68. $attn = $attn ;
  69. $subject = $attn;
  70.  
  71. $notes = stripcslashes($notes);
  72.  
  73. $message = " $todayis [EST] \n
  74. Attention: $attn \n
  75. Message: $notes \n
  76. From: $visitor ($visitormail)\n
  77. Additional Info : IP = $ip \n
  78. Browser Info: $httpagent \n
  79. Referral : $httpref \n
  80. ";
  81.  
  82. $from = "From: $visitormail\r\n";
  83.  
  84.  
  85. mail("test2@myemail.com", $subject, $message, $from)}
  86.  
  87. ?>
  88. </body>
  89. </html>
  90.  
If I am missing something? Thanks in advance!
Sep 17 '09 #1
4 1859
TheServant
1,168 Expert 1GB
Main reason:
You have not closed your if statement with curly brackets. Should be like:
Expand|Select|Wrap|Line Numbers
  1. if (x=1) {
  2. //do something
  3. }
  4. elseif (x=2) {
  5. //do something else
  6. }
  7. else {
  8. //do yet another thing
  9. }
Hope that helps. Welcome to Bytes.
Sep 17 '09 #2
Atli
5,058 Expert 4TB
Hey. Welcome to Bytes!

I see 2 errors there.

On lines #62 and #85 you are missing the semi-colon (;) to end the line.

And, before the "else if " on line #65, you are missing the } to close the if preceding it.

An if structure should look like:
Expand|Select|Wrap|Line Numbers
  1. if {
  2.   // statementes
  3. }
  4. else if {
  5.   // statements
  6. }
Yours is like this:
Expand|Select|Wrap|Line Numbers
  1. if {
  2.   // statementes
  3. else if {
  4.   // statements
  5. }
Sep 17 '09 #3
Atli
5,058 Expert 4TB
@TheServant
Actually, both work in the typical curly-braces structure.
(See elseif/else if - PHP manual)
Sep 17 '09 #4
TheServant
1,168 Expert 1GB
@Atli
Point taken. Thanks. And well done with the ";" pick-up ;)
Sep 17 '09 #5

Sign in to post your reply or Sign up for a free account.

Similar topics

2
by: Steven | last post by:
I got a "Parse error: parse error in ..." in this line: if(empty($_POST){ ..... But if I fist assign $ssn=$_POST; and then if(empty($ssn){ ... it is working. Any advice? Thanks in advance.
1
by: H.L Bai | last post by:
hi, everybody i meet a parse error when i used the xml4c. any proposal is helpful. The error is following .../XMLRegionHandler.h:59 parse error before '*' .../XMLRegionHandler.h:60 parse...
2
by: Vittal | last post by:
Hello All, I am trying to compile my application on Red Hat Linux 8 against gcc 3.2.2. Very first file in application is failing to compile. I tried compiling my application on Linux 7.2...
21
by: BWIGLEY | last post by:
Basically I've just started making a game. So far it makes an array 25 by 20 and tries to make five rooms within it. In scr_make_room() there's parse errors: 20 C:\c\Rooms\Untitled1.c parse error...
6
by: trevor | last post by:
Incorrect values when using float.Parse(string) I have discovered a problem with float.Parse(string) not getting values exactly correct in some circumstances(CSV file source) but in very similar...
5
AdrianH
by: AdrianH | last post by:
Assumptions I am assuming that you know or are capable of looking up the functions I am to describe here and have some remedial understanding of C++ programming. FYI Although I have called...
1
AdrianH
by: AdrianH | last post by:
Assumptions I am assuming that you know or are capable of looking up the functions I am to describe here and have some remedial understanding of C programming. FYI Although I have called this...
1
by: Phaelle | last post by:
Parse error: parse error, expecting `T_STRING' or `T_VARIABLE' or `T_NUM_STRING' What does that error mean? I can´t find the mistake!! In another script, I have got another kind of mistake : parse...
2
by: Lawrence Krubner | last post by:
Imagine a template system that works by getting a file, as a string, and then putting it through eval(), something like this: $formAsString = $controller->command("readFileAndReturnString",...
5
by: goldtech | last post by:
SAX XML Parse Python error message Hi, My first attempt at SAX, but have an error message I need help with. I cite the error message, code, and xml below. Be grateful if anyone can tell me...
0
by: Charles Arthur | last post by:
How do i turn on java script on a villaon, callus and itel keypad mobile phone
0
BarryA
by: BarryA | last post by:
What are the essential steps and strategies outlined in the Data Structures and Algorithms (DSA) roadmap for aspiring data scientists? How can individuals effectively utilize this roadmap to progress...
1
by: nemocccc | last post by:
hello, everyone, I want to develop a software for my android phone for daily needs, any suggestions?
0
by: Hystou | last post by:
There are some requirements for setting up RAID: 1. The motherboard and BIOS support RAID configuration. 2. The motherboard has 2 or more available SATA protocol SSD/HDD slots (including MSATA, M.2...
0
marktang
by: marktang | last post by:
ONU (Optical Network Unit) is one of the key components for providing high-speed Internet services. Its primary function is to act as an endpoint device located at the user's premises. However,...
0
by: Hystou | last post by:
Most computers default to English, but sometimes we require a different language, especially when relocating. Forgot to request a specific language before your computer shipped? No problem! You can...
0
jinu1996
by: jinu1996 | last post by:
In today's digital age, having a compelling online presence is paramount for businesses aiming to thrive in a competitive landscape. At the heart of this digital strategy lies an intricately woven...
0
tracyyun
by: tracyyun | last post by:
Dear forum friends, With the development of smart home technology, a variety of wireless communication protocols have appeared on the market, such as Zigbee, Z-Wave, Wi-Fi, Bluetooth, etc. Each...
0
agi2029
by: agi2029 | last post by:
Let's talk about the concept of autonomous AI software engineers and no-code agents. These AIs are designed to manage the entire lifecycle of a software development project—planning, coding, testing,...

By using Bytes.com and it's services, you agree to our Privacy Policy and Terms of Use.

To disable or enable advertisements and analytics tracking please visit the manage ads & tracking page.