undefine index:no in c:/wamp/www/Hotel 18\confirm_booking.php on line 3

Question

error : undefine index:no in c:/wamp/www/Hotel 18\confirm_booking.php on line 3.

    $roomno=$_GET['no'];
    include('connection.php');
    if(isset($_Request['btnconfirm']))
    {

    $cust=$_POST['custname'];
    $room=$_POST['custroom'];
    $ar=$_POST['arrv'];
    $dep=$_POST['depr'];
    $total=$_POST['total'];
    echo mysql_query("inserted into checkout(customer_name,room_no,arrival_time,departure_time,total,status) values('$cust','$room','$ar','$dep','$total','PAID')");
    mysql_query("update room_booking set status='Clear' where name='$cust' and roomNo='$room'");
 }
 ?>

note: i have taken 'no' as a reference from a page, and it is working quit well, but on submitting the form, still an error occurred related to the $_GET['no'] what mistake i did ?**

why are you using $_GET and $_POST at the same time ? and you are not checking if isset **no** or not — CS GO, Mar 18 '14 at 05:11
You can have a URL query under any method. It may not be his style to properly check for values. Stay on topic. — Dustin Oprea, Mar 18 '14 at 05:15

score 0 · Answer 1 · answered Mar 18 '14 at 05:19

0

Obviously "no" isn't being passed within the query string, at least some of the time.

answered Mar 18 '14 at 05:19

Dustin Oprea

9,673
13
65
105

score 0 · Accepted Answer · edited May 23 '17 at 11:57

This line:

if(isset($_Request['btnconfirm']))

$_Request is a superglobal which must be in uppercase $_REQUEST

change it to:

if(isset($_REQUEST['btnconfirm']))

I suggest that you switch to using mysqli_* with prepared statements or PDO. mysql_* functions are deprecated and will be deleted from future PHP releases.

As it stands now, you are open to SQL injection.

Read this article on how to prevent SQL injection.

undefine index:no in c:/wamp/www/Hotel 18\confirm_booking.php on line 3

2 Answers2