[PHP]dropdownlist neemt waarde uit DB niet aan.

[Verwijderd]

....ik van week verander.

Ik heb volgend probleem. Ik heb een jaar en week dropdownlijst. Zodra ik deze aanpas worden de desbtreffende gegevens uit de database gehaald. Het gaat hier om gegevens zoals klanten, projecten, rollen en uren per week.

Ik zie wel dat de juiste gegevens worden opgehaald als ik van week verander. Maar deze worden niet geplaatst in de dropdownlijsten.

Hier mijn code om de projecten op te halen en te tonen in de dropdownlijst.

PHP:

            // Get existing klant from db at selected year and week     
            $sqlSearch = "SELECT klant_id FROM tijden where jaar = $jaar and week = $week LIMIT $i, 1 ";
            $resultSearch = mysql_query($sqlSearch);
            
            $klant = mysql_fetch_row($resultSearch);
            echo trim($klant[0]);
                        
            $sql = "SELECT * FROM klanten";
            
            $result = mysql_query($sql)
                or die('Fout bij ophalen omschrijvingen');
?>
            
                <TD>
                <?php
                    $klantenR = 0;
                    if(isset($_REQUEST["week"])) {
                        echo "week";
                        if($_REQUEST["mySelectKlanten"."".$i]) {
                            echo $_REQUEST["mySelectKlanten"."".$i];
                            if($_REQUEST["mySelectKlanten"."".$i] != $klant[0]) {
                                $klantenR = $_REQUEST["mySelectKlanten"."".$i];
                            } else {
                                $klantenR = $klant[0];
                            }
                        } else {
                            echo "klant" .$klant[0];
                            if($klant[0]) {
                                echo "klant aanwezig";
                                $klantenR = $klant[0];
                            } else {
                                echo "klant NIET";
                                $_REQUEST["mySelectKlanten"."".$i] = 0;
                                $klantenR = $_REQUEST["mySelectKlanten"."".$i];
                            }
                        }
                    }
                    
                ?>
                     <!-- submit page to itself             --> 
                            
                    <select name = "<?php echo "mySelectKlanten"."".$i; ?>" 
                         onchange="document.myForm.action = 'uren.php';document.myForm.submit();">
                    <option value="0">--- Select ---</option>
                <?php
                    while ($line = mysql_fetch_assoc($result)) {    
                    ?>
                        <option value = "<?php echo trim($line['klant_id']); ?>" 
                        <?php   
                            //echo $klantenR;
                            //echo trim($line['klant_id']);
                            if(trim($line['klant_id']) == trim($klantenR)) {
                                echo "selected='selected'"; 
                            } 
                        ?>> 
                        <?php echo $line['bedrijfsnaam'];?>     
                        </option>
                       
                                          
                 <?php
                        
                     }
                 ?>
                    </select>
                 
                </TD>
            <?php
                    
        }
    }
?>

Deze functie wordt vanuit een andere pagina aangeroepen.
Het trucje zit in deze code

PHP:

                <?php
                    $klantenR = 0;
                    if(isset($_REQUEST["week"])) {
                        echo "week";
                        if($_REQUEST["mySelectKlanten"."".$i]) {
                            echo $_REQUEST["mySelectKlanten"."".$i];
                            if($_REQUEST["mySelectKlanten"."".$i] != $klant[0]) {
                                $klantenR = $_REQUEST["mySelectKlanten"."".$i];
                            } else {
                                $klantenR = $klant[0];
                            }
                        } else {
                            echo "klant" .$klant[0];
                            if($klant[0]) {
                                echo "klant aanwezig";
                                $klantenR = $klant[0];
                            } else {
                                echo "klant NIET";
                                $_REQUEST["mySelectKlanten"."".$i] = 0;
                                $klantenR = $_REQUEST["mySelectKlanten"."".$i];
                            }
                        }
                    }
                    
                ?>

$klant[0] bevat de waarde van de klant uit de database.

