[PHP/MySQL] UPDATE neemt verkeerde waardes over*

[Verwijderd]

Ik ben bezig met een admin panel. Ik ben bijna klaar en heb al veel pagina's gemaakt waar tabellen geupdate, bewerkt en verwijderd worden. Ik ben nu echter bezig met een tabel dat menu-items aan de lay out moet toevoegen.
Het aanmaken en het plaatsen van menu items in de lay out gaat perfect. echter gaat het mis bij het bewerken van een menu item. Op een of andere manier veranderd de bewerkte ' menuitem_naam' steeds naar een waarde (andere menuitem_naam) die al in het tabel staat. Ik snap niet wat er mis is, want ik gebruik dit script en tabelopbouw op verschillende pagina's en daar werkt het wel perfect.

Weet iemand het probleem

dit is de tabel genaamd admin_menuitem

code:

CREATE TABLE `admin_menuitem` (
  `menuitem_id` int(200) NOT NULL auto_increment,
  `menuitem_naam` varchar(255) NOT NULL default '',
  `menuitem_url` varchar(255) NOT NULL default '',
  `menuitem_onderwerp` varchar(255) NOT NULL default '',
  PRIMARY KEY  (`menuitem_id`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 AUTO_INCREMENT=0 ;

Dit is het 'bewerk' script

code:

<?   
} 
elseif($actie == "bewerk")   
{ 

                if (isset($_POST['bewerk'])){ 
    
    $query = "UPDATE admin_menuitem SET menuitem_naam='$menuitem_naam', menuitem_url='$menuitem_url', menuitem_onderwerp='$menuitem_onderwerp' where menuitem_id = $menuitem_id"; 
    $query1 = mysql_query($query); //or die(mysql_error()); 
    echo "<div class='warning' align='center'>Deze menu item bericht is perfect bewerkt</div>";
        echo "<META HTTP-EQUIV=\"REFRESH\" CONTENT=\"1; URL=javascript:history.go(-2)\">";
 
                }
        else { 
                    $queryselect = 'SELECT * FROM admin_menuitem WHERE menuitem_id=' . $_GET["menuitem_id"];   
                    $getsel = mysql_query($queryselect); 
                    $row = mysql_fetch_assoc($getsel);
             
?>
<!--- lot of HTML meuk --->
<?
}
?>

[ Voor 47% gewijzigd door Verwijderd op 21-12-2005 00:28 ]

[teun-v]

Mijn glazen bol zecht dat je in je "bewerk" script even op regel nummer 9 de combinatie ; // voor "or die" moet vervangen door een spatie. Volgens mij wordt er namelijk geen een veld geupdate. Kijk dan ff wat voor een error hij geeft.

[Verwijderd]

Dan geeft ie geen error.
Als ik een item in tabel heb staan, dan kan ik deze perfect bewerken. Bij twee gat het ook nog.
Daarna heb ik alle items toegevoegd en plots gaat hij na het bewerken de bewerkte item vervangen voor een bestaande item uit het tabel

[MueR]

Even terzijde: kan je die html meuk niet even weghalen?

Print je queries eens uit, ipv de stappen echt uit te voeren. Dat helpt vaak erg goed.

[teun-v]

in dat ge val heb je globals in je php.ini aan aan staan? Ik zie verder namelijk voor je vars buiten deze: $_POST['bewerk'] nergens terug komen. Verder probeer te debuggen door bijvoorbeeld je $query ook in een echo te zetten.

MueR schreef op woensdag 21 december 2005 @ 00:23:
Even terzijde: kan je die html meuk niet even weghalen?

Print je queries eens uit, ipv de stappen echt uit te voeren. Dat helpt vaak erg goed.

je was me net ff voor

[ Voor 40% gewijzigd door teun-v op 21-12-2005 00:25 ]

[Verwijderd]

ok html meuk weggehaald

$_POST['bewerk'] wordt onder de php code in het <form> benoemd door de submitknop.
Echter zien jullie dit nu niet terug omdat me aangeraden werd ff die html code weg moest halen.

[ Voor 37% gewijzigd door Verwijderd op 21-12-2005 00:31 ]

[teun-v]

Verwijderd schreef op woensdag 21 december 2005 @ 00:30:
ok html meuk weggehaald

$_POST['bewerk'] wordt onder de php code in het <form> benoemd door de submitknop

en de rest???

[Verwijderd]

hetzelfde geld ook voor de rest. Allemaal in het formulier dat er onder staat.

code:

name="menuitem_naam"
name="menuitem_url"
name="menuitem_onderwerp"

[teun-v]

Verwijderd schreef op woensdag 21 december 2005 @ 00:32:
hetzelfde geld ook voor de rest. Allemaal in het formulier dat er onder staat.

PHP:

name="menuitem_naam"
name="menuitem_url"
name="menuitem_onderwerp"

ik bedoel in de phpcode deze:

PHP:

$_POST['menuitem_naam']
$_POST['menuitem_url']
$_POST['menuitem_onderwerp']

maar als je die debug echo van iniedergeval je query doet weet je snel genoeg wat er waarschijnlijk fout gaat

[ Voor 20% gewijzigd door teun-v op 21-12-2005 00:37 ]

[Verwijderd]

Ja, dat zijn toch verwijzingen naar de waardes die in de input typ velden zijn ingevuld binnen het formulier. Of heb ik het nou fout?

[disjfa]

Als het de namen in je html is zeker maar zoals je al eerder zei krijg je een error in je query. Zou je dan niet eerder even kijken wat daar fout aan gaat.

[Verwijderd]

Hmm van het precies debuggen heb ik niet veel eieren gegeten hoe pak ik het precies aan.
Moet ik voor elke waarde controleren of welke waarde het bevat?
of hoe pak ik het debuggen bij dit geval aan?

[teun-v]

Dat klopt. Deze moet je gebruiken in plaats van $menuitem_naam als je globals in je php.ini uitgeschakeld hebt. Het is overigens wel verstandig om deze waardes te valideren om bijvoorbeeld na die validatie $menuitem_naam = $_POST['menuitem_naam'] te doen.

disjfa schreef op woensdag 21 december 2005 @ 00:41:
Als het de namen in je html is zeker maar zoals je al eerder zei krijg je een error in je query. Zou je dan niet eerder even kijken wat daar fout aan gaat.

TS geeft aan geen error te krijken ook niet met "or die(mysql_error());"

maar die echo als je die eerst nu doet, kun je sneller weten wat er fout gaat

[ Voor 8% gewijzigd door teun-v op 21-12-2005 00:46 ]

[disjfa]

Verwijderd schreef op woensdag 21 december 2005 @ 00:45:
Hmm van het precies debuggen heb ik niet veel eieren gegeten hoe pak ik het precies aan.
Moet ik voor elke waarde controleren of welke waarde het bevat?

Dat is idd meestal de bedoeling van debuggen. Kijken welke waarde fout is of welke query fout is. Is best handig namelijk.

[Verwijderd]

maar hoe pak ik dat debuggen aan, en waar zeg ik die echo precies?

code:

<?
if( isset($_POST["menuitemnieuw"]))
    {

        if($_POST["menuitem_naam"] != "" && $_POST["menuitem_url"] != "" && $_POST["menuitem_onderwerp"] != "")
            {'
                   echo"<div class='warning' align='center'>De menu item is toegevoegd.</div>";
                                    
                 mysql_query("INSERT INTO admin_menuitem (menuitem_naam,menuitem_url,menuitem_onderwerp) values ('".$_POST["menuitem_naam"]."','".$_POST["menuitem_url"]."','".$_POST["menuitem_onderwerp"]."')") or die(mysql_error());


                }
            
                else
                {
                    echo "<div class='warning' align='center'>U heeft iets niet ingevult.</div>";
                    echo "<META HTTP-EQUIV=\"REFRESH\" CONTENT=\"1; URL=javascript:history.go(-1)\">";
                }
    }
            
?>

[teun-v]

bijvoorbeeld

PHP:

    $query = "UPDATE admin_menuitem SET menuitem_naam='$menuitem_naam', menuitem_url='$menuitem_url', menuitem_onderwerp='$menuitem_onderwerp' where menuitem_id = '$menuitem_id'"; 
    $query1 = mysql_query($query) or die(mysql_error()); 
    
    // letop hier onder :

    echo $query;

[ Voor 20% gewijzigd door teun-v op 21-12-2005 01:02 ]

[Verwijderd]

wokee ik ben echt heel dom bezig. In al mijn enthausiasme heb ik perongeluk het verkeerde deel van de pagina hier gepost. Het gaat om het volgende stuk:

code:

<?   
} 
elseif($actie == "bewerk")   
{ 

                if (isset($_POST['bewerk'])){ 
    
    $query = "UPDATE admin_menuitem SET menuitem_naam='$menuitem_naam', menuitem_url='$menuitem_url', menuitem_onderwerp='$menuitem_onderwerp' where menuitem_id = $menuitem_id"; 
    $query1 = mysql_query($query) or die(mysql_error()); 
    echo "<div class='warning' align='center'>Deze menu item bericht is perfect bewerkt</div>";
        echo "<META HTTP-EQUIV=\"REFRESH\" CONTENT=\"1; URL=javascript:history.go(-2)\">";
 
                }
        else { 
                    $queryselect = 'SELECT * FROM admin_menuitem WHERE menuitem_id=' . $_GET["menuitem_id"];   
                    $getsel = mysql_query($queryselect); 
                    $row = mysql_fetch_assoc($getsel);
             
?>

Het stuk dat ik net geplaatst had was om iets toe te voegen.
excuus voor deze verwarring

[NMe]

Heel leuk, maar wat is het resultaat van een echo van je query nou? Echo je query, en voer die eens uit in een admin programma (phpMyAdmin, en MySQL admin, whatever). Doet die het daar wel? Lees ook P&W FAQ - Leer **** debuggen!! eens even door, dat zou je wel wat goed doen.

[Verwijderd]

-NMe- schreef op woensdag 21 december 2005 @ 02:05:
Heel leuk, maar wat is het resultaat van een echo van je query nou? Echo je query, en voer die eens uit in een admin programma (phpMyAdmin, en MySQL admin, whatever). Doet die het daar wel? Lees ook P&W FAQ - Leer **** debuggen!! eens even door, dat zou je wel wat goed doen.

Ik heb dat topic doorgenomen en ik snap de bedoeling van debuggen, maar ik weet echter niet hoe ik het moet aanpakken. waar moet ik beginnen met debuggen en hoe Echo ik mijn query?
ik heb niet veel kaas gegeten van php en sql, maar ben ooit begonnen en begrijp dat ik hier iets nieuws kan leren. als iemand me ff wil helpen of op weg helpen.

[whoami]

Ik denk dat je met bovenstaande wel verder moet kunnen.
Verder hebben we in onze FAQ ook wel een en ander aan informatief materiaal ivm debuggen.

[ Voor 72% gewijzigd door whoami op 21-12-2005 12:28 ]

[Verwijderd]

Tja natuurlijk.... na ff wat rond gegoogled te hebben vond ik het volgende.

print_r ($_POST);

deze heb ik in de code gezet nadat op submit geklikt is.
het gaf het volgende weer.

Array ( [menuitem_naam] => n [menuitem_url] => test [menuitem_onderwerp] => Admin [bewerk] => [ Bewerk ] )

kan ik daaruit concluderen dat er wel waardes meegezonden worden na het klikken op submit

[ Voor 7% gewijzigd door Verwijderd op 21-12-2005 12:58 ]

[NMe]

Geen waardes? menuitem_naam heeft toch zeker de waarde "n"? En menuitem_url de waarde "test"? Enz. Snel je post wijzigen he?

Verder geeft [rml]teun-v in "[ PHP/MySQL] UPDATE neemt verkeerde waard..."[/rml] allang aan hoe je je query kan uitvoeren, dus probeer dat eerst maar eens.

[ Voor 6% gewijzigd door NMe op 21-12-2005 12:58 ]

[Verwijderd]

hahaha jah ik las ff verkeerd.

Ik heb nu echo $query; geprobeerd en toen kreeg ik het volgende

UPDATE admin_menuitem SET menuitem_naam='Vissoorten', menuitem_url='test', menuitem_onderwerp='Admin' where menuitem_id = 29

Wanneer ik bij 'menuitem_naam' vissoorten had ingevoerd dan klopte het, maar laat ik daar nu juist ietas anders ingevuld hebben. Alles springt automatisch naar vissoorten (alleen dus bij menuitem_naam).
Wanneer ik de menuitem_url of menuitem_onderwerp bewerk gebeurt dit wel correct. Echter veranderd dan ook menuitem_naam naar vissoort

[ Voor 19% gewijzigd door Verwijderd op 21-12-2005 13:06 ]

[NMe]

Waar komen de variabelen $menuitem_naam enz. vandaan? Hoe vul je die?

[Verwijderd]

Maak je niet toevallig voor de update query nog een keer gebruik van '$menuitem_naam'? zodat de origineel gepostte $menuitem_naam overschreven wordt, waardoor die standaard op vissoorten staat?

[Verwijderd]

Dit is de hele pagina die ik gebruik.
Dus incl. html

code:

<?   
} 
elseif($actie == "bewerk")   
{
                    $queryselect = 'SELECT * FROM admin_menuitem WHERE menuitem_id=' . $_GET["menuitem_id"];   
                    $getsel = mysql_query($queryselect); 
                    $row = mysql_fetch_assoc($getsel);
             
?>
<div align="center">
<br/>
<table class="content" cellpading="0" cellspacing="0">
  <tr> 
    <td><span class="titel">Aquawish Menu item bewerk pagina</span><br/>

    U bevindt zich nu op de bewerk pagina van het menu item <i> <? echo $row[menuitem_naam]; ?></i>. Zorgt u dat u naar alle zorgvuldigheid alle velden invuld en bewerkt.
    Als u alles bewerkt heeft diend u op de knop 'bewerken' te drukken.

    </td>
  </tr>
</table>
<br/>



<form action="?p=admin_menuitem&actie=bewerk&menuitem_id=<? echo $row[menuitem_id]; ?>" method="post" name="postMsg">
<table class="content" cellpading="0" cellspacing="0"> 
  <tr>
    <td class="contenttitelborder1" colspan="2">menu item aanpassen</td> 
  </tr> 
  <tr>
    <td class="contenttitelborder2">menu item titel:</td>
    <td class="contenttitelborder3"><input name="menuitem_naam" type="text" id="menuitem_naam" size="50" value="<? echo $row[menuitem_naam]; ?>"></td>
  </tr>
  <tr>
    <td class="contenttitelborder2">menu item URL:</td>
    <td class="contenttitelborder3"><input name="menuitem_url" type="text" id="menuitem_url" size="50" value="<? echo $row[menuitem_url]; ?>"></td>
  </tr>
  <tr>
    <td class="contenttitelborder2">menuitem onderwerp:</td>
    <td class="contenttitelborder3">
    <select name="menuitem_onderwerp" id="menuitem_onderwerp"  STYLE="width: 175px">
    <option><? echo $row[menuitem_onderwerp]; ?><option value="afblijven" disabled">-----------------------
    <option>Algemeen
    <option>Admin
    <option>Planten
    <option>Vissen
    <option>Aquarium
    <option>Algemeen
    <option>Overige
    </select>
    </td>
  </tr>
  <tr>
    <td class="contenttitelborder3" colspan="2">
<input name="refresh" class="inputfield" type="reset" id="refresh" value="[ Refrehs ]">
<input name="bewerk" class="inputfield" type="submit" id="bewerk" value="[ Bewerk ]">
    </td>
  </tr>
</form>
  <tr>
    <td align="right">
      <br/><? echo "$terug"; ?>
    </td>
  </tr> 
</table>
</div>
<br/>
<?
{ 

                if (isset($_POST['bewerk'])){ 
    
    $query = ("UPDATE admin_menuitem SET menuitem_naam='$menuitem_naam', menuitem_url='$menuitem_url', menuitem_onderwerp='$menuitem_onderwerp' where menuitem_id = $menuitem_id") or die(mysql_error()); 
    $query1 = mysql_query($query) or die(mysql_error()); 
    echo $query;
    echo "<div class='warning' align='center'>*Deze menu item bericht is perfect bewerkt</div>";

 
                }


}
?>

[teun-v]

Als dit het hele php script is mis ik echt nog behoorlijk wat dingen. Bijvoorbeeld een verbinding met je mysql server. hoe onvang je de post's en dergelijke. is dit soms een include op een andere pagina? In dat geval kan de fout namelijk in het php script zitten waar dit script in geinclude wordt.

[Verwijderd]

nee ok dit is niet het hele script.
Ik heb een config bestand waarin de database verbinding tot stand komt.

in de index.php pagina staat hetvolgende

code:

<!-- content gedeelte -->
                    <?
                    ob_start();
                    include("config.php");
                    if(isset($_GET["p"])){
                        $page = stripslashes($_GET["p"]);
                    }
                    if(isset($_GET["page"])){
                        $pagina = stripslashes($_GET["page"]);
                    }
                    else{
                        $pagina = "";
                    }
                    ?>               
                    <?
                    if(!isset($page)){
                        $page = "home";
                    }
                    if(!file_exists($page.".php")){
                        include("home.php");
                    }
                    else{
                        include($page.".php");
                    }
                    ?>

dit staat op de plek waar de content pagina's komen.
Het hierboven geplaatste 'bewerk' script is zo'n pagina.

[NMe]

En ik zie nog steeds niet hoe je nou die menuitem_xxx variabelen vult.

[Verwijderd]

-NMe- schreef op woensdag 21 december 2005 @ 16:24:
En ik zie nog steeds niet hoe je nou die menuitem_xxx variabelen vult.

de velden van het formulier vullen toch na het klikken op submit de variabelen?

code:

<input name="menuitem_url" type="text" id="menuitem_url" size="50" value="<? echo $row[menuitem_url]; ?>">

dat ook weer terug komt in de update query

code:

$query = ("UPDATE admin_menuitem SET menuitem_naam='$menuitem_naam', menuitem_url='$menuitem_url', menuitem_onderwerp='$menuitem_onderwerp' where menuitem_id = $menuitem_id") or die(mysql_error());

of begrijp ik je nou verkeerd

[ Voor 11% gewijzigd door Verwijderd op 21-12-2005 16:30 ]

[stekkel]

je moet natuurlijk bij een formpost de variabelen die je update wel uit $_POST vissen. Ik heb zo'n vermoeden dat je de variablen die je in de UPDATE sql query gebruikt de oude variebelen zijn die je met een select call aan de hand van $_GET["menuitem_id"] ophaalt. Je schrijft dan gewoon de originele waarden terug i.p.v. de aangepaste $_POST waarden.

Daarnaast zou ik eens error_reporting op E_ALL zetten want je script zal nogal wat warnings genereren.
Ik heb ook zo'n vermoeden dat je script gevoelig is voor sql injectie .....

[NMe]

P&W FAQ - PHP:Mijn POST en GET forms werken niet (meer)

[Verwijderd]

Wanneer ik de query verander naar menuitem_url='$_POST[menuitem_url]' en dat ook voor de andere items, dan doet ie het perfect.

[teun-v]

Dan zit er in een van al je andere includes die uitgevoerd worden voor dat je gaat updaten iets wat de var $menuitem_naam overschrijft met iets dat je niet wilt.

Sorry begrijp ik uit dit:

Verwijderd schreef op woensdag 21 december 2005 @ 17:02:
Wanneer ik de query verander naar menuitem_url='$_POST[menuitem_url]' en dat ook voor de andere items, dan doet ie het perfect.

dat het script werkt zoals het hoord??

anders geld het doorgestreepte nog steeds.

[ Voor 54% gewijzigd door teun-v op 21-12-2005 17:23 ]

[Verwijderd]

teun-v schreef op woensdag 21 december 2005 @ 17:15:
Dan zit er in een van al je andere includes die uitgevoerd worden voor dat je gaat updaten iets wat de var $menuitem_naam overschrijft met iets dat je niet wilt.

En kan ik daar met een debug methode achterkomen?

[teun-v]

Als het niet werkt zit er niks anders op dan al je includes die voor het update komen te doorzoek op de vars:

• $menuitem_naam
• $_POST['menuitem_naam']

Als deze overschreven worden klopt het dat je niet krijgt wat je wil.Maar lees mijn edit van het vorige bericht nog ff.

[ Voor 13% gewijzigd door teun-v op 21-12-2005 17:32 ]

[Verwijderd]

Nee het script werkt nu goed.
bedankt allemaal alvast!

Maar als ik het goed begrijp moet ik ook de includes van het formulier wijzigen

van:
$menuitem_naam
naar:
$_POST['menuitem_naam']

[NMe]

Goed, dit topic is dus samen te vatten in een zinnetje: "FAQ's lezen kan geen kwaad." DragonWish, voordat je ergens een topic opent, is het toch op zijn minst gangbaar om de daar geldende FAQ's te lezen. Daar staan in dit geval twee inhoudelijke dingen in, die je hadden kunnen helpen bij het oplossen van dit probleem. Verder staat register_globals al jaren standaard uit, dus als je PHP uit een boek of tutorial aan het leren bent, dan doe je er goed aan een meer recentere bron te zoeken.

Hoe dan ook, er schort nogal wat aan dit topic, om te beginnen dus het feit dat je de FAQ's niet gelezen hebt, en verder geef je in dit topic erg weinig informatie, waarbij verschillende mensen, waaronder ikzelf, meer dan eens om dezelfde informatie moesten vragen, zelfs als je gezegd werd hoe je aan die informatie kan komen.

Hoe dan ook doe je er goed aan de FAQ nog eens door te nemen, want er staan natuurlijk nog meer bruikbare dingen in. Lees ook P&W FAQ - De "quickstart" even door.