læsning af pdf fil, going wrong

Tags: upload

Bruger #16555 @ 16.08.11 16:43

Hej folkens,

Jeg har lavet et script der uploader pdf filer til en database.

Nu skal jeg så læse disse filer på en anden side, og her er der problemer, da jeg faktisk ikke kan få filerne frem:

Her er min add_pdfs.php side:

PHP kode

 <?php require_once("includes/config.php"); ?>
 <?php require(MYSQL); ?>
 <?php $pdf_errors = array(); ?>
 <?php error_reporting(NULL); ?>
 <?php
 if(isset($_POST['submit'])){    
 
 if(empty($_POST['title'])){
     $pdf_errors[1] = '<small style="color:red;">Husk at indtaste en titel!</small>';    
 }else{
     $t = mysqli_real_escape_string($connection, strip_tags($_POST['title']));    
     }
 if(empty($_POST['description'])){
     $pdf_errors[2] = '<small style="color:red;">Husk at indtaste noget i description!</small>';    
 }else {
     $d = mysqli_real_escape_string($connection, strip_tags($_POST['description']));    
     }
     
 if(is_uploaded_file($_FILES['pdf']['tmp_name']) && ($_FILES['pdf']['error']) == UPLOAD_ERR_OK){
 $file = $_FILES['pdf'];
 }
 $size = round($file['size']);
 if($size > 1048576 ){// 1MB in BYTES
     $pdf_errors[3] = '<small style="color:red;">Filen var for stor - max størrelse er 1000K!</small>';    
             // THIS ERROR MESSAGES IS NOT BEING REGISTERED!
         }    
 if(($file['type']!= 'application/pdf') && (substr($file['name'],-4)!='.pdf'))
     {
     $pdf_errors[4] = '<small style="color:red;">Det skal være et PDF dokument, prøv igen!</small>';    
     }
 }
 
 if(isset($_POST['submit']) && empty($pdf_errors)){
     $destination = 'PDFS_DIR' .$tmp_name.'_tmp';
     $tmp_name = sha1($file['name'].uniqid('', true));
     
     move_uploaded_file($tmp_name, $destination);
     
     // MOVING THE FILE:
     if(move_uploaded_file($file['tmp_name'], $destination)){
         $_SESSION['pdf']['tmp_name'] = $tmp_name;
         $_SESSION['pdf']['size'] = $size;
         $_SESSION['pdf']['file_name'] = $file['name'];
         $upload_succes = '<h4>The file has been uploaded!</h4>';
         }else {
         trigger_error('<p>The file could not be moved!</p>');
         unlink($file['tmp_name']);    
         }                
             
 // Add pdf into database:
 $fn = mysqli_real_escape_string($connection, $_SESSION['pdf']['file_name']);
 $tmp_name = mysqli_real_escape_string($connection, $_SESSION['pdf']['tmp_name']);
 $size = mysqli_real_escape_string($connection, $_SESSION['pdf']['size']);
 $query = "INSERT INTO pdfs(tmp_name,title,description,file_name,size)
 VALUES
 ('$tmp_name','$t','$d','$fn','$size')";
 $result = mysqli_query($connection, $query);
 
 $succes = '<p>The file has been succesfully uploaded to the db!</p>';
 }    
 ?>

Så er filerne endt i databasen:
derefter har jeg lavet en side hvor jeg viser filerne, som via et link sender bruger videre til den side hvor hele pdf filen (Den valgte, red.) skal vises (OG SOM ER DER HVOR DET GÅR GALT)

PHP kode

 <?php
 // GET the PDFS:
 $query = "SELECT tmp_name, title, description, size FROM pdfs ORDER BY date_created DESC";
 $result = mysqli_query($connection, $query);
 if(mysqli_num_rows($result)>0){
 while($row = mysqli_fetch_array($result, MYSQLI_ASSOC)){
 echo "<div><h4><a href=\"view_pdf.php?id={$row['tmp_name']}\">
 {$row['title']}</a>&nbsp;&nbsp;&nbsp;({$row['size']}kb)</h4><p>{$row['description']}</p>
 </div>\n<br />";
     }
 }
 ?>

