SQL syntax

Det kunne tyde på at de værdier du forsøger at indsætte i database, indeholder et tegn som SQL ikke kan li'. F.eks et en quote: ' - som du også bruger til at definerer streng start og slut i dine SQLs.

Løsningen kan være:

$headline = mysql_escape_string($_POST['headline']);
$bodytext = mysql_escape_string($_POST['bodytext']);

Helvede er dog at MySQL driveren er under udfasning og erstattes med den nye MySQLi driver. Så du møder et stort problem den da MySQL driveren forsvinder helt.

print_r kan ikke stå alene, den viser kun indholdet af af arrayet, ikke syntax fejl, og manglende erklæringer af variable, manglende $ tegn mm
så under udvikling SKAL error_reporting være slået til.

hvis du har fuldt indlægget 01.12.13 14:14, så kommer den ene fil op på serveren, og der står det rigtige i sql sætningen,

hvordan ser din komplette code ud

prøv at lave dette

$query = ("INSERT INTO news (headline, bodytext, date) VALUES ('{$headline}', '{$bodytext}', now())") or die(mysql_error());
$result = mysql_query($query, $connection);

om til

$query = "INSERT INTO news (headline, bodytext, date) VALUES ('$headline', '$bodytext', NOW())";
$result = mysql_query($query, $connection) or die(mysql_error());

kig på linje 7

Kode 

