PHP udfordring - images upload driller

"Mit php upload script virker fint. Men hvis jeg vælger ikke at medtage et image i mit upload, så bliver mit upload ikke gennemført"??? Øhh, nej det gør det ikke.

Får du en fejlmeddelse, eller gennemføres scripted bare uden at uploade noget?

@skelboe & @michael_stim beklager min ringe forklaring.
mit upload system fungere på den måde, at du indtaster nogle oplysninger samt vedhæftet et billede og derefter uploader du det. Og som det er nu virker det fint. men hvis jeg vælger ikke at browse efter et billede men bare uploadet tekst, virkeer det ikke, den springer bare ud, rent exit uden fejlmelddelse.

Problemet er vel at dit script kun eksekveres ved filupload

Ved ikke hvor denne kommer fra:



Men er det den der trigger knappen, så skal den flyttes op over:

Nu har jeg flyttet if(isset($_POST['newfounding'])) {
som beskrivet men det hjælper ikke.
Hvis jeg vælger at indtaste data i min form og vælger et billede så virker det som det skal. Men jeg vil også gerne uploade tekst uden at vælge et billede.

Jeg kan se at hvis jeg ikke vælger at uploade et billede resten af felterne. bliver mit script stoppet her ->

header("location:newfoundings.php");
echo "Der kan kun anvendes filer af typen Gif & Jpeg";
exit;

Andre mulige løsninger ?

Vi ved ikke hvordan din kode ser ud nu. Mistænker at du ikke gør som jeg siger.

Har kun ændret det som du beskrev. Koden er som nedenstående :

$uploadDir = 'upload/';

//Hvis der er trykket på Upload
if(isset($_POST['newfounding'])) {

//Hvis filen findes og er større end 0
if(is_uploaded_file($_FILES['userfile']['tmp_name']) 
&& ($_FILES['userfile']['size'] >= 0))


$fileName = $_FILES['userfile']['name'];
$tmpName = $_FILES['userfile']['tmp_name'];
$fileSize = $_FILES['userfile']['size'];
if ((($_FILES["userfile"]["type"] == "image/gif") ||
    ($_FILES["userfile"]["type"] == "image/jpeg") ||
    ($_FILES["userfile"]["type"] == "image/png") ||
    ($_FILES["userfile"]["type"] == "image/pjpeg"))
&& ($_FILES["file"]["size"] < 2000000))
{
$fileType = $_FILES['userfile']['type'];
}
else
{
header("location:newfoundings.php");
echo "Der kan kun anvendes filer af typen Gif & Jpeg";
exit;

}

// get the file extension first
$ext = substr(strrchr($fileName, "."), 1);

// make the random file name
$randName = md5(rand() * time());

// and now we have the unique file name for the upload file
$filePath = $uploadDir . $randName . '.' . $ext;

$result = move_uploaded_file($tmpName, $filePath);

if (!$result) {
echo "Fejl, prøv igen.";
exit;
}

if(!get_magic_quotes_gpc())
{
$fileName = addslashes($fileName);
$filePath = addslashes($filePath);
}

mysql_query("INSERT INTO Foundings (picture_url, name, gps, users_id, dato, info, type, dane) VALUES ('$filePath', '$name', '$gps', '$users_id', '$dato', '$info', '$fundtype', '$dane')") or die(mysql_error());

}

Du har jo netop ikke gjort hvad jeg har sagt ;o)

if(is_uploaded_file($_FILES['userfile']['tmp_name'])
&& ($_FILES['userfile']['size'] >= 0))

Hvornår starter og slutter denne if? ;o)

@michael_stim, Ja den har jeg vist overset.
jeg har nu indsat { efter
if(is_uploaded_file($_FILES['userfile']['tmp_name'])
&& ($_FILES['userfile']['size'] >= 0))
Men det ser ikke lige ud til at virke.
Jeg får en Parse error: syntax error, unexpected $end på linje 104. Som er min slut header header("Location:fund.php");
Der var en slut }.

Og du afslutter også din while med en }?

@michael_stim, While ? jeg har ikke anvendt  en while funktion, men måske jeg skulle ?

If'en selvfølgelig. Den du satte { ved.

Arhh tak michael_stim , nu virker den som den skal.
Men er det muligt at indsætte en if således at hvis der ikke bliver tilføjet noget billede at man kan erstatte det med en standard ?

Her er min fulde kode ->

$uploadDir = 'upload/';

//Hvis der er trykket på Upload
if(isset($_POST['newfounding'])) {

//Hvis filen findes og er større end 0
if(is_uploaded_file($_FILES['userfile']['tmp_name']) 
&& ($_FILES['userfile']['size'] > 0)) {


$fileName = $_FILES['userfile']['name'];
$tmpName = $_FILES['userfile']['tmp_name'];
$fileSize = $_FILES['userfile']['size'];
if ((($_FILES["userfile"]["type"] == "image/gif") ||
    ($_FILES["userfile"]["type"] == "image/jpeg") ||
    ($_FILES["userfile"]["type"] == "image/png") ||
    ($_FILES["userfile"]["type"] == "image/pjpeg"))
&& ($_FILES["file"]["size"] < 2000000))
{
$fileType = $_FILES['userfile']['type'];
}
else
{
header("location:newfoundings.php");
echo "Der kan kun anvendes filer af typen Gif & Jpeg";
exit;

}

// get the file extension first
$ext = substr(strrchr($fileName, "."), 1);

// make the random file name
$randName = md5(rand() * time());

// and now we have the unique file name for the upload file
$filePath = $uploadDir . $randName . '.' . $ext;

$result = move_uploaded_file($tmpName, $filePath);

if (!$result) {
echo "Fejl, prøv igen.";
exit;
}

if(!get_magic_quotes_gpc())
{
$fileName = addslashes($fileName);
$filePath = addslashes($filePath);
}}}

mysql_query("INSERT INTO Foundings (picture_url, name, gps, users_id, dato, info, type, dane) VALUES ('$filePath', '$name', '$gps', '$users_id', '$dato', '$info', '$fundtype', '$dane')") or die(mysql_error());



header("Location:fund.php");

smider du et point svar -> michael_stim