Problem med bilduppladdning

Du har en massa odefinerade variabler. Dom måste innehålla ett värde innan du använder dom. Kolla i originalskriptet och se hur dom är definerade (tilldelade värden) där. ex:

$new_image_large = ...

Då borde du också se vilka värden som borde finnas i dom.

header("Location: product_add.php");

Klagar den på för att du fått en massa felmedelanden. När dom är fixade försvinner den.

Aha okej.
Jag har kollar hur originalscriptet ser ut och kommit fram till att samma felmess dyker upp om jag skulle provköra det.
Jag antar då att det räcker med att sätta $variabel = ""; för dessa som det gbälls om, eftersom de inte används om bilden _inte_ behöver förminskas.

Återkommer...

Hmm, det där funkade inte riktigt...
Vilka är argumenten som ska användas till:

imagejpeg($new_image_large, $save_img_large, $img_compress);
imagejpeg($new_image_small, $save_img_small, $img_compress);

Det verkar som om felet uppstår när bilden alltså inte behöver förminskas och då eftersom ovanstående variabler (bortsett från $img_compress) aldrig sätts.

Vad ska jag sätta dom till i mitt script ovan, för att det ska bli rätt?

Att sätta variablen till "" gör ingen nytta. Det är samma sak som den innehåller om den inte är satt.

Ska kolla lite i originalskriptet.

/D

Jag skulle nog prova att flytta dom raderna lite uppåt. Till alldeles efter

// Kopiera bilden till sin mapp
$save_img_large = $fsize_dir . 'product_' . $last_id . $product_photo_ext;
$save_img_small = $thumb_dir . 'product_' . $last_id . $product_photo_ext;
				
imagedestroy($image);

Här är mitt förslag till ditt skript. Inte testad och hunden vill ut. Prova.

<?php
if(isset($_GET['do']) && $_GET['do'] = "add_product"){
	$product_cat			= fix_sql($_POST['product_cat']);
	$product_name			= fix_sql($_POST['product_name']);
	$product_article		= fix_sql($_POST['product_article']);
	$product_photo_name		= $_FILES['product_photo']['name'];
	$product_photo_tmp_name	= $_FILES['product_photo']['tmp_name'];
	$product_photo_type		= $_FILES['product_photo']['type'];
	$product_photo_ext		= strtolower(substr($product_photo_name, (strrpos($product_photo_name, '.'))));
	$product_prize			= fix_sql($_POST['product_prize']);
	$product_about			= fix_sql($_POST['product_about']);

	$fsize_dir = './../pics/product/';
	$thumb_dir = './../pics/product/thumbs/';
	$new_img_width_large = 480;
	$new_img_width_small = 110;
	$img_compress = 85;
	
	$query = "INSERT INTO site_products "
			."(product_cat, product_name, product_article, product_prize, product_about) "
			."VALUES "
			."("
			."" . $product_cat . ", "
			."'" . $product_name . "', "
			."'" . $product_article . "', "
			."" . $product_prize . ", "
			."'" . $product_about . "'"
			.")";
	mysql_query($query, $the_link) or die("<p>Sql frågan genererade ett fel:<br /><span>Orsak: " . mysql_error() . "</span></p></body></html>");
	$last_id = mysql_insert_id($the_link);
	if(isset($last_id) && is_numeric($last_id) && !empty($product_photo_name))
	{
		if ($product_photo_type == "image/pjpeg" || $product_photo_type == "image/gif" || $product_photo_type == "image/jpeg")
		{
			// Proceed img_upload...
			$query =	 "UPDATE site_products SET "
						."product_picture = 'product_" . $last_id . $product_photo_ext . "' "
						."WHERE product_id = " . $last_id . " "
						."LIMIT 1";
			mysql_query($query) or die(mysql_error());
			$img_x_y_size =  getimagesize($product_photo_tmp_name);

			if ($new_img_width_large < $img_x_y_size[0])
			{
				$new_img_height_large = ceil(($new_img_width_large / $img_x_y_size[0]) * $img_x_y_size[1]);
				$new_img_height_small = ceil(($new_img_width_small / $img_x_y_size[0]) * $img_x_y_size[1]);

				// Gör om bildens storlek
				$new_image_large = imagecreatetruecolor($new_img_width_large, $new_img_height_large);
				$new_image_small = imagecreatetruecolor($new_img_width_small, $new_img_height_small);

				// Kontrollera vilken bildtyp det är (1=gif , 2=jpg , 3=png)
				switch($img_x_y_size[2])
				{
					case 1:
						$image = imagecreatefromgif($product_photo_tmp_name);
					break;
					case 2:
						$image = imagecreatefromjpeg($product_photo_tmp_name);
					break;
					case 3:
						$image = imagecreatefrompng($product_photo_tmp_name);
					break;
				}

				imagecopyresampled($new_image_large, $image, 0, 0, 0, 0, $new_img_width_large, $new_img_height_large, $img_x_y_size[0], $img_x_y_size[1]);
				imagecopyresampled($new_image_small, $image, 0, 0, 0, 0, $new_img_width_small, $new_img_height_small, $img_x_y_size[0], $img_x_y_size[1]);

				// Kopiera bilden till sin mapp
				$save_img_large = $fsize_dir . 'product_' . $last_id . $product_photo_ext;
				$save_img_small = $thumb_dir . 'product_' . $last_id . $product_photo_ext;
				
				imagedestroy($image);

				imagejpeg($new_image_large, $save_img_large, $img_compress);
				imagejpeg($new_image_small, $save_img_small, $img_compress);
			
				imagedestroy($new_image_large);
				imagedestroy($new_image_small);
			}
			else
			{
				move_uploaded_file($product_photo_tmp_name, $fsize_dir . 'product_' . $last_id . $product_photo_ext);
				move_uploaded_file($product_photo_tmp_name, $thumb_dir . 'product_' . $last_id . $product_photo_ext);
			}
		}
		else
		{
			// Inte en bild...
		}
	}
	header("Location: product_add.php");
	exit;
}
?>