Gedrag applicatie:
1. Ik log in week 1 , jaar 2009 heeft klanten. Deze worden netjes getoond.
2. Ik verander van week 1 naar week 2. I zie dat de juiste database waarden worden opgehaald. Echter zie ik nog de klanten van week 1 staan.
3. Ga ik naar een week waar er geen klanten in de database zijn, dan toont hij de laatste geselecteerde waarde. Dit kan hij gehaald hebben uit de database of de handmatige selectie van klanten.

Wie kan mij hiermee helpen? Ik struggle hier al een tijdje mee, maar heb nergens nog een goede post hierover kunnen vinden.

[MueR]

Ik zie nergens waar je de week instelt? Alleen in regel 1 zie ik een var $week ?

offtopic:
Ga eens gewoon netjes met $_POST en $_GET werken :x

[Verwijderd]

Hoi,

Week wordt in uren.php gemaakt. Het is OO-opgezet. Althans dat heb ik geprobeerd om te doen.

Dit staat er in uren.php voor week ddl.

PHP:

<FORM name="myForm" method="get" target="_self">
<?php
    $tijd = new tijden();
?>
    <TR>
        <TD><?php $yearR = !isset($_REQUEST['year'])? NULL : $_REQUEST['year']; ?>
                Jaar <select name="year" onChange="myForm.submit()"> 
                <option value="2009" <?php if($yearR == "2009") echo "selected='selected'";?>> 2009 </option>
                <option value="2010" <?php if($yearR == "2010") echo "selected='selected'";?>> 2010 </option>
                <option value="2011" <?php if($yearR == "2011") echo "selected='selected'";?>> 2011 </option>
               
             </select>
        </TD>
        <TD>
        <?php
            if(isset($_REQUEST["year"])) {
                
                $yearR = $_REQUEST["year"];
                $week = get_week_number($yearR);    
                //echo $week;
            }
            
            $weekR = !isset($_REQUEST['week'])? NULL : $_REQUEST['week']; 
        ?>
        
            Week <select name = "week" onChange="document.myForm.submit();">
                 
          <?php
            if(isset($week)) {
                for($i=1; $i <= $week; $i++) {
                ?>
                    <option value="<?php echo $i; ?>" 
                        <?php if($weekR == $i) echo "selected='selected'"; ?>><?php echo $i; ?>
                    </option>
                <?php
                }
                
            }
        ?>  
            </select>
          </TD>
      </TR>

[Verwijderd]

Waar gaat het fout, welke regel? Waar verwacht je resultaten die je niet krijgt? Je geeft aan dat je klant gegevens wel terug krijgt maar deze niet in het drop down menu kan printen, kan je ze er wel buiten printen? Zo nee, dan krijg je dus geen klant gegevens terug?

Je haalt klant gegevens uit de database en uit de request variabelen? Welke wil je nou gebruiken. Ik snap niets van je code.
Als je code post gebruik dan ook de language tag:

code:

[code=php]...[/code]

Dit maakt het al enigszins leesbaarder voor ons.

[Matis]

Verwijderd schreef op donderdag 19 maart 2009 @ 09:39:

code:

[code=php]...[/code]

Dit maakt het al enigszins leesbaarder voor ons.

Ja graag, ook voor mij. Zonder specifieke outputs e.d. kan ik er niet veel mee. Probeer je query eens te echo-en voor verschillende weken.

probeer daarnaast ook wat secuurder te zijn in je query opbouw.

De ene keer gebruik je de SQL-commando's in hoofdletters (wat ik alleenmaar aanraadt) en daarna weer klein.

Persoonlijk ga ik voor de gouden regel --> database's en velden altijd lowercase; SQL-commandos altijd UPPERCASE.

Dat maakt het voor jezelf ook wat makkelijker.

PHP:

$sqlSearch = "SELECT klant_id FROM tijden WHERE jaar = $jaar AND week = $week LIMIT $i, 1 ";

Stan je jaar en week waardes wel als integer in de DB?

[Woy]

Welkom op GoT,

Je post hier een hele lap code, maar hier op GoT hebben we graag dat je alleen relevante code post. Lees ook het PRG beleid, "Een goede topicstart: De Quickstart" en *** Over topictitels in PRG - lezen voor topic openen!!! *** even door.