Her er så siden hvor jeg vil vise pdf filerne, efter de er klikket på via ovenstående side:
Men jeg får en undefined variable= $row i browseren og INGEN pdf vises.

PHP kode

 <?php require_once("includes/config.php"); ?>
 <?php require(MYSQL); ?>
 <?php $valid =''; ?>
 <?php
 if(isset($_GET['id'])&&(strlen($_GET['id'])==40)&&(substr($_GET['id'],0,1)!='.')){
 $file = 'PDFS_DIR'.$_GET['id'];
 //echo $file;
 if(file_exists($file)&&(is_file($file))){
 $query = 'SELECT title, description, file_name, FROM pdfs WHERE tmp_name='.mysqli_real_escape_string($connection, $_GET['id']).'';
 $result = mysqli_query($connection, $query);
 if($mysqli_num_rows($result) == 1){//OK!
      $row = mysqli_fetch_array($result, MYSQLI_ASSOC);
      echo $row['tmp_name'];
      $valid = true;
      
      $fs = filesize($file);
      
      header('Content-Length:$fs\n');
      header('Content-type:application/pdf');
      header('Content-Disposition: inline;filename="'.$row['file_name'].'"');     
      
 readfile($file);
 //exit(); 
         }
     }
 }
 ?>
 // Længere nede vil jeg så udskrive $row['description'] - og får en 
 // fejl herved
 <?php
 echo "<div>{$row['description']}</div>";
 if(!$valid){
 echo'<h4>Der er sket en fejl og PDF-filen kunne ikke vises. 
 Vi beklager ulejligheden, og vil løse problemet hurtigst muligt, da vores Administrator er blevet automatisk informeret!</h4>';    
 }
 ?>

