[PHP] Uploaden van een bestand lukt niet

[LittleWan]

beste,

ik probeer een bestand up te loaden met een form, om deze daarna te kunnen parsen en de inhoud in een mysql database te steken.
Nu lukt het uploaden van het bestand niet.
Ik gebruik volgende code

code:

    <form id="form3" name="form3" method="post" action="menu.php?action=upload">
      <p>Menu Upload</p>
      <label for="file">Bestand:</label>
      <input name="file" type="file" id="file" />
      <br /> 
      <input type="submit" name="submit" value="Upload" />
    </form>

code:

if($action == "upload"){
    if(isset($_FILES['file']))  {
        echo "bestand upgeload";
    } else {
        echo "bestand NIET upgeload";
}

wat zou ik mis doen?
de array $_FILES['file'] bevat nooit iets wat ik ook doe of verander

[Verwijderd]

Er gaan hier meerdere dingen fout, de waarde van method is sowieso al fout, en met de PHP-code kom je ook al niet ver.

Lees anders gewoon even een paar tutorials door, hieronder een:

http://www.phphulp.nl/php/scripts/2/96/

[ID-College]

Er mist zoiezo een } bij else als ik het zo zie hoor.
En heel de constructie is zoiezo fout want ik gok dat er zo weinig gebeurt
En post zoiezo de relevante code want hier heeft niemand iets aan

[ Voor 20% gewijzigd door ID-College op 12-12-2006 00:05 ]

[Verwijderd]

Wellicht nog een enctype="multipart/form-data" meegeven aan je form?

[DizzyWeb]

method="post" is imho gewoon goed hoor.

Ik mis wel een enctype="multipart/form-data" in het form.

Je gebruikt $action, maar staat register_globals wel aan? Zo niet, $_GET['action'] gebruiken.

[ID-College]

DizzyWeb schreef op dinsdag 12 december 2006 @ 00:06:
method="post" is imho gewoon goed hoor.

Ik mis wel een enctype="multipart/form-data" in het form.

Je gebruikt $action, maar staat register_globals wel aan? Zo niet, $_GET['action'] gebruiken.

<action="menu.php?action=upload">

[LittleWan]

ik weet dat de code niet volledig is, maar de volledige file bestaat al uit 300 regels code die eigenlijk niets met dit te maken hebben.

Ik maakte ook eerst een zo eenvoudig mogelijk test situatie, om elke andere fout uit te sluiten,

enctype="multipart/form-data" did the trick, voorlopig werk het nu!

bedankt

[Verwijderd]

ID-College schreef op dinsdag 12 december 2006 @ 00:08:
[...]

<action="menu.php?action=upload">

Wat wil je hier nou mee zeggen vraag ik me af? Niet elke server accepteerd dit.
Overgens wordt dit gezien als "bad practise".

Google is your friend..

[programist]

Gebruik het volgende:

PHP:

 <form id="form3" name="form3" method="post" action="menu.php" enctype="multipart/form-data">
      <p>Menu Upload</p>
      <label for="file">Bestand:</label>
      <input name="file" type="file" id="file" />
      <br /> 
      <input type="submit" name="submit" value="Upload" />
    </form>

PHP:

if(isset($_POST['file'])){
    if($_POST['file'] != null){
        //doe je ding           
    }
}

Dit zou je iets moeten oplerveren als ik het niet mis heb

[XWB]

@programist, $_FILES['file'] ipv $_POST['file']

PHP:

if ( isset ( $_FILES['file'] ) && $_FILES['file']['size'] > 0 )
{
    //doe je ding            
}

[Janoz]

GET parameters in je action is niet alleen lelijk, het is ook nog eens onjuist en makkelijk te voorkomen.

code:

<form action="menu.php" ...>
  <input type="hidden" name="action" value="upload"/>
 ...
</form>

[ Voor 3% gewijzigd door Janoz op 12-12-2006 11:31 ]

[Tanuki]

Waarom is het lelijk?

Als jij op index.php?p=blabla een upload formulier hebt en je wilt daar een POST naar versturen zul je toch action="index.php?p=blabla" moeten gebruiken, want door "p" op te nemen in je formulier krijg je $_POST['p'] in plaats van $_GET['p'].

[Janoz]

Als je naar de HTTP specificaties kijkt, en dan specifiek naar hoe een request opgebouwd wordt, dan zie je dat een POST en een GET twee apparte request types zijn die je naar de server kunt versturen. Beiden hebben een iets andere opmaak en iets andere betekenis. Ze zijn volgens de specificaties onmogelijk te mixen. Dat het uberhaupt werkt is omdat webservers er eventueel wat coulant mee omgaan.

[LittleWan]

wel, ik gebruik die 'action=' bij nog andere zaken waarbij geen forumuleer gebruikt wordt... daarom is het makelijker om het bij een POST te houden. of toch niet? ik krijg wel werkende code, maar ik weet niet of het ook correcte code is. Ik heb alles uit zelfstudie...
is er een manier om met een url iets mee te geven vb: action=edit, zonder dat dit bovenaan in de url komt? ik wil dus enkel menu.php zien ipv menu.php?action=edit ?

[Verwijderd]

LittleWan schreef op dinsdag 12 december 2006 @ 14:16:
wel, ik gebruik die 'action=' bij nog andere zaken waarbij geen forumuleer gebruikt wordt... daarom is het makelijker om het bij een POST te houden. of toch niet? ik krijg wel werkende code, maar ik weet niet of het ook correcte code is. Ik heb alles uit zelfstudie...
is er een manier om met een url iets mee te geven vb: action=edit, zonder dat dit bovenaan in de url komt? ik wil dus enkel menu.php zien ipv menu.php?action=edit ?

Wanneer je een link naar die specifieke pagina plaatst en een click met JS afvangt, kan je een post meegeven via JS. Anders ben je altijd aangewezen tot het gebruik van GET's. Een alternatief daarvoor is eventueel het gebruik van URI's waarbij je de GET's eigenlijk met /var1/var2/var3 opbouwd, dat werkt met Mod Rewrite als ik het goed heb, of bijvoorbeeld het gebruik van een framework zoals Code Igniter, waarbij je met dezelfde URI structuur bepaalde classes en functies binnen deze classes aanspreekt met parameters voor deze functies /class/functie/par1/par2/par3 enz. (www.codeignitor.com)