Fejl i mit upload script

Hov tror fejlen ligger i min .htaccess fil i mappen, den giver nemlig kun en speciel fil lov til at komme ind. Det kode fra .htaccess filen er:

##Start rewrite engine
RewriteEngine on
##Hvis loades fra side.com/index.php med valgfri www samt mulighed for subdomain
RewriteCond %{HTTP_REFERER} !^http://(www\.)?([^.]+\.)?babegalleri.dk/images/users/hotornot/index.php [nc]
##send brugeren til main domain
RewriteRule /* http://www.babegalleri.dk [R,L]

AAAAAARGH lær at bruge indrykning!
Jeg tror det er fordi typen er forkert, prøv:

<?

    if (!isset($_SESSION['username']))
    {
        include('login_required.php');
    }
    else
    {
?>

    <p align="center">Tilføj billede</p>
<?

        if(isset($_POST['UploadBotton']))
        {
       
            $warning[] = array();
            $image = $_FILES["image"]["tmp_name"];
           
            if($file = @getimagesize($image))
            {
                if($file[2] == 1)
                {
                    $type = ".gif";
                }
                else if($file[2] == 2)
                {
                    $type = ".jpg";
                }
                else
                {
                    $warning[] = "Kun .jpg eller .gif billedetyper bliver godkendt!";
                }
            }
            else
            {
                $warning[] = "Kun .jpg eller .gif billedetyper bliver godkendt!";
            }
       
            if ( count( $warning ) > 0 )
            {
                die($warning[0]);
            }

            if($type == ".gif")
            {
                $img = imagecreatefromgif($image);
            }
            else
            {
                $img = imagecreatefromjpeg($image);
            }
       
            $big_width = imagesx($img);
            $big_height = imagesy($img);
            if($big_width < 340)
            {
                $warning[] = "Billedet er for smalt, det skal minimum være 340 pixel bredt.";
            }
   
            if(!is_array($warning) && count($warning)>0)
            {
                echo 'Dit billede blev ikke uploadet, dette skyldes nedenstående fejl:<br><font color="#FF0000">';
                echo implode("-" , $warning);
                echo '</font>';
            }
            else
            {
       
                mysql_query("INSERT INTO hotornot (userid,img_type,img_accepted) VALUES ('$_SESSION[userid]','$type','no')") or
                    die(mysql_error());
       
                $big_img_to = "images/users/hotornot/images/".mysql_insert_id()."_big".$type;
                $img_to = "images/users/hotornot/images/".mysql_insert_id().$type;
       
                copy($_FILES["image"]["tmp_name"],$big_img_to);
       
                $new_width = 340 / $big_width;
                $new_height = round($big_width * $new_width,0);
       
                $dst_img = imagecreatetruecolor($new_width,$new_height);
       
                if($type == ".gif")
                {
                    $src_img = ImageCreateFromGif($big_img_to);
                }
                else
                {
                    $src_img = ImageCreateFromJpeg($big_img_to);
                }
       
                ImageCopyResampled($dst_img,$src_img,0,0,0,0,$new_width,$new_height,$big_width,$big_height);
       
                if($type == ".gif")
                {
                    ImageGif($dst_img, $img_to, 100);
                }
                else
                {
                    ImageJpeg($dst_img, $img_to, 100);
                }
       
                unlink("images/users/hotornot/images/".mysql_insert_id()."_big".$type);
       
            }
        }
        else
        {

?>
    <script type="text/javascript">
    <!--
    function CheckUpload(f)
    {
   
        if (f.image.value == "")
        {
            alert("Du skal vælge et billede!");
            f.image.focus();
            return false;
        }
        f.UploadButton.disabled=true;
        f.UploadButton.value = 'Øjeblik!';
        return true;
    }
    -->
    </script>
   
    <form method="POST" onsubmit="return CheckUpload(this);" action="main.php?page=img_add" enctype="multipart/form-data">
    <p>Alle billeder der bliver tilføjet her, vil blive vist på Hot or Not siden og evt. på toplisterne, hvis den får en god
   
    bedømmelse.<br><br>Billede:<br><input type="file" name="image"><br><br><input type="submit" value="Tilføj"
   
    name="UploadBotton"></p></form>
<?

        }

    }

?>

Hmm.. hvorfor indryk? Gør mig bare forviret, men nu lige meget om jeg bruger .jpg, .gif eller en anden filtype skriver den bare Array også skriver den ikke mere.

Den første:

$warning[] = array();

skal bare være:

$warning = array();

Prøv så igen - Indrykning er for at gøre det mere overskueligt - Det er jo umuligt at overskue kode der bare står på samme niveau ;P

Jeg tror faktisk at getimagesize ikke virker på mit webhotel, hvad kunne det ellers være, fordi før var der jo heller ingenting der kunne få fat i $type

Selv om jeg prøver med .jpg, .txt, .gif, .bmp eller noget, så får jeg fejlen: Kun .jpg eller .gif billedetyper bliver godkendt!

Prøv manuelt at uploade et jpg billede med ftp, og brug getimagesize på det. Det burde da bare virke ;P
Evt. dump noget debug:

var_dump( $_FILES );

evt...

Har prøvet med dette:

<?
if($file = @getimagesize(images/logo.jpg))
            {
                if($file[2] == 1)
                {
                    $type = ".gif";
                }
                else if($file[2] == 2)
                {
                    $type = ".jpg";
                }
                else
                {
echo "Kun .jpg eller .gif billedetyper bliver godkendt!";
                }
            }
            else
            {
echo "Kun .jpg eller .gif billedetyper bliver godkendt!";
            }
echo $type;
?>

http://www.babegalleri.dk/test/test.php

billedet:
http://www.babegalleri.dk/test/images/logo.jpg

Prøv lige at fjerne @'et

Nu giver den fejl:
http://www.babegalleri.dk/test/test.php

satte lige "" uden om, nu virker den, nu skriver den .jpg

hahha ja det havde jeg slet ikke set :P

Men skal jeg så bare fjerne @'et eller hvad?

Nej, @ må gerne blive der, hvis der er .txt-filer osv, det var bare lige for at se om den skrev noget! :)

Men hvad er fejlen så i den lange kode? Skal jeg bare smie "" om $image?

Ja, det ved vi jo stadig ikke. Prøv at lave en:

var_dump( $_FILES );

Ved ikke lige hvad du mener med at lave en var_dump( $_FILES );?

prøv at skrive:

echo "<pre>;" var_dump( $_FILES ); echo "</pre>";
exit;

i toppen af scriptet, så kan du se om der kommer noget med

array(0) {
}

Ja, så bliver der ikke uploadet noget ;)
Hvordan ser din form ud? Har du husket enctype="multipart/form-data" ?

Vidste godt at der var små fejl i, men dem havde jeg ellers tænkt mig at rette bagefter:

    <script type="text/javascript">
    <!--
    function CheckUpload(f)
    {
   
        if (f.image.value == "")
        {
            alert("Du skal vælge et billede!");
            f.image.focus();
            return false;
        }
        f.UploadButton.disabled=true;
        f.UploadButton.value = 'Øjeblik!';
        return true;
    }
    -->
    </script>
   
    <form method="POST" onsubmit="return CheckUpload(this);" action="main.php?page=img_add" enctype="multipart/form-data">
    <p>Alle billeder der bliver tilføjet her, vil blive vist på Hot or Not siden og evt. på toplisterne, hvis den får en god
   
    bedømmelse.<br><br>Billede:<br><input type="file" name="image"><br><br><input type="submit" value="Tilføj"
   
    name="UploadBotton"></p></form>

Den viser heller ikke Øjeblik, som den skal.

Endelig kom siden op igen. Kan du se min fejl? Kan nemlig ikke selv :(

Nej, ikke umiddelbart :( Så vidt jeg lige kan se burde der blive uploadet en fil - Hvorfor det ikke sker ved jeg ikke lige...

Prøver lige med:

<form method="POST" action="main.php?page=img_add" enctype="multipart/form-data"><input type="file" name="image"><input type="submit" value="Tilføj" name="UploadBotton"></form>

Det virker ikke engang :S

Prøv lige et nyt script:

<?php

  if ( isset( $_POST['upl']) )
  {
    var_dump( $_FILES );
  }

?>
<form method="post" action="<?= $_SERVER['PHP_SELF'] ?>" enctype="multipart/form-data">
<input type="file" name="minfil">
<input type="submit" value="test" name="upl">
</form>

Nu har jeg lige prøvet med:

<?
if(isset($_GET[ok])) {
echo "<pre>";
var_dump( $_FILES );
echo "</pre>";
exit;

} else {

echo '<form method="POST" action="?ok" enctype="multipart/form-data">';

echo '<p>Fil:<br><input type="file" name="fil"><p>';

echo '<p><input type="submit" value="Upload" name="submit"></p>';
echo '</form>';

}
?>

Den skriver også bare:

array(0) {
}

Din giver det samme: array(0) { }

Der er noget galt med dit webhotel, for det der burde virke :P

Damn havde jeg også på fornemmelsen, har lige sendt et brev til powerhosting.

Men har jeg ikke andre muligheder?

For hvad? Som det der nu kan du ikke uploade! :)

Hele min side handler om at folk skal kunne uploade :(

hehe så må du jo vente på at din udbyder retter fejlen ;)

Så har de aktiveret det igen, de sagde at det var noget sikkerheds haløj, men når får jeg:

array(1) {
  ["image"]=>
  array(5) {
    ["name"]=>
    string(10) "SIMONE.jpg"
    ["type"]=>
    string(11) "image/pjpeg"
    ["tmp_name"]=>
    string(14) "/tmp/phpsnvgHe"
    ["error"]=>
    int(0)
    ["size"]=>
    int(79034)
  }
}

Fint, nu kan du fjerne:

echo "<pre>;" var_dump( $_FILES ); echo "</pre>";
exit;

Og se om det virker ;P

http://babegalleri.dk/test/img_add.phps - Der kan du se koden
http://babegalleri.dk/test/ - Brugernavn: WooX - Kode: asdasd

Gå ind i tilføj billede og prøv, den skriver bare hver gang at man kun må uploade gif eller jpg :S

Prøv lige at ændre:

            if($file = @getimagesize($image))
            {
                if($file[2] == 1)
                {
                    $type = ".gif";
                }
                else if($file[2] == 2)
                {
                    $type = ".jpg";
                }
                else
                {
                    $warning[] = "Kun .jpg eller .gif billedetyper bliver godkendt!";
                }
            }
            else
            {
                $warning[] = "Kun .jpg eller .gif billedetyper bliver godkendt!";
            }


til:

            if($file = @getimagesize($image))
            {
                if($file[2] == 1)
                {
                    $type = ".gif";
                }
                else if($file[2] == 2)
                {
                    $type = ".jpg";
                }
                else
                {
                    $warning[] = "Kun .jpg eller .gif billedetyper bliver godkendt!";
                }
            }
            else
            {
                $warning[] = "Du skal uploade et .jpg eller .gif billede!";
            }

Evt. lige efter:

$image = $_FILES["image"]["tmp_name"];

smid:

echo "<pre>"; var_dump( $_FILES ); echo "</pre>";

array(1) {
  ["image"]=>
  array(5) {
    ["name"]=>
    string(10) "SIMONE.jpg"
    ["type"]=>
    string(11) "image/pjpeg"
    ["tmp_name"]=>
    string(14) "/tmp/phpuvuTP1"
    ["error"]=>
    int(0)
    ["size"]=>
    int(79034)
  }
}

Du skal uploade et .jpg eller .gif billede!

Weird. Lige efter:

            if($file = @getimagesize($image))
            {

Prøv at skrive:

              echo "<pre>"; var_dump( $file ); echo "</pre>";

Ved du om billedet er gemt fra photoshop?

Ja det er fra photoshop, det er en falsk check, som mine forældre skal give min søster i konfimations gave i morgen.

Skummelt hvsi jeg smider det lige efter

            if($file = @getimagesize($image))
            {

så skriver den bare:

Du skal uploade et .jpg eller .gif billede!

Paste lige hele koden, eller lav en .phps

PS. Hvis du bruger photoshop, *SKAL* du gemme vha. "Save for web" - Photoshop kan ikke finde ud af at lave ordentlige jpeg-billeder hvis man bruger "Save as" eller "Save as a copy".

http://babegalleri.dk/test/img_add.phps

Men jeg har prøvet med billeder taget med mit digital kamera, og den vil heller ikke rigtigt.

Dem har du tilfældigvis ikke behandlet i photoshop? :)

Hmm, det er underligt. Det lader ikke til at getimagesize() vil læse dem - Prøv den lige i et script for sig selv, med et billede der findes på serveren:

<?php

  $inf = getimagesize( "../images/logo.jpg" );

  echo "<pre>"; var_dump( $inf ); echo "</pre>";

?>

Den burde give dette output:

array(7) {
  [0]=>
  int(750)
  [1]=>
  int(150)
  [2]=>
  int(2)
  [3]=>
  string(24) "width="750" height="150""
  ["bits"]=>
  int(8)
  ["channels"]=>
  int(3)
  ["mime"]=>
  string(10) "image/jpeg"
}

Nej det mener jeg da ikke, men du kan da prøve, hvis du har et billede.
http://babegalleri.dk/test/ - Brugernavn: WooX - Kode: asdasd

Jeg tror faktisk slet ikke at den forfat i noget med billedet, for med .txt filer eller .png filer, som jeg lige har testet det med får jeg stadig:

Du skal uploade et .jpg eller .gif billede!

array(7) {
  [0]=>
  int(750)
  [1]=>
  int(150)
  [2]=>
  int(2)
  [3]=>
  string(24) "width="750" height="150""
  ["bits"]=>
  int(8)
  ["channels"]=>
  int(3)
  ["mime"]=>
  string(10) "image/jpeg"
}

Præcis som du siger.
http://babegalleri.dk/test/coderdk.php

Alt for weird. Prøv lige at fjerne @'et og se hvad den så siger

Hvis jeg prøver med .txt:

Warning: getimagesize(): open_basedir restriction in effect. File(/tmp/phpg51Evo) is not within the allowed path(s): (/var/www/babegalleri) in /mnt/raid/www/www/babegalleri/test/img_add.php on line 20

Warning: getimagesize(/tmp/phpg51Evo): failed to open stream: Operation not permitted in /mnt/raid/www/www/babegalleri/test/img_add.php on line 20
Du skal uploade et .jpg eller .gif billede!


Med .jpg:

Warning: getimagesize(): open_basedir restriction in effect. File(/tmp/phpSb2Mdi) is not within the allowed path(s): (/var/www/babegalleri) in /mnt/raid/www/www/babegalleri/test/img_add.php on line 20

Warning: getimagesize(/tmp/phpSb2Mdi): failed to open stream: Operation not permitted in /mnt/raid/www/www/babegalleri/test/img_add.php on line 20
Du skal uploade et .jpg eller .gif billede!

Well, DUH! Din udbyder har slået open_basedir til. Derfor er du nødt til at flyttet filen ind i dit eget bibliotek før du kan bruge getimagesize()...

Ændr:

    $image = $_FILES["image"]["tmp_name"];

Til:
   
    if ( is_uploaded_file( $_FILES["image"]["tmp_name"] ) )
    {
        $image = 'images/users/hotornot/images/' . uniqid() . uniqid();
        move_uploaded_file( $_FILES["image"]["tmp_name"], $image )
    }
    else
    {
        $warning[] = "Du skal uploade en fil.";
    }

Og så skal du bruge $fname hele vejen ned - Ikke noget med $_FILES['image']['tmp_name']

Sidste linje skulle jo have været:

Og så skal du bruge $image hele vejen ned - Ikke noget med $_FILES['image']['tmp_name']

Parse error: parse error, unexpected '}' in /mnt/raid/www/www/babegalleri/test/img_add.php on line 23

http://babegalleri.dk/test/img_add.phps

Hov nogen havde glemt et ; :P

Nu får jeg denne fejl:

Warning: uniqid() expects at least 1 parameter, 0 given in /mnt/raid/www/www/babegalleri/test/img_add.php on line 21

Warning: uniqid() expects at least 1 parameter, 0 given in /mnt/raid/www/www/babegalleri/test/img_add.php on line 21

Warning: move_uploaded_file(images/users/hotornot/images/): failed to open stream: Is a directory in /mnt/raid/www/www/babegalleri/test/img_add.php on line 22

Warning: move_uploaded_file(): Unable to move '/tmp/php7ix1MT' to 'images/users/hotornot/images/' in /mnt/raid/www/www/babegalleri/test/img_add.php on line 22

Warning: getimagesize(): Read error! in /mnt/raid/www/www/babegalleri/test/img_add.php on line 29
Du skal uploade et .jpg eller .gif billede!

Den skal bare have et eller andet prefix:

$image = 'images/users/hotornot/images/' . uniqid('tmpimg') . uniqid('a');

Nu virkede det.

array(7) {
  [0]=>
  int(144)
  [1]=>
  int(176)
  [2]=>
  int(2)
  [3]=>
  string(24) "width="144" height="176""
  ["bits"]=>
  int(8)
  ["channels"]=>
  int(3)
  ["mime"]=>
  string(10) "image/jpeg"
}

MEN!! den har også uploadet en fil med navnet tmpimg427d1df60d04aa427d1df611e71

Den laver heller ikke det billede jeg prøver at uploade ordenligt.

Hvis jeg bruger et billede fra photoshop får jeg denne fejl:

Warning: imagecreatetruecolor(): Invalid image dimensions in /mnt/raid/www/www/babegalleri/test/img_add.php on line 92

Warning: imagecopyresampled(): supplied argument is not a valid Image resource in /mnt/raid/www/www/babegalleri/test/img_add.php on line 103

Warning: imagejpeg(): supplied argument is not a valid Image resource in /mnt/raid/www/www/babegalleri/test/img_add.php on line 111

Ja, det er meningen at du skal arbejde videre med den fil, hvis det f.eks. ikke er et jpeg/gif, skal du selv slette det igen med unlink()

Prøv at debugge, hvad tror scriptet at bredden og højden er? Hvad tror scriptet at new_width og new_height skal være? De to sidste fejl er følger af den første.

Så kan jeg vel bare slette den igen med unlink($image);

Må man spørge hvad u mener med debugge? Og hvordan skal jeg tjekke hvad scriptet tror bredden og højden er?

Kunne du lave en lille guide?

Vil smutte i seng, godnat!

erh, prøv at skrive dem ud?

Har prøvet at unlinke her:

        $warning[] = "Du skal uploade en fil.";
unlink($image);
    }

Men den sletter det ikke, så har jeg kigget på de 2 billeder den uploader, den der heder tmp er hvis jeg skriver.jpg på det 100% rigtige billede, som jeg uploadede, men det andet bliver 2 bredt og 340 højt.

Den gav heller ingen fejl med obenstående, selv om billede var for lille for kravene, hvordan kan det være?

http://babegalleri.dk/test/img_add.phps

Nu virker det fandme :D

Godt jeg lige selv kiggede alt i gennem :D Photoshop filer virker også nu, en af de største fejl var da jeg regnede de nye højder og breder ud, det gav 0,4454576567 eller ngoet i den stil.

Tusind tak for din hjælp, vil du lige smide et svar?

Yep :)

Det var godt nok dejligt at få fixet, det må vel være oven på en dag med så meget god mad :D