Det virker til at denne linie ikke bliver læst:
if(file_exists($file)&&(is_file($file))){...

Kan nogen se hvad der skal laves for at få det til at virke?

Det er lidt en hastesag, og jeg har desværre ikke meget erfaring i file uploads, og visning af PDF'er.

Mvh,
Jan Klemmensen

Indlæg senest redigeret d. 16.08.2011 16:45 af Bruger #16555

8 svar postet i denne tråd vises herunder
0 indlæg har modtaget i alt 0 karma

Sorter efter stemmer Sorter efter dato

Bruger #10216 @ 16.08.11 16:59

4.283

Du har en parantes for meget på linje 8 i sidste kodeblok (den linje du selv fremhæver).

Men sikke da et rod, du bør da få ryddet op i din kode. Tror du ville have fundet fejlen selv, hvis din kode var organiseret

Eksempel på sidste blok

PHP kode

 <?php
 // PHP FØRST!!!
 require_once("includes/config.php");
 require(MYSQL);
 
 $valid = false;
 
 if(isset($_GET['id']) && strlen($_GET['id'])==40 && substr($_GET['id'],0,1) != '.') {
     $file = 'PDFS_DIR'.$_GET['id'];
  //echo $file;
     if(file_exists($file) && is_file($file)) {
         $sql = 'SELECT title, description, file_name, FROM pdfs WHERE tmp_name='.mysqli_real_escape_string($connection, $_GET['id']).'';
         $query = mysqli_query($connection, $sql);
         if($mysqli_num_rows($result) == 1){//OK!
             $result = mysqli_fetch_array($result, MYSQLI_ASSOC);
             
             echo $result['tmp_name']; // Cannot output anything other than the file.
             
             $valid = true;
       
             $fs = filesize($file);
       
             header('Content-Length:$fs\n');
             header('Content-type:application/pdf');
             header('Content-Disposition: inline;filename="'.$row['file_name'].'"');     
       
             readfile($file);
             die(); // die() stops here, exit() does not.  
          }
      }
 }
 ?>
 <!-- HTML SKAL KOMME TIL SIDST. -->
 <html>
     <body>
         <?php if(!$valid): ?>
         <h4>Der er sket en fejl</h4>
         <p>PDF-filen kunne ikke vises. Vi beklager ulejligheden, og vil løse problemet hurtigst muligt, da vores Administrator er blevet automatisk informeret!</p>    
         <?php endif; ?>
     </body>
 </html>

Indlæg senest redigeret d. 16.08.2011 17:01 af Bruger #10216

Bruger #16555 @ 16.08.11 17:14

Hej Michael,

Mange tak for dit svar!

Ja jeg er enig i at det er lidt en rode butik, jeg vil lave mere struktur på det når det funger..

Men nu har jeg det her kode, og får den samme fejl:

PHP kode

 <?php require_once("includes/config.php"); ?>
 <?php require(MYSQL); ?>
 <?php $valid = false; ?>
 <?php
 if(isset($_GET['id'])&&(strlen($_GET['id'])==40)&&(substr($_GET['id'],0,1)!='.')){
 
 $file = 'PDFS_DIR'.$_GET['id'];
 
 if(file_exists($file) && is_file($file)){
 $query = 'SELECT title, description, file_name, FROM pdfs WHERE tmp_name='.mysqli_real_escape_string($connection, $_GET['id']).'';
 
 $result = mysqli_query($connection, $query);
 if($mysqli_num_rows($result) == 1){//OK!
 $row = mysqli_fetch_array($result, MYSQLI_ASSOC);
 
 $valid = true; 
      
 $fs = filesize($file);
      
 header("Content-Length:$fs\n");     
 header('Content-type:application/pdf');
 header('Content-Disposition:inline;filename='.$row['file_name'].'');    
      
 readfile($file);
      
 die();      
       }
    }
 }
 ?>
 <!DOCTYPE HTML>
 <html>
 <head>
 <meta charset="utf-8">
 <title>View PDF artikel!</title>
 <style type="text/css">
 * {margin:0px; padding:0px;}
 #wrapper {margin-left:auto; margin-right:auto; width:600px; height:auto; margin-top:15px;}
 #content {width:598px; height:598px; border:1px solid #999; background-color:#CCC;}
 </style>
 </head>
 
 <body>
 <div id="wrapper">
 
         <div id="content">
         <h1>Den valgte fil:</h1>
         <a href="pdfs.php">Se oversigten over uploadede PDF'er her!</a>
         <br /><br />
 <?php
 if(!$valid){
 echo'<h4>Der er sket en fejl og PDF-filen kunne ikke vises. 
 Vi beklager ulejligheden, og vil løse problemet hurtigst muligt, da vores Administrator er blevet automatisk informeret!</h4>';    
 }else {
 echo "<div>{$row['description']}</div>";    
 }
 ?>
 </div>
         
 </div>
 </body>
 </html>

Som du skriver, så skal php ført, og derefter html, hvilket jeg jo også har i mit eksempel?

Til sidst har jeg php i html, da jeg vel skal echo $row['description']; så brugerne kan læse pdf'en.

Men jeg får $row = undefined variable, og filen læses ikke.

Kan du se hvordan det kan være?
Er det forkert grebet an?

jeg har læst mig frem til ovenstående da jeg ikke har uploadet filer til DB før.

Hvordan kan det være at der skal laves: die(); i det hele taget? Kan man ikke bare køre videre gennem sriptet, og echo det der nu skal echo'es ud længere nede?

Kan jeg have defineret PDFS_DIR forkert? Hvis du vil se på det?
Det er meningen jeg vil tilbage til min root folder, og ind i includes/pdfs.

PHP kode

 define('PDFS_DIR', $_SERVER['DOCUMENT_ROOT'] .'includes/pdfs');

Er dette korrekt?

Indlæg senest redigeret d. 16.08.2011 17:33 af Bruger #16555

Bruger #10216 @ 16.08.11 17:44

4.283

Når du bruger konstanter så skal det være uden gnyffer rundt om. Det er fint når du definerer dem, men herefter skal du ikke...

Bruger #16555 @ 16.08.11 18:37

Okay, jeg har fjernet de single quotes der var omkring, men det resulterer i det samme, hvilket ikke er så prangende meget..

Dvs. at det stadig ikke fungerer :roll:

Hvis der skulle være en der har en halv time, så sidder jeg bundet fast til stolen, og venter på at blive klogere, og ikke midst få fundet ud af hvordan jeg kan læse den PDF fra databasen

Bruger #10576 @ 16.08.11 19:12

287

$query = 'SELECT title, description, file_name, FROM pdfs WHERE tmp_name='.mysqli_real_escape_string($connection, $_GET['id']).'';

Det ser pænt mærkeligt ud, er det virkelig navnet på en pdf fil du der får retur ?

Da du senere bruger din "row" er du ude af din if sætning, så der eksisterer din variablen jo ikke længere, den er kun defineret i din funktion deroppe.

Indlæg senest redigeret d. 16.08.2011 19:17 af Bruger #10576

Bruger #16555 @ 16.08.11 19:34

Jeg har gjort det, at når en pdf bliver uploaded til databasen, så går navnet gennem en sha1, og det er så det jeg har trukket ned fra databasen og brugt i $_GET, dvs de links der bliver lavet.

Men som sagt så har jeg ikke lavet uploads før, så jeg kan ikke lige gennemskue pt hvad der ellers er den smarteste måde at gøre det på.

Forslag er velkomne.. :-)

