[php] uploaden functioneerd niet

[Verwijderd]

Ik ben voor m'n vriendin een klein siteje in elkaar aan het bouwen, welke ze zelf kan adminnen, foto's uppe, teksten editen etc. niet zo heel spannend, nu had ik portfolio ding werkend, allemaal geen punt, toen ging ik het verbeteren en nu is het stuk en ik heb geen flauw idee waarom, misschien een frisse blik die opheldering kan geven, hij bereikt deze if statement wel, maar ik denk dat het fout gaat met m'n foreach en zou zo niet weten hoe te fixen, en ja, ik weet dat ik niet 100% nette php code schrijf

PHP:

if ($_POST["submit"] && $_POST[file]) {
  foreach ($_FILES as $plaatje) {
    $allowed_types = array("image/pjpeg","image/jpeg");
    $code = rand(100000,999999);

    if (in_array($plaatje[type],$allowed_types)) {
      $fullimage = ImageCreateFromJPEG($plaatje[tmp_name]);

      $fullwidth = imagesx($fullimage);
      $fullheigth = imagesy($fullimage);

      if ($fullwidth > 640) {
        echo TableTop("Foto aan portfolio toevoegen...");
        echo TableBot("Helaas.... :(\n\nHet is mislukt, je foto is groter dan 640 pixels breed.");
        include("../inc/right.inc.php");
        include("../inc/footer.inc.php");
        exit;
      }

      $thumbratio = imagesx($fullimage)/150;
      $thumbwidth = imagesx($fullimage)/$thumbratio;
      $thumbheight = imagesy($fullimage)/$thumbratio;

      $thumbnail = ImageCreateTrueColor($thumbwidth,$thumbheight);
      imagefill($thumbnail, 0, 0, ImageColorAllocate($thumbnail, 255, 255, 255));
      imagecopyresampled($thumbnail,$fullimage,0,0,0,0,$thumbwidth,$thumbheight,imagesx($fullimage),imagesy($fullimage));
      imagejpeg($thumbnail, "../img/portfolio/thumb$code$plaatje[name]");

      move_uploaded_file($plaatje[tmp_name], "../img/portfolio/full$code$plaatje[name]");

      mysql_query("INSERT into fotos (path, width, height) VALUES ('$code$plaatje[name]','$fullwidth','$fullheigth')") or die(mysql_error());
      
      echo TableTop("Foto aan portfolio toevoegen...");
      echo TableBot("Het is gelukt! :D\n\nJe foto is toegevoegd aan de portfolio!\n\n[url2=$PHP_SELF]Terug naar de portfolio[/url]");
    }

[whoami]

Gewoon zelf eens debuggen. 't is een onderdeel van het programmeerproces.

[Verwijderd]

als jij je eigen FAQ gelezen zou je weten dat als mensen d'r echt niet uit komen ze de vraag mogen stellen.

En gezien ik hier al 2 dagen mee bezig ben om dit waarschijnlijk kleine foutje te vinden denk ik dat jouw reply een bullshit reply was!

[whoami]

Verwijderd schreef op 31 oktober 2002 @ 12:57:
als jij je eigen FAQ gelezen zou je weten dat als mensen d'r echt niet uit komen ze de vraag mogen stellen.

En gezien ik hier al 2 dagen mee bezig ben om dit waarschijnlijk kleine foutje te vinden denk ik dat jouw reply een bullshit reply was!

1. Het is mijn FAQ niet.
2. Met er gewoon op te kijken kom je er niet. Aangezien jij niet weet waar de fout precies ligt, wil dat zeggen dat je nog niet eens gedebugged hebt.
3. Mijn reply is geen bullshit reply, maar een zeer goede tip voor jou.
4. Jij moet niet zo geaggiteerd reageren omdat ik of iemand anders je een hint geef ipv direct de oplossing te posten.

[whoami]

Ow, en
5. Jij moet de P&W Quickstart eens lezen, zodat je een duidelijke post maakt waar je duidelijk in uitlegt wat er wel en wat er niet werkt. En welke fout je eventueel krijgt, enz....

[Verwijderd]

2.k weet dondersgoed waar de fout ligt, als jij die boventekst had gelezen zie je vast het woordje foreach staan.

3. slaat precies op je signature
4. slaat weer op dit puntje 3

[Bosmonster]

Wij kunnen zo ook geen fout zien in een hele lap code.. Dan zouden ook wij het moeten copy/pasten en uitvoeren en debuggen.

Je zegt niet eens wat er nu precies fout gaat, of wat voor melding op welke regel je krijgt.

[Altaphista]

doe op elke regel es een print/echo.
post je wel goede enctype?

[Verwijderd]

ok, komt ie:

de if statement (helemaal bovenin dus) die bereikt, hij als ik een echo deronder plaats dan krijg ik dat gewoon te zien, voor de rest als ik ook een echo in m'n foreach zet wordt die niet geparsed.

dus hij bereikt de if wel, maar de foreach schijnt niet aan de criteria te voldoen, na debuggen ben ik daar allang achter gekomen.

maar nu de oplossing, met RTFM kom ik d'r niet uit, dus daarom ook dit topic, wat is er fout, de rest van code is om aan te geven wat het doet.

[Verwijderd]

PHP:

  echo '</table>';
  echo '<form enctype="multipart/form-data" action="'.$PHP_SELF.'" method="post">';
  echo '<table class="normaltable" align="center" cellspacing="0" cellpadding="0" border="0" width="95%">';
  echo '<tr><td><b>&nbsp;Portfolio foto\'s uploaden...</b><br>&nbsp;</td></tr>';
  echo '<tr><td>&nbsp;<input type="file" name="file" class="width: 250px"></td></tr>';
  echo '<tr><td>&nbsp;<input type="submit" name="submit" value="Uploaden">';
  echo '</table></form>';

das m'n form

[LuCarD]

plaats eens een print_r($_FILES) onder jouw eerste if

[Bosmonster]

Lees deze ook eens door:

http://gerard.yoursite.nl/got/php-tiplist/#3

Associative array indices moeten gequote worden.

[Verwijderd]

LuCarD schreef op 31 oktober 2002 @ 13:09:
plaats eens een print_r($_FILES) onder jouw eerste if

Parse error: parse error, unexpected T_FOREACH in /home/john/HTML/www.linda-online.nl/html/admin/portfolio.php on line 7

[whoami]

Als hij niet in de foreach raakt, zal dat waarschijnlijk willen zeggen dat er niets in de $_FILES zit waar hij door moet loopen.

3. slaat precies op je signature

Welk puntje uit m'n signature precies?

[Verwijderd]

$_FILES is toch de superglobal voor files die gepost worden

met deze form enctype en <input type="file">

[Verwijderd]

whoami schreef op 31 oktober 2002 @ 13:13:
Als hij niet in de foreach raakt, zal dat waarschijnlijk willen zeggen dat er niets in de $_FILES zit waar hij door moet loopen.


[...]

Welk puntje uit m'n signature precies?

<Yarvieh> litemods zijn niet allemaal arrogante zakken

[whoami]

Verwijderd schreef op 31 oktober 2002 @ 13:14:
[...]


<Yarvieh> litemods zijn niet allemaal arrogante zakken

Idd..... wat dus wil zeggen dat ik geen arrogante zak ben.

Debuggen heeft trouwens niet enkel te maken met 'waar loopt het mis', maar ook met 'waarom loopt het mis'. Dat je al weet waar het probleem zich situeert is al een stap in de goeie richting, maar dan heb je de volgende stap nog niet uitgevoerd: nl. waarom loopt het mis. Check dan of er wat in die array oid zit waar die foreach moet doorlopen.

[Verwijderd]

zoals ik al zei, volgens mij is die array gewoon gevuld. Tenminste, daar mag ik vanuit gaan omdat volgens mij m'n form gewoon goed is en files worden gepost (vanwege enctype).

Ik heb niemand gehoord die anders wist te vertellen

[packman]

Verwijderd schreef op 31 oktober 2002 @ 13:11:
[...]


Parse error: parse error, unexpected T_FOREACH in /home/john/HTML/www.linda-online.nl/html/admin/portfolio.php on line 7

ROFLMAO

mischien niet gewoon letterlijk copy/pasten maar ook effe nadenken? een ";" doet soms wonderen...

En blijkbaar lees jij ook niet goed - indices van associative arrays moeten dus GEQUOTE worden. zo dus:

PHP:

$mijnarray['mijnindex'] = 'blabla';

en dus niet zo:

PHP:

$mijnarray[mijnindex] = 'blabla';

[whoami]

Verwijderd schreef op 31 oktober 2002 @ 13:18:
zoals ik al zei, volgens mij is die array gewoon gevuld. Tenminste, daar mag ik vanuit gaan omdat volgens mij m'n form gewoon goed is en files worden gepost (vanwege enctype).

Ik heb niemand gehoord die anders wist te vertellen

Volgens jij.... Tja, misschien ben jij wel verkeerd en moet je het toch ff checken.
Ga nooit uit van veronderstellingen.

[Verwijderd]

print_r($HTTP_POST_FILES) heeft als output: Array ( )
$_POST['file'] geeft lokatie van foto op m'n eigen skijf

[packman]

PHP:

imagejpeg($thumbnail, "../img/portfolio/thumb$code$plaatje[name]");

?? Ik veronderstel dat je dit bedoeld:

PHP:

imagejpeg($thumbnail, "../img/portfolio/thumb$code{$plaatje['name']}");

ik zou zo de hele code is doorlopen, en de php-manual nog eens vastnemen voor je aan zoiets begint...

[Verwijderd]

packman schreef op 31 oktober 2002 @ 13:34:

PHP:

imagejpeg($thumbnail, "../img/portfolio/thumb$code$plaatje[name]");

?? Ik veronderstel dat je dit bedoeld:

PHP:

imagejpeg($thumbnail, "../img/portfolio/thumb$code{$plaatje['name']}");

ik zou zo de hele code is doorlopen, en de php-manual nog eens vastnemen voor je aan zoiets begint...

dan altijd nog whitespaces

maar als hij niet eens in die foreach komt (1 regel onder foreach een echo 'HEY!'; zetten wordt niet geparsed)

dan lijkt me dat op dit moment onzinnig

[packman]

Verwijderd schreef op 31 oktober 2002 @ 13:05:

PHP:

// allemaal code..

das m'n form

Plain HTML leren kan anders ook wel van pas komen soms... wat jij doet is:

code:

<table><form ...>{inhoud van tabel en form}</table></form>

Je sluit dus je tags in de verkeerde volgorde af... Maak daar eens:

code:

<table><form ...>{inhoud van tabel en form}</form></table>

of

code:

<form ...><table>{inhoud van tabel en form}</table></form>

van...

[ Voor 0% gewijzigd door packman op 31-10-2002 13:54 . Reden: form zever ]

[Altaphista]

kijk es of je op een andere manier de geposte vars kunt uitlezen...
for each posted var as $varnaam echo $varnaam oid. (zit ff te diep in asp, java en jsp om php zo uit mn hoofd te kunnen )

[Verwijderd]

PHP:

if ($_POST["submit"]) {
echo $_POST['file'];
echo $_POST['submit'];
print_r ($HTTP_POST_FILES); 
  foreach ($HTTP_POST_FILES as $plaatje) {

heeft als output:

C:\\Documents and Settings\\John\\Mijn documenten\\Mijn afbeeldingen\\3 oktber\\3 oktber 006.jpgUploadenArray ( )

[Verwijderd]

packman schreef op 31 oktober 2002 @ 13:40:
[...]


Plain HTML leren kan anders ook wel van pas komen soms... wat jij doet is:

code:

<table><frame ...>{inhoud van tabel en frame}</table></frame>

Je sluit dus je tags in de verkeerde volgorde af... Maak daar eens:

code:

<table><frame ...>{inhoud van tabel en frame}</frame></table>

of

code:

<frame ...><table>{inhoud van tabel en frame}</table></frame>

van...

<frame> ??

[Altaphista]

jonge, ga zelf ff debuggen... en trek daar je conclusies uit... probeer opnieuw etc...

edit: functioneerd moet met een t

[packman]

Watch and learn...

PHP:

print_r($_POST);
print_r($_FILES);

if ($_POST['submit'] && $_POST['file']) {
  echo 'SUBMITTED: OK<br />\n';
  
  foreach ($_FILES as $plaatje) {
    echo 'IMAGE: {$plaatje['name']} LOOP<br />\n';
    
    $allowed_types = array("image/pjpeg","image/jpeg");
    $code = rand(100000,999999);

    if (in_array($plaatje['type'], $allowed_types)) {
      echo 'IMAGE: {$plaatje['type']} ALLOWED<br />\n';
      $fullimage = ImageCreateFromJPEG($plaatje['tmp_name']);

      $fullwidth = imagesx($fullimage);
      $fullheigth = imagesy($fullimage);

      if ($fullwidth > 640) {
        echo 'IMAGE: $fullwidth > 640 <br />\n';
        echo TableTop("Foto aan portfolio toevoegen...");
        echo TableBot("Helaas.... :(\n\nHet is mislukt, je foto is groter dan 640 pixels breed.");
        include('../inc/right.inc.php');
        include('../inc/footer.inc.php');
        exit();
      }
      echo 'IMAGE: $fullwidth OK<br />\n';
      $thumbratio = imagesx($fullimage)/150;
      $thumbwidth = imagesx($fullimage)/$thumbratio;
      $thumbheight = imagesy($fullimage)/$thumbratio;

      $thumbnail = ImageCreateTrueColor($thumbwidth, $thumbheight);
      imagefill($thumbnail, 0, 0, ImageColorAllocate($thumbnail, 255, 255, 255));
      imagecopyresampled($thumbnail, $fullimage, 0, 0, 0, 0, $thumbwidth, $thumbheight, imagesx($fullimage), imagesy($fullimage));
      imagejpeg($thumbnail, "../img/portfolio/thumb$code{$plaatje['name']}");

      move_uploaded_file($plaatje[tmp_name], "../img/portfolio/full$code{$plaatje['name']}");

      mysql_query("INSERT into fotos (path, width, height) VALUES ('$code{$plaatje['name']}', '$fullwidth', '$fullheigth')") or die(mysql_error());
      
      echo TableTop("Foto aan portfolio toevoegen...");
      echo TableBot("Het is gelukt! :D\n\nJe foto is toegevoegd aan de portfolio!\n\n[url2=$PHP_SELF]Terug naar de portfolio[/url]");
    }
}

Zo debug je dus...

[packman]

Verwijderd schreef op 31 oktober 2002 @ 13:45:
[...]


<frame> ??

ach jong - zeik niet - je weet best wat ik bedoel...

[whoami]

GoT is geen helpdesk service of 'we pakken je bij het handje'-forum. Je mag zelf wel wat meer initiatief vertonen en de gegeven hints (want dat is toch meestal wat er gepost wordt, een hint, en geen kant en klare oplossing) zelf verder uitspitten/verwerken en er wat mee doen.
Jij wilt blijkbaar dat alles voor jou voorgekauwd wordt, en zo werkt het hier niet. Ik denk dat je nu wel al een hoop tips gekregen hebt waarmee je verder op weg kunt.

Dus:
6. Deze thread gaat dicht voor het uitaard in over en weer geflame.