1

I've been working on this for hours now and including rebuilding my whole code I cannot figure out what is wrong. The HTML form is supposed to populate the patients SQL database but it is not working. I have a similar form that I am using on another part of the website that works flawlessly but this on doesn't seem to and a can't figure out why.

thanks for your help

Here is a picture of thedatabase fields : Database fields

Heres the code:

<?php

//session_start();

//if( isset($_SESSION['user_id']) ){
//  header("Location: /");
//}

require 'connection.php';

$message = '';

if(!empty($_POST['name']) && !empty($_POST['dob']) && !empty($_POST['nok']) && !empty($_POST['nok_add']) && !empty($_POST['nok_phone'])&& !empty($_POST['doa'])&& !empty($_POST['allergies'])):
  
  // Enter the new user in the database
  $sql = "INSERT INTO patients (name, dob, nok, nok_add, nok_phone, doa, allergies) VALUES (:name, :dob, :nok, :nok_add, :nok_phone, :doa, :allergies)";
  $stmt = $conn->prepare($sql);

  $stmt->bindParam(':name', $_POST['name']);
  $stmt->bindParam(':dob', $_POST['dob']);
  $stmt->bindParam(':nok', $_POST['nok']);
  $stmt->bindParam(':nok_add', $_POST['nok_add']);
  $stmt->bindParam(':nok_phone', $_POST['nok_phone']);
  $stmt->bindParam(':doa', $_POST['doa']);
  $stmt->bindParam(':allergies', $_POST['allergies']);

  if( $stmt->execute() ):
    $message = 'Successfully created new user';
  else:
    $message = 'Sorry there must have been an issue creating your account';
  endif;

endif;



?>


<html>
  <head>
    <title>Patient Add</title>
    <meta name="viewport" content="width=device-width, initial-scale=1">
      <link rel="stylesheet" type="text/css" href="assets/css/style.css">
      <link href="//fonts.googleapis.com/css?fami=Roboto:400,300,200,100&subset=latin,cyrillic" rel="stylesheet">
<style type="text/css">      
        #headline {
        padding: 0.8em;
        color: white;
        font-family: Roboto, helvetica, arial, sans-serif;
        background-color: black;
        background-image: url(backgroundimage.jpg);
        background-size: cover;
      } 

</style>

  </head>
  <body>

    <div id="headline">
      <div class="container">
        <header>
          <h1>Patient Add</h1>
          <p></p>
        </header>



    <button onClick="window.location='manager_select.php';" class="_button">Back</button>
    <form method="POST" id="add">
          
      <h2>Enter the Patient's Details</h2>

      <label for="name">Name:</label>
      <input type="text" name="name">

      <label for="dob">Date of Birth:</label>
      <input type="text" name="dob">

      <label for="nok">Next of Kin:</label>
      <input type="text" name="nok">

      <label for="nok_add">Address:</label>
      <input type="text" name="nok_add">

      <label for="nok_phone">Phone:</label>
      <input type="text" name="nok_phone">

      <label for="doa">Date of Admission:</label>
      <input type="text" name="dob">

      <label for="allergies">Allergies:</label>
      <input type="text" name="allergies">

      <?php if(!empty($message)): ?>
        <p><?= $message ?></p>
      <?php endif; ?>
      <input type="submit">

    </form>
        <br>
      </div>
    </div>

  <!-- // [START footer] -->
      <footer>
        <div class="container">
          <p>Copyright Ghyll Court Residental Home 2017</p>
        </div>
      </footer>
      <!-- // [END footer] -->

  </script>
      <script type="text/javascript">
        function init() {
          window.matchMedia("(max-width: 600px)").addListener(hitMQ);
        }

        function hitMQ(evt) {
          sampleCompleted("GettingStarted-ContentWithStyles");
        }

        init();

      </script>

  </body>
</html>

2 Answers2

1

As answered on your duplicate post (also answering here for completeness), you're checking for a "doa" field in your if statement, which doesn't exist. Your form defines "dob" twice: once for Date of Birth, and once for Date of Admission:

<label for="dob">Date of Birth:</label>
<input type="text" name="dob">

...

<label for="doa">Date of Admission:</label>
<input type="text" name="dob">
Community
  • 1
  • 1
rickdenhaan
  • 10,857
  • 28
  • 37
  • this is the right answer as far as i can see, as he said you have no field for "doa" so it will always be "empty" and your if() statement will never execute. – ATechGuy Mar 30 '17 at 22:10
  • @rickdenhann thankyou so much for you help its the little things that you don't see! – charles verfuerth Mar 30 '17 at 22:10
0

I am not completely sure about this being the safest way but instead of using MSQLI PDO why don't you try simple mysqli_query($conn,$sql);