PS: Jeg prøvede lige at echo $row['description']; inde i if sætningen, og der skete intet. Så der bliver vel ikke fundet noget som helst.

Kender du til en løsning eller en tutotial der viser præcist det jeg prøver på at lave?

Tak for svar!

Bruger #10576 @ 16.08.11 19:40

287

1) Har du kørt din SELECT og set at der kommer data ud fra databasen ?
2) Hvad melder den af fejl nu?
3) Lav fx en if(isset($row]))
echo "OK";
else
echo "FEJL";

når du prøver at hente data ud, så du ved om du har fået data.
4) hvis ikke, check at du har en database forbindelse.
5) hvis der er db-forbindelse, check at du er på den rigtige database.
6) skriv igen

PS: Prøv også lige at hent ud med indexes, altså med $row[0] og se om den kan spytte data ud sådan i stedet evt.

Indlæg senest redigeret d. 16.08.2011 19:46 af Bruger #10576

Bruger #16555 @ 16.08.11 20:26

Jeg har lige gået igennem tjeck listen :-)

Der sker ikke noget hverken når jeg tjekker om $row er set, eller ved at bruge indexes ved output.

Alt der kører efter:
9. if(file_exists($file) && is_file($file)){....

bliver ikke kørt virker det til. Så vidt jeg kan se, kan det så være noget med hvordan filerne bliver uploadet ELLER hvordan/HVOR de bliver gemt?

Altså hvordan jeg definere der hvor de skal gemmes midlertidigt? Hvis jeg har forstået processen korrekt.

Jeg kan prøve at lægge alt ud, dvs.

1)Filen hvor pdf'erne bliver uploadet.
2)Filen hvor pdf'erne bliver lavet til links, der går til siden hvor jeg gerne vil vise hele filen når den er valgt.
3)Filen som skal vise pdf'en (Som ikke kan finde den, eller hvad der sker)

1)

