Switching to PDO but having issues

myliberty

I am switching to PDO as per everyone's advice. The below script uploads the photo but nothing is added to the DB

<?php

///DEBUGGING
if (!empty($_POST)) {
   print "<pre>This is your \$_POST array \n\n".print_r($_POST,true)."</pre>"; 
}

//This gets all the other information from the form
   $Id        = isset($_POST['Id'])        ? $_POST['Id']        : 'NULL'; 
   $Date      = isset($_POST['Date'])      ? $_POST['Date']      : ''; 
   $Category      = isset($_POST['Category'])      ? $_POST['Category']      : ''; 
   $Title      = isset($_POST['Title'])      ? $_POST['Title']      : ''; 
   $Author  = isset($_POST['Author'])  ? $_POST['Author']  : ''; 
   $Ctime  = isset($_POST['Ctime'])  ? $_POST['Ctime']  : ''; 
   $Temp  = isset($_POST['Temp'])  ? $_POST['Temp']  : ''; 
   $Prep  = isset($_POST['Prep'])  ? $_POST['Prep']  : '';
   $Serves  = isset($_POST['Serves'])  ? $_POST['Serves']  : '';
   $Details     = isset($_POST['Details'])     ? $_POST['Details']     : ''; 
   $Ingredients     = isset($_POST['Ingredients'])     ? $_POST['Ingredients']     : ''; 
   $Instructions     = isset($_POST['Instructions'])     ? $_POST['Instructions']     : ''; 
   $Notes     = isset($_POST['Notes'])     ? $_POST['Notes']     : '';  

   $nutrition     = isset($_POST['nutrition'])     ? $_POST['nutrition']     : '';  

   $photo=($_FILES['photo']['name']);  

   $folder = $_POST['Category'];  

// Connects to your Database
 require('dbcadmin.php');


/////////////////////
//Writes the information to the database
   $stmt = $conn->prepare("INSERT INTO recipes (Id, Date, Category, Title, Author, Ctime, Temp, Photo, Prep, Serves, Details, Ingredients, Instructions, Notes, nutrition) \n". 
                              " VALUES (:Id, :Date, :Category, :Title, :Author, :Ctime, :Temp, :Photo, :Prep, :Serves, :Details, :Ingredients, :Instructions, :Notes, :nutrition) "); 

  $success=$stmt->execute(array(':Id'=>$Id, ':Date'=>$Date, ':Category'=>$Category, ':Title'=>$Title, ':Author'=>$Author, ':Ctime'=>$Ctime, ':Temp'=>$Temp, ':Photo'=>$Photo, ':Prep'=>$Prep, ':Serves'=>$Serves,  ':Details'=>$Details, ':Ingredients'=>$Ingredients, ':Instructions'=>$Instructions, ':Notes'=>$Notes, ':nutrition'=>$nutrition)); 
/////////////////////
  include('class.upload.php');
//Check that we have a file

   // we instanciate the class for each element of $file
         $handle = new Upload($_FILES['photo']);

    // then we check if the file has been uploaded properly
    // in its *temporary* location in the server (often, it is /tmp)
    if ($handle->uploaded) {
   $handle->auto_create_dir = false;
   $handle->image_resize         = true;
   $handle->image_x              = 400;
   $handle->image_ratio_y        = true;

        // now, we start the upload 'process'. That is, to copy the uploaded file
        // from its temporary location to the wanted location
        // It could be something like $handle->Process('/home/www/my_uploads/');
    $handle->process('/home/xxxxxxxxx/public_html/family/recipes/images/'.$folder);

        // we check if everything went OK
          if ($handle->processed) {
       echo '<center><p>Image resized and uploaded to ' .$folder;
            // everything was fine !
         echo ' &nbsp; <a href="recipesview-pages.php">Go Back</a></p>';
        } else {
            // one error occured
            echo '<p class="result">';
            echo '  <b>File not uploaded to the wanted location</b><br />';
            echo '  Error: ' . $handle->error . '';
            echo '</p>';
        }


    } else {
        // if we're here, the upload file failed for some reasons
        // i.e. the server didn't receive the file
        echo '<center><p class="result">';
        echo '  <b>No file chosen for upload</b><br />';
        echo ' ' . $handle->error . '';
        echo '</p>';
    }


?>

The values are all posting and the photo is uploaded but that's as far as it goes. Any assistance would be appreciated. Thanks!

dalecosp

Not wanting to "rag on ya", but have you heard about testing for errors when you do stuff?

I don't see $conn initialized ... so I'll assume it's in the require()d file ... which means the script would "fatal error out" if the file was unavailable.

However, later on I see $success, and you never test the value of that to see if there was an error ... so how can I trust that you're actually checking that $conn is valid, either?

A short example:

$success=do_something();  // do_something should return a value, boolean, probably
                        //or something that can be interpreted as a pass/fail test
if (!$success) {
    echo "OMG!  I couldn't do_something!!!!<br>\n";
}

Like I said, not trying to be mean ... but do you know what I'm talking about and use some strategy for this?

Weedpacket

And if [font=monospace]$success[/font] is false, both [font=monospace]$stmt[/font] and [font=monospace]$conn[/font] have [font=monospace]errorInfo()[/font] methods that can be queried to find out why.

myliberty

dalecosp;11037057 wrote:
Not wanting to "rag on ya", but have you heard about testing for errors when you do stuff?

I don't see $conn initialized ... so I'll assume it's in the require()d file ... which means the script would "fatal error out" if the file was unavailable.

However, later on I see $success, and you never test the value of that to see if there was an error ... so how can I trust that you're actually checking that $conn is valid, either?

A short example:
$success=do_something();  // do_something should return a value, boolean, probably
                        //or something that can be interpreted as a pass/fail test
if (!$success) {
    echo "OMG!  I couldn't do_something!!!!<br>\n";
}
Like I said, not trying to be mean ... but do you know what I'm talking about and use some strategy for this?

So I added that to my code and I get

     echo "OMG!  I couldn't do_something!!!!<br>\n";

My Connection file is like this

// Connection data (server_address, database, username, password)
$hostdb = 'localhost';
$namedb = 'mysite_recipes';
$userdb = 'xxxx_xxxxx';
$passdb = 'xxxxxxxx';

// Display message if successfully connect, otherwise retains and outputs the potential error
try {
  $conn = new PDO("mysql:host=$hostdb; dbname=$namedb", $userdb, $passdb);
  echo 'Connected to database';
	  }
catch(PDOException $e)
 {
  echo $e->getMessage();
}

johanafm

And the output of the actual error information would be…?

myliberty

johanafm;11037139 wrote:
And the output of the actual error information would be…?

I got it somewhere I had a typo because I rewrote everything and got it working. So thank you to all who helped.