foreach($files as $file);
{ 

hvad laver det semicolon i slutningen af din foreeach ??
for hver file laver du ingen ting, hvorefter du starter en block.
start med at fjerne semicolon, så blokken bliver udført med data

foreach($files as $file)
{ 

mens du er igang med rettelser så skift dit mysql API
http://php.net/manual/en/mysqlinfo.api.choosing.php

Hej Ronny.

Tak for det hurtige svar, men desværre hjalp dette hellere ikke.
Der kommer intet ind i newsimg tabel i databasen eller i folderen.

Den du har set på er "news" som virker fint, så det går galt derefter.
Her kan du se hele mit script:

 <?php require_once("../login/includes/connection.php");
     $headline = $_POST['headline']; 
     $bodytext = $_POST['bodytext'];
     $files = array(); 
     $error = ""; 
      
     if(isset($_POST['submit']))
     { 
             if($headline == ""){
                     $error .="Du skal angrive en overskift.<br />";
             } 
             if($bodytext == ""){
                     $error .="Du kan ikke indsende en tom nyhed.<br />";
             }
      
             // Er alt ok fra formularen?
             if ($error == "")
             {
                     // Da vi uploader flere end én fil bliver vi nødt til at lave en løkke for at køre dem alle igennem
                     // $i indikerer hvilken fil vi arbejder med
                     for($i = 0; $i < count($_FILES); $i++)
                     {
                         // Er der en fil uploadet? 
                         if (! file_exists($_FILES["file"]["tmp_name"][i])) { 
                         // Fortsæt til næste løkke 
                         continue; 
                         }
                             $filenameArray = explode(".", $_FILES['file']["name"][i]);
                  
                     // Hiv fat i sidste ende af filnavnet (feks png)
                     $extension = end($filenameArray);
                  
                     // Angiv nyt unikt filnavn så filerne ikke overskriver hinanden
                     $newfile = uniqid() . "." . $extension;
                  
                     // Mappen filen skal ligge i
                     $folder = 'newsimg/';
                  
                     // Tilladte filtyper
                     $allowedTypes = array('image/gif', 'image/jpeg', 'image/jpg', 'image/pjpeg', 'image/x-png', 'image/png');
                  
                     // Er filen et billede? (baseret på filtyperne foroven)
                     if ( ! in_array($_FILES['file']["type"][$i], $allowedTypes))
                     {
                         $error = "Du må kun uploade billeder";
                     }
                     else
                     {
                         // Er filen for stor? Maks 2MB
                         if (($_FILES['file']["size"][$i] > 5097152))
                         {
                             $error = "Filen er for stor. Maks 5MB";
                         }
                         else
                         {
                             // Er der en fejl ved upload?
                             if ($_FILES["file"]["error"][$i] > 0)
                             {
                                 $error = "Fejl ved upload: " . $_FILES["file"]["error"][$i];
                             }
                             else
                             {
                                 // Kan vi gemme den nye fil? (@ gør at funktionen ikke udskriver en fejl på siden, hvis der er en fejl)
                                 if (@move_uploaded_file($_FILES["file"]["tmp_name"][$i], $folder . $newfile) )
                                 {
                                     $files[] = $newfile;
                                 }
                                 else
                                 {
                                     $error = "Kunne ikke uploade filen. Kontakt admin";
                                     break; // Stop løkken
                                 }
                  
                             }
                         }
                     }
                     }
      
                     // Er alt ok ved filerne?
                     if ($error == "")
                     {
                         $query = "INSERT INTO news (headline, bodytext, date) VALUES ('$headline', '$bodytext', NOW())";
 $result = mysql_query($query, $connection) or die(mysql_error()); 
 
                         $result = mysql_query($query, $connection);
                             if ($result) {
                                     // Gem id på nyhed i en variabel til brug senere
                                     $newsid = mysql_insert_id();
                                     // Indsæt filerne nu hvor nyheden er oprettet
                                     foreach($files as $file)
                                     {
                                         mysql_query("INSERT INTO newsimg (newsID, images) VALUES ('$newsid', '$file'") or die(mysql_error());
                                     }
                                     // Done
                                     echo "Nyhed oprettet.";
                             }
                             else {
                                     $error = "Noget gik galt, nyhed ikke oprettet. Prøv igen.";
                                     $error .= "<br />" . mysql_error();
                             }
                     }
             }
     }
     if (!empty($error)) {echo "<p>" . $error . "</p>";}
      
     ?>
     <form action="" method="post" enctype="multipart/form-data">
     Overskift:<br/>
     <input type="text" name="headline" maxlength="30" value="<?php echo $headline; ?>" /><br/>
     Tekst:<br/>
     <textarea name="bodytext" rows="9" cols="40"><?php echo $bodytext; ?></textarea><br/>
     Billeder:<br/>
     <input name="userfile[]" type="file" /><br />
     <input name="userfile[]" type="file" /><br />
     <input name="userfile[]" type="file" /><br />
     <input name="userfile[]" type="file" /><br />
     <input name="userfile[]" type="file" /><br />
     <input type="submit" name="submit" value="Opret nyhed" />
     </form>
      
     </body>
     </html>
     <?php
     if (isset($connection)){
             mysql_close($connection);}
     ?>

Kan du se, hvad det er der går galt? :s

Ps. Jeg var godt det med mysql - mysqli. Dette er dog noget til min eksamen, og har valgt at holde mig til det jeg 100% kender

Hej Michael.

Det er som sagt ikke news det er galt med, det er billederne. Det andet går ind uden problemer Det er newsimg, nyheden headline og bodytext fungere uden problemer. Det er KUN billederne der ikke kommer videre.

hvordan ser din connection.php ud ??
hvis du har navngivet din $connection i connection.php skal du også anvende navnet i alle dine mysql_query
jeg kan se du anvender $connection i linje 85

linje 85
$result = mysql_query($query, $connection);

linje 92
mysql_query("INSERT INTO newsimg (newsID, images) VALUES ('$newsid', '$file'") or die(mysql_error());
skal se sådan ud
mysql_query("INSERT INTO newsimg (newsID, images) VALUES ('$newsid', '$file'", $connection) or die(mysql_error());


linje referancerer er fra indlæg Sassy @ 30.11.13 19:24

i det gamle API havde man ikke behøvet at navngive sin connection, da den så blev en super global connection, der blev anvendt som default parameter til mysql_query, men hvis man havde navngivet den skal den angives i alle mysql_query
det er en udemærket ide at vænne sig til at navngive sin connection, da når du skifter til et nyere API SKAL du angive din connection

hvad uddannelse er du i gang med ??, hvorhenne ??
hvis din uddannelse skal bruges til noget skal den opdateres til nutidens code, det nytter ikke at anvende matrialer der ikke er opdateret i mange år

ved at forsætte med at bruge det gamle API skyder du dig selv i foden med stor kaliber.

hvis du har opdateret din PHP til min Version 5.5.0 frigivet 20-Jun-2013, http://www.php.net/ChangeLog-5.php#5.3.27 har du fået advarsel, hvis du har slået error_reporting til

mysql:
This extension is now deprecated, and deprecation warnings will be generated when connections are established to databases via mysql_connect(), mysql_pconnect(), or through implicit connection: use MySQLi or PDO_MySQL instead

Skulle lige kigge en ekstra gang - du har en syntaksfejl i din SQL. :o)

"INSERT INTO newsimg (newsID, images) VALUES ('$newsid', '$file'"

Mangler en slut parentes.

som micharl skrev mangler der lige en slut parantes
mysql_query("INSERT INTO newsimg (newsID, images) VALUES ('$newsid', '$file'", $connection) or die(mysql_error());

skal se sådan ud (utested)
mysql_query("INSERT INTO newsimg (newsID, images) VALUES ('$newsid', '$file')", $connection) or die(mysql_error());


sådan vil jeg skrive det, så ligner det også den anden sql du har

$sql = "INSERT INTO newsimg (newsID, images) VALUES ('$newsid', '$file')";
mysql_query($sql, $connection) or die(mysql_error());

og det er nemt at echo sql sætningen ud, som kontrol om det er det du forvendter der står der

jeg gider ikke teste det gamle API

Hej igen alle sammen

Tak for jeres svar, som er læst igennem og test - men det virker ikke endnu :s Andre forslag? Og tak fordi I gider hjælpe. Jeg synes bare det er underligt, at det slet ikke vil virke.

Nu bliver jeg ved med at få:
"Query was empty"

hvis du skriver sin sql sådan

$sql = "INSERT INTO newsimg (newsID, images) VALUES ('$newsid', '$file')";
echo $sql; // test hvad står der her ??? 
mysql_query($sql, $connection) or die(mysql_error());

hvilke data type er newsID og images