PHP kode

 <?php require_once("includes/config.php"); ?>
 <?php require(MYSQL); ?>
 <?php $pdf_errors = array(); ?>
 <?php error_reporting(NULL); ?>
 <?php
 if(isset($_POST['submit'])){    
 
 if(empty($_POST['title'])){
 $pdf_errors[1] = '<small style="color:red;">Husk at indtaste en titel!</small>';    
 }else{
 $t = mysqli_real_escape_string($connection, strip_tags($_POST['title']));    
 }
 if(empty($_POST['description'])){
 $pdf_errors[2] = '<small style="color:red;">Husk at indtaste noget i description!</small>';    
 }else {
 $d = mysqli_real_escape_string($connection, strip_tags($_POST['description']));    
 }
     
 if(is_uploaded_file($_FILES['pdf']['tmp_name']) && ($_FILES['pdf']['error']) == UPLOAD_ERR_OK){
 $file = $_FILES['pdf'];
 }
 $size = round($file['size']);
 if($size > 1048576 ){// 1MB in BYTES
 $pdf_errors[3] = '<small style="color:red;">Filen var for stor - max størrelse er 1000K!</small>';    
 // THIS ERROR MESSAGES IS NOT BEING REGISTERED!
         }    
 if(($file['type']!= 'application/pdf') && (substr($file['name'],-4)!='.pdf'))
 {
 $pdf_errors[4] = '<small style="color:red;">Det skal være et PDF dokument, prøv igen!</small>';    
 }
 }
 
 if(isset($_POST['submit']) && empty($pdf_errors)){
 $destination = 'PDFS_DIR' .$tmp_name.'';
 $tmp_name = sha1($file['name'].uniqid('', true));
     
 move_uploaded_file($tmp_name, $destination);
     
 // MOVING THE FILE:
 if(move_uploaded_file($file['tmp_name'], $destination)){
 $_SESSION['pdf']['tmp_name'] = $tmp_name;
 $_SESSION['pdf']['size'] = $size;
 $_SESSION['pdf']['file_name'] = $file['name'];
 $upload_succes = '<h4>The file has been uploaded!</h4>';
 }else {
 trigger_error('<p>The file could not be moved!</p>');
 unlink($file['tmp_name']);    
 }                
             
 // Add pdf into database:
 $fn = mysqli_real_escape_string($connection, $_SESSION['pdf']['file_name']);
 $tmp_name = mysqli_real_escape_string($connection, $_SESSION['pdf']['tmp_name']);
 $size = mysqli_real_escape_string($connection, $_SESSION['pdf']['size']);
 $query = "INSERT INTO pdfs(tmp_name,title,description,file_name,size)
 VALUES
 ('$tmp_name','$t','$d','$fn','$size')";
 $result = mysqli_query($connection, $query);
 
 $succes = '<p>The file has been succesfully uploaded to the db!</p>';
 }    
 ?>

PHP kode

 <?php
 // GET the PDFS:
 $query = "SELECT tmp_name, title, description, size FROM pdfs ORDER BY date_created DESC";
 $result = mysqli_query($connection, $query);
 if(mysqli_num_rows($result)>0){
 while($row = mysqli_fetch_array($result, MYSQLI_ASSOC)){
 echo "<div><h4><a href=\"view_pdf.php?id={$row['tmp_name']}\">
 {$row['title']}</a>&nbsp;&nbsp;&nbsp;({$row['size']}kb)</h4><p>{$row['description']}</p>
 </div>\n<br />";
 }
 }        
 ?>

PHP kode

 <?php require_once("includes/config.php"); ?>
 <?php require(MYSQL); ?>
 <?php $valid = false; ?>
 <?php
 if(isset($_GET['id'])&&(strlen($_GET['id'])==40)&&(substr($_GET['id'],0,1)!='.')){
 $file = PDFS_DIR .$_GET['id'];
 //echo $file;
 if(file_exists($file) && (is_file($file))){
 $query = 'SELECT title, description, file_name, FROM pdfs WHERE tmp_name='.mysqli_real_escape_string($connection, $_GET['id']).'';
 $result = mysqli_query($connection, $query);
 if(mysqli_num_rows($result) == 1){//OK!
 $row = mysqli_fetch_array($result, MYSQLI_ASSOC);
 echo $row[3];
 $valid = true; 
 $fs = filesize($file);
      
 header("Content-Length:$fs\n");     
 header('Content-type:application/pdf');
 header('Content-Disposition:inline;filename="'.$row['file_name'].'"');    
      
 readfile($file);    
 die();      
 }
 }
 }
 ?>

I config.php, som inkluderers i toppen af hvert script, definerer jeg PDFS_DIR - Som jeg bruger i scriptet. Jeg er ikke sikker på jeg har gjort det rigtigt?

Her er den constant:

PHP kode

 define('PDFS_DIR', $_SERVER['DOCUMENT_ROOT'] .'includes/pdfs');

Når jeg har uloadet nogen test pdf'er, så er det ikke i den folder der er kommet en fil, men ude i root niveau. Selvom jeg har forsøgt..at sende pdf'erne (tmp_name, er det ikke korrekt?) ind til includes/pdfs

Spørg endelig hvis denne forklaring ikke gav mening.

Men det er det jeg har nu, og der kan godt uploades, det er det sidste script, hvor jeg vil vise filerne der ikke virker. Men måske er der er sammenhæng hen over de 3 filer der ikke er ok?

Indlæg senest redigeret d. 16.08.2011 20:28 af Bruger #16555

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler