Re: DATABASE error on form submission.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Can you access the other POST variables, if not put quotation marks around method="post", also try using instead of op something else...
 
 
Kind Regards
Arné Klopper
 
------------------------------------
NetCB Solutions (Pty) Ltd.
www.netcb.com
Office: +27 12 997 3007
Fax: 27 12 997 3629
Mobile: +27 84 513 4119

>>> <Remember14a@xxxxxxx> 2004/08/31 09:58:39 AM >>>

Dear Friends,

I have a reply form which writes to database, however it gives an error 
posted below.
Guidance, please.

------------------------------------------------------------------------------
---
Error when I keep op like 'op'
------------------------------------------------------------------------------
-
Notice: Undefined index: op in 
\\premfs15\sites\pre15\mrbupeace\webroot\replytopost.php on line 7
----------------------------------------------------------------------------
error when I keep op like op without inverted comma's gives two different 
errors
------------------------------------------------------------------------------
----
Notice: Use of undefined constant op - assumed 'op' in 
\\premfs15\sites\pre15\mrbupeace\webroot\replpost.php on line 7

Notice: Undefined index: op in 
\\premfs15\sites\premium15\mrbushforpeace\webroot\replytopost.php on line 7
------------------------------------------------------------------------------
----------
Code of the script
-------------------------------------------------------------
<?php
//connect to server and select database; we'll need it soon
$conn = mysql_connect("orf-kster.com", "mr", "p") or die(mysql_error());
mysql_select_db("mr",$conn)  or die(mysql_error());

//check to see if we're showing the form or adding the post
if ($_POST['op'] != "addpost") {
   // showing the form; check for required item in query string
   if (!$_GET['post_id']) {
        header("Location: topiclist.php");
        exit;
   }

   //still have to verify topic and post
   $verify = "select ft.topic_id, ft.topic_title from forum_posts as fp left 
join forum_topics as ft on fp.topic_id = ft.topic_id where fp.post_id = 
$_GET[post_id]";
   $verify_res = mysql_query($verify, $conn) or die(mysql_error());
   if (mysql_num_rows($verify_res) < 1) {
       //this post or topic does not exist
       header("Location: topiclist.php");
       exit;
   } else {
       //get the topic id and title
       $topic_id = mysql_result($verify_res,0,'topic_id');
       $topic_title = stripslashes(mysql_result($verify_res, 
0,'topic_title'));

       print "
       <html>
       <head>
       <title>Post Your Reply in $topic_title</title>
       </head>
       <body>
       <h1>Post Your Reply in $topic_title</h1>
       <form method=post action=\"$_SERVER[PHP_SELF]\">
       <p><strong>Your E-Mail Address:</strong><br>
       <input type=\"text\" name=\"post_owner\" size=40 maxlength=150>

       <P><strong>Post Text:</strong><br>
       <textarea name=\"post_text\" rows=8 cols=40 wrap=virtual></textarea>

       <input type=\"hidden\" name=\"op\" value=\"addpost\">
       <input type=\"hidden\" name=\"topic_id\" value=\"$topic_id\">

       <P><input type=\"submit\" name=\"submit\" value=\"Add Post\"></p>

       </form>
       </body>
       </html>";
   }
} else if ($_POST[op] == "addpost") {
   //check for required items from form
   if ((!$_POST['topic_id']) || (!$_POST['post_text']) || 
(!$_POST['post_owner'])) {
       header("Location: topiclist.php");
       exit;
   }

   //add the post
   $add_post = "insert into forum_posts values ('', '$_POST[topic_id]', 
'$_POST[post_text]', now(), '$_POST[post_owner]')";
   mysql_query($add_post,$conn) or die(mysql_error());

   //redirect user to topic

   exit;
}
?>



[Index of Archives]     [PHP Home]     [PHP Users]     [Postgresql Discussion]     [Kernel Newbies]     [Postgresql]     [Yosemite News]

  Powered by Linux