Verwijderd schreef op woensdag 18 maart 2009 @ 15:55:
Ik zie wel dat de juiste gegevens worden opgehaald als ik van week verander. Maar deze worden niet geplaatst in de dropdownlijsten.

.... Hele lap code ...

Als je dus weet dat het mis gaat bij het plaatsen van de Data in de dropdownlijst, dan willen we graag alleen code zien waar je dat probeert te doen.

Als je niet precies weet waar het fout gaat: Debuggen: Hoe doe ik dat?

[ Voor 4% gewijzigd door Woy op 19-03-2009 10:28 ]

[Verwijderd]

Dank voor de tips van hoe ik hier code in het forum moet plaatsen. Ik ben hier nieuw, dus moet nog even weten wat precies hier wel en niet mag.

Wellicht handig om wat meer te vertellen over mijn applicatie.
Ik heb bovenaan een dropdownlijst met jaren en een dropdownlijst met weken.
Hieronder heb ik 5 regels. Elke regel bestaat uit:
- een ddl met klanten
- een ddl met projecten
- een ddl met rollen
- 7 inputvelden voor de dagen in een week.

Selecteer ik een klant dan wordt de ddl met projecten dynamisch opgebouwd. Hetzelfde geldt voor indien ik projecten selecteerd wordt dynamisch de ddl van projecten gevuld.

Het gaat om deze code:

PHP:

  $klantenR = 0;
                    if(isset($_REQUEST["week"])) { //indien ik week selecteer
                        echo "week";
                        if($_REQUEST["mySelectKlanten"."".$i]) { //indien ik klant selecteer
                            echo $_REQUEST["mySelectKlanten"."".$i];
                            if($_REQUEST["mySelectKlanten"."".$i] != $klant[0]) { //kijk of waarde in klant ddl ongelijk is aan waarde uit database. Indien ongelijk toon geselecteerde waard
                                $klantenR = $_REQUEST["mySelectKlanten"."".$i];
                            } else { //anders toon de waarde uit de database
                                $klantenR = $klant[0];
                            }
                        } else {
                            echo "klant" .$klant[0];
                            if($klant[0]) { //indien waarde uit database. Toon deze in ddl klanten
                                echo "klant aanwezig";
                                $klantenR = $klant[0];
                            } else {
                                echo "klant NIET";
                                $_REQUEST["mySelectKlanten"."".$i] = 0; //indien geen waarde uit database, zet toon waarde 0 dus (-select-) in klanten ddl.
                                $klantenR = $_REQUEST["mySelectKlanten"."".$i];
                            }
                        }
                    }

Met $_REQUEST["mySelectKlanten"."".$i] en $_REQUEST["week"] haal ik waarden uit de dropdownlijsten op.

$klant[0] bevat de waarde van de klant uit de database. Deze krijg ik correct uit de database. En ik kan ze ook buiten de ddl printen.

Vervolgens bouw ik de select option op

PHP:

 <select name = "<?php echo "mySelectKlanten"."".$i; ?>" 
                            onchange="document.myForm.action = 'uren.php';document.myForm.submit();">
                    <option value="0">--- Select ---</option>
                <?php
                    while ($line = mysql_fetch_assoc($result)) {     
                    ?>
                        <option value = "<?php echo trim($line['klant_id']); ?>" 
                        <?php     
                            if(trim($line['klant_id']) == trim($klantenR)) {
                                echo "selected='selected'"; 
                            } 
                        ?>>    
                        <?php echo $line['bedrijfsnaam'];?>     
                        </option>
                       
                                          
                 <?php
                         
                     }
                 ?>
                    </select>

Waar gaat het nu fout:
- zodra ik van week 1 naar week 2 ga, worden in klant ddl de waarde uit week 1 getoond. Het lijkt alsof

HTML:

onchange="document.myForm.action = 'uren.php';document.myForm.submit();">

niet goed de roundtrip uitvoert.
- of dat in bovenstaande eerste stuk code iets niet goed zit.