/D

Nu är jag nästa hemma.
Det funkar nu att ladda upp utan att få felmeddelandet, men...
När jag laddar upp en bild som inte ska förminskas så skapas det ingen thumb.
Nu vet jag inte om scriptet gjorde det från början heller, men det är iaf. meningen.
Oavsett om bilden man laddar upp ska förminskas eller inte, så måste det alltid skapas en miniatyr.

Vore även glad om du kunde förklara hur du gjorde om scriptet, så att det slutade strula.

thumbs skapas i thumb.php som finns i samma paket, inte i photo.php.

Ska se var och hur det lämpligast stoppas in, om en stund.

Som jag skrev i mitt förra inlägg så flyttade jag raderna du fick felmeddelanden på. Jag flyttade upp dom så dom bara anropas om bildens storlek ändras, annars gör dom iaf ingen nytta.

Platsen där dom var innan är en bra plats att skapa thumbs på.

edit: thumb.php bara roterar bilden.

/J

Prova byta ut

            else
            {
                move_uploaded_file($product_photo_tmp_name, $fsize_dir . 'product_' . $last_id . $product_photo_ext);
                move_uploaded_file($product_photo_tmp_name, $thumb_dir . 'product_' . $last_id . $product_photo_ext);
            }

i mitt tidigare förslag mot.

			else
			{
				$new_img_height_small = ceil(($new_img_width_small / $img_x_y_size[0]) * $img_x_y_size[1]);
				$new_image_small = imagecreatetruecolor($new_img_width_small, $new_img_height_small);
				switch($img_x_y_size[2])
				{
					case 1:
						$image = imagecreatefromgif($product_photo_tmp_name);
					break;
					case 2:
						$image = imagecreatefromjpeg($product_photo_tmp_name);
					break;
					case 3:
						$image = imagecreatefrompng($product_photo_tmp_name);
					break;
				}
				imagecopyresampled($new_image_small, $image, 0, 0, 0, 0, $new_img_width_small, $new_img_height_small, $img_x_y_size[0], $img_x_y_size[1]);
				$save_img_small = $thumb_dir . 'product_' . $last_id . $product_photo_ext;
				imagedestroy($image);
				imagejpeg($new_image_small, $save_img_small, $img_compress);
				imagedestroy($new_image_small);

				move_uploaded_file($product_photo_tmp_name, $fsize_dir . 'product_' . $last_id . $product_photo_ext);
			}

Det är varken speciellt snyggt eller testat men det borde funka (tror jag). Jag har bara kopierat lite grejer från delen som förminskar bilden om den är för stor. Det verkar som att originalskriptet förutsatte att bilderna skulle vara för stora.

/D

Tackar!
Det fungerar som en dans.
Man kanske kan lägga det där i en egen fil som man infogar på sidan (eller i en funktion) om man vill ha det lite snyggare.
I det här fallet spelar det dock ingen roll om det är snyggt eller inte, det är funktionen jag är ute efter och den har jag nu.

Tack!