Als er nog onduidelijkheden zijn, meld het dan, ik probeer het zoveel mogelijk duidelijk voor je op te schrijven. Ik hoop dat iemand mij met het probleem kan helpen.

Trouwens jaar en week staan als text in database

[Matis]

Verwijderd schreef op donderdag 19 maart 2009 @ 11:11:
Trouwens jaar en week staan als text in database

Wellicht scheelt het om in dat geval single quotes om de jaar en maand waardes heen te zetten.

PHP:

<?php
$sqlSearch = "SELECT klant_id FROM tijden WHERE jaar = ' $jaar' AND week = '$week' LIMIT $i, 1 ";
?>

Ook weet ik niet waar je $i vandaan komt...

[Verwijderd]

ik zie in uren.php niet terug waar je de gebruikers wilt printen. Dit stukje code ontbreekt.
Je form wordt gesubmit en doorgestuurd naar uren.php.

Print daar eens je query in platte tekst waarmee je de klant gegevens ophaalt. Knal deze query dan eens in phpmyadmin ofzo. Krijg je dan de verwachte resultaten terug? zo nee? waarom niet?

als de resultaten van de eerste week nogsteeds worden geprint vermoed ik dat een waarde niet goed wordt doorgegeven.

[MueR]

Ik zal het nog eens zeggen: Stap van $_REQUEST af. Die superglobal is niet veilig. Gebruik netjes $_GET en $_POST.

[Cartman!]

Ik denk sowieso dat de TS wat basis tutorials opnieuw mag doorlezen:

PHP:

$_REQUEST["mySelectKlanten"."".$i];

En verder kan ik t niet meer eens zijn met MueR

[Verwijderd]

@tomatis: query aangepast. Ik heb de jaar en week naar int waardes gezet in de database.
$i komt van uren.php. In een for loop bouw ik de 5 regels op. Dus bijvoorbeeld de 5 klanten ddl krijgen de mySelectKlanten0, mySelectKlanten1.......en mySelectKlanten4.

@Khainder: De query wordt goed uitgevoerd. De juiste data komt uit de database. Heb ik getest en gepriont. Dus het ligt niet in de communicatie tussen database en applicatie. De juiste data komt voor iedere week binnen in de applicatie.

@MueR: Indien ik $_REQUEST vervang door $_POST krijg ik undefined index errors? Waarom?

[Verwijderd]

@Cartman: Is hier wat fout $_REQUEST["mySelectKlanten"."".$i];
Indien ik dit uitprint krijg ik namelijk wel het gewenste resultaat. Of bedoel je wat anders?

Voordat we helemaal afdwalen.
Iets in deze code klopt niet:

PHP:

<?php 
  $klantenR = 0; 
                    if(isset($_REQUEST["week"])) { //indien ik week selecteer 
                        echo "week"; 
                        if($_REQUEST["mySelectKlanten"."".$i]) { //indien ik klant selecteer 
                            echo $_REQUEST["mySelectKlanten"."".$i]; 
                            if($_REQUEST["mySelectKlanten"."".$i] != $klant[0]) { //kijk of waarde in klant ddl ongelijk is aan waarde uit database. Indien ongelijk toon geselecteerde waard 
                                $klantenR = $_REQUEST["mySelectKlanten"."".$i]; 
                            } else { //anders toon de waarde uit de database 
                                $klantenR = $klant[0]; 
                            } 
                        } else { 
                            echo "klant" .$klant[0]; 
                            if($klant[0]) { //indien waarde uit database. Toon deze in ddl klanten 
                                echo "klant aanwezig"; 
                                $klantenR = $klant[0]; 
                            } else { 
                                echo "klant NIET"; 
                                $_REQUEST["mySelectKlanten"."".$i] = 0; //indien geen waarde uit database, zet toon waarde 0 dus (-select-) in klanten ddl. 
                                $klantenR = $_REQUEST["mySelectKlanten"."".$i]; 
                            } 
                        } 
                    } 
?>

[ Voor 163% gewijzigd door Verwijderd op 19-03-2009 16:12 ]

[Cartman!]

PHP:

$_REQUEST["mySelectKlanten"."".$i];

of

PHP:

$_GET['mySelectKlanten' . $i];

zie je t verschil de verschillen?

ennuh...2 posts achter elkaar is not done, daarvoor bestaat de edit functie

[ Voor 30% gewijzigd door Cartman! op 19-03-2009 17:55 ]

[Creepy]

Verwijderd schreef op donderdag 19 maart 2009 @ 15:45:
@Cartman: Is hier wat fout $_REQUEST["mySelectKlanten"."".$i];
Indien ik dit uitprint krijg ik namelijk wel het gewenste resultaat. Of bedoel je wat anders?

Voordat we helemaal afdwalen.
Iets in deze code klopt niet:
*snip*

"iets in deze code klopt niet". Eeeh.. okay... en toen? Wat heb je zelf algeprobeerd? Wat lukte daar niet mee? Waar denk je zelf dat de fout zit?

Het is hier geen helpdesk om even snel een oplossing te scoren. We verwachten hier zelf wat inzet maar de eigen inzet is wat moeilijk te ondekken. Dus geef eens wat meer info over wat je gedaan hebt etc.
Lees Het PRG Beleid en Het algemeen beleid #topicplaatsen eens rustig door dan weet je wat hier wel en niet kan.

[ Voor 54% gewijzigd door Creepy op 19-03-2009 20:47 ]

[Verwijderd]

Ik heb al zoveel geprobeerd. Ach ja...dat kunnen jullie niet weten hier wat stukje codes welke ik heb geprobeerd. Je denkt toch niet dat ik zomaar een post plaatst zonder hier eerst mee geworsteld te hebben.

PHP:

                    if(isset($_REQUEST["week"])) {
                        
                        echo $_REQUEST['mySelectKlanten'.$i];
                        if($klant[0]) {
                            if($_REQUEST['mySelectKlanten'.$i] != trim($klant[0]) && $_REQUEST['mySelectKlanten'.$i] > 0) {
                                $klantenR = $_REQUEST['mySelectKlanten'.$i];
                                echo "KLant Ongelijk " .$klantenR;
                            } else{
                                $klantenR = $klant[0];
                                echo "Klant gelijk " .$klantenR;                            
                            }
                        } else {
                            $klantenR = $_REQUEST['mySelectKlanten'.$i];
                            echo "gelijk " .$klantenR;
                        }
                    }

Indien ik inlog zie ik in week 1 de juiste waardes staan.
Ga ik naar week 2 worden de waarde uit week 1 nog steeds getoond. Terwijl ik bovenaan in de pagina met een print zie dat week 2 andere waardes bevat.

Indien éénmaal een ddl is gevuld, wordt hij bij het veranderen van een week niet de juiste waardes gezet. Hij neemt gewoon de waarde van de vorige ingevulde week mee.

[Cartman!]

Ik heb al zoveel geprobeerd. Ach ja...dat kunnen jullie niet weten hier wat stukje codes welke ik heb geprobeerd. Je denkt toch niet dat ik zomaar een post plaatst zonder hier eerst mee geworsteld te hebben.

Dan moet je toch beter uitleggen wat je allemaal geprobeerd hebt, dat kunnen we inderdaad niet weten zoals je zelf al zegt

Verder lijkt het me gewoon n kwestie van goed debuggen.

[Verwijderd]

Je geeft aan dat er in dat stukje code wat fout gaat maar daar worden helemaal geen ddl's gevuld. Als je al weet waar het fout gaat WAAR gaat het in dat stukje code dan fout?
Ik zie in dat stukje code overigens ook geen query resultaten opgehaald en/of gebruikt worden. Alleen maar waarden uit de $_REQUEST array.

Voor jou steekt het allemaal logisch in elkaar omdat jij het geschreven hebt. Je laat veel voor ons onbekende informatie weg waardoor het voor ons heel moeilijk wordt om te achterhalen waar het fout gaat.

Dat terzijde wordt hier meerdere malen gesuggereerd om de $_REQUEST array niet te gebruiken. Het is netter om de $_GET en $_POST array te gebruiken, dan heb je wat meer controle en kan je makkelijk zien waar fouten optreden omdat je gecontroleerd waarden wilt gebruiken en je daardoor precies weet waar ze vandaan komen. Ook gebruik je de $_REQUEST array om waarden in op te slaan en te gebruiken over meerdere pagina's, hiervoor kan je beter de $_SESSION array gebruiken.
meer leesvoer over sessions alhier.

[Creepy]

Verwijderd schreef op vrijdag 20 maart 2009 @ 09:26:
Ik heb al zoveel geprobeerd. Ach ja...dat kunnen jullie niet weten hier wat stukje codes welke ik heb geprobeerd. Je denkt toch niet dat ik zomaar een post plaatst zonder hier eerst mee geworsteld te hebben.

Het probleem is dat er meer mensen zijn die roepen al dagen bezig te zijn en vanalles geprobeerd te hebben maar dat duidelijk niet hebben gedaan. Ook voorkom je dat wij dingen gaan vragen die je zelf al lang hebt geprobeerd. Dus ook voor jezelf is het handiger om gewoon meer informatie te geven. Het geeft ons ook meer inzicht in jouw denkwijze.

Indien éénmaal een ddl is gevuld, wordt hij bij het veranderen van een week niet de juiste waardes gezet. Hij neemt gewoon de waarde van de vorige ingevulde week mee.

Blijkbaar worden de variabelen niet aangepast. Wilde gok: ergens doet een if niet wat jij denkt dat ie doet. Met wat debugwerk, strategisch plaatsen van wat printf of echo's e.d. zou je daar vrij snel achter moeten zijn.

Overiges geef je nu weer geen informatie over hoe en wat je nu zelf al hebt geprobeerd. Als je weet wat er mis gaat dan heb je als het goed is zelf ook al wat geprobeerd om dat op te lossen, dus wat is dat dan geweest? Nogmaals: neem die quickstart en het PRG beleid eens door want je geeft nog steeds erg weinig info.

[ Voor 36% gewijzigd door Creepy op 20-03-2009 10:12 ]

[MueR]

Verwijderd schreef op vrijdag 20 maart 2009 @ 09:39:
Ook gebruik je de $_REQUEST array om waarden in op te slaan en te gebruiken over meerdere pagina's, hiervoor kan je beter de $_SESSION array gebruiken

Incorrect. $_REQUEST is een mix van $_GET, $_POST en $_COOKIE. Maar door het gebruik hiervan weet je niet waar je data vandaan komt.

[Cartman!]

Waardoor het er ongeveer weer op neerkomt dat je soortgelijk werkt als dat je register_globals aan hebt staan, laat dat nou net geen goed idee zijn.

[Verwijderd]

MueR schreef op vrijdag 20 maart 2009 @ 10:35:
[...]

Incorrect. $_REQUEST is een mix van $_GET, $_POST en $_COOKIE. Maar door het gebruik hiervan weet je niet waar je data vandaan komt.

Ik wist niet dat $_COOKIE er ook in zat.
Het was overigens geen stelling hoe de $_REQUEST gebruikt dient te worden, maar een verwijzing naar hoe Vandana de $_REQUEST array gebruikt. Met als doel hem erop te wijzen dat dit niet correct is =)

[Verwijderd]

Iedereen bedankt voor de tips. Ik ga deze toepassen in mijn code. Ik kom er daarna op terug.

[EnigmA-X]

MueR schreef op vrijdag 20 maart 2009 @ 10:35:
[...]

Incorrect. $_REQUEST is een mix van $_GET, $_POST en $_COOKIE. Maar door het gebruik hiervan weet je niet waar je data vandaan komt.

Ik denk dat het nóg belangrijker is om te zeggen dat de waardes in deze superglobal clientside te beinvloeden zijn, waardoor je de hele bups dus niet als 'veilige bron' kan zien. (Voor velen wellicht inkoppertje, voor de mensen die wat nieuwer zijn in de PHP-wereld misschien een fijne toevoeging)