login  Naam:   Wachtwoord: 
Registreer je!
 Forum

Query Fout?

Offline roy - 20/09/2005 20:58 (laatste wijziging 21/09/2005 09:45)
Avatar van royPHP gevorderde
Citaat:
pages/tekstwijzigen.php?kop=1

Via deze link kom je op de volgende pagina.

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?
  2.  $_SESSION['kop'] = $_GET['kop']; 
  3.  
  4.  
  5. if(isset($_POST["submitreactie"])){ 
  6. $query2 = mysql_query("UPDATE tekst SET tekst='".$bericht."' WHERE id='".$_SESSION['kop']."'") or die(mysql_error("Kon geen verbinding maken met MySQL")); 
  7.      echo "De tekst is succesvol gewijzigd.";
  8.  
  9. 	}
  10.   ?>



Ik heb deze query, na op submit geklikt te hebben krijg je de melding "De tekst is succesvol gewijzigd.".. maar in de database is niet veranderd. wat kan er fout zijn?

MVG,
Roy de Kleijn

13 antwoorden

Gesponsorde links
Offline Rens - 20/09/2005 21:02
Avatar van Rens Gouden medaille

Crew algemeen		 $bericht bestaat niet?
Waarom zet je $_GET['kop'] in een sessie?
Offline roy - 20/09/2005 21:04 (laatste wijziging 21/09/2005 16:11)
Avatar van roy PHP gevorderde
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <? 
  4.  
  5.  $_SESSION['kop'] = $_GET['kop']; 
  6. ?>
  7.  
  8. <?
  9.  
  10. if(isset($_POST["submitreactie"])){ 
  11. print  $query2 = mysql_query("UPDATE tekst SET inhoud='".$bericht."' WHERE id='".$_SESSION['kop']."'") or die(mysql_error("Kon geen verbinding maken met MySQL")); 
  12.      echo "De tekst is succesvol gewijzigd.";
  13.  
  14. 	}
  15.  
  16.  
  17.  
  18. ?>
  19. <?
  20. $query="SELECT * FROM `tekst` WHERE id='".$_SESSION['kop']."'"; 
  21.  $sql=mysql_query($query) or die (mysql_error()); 
  22.  
  23.     while ($obj=mysql_fetch_object($sql)) { 
  24.  
  25. $tekst = $obj->tekst;
  26.  
  27.  
  28. ?>
  29.  
  30. <script language="JavaScript" src="java/ubb.js"></script>
  31. <style type="text/css">
  32.  
  33. html,body {
  34.     background-image: url(images/bg.gif);
  35.     font-family: Verdana;
  36.     font-size: 10pt;
  37.     color: #000000;
  38.     margin-left: 0px;
  39.     margin-top: 0px;
  40.     margin-right: 0px;
  41.     margin-bottom: 0px;
  42. }
  43. input, textarea, option {
  44.  border:1px solid #999999;
  45.  background-color: #FFFFFF;
  46. }
  47. a {
  48.  color: #000000;
  49.  text-decoration: none;
  50. }
  51. a:hover {
  52.  text-decoration: Underline; 
  53.  height: 1px;
  54. } 
  55.  
  56. .Table_Kader { 
  57.  border-top: 1px solid #B7B7B7;
  58.  border-left: 1px solid #B7B7B7;
  59.  border-right: 1px solid #B7B7B7;
  60.  border-bottom: 1px solid #B7B7B7;
  61. }
  62. .Cell_top {
  63.  border-top: 1px solid #B7B7B7;
  64. }
  65. .box {
  66.  background-color: #EFEDDE;
  67.  font-size: 10px;
  68.  border:1px solid blue #EFEDDE;
  69. }
  70. -->
  71. </style>
  72. </head>
  73. <body bgcolor="#FFFFFF" leftmargin="0" topmargin="0" marginwidth="0" marginheight="0">
  74. <form name="bfrm" action="<?php print $_SERVER['PHP_SELF'];  ?>" method="post">
  75.   <table width="570"  border="0" align="center" cellpadding="0" cellspacing="0" class="Table_Kader">
  76.     <tr>
  77.       <td width="574"><table width="574" height="23" border="0" cellpadding="0" cellspacing="0" bgcolor="EFEDDE" id="Table_01">
  78.           <tr>
  79.             <td width="23"><a href="#"
  80.                 onmouseover="window.status='Stijl en opmaak';  return true;"
  81.                 onmouseout="window.status='';  return true;"> <img src="images/Stijl-en-opmaak.gif" width="22" height="23" border="0" alt="Stijl en opmaak"></a></td>
  82.             <td width="271" bgcolor="EFEDDE"><table width="267" border="0" cellpadding="0" cellspacing="0" background="images/nav_02.gif">
  83.                 <tr>
  84.                   <td><select name="kop" onChange="addkop(this, 'bfrm', 'bericht');" class="box">
  85.                       <option value="0" selected>Standaard</option>
  86.                       <option value="1">Kop 1</option>
  87.                       <option value="2">Kop 2</option>
  88.                       <option value="3">Kop 3</option>
  89.                     </select>
  90.                       <select name="letter" onChange="addtype(this, 'bfrm', 'bericht');" class="box">
  91.                         <option value="Arial" selected>Arial</option>
  92.                         <option value="TRoman">Times New Roman</option>
  93.                         <option value="TMS">Trebuchet MS</option>
  94.                         <option value="Verdana" selected>Verdana</option>
  95.                       </select>
  96.                       <select name="size" onChange="addsize(this, 'bfrm', 'bericht');" class="box">
  97.                         <option value="10" selected>10</option>
  98.                         <option value="11">11</option>
  99.                         <option value="12">12</option>
  100.                         <option value="16">16</option>
  101.                         <option value="20">20</option>
  102.                       </select>
  103.                   </td>
  104.                 </tr>
  105.             </table></td>
  106.             <td width="22"><a href="#"
  107.                 onmouseover="window.status='Vet';  return true;"
  108.                 onmouseout="window.status='';  return true;"> <img src="images/Vet.gif" width="22" height="23" border="0" name="b" onMouseDown="klik('b','bfrm','bericht');" onMouseUp="foc('bfrm','bericht');"  alt="Vet"></a></td>
  109.             <td width="23"><a href="#"
  110.                 onmouseover="window.status='Cursief';  return true;"
  111.                 onmouseout="window.status='';  return true;"> <img src="images/Cursief.gif" width="23" height="23" border="0" name="i" onMouseDown="klik('i','bfrm','bericht');" onMouseUp="foc('bfrm','bericht');" alt="Cursief"></a></td>
  112.             <td width="24"><a href="#"
  113.                 onmouseover="window.status='Onderstrepen';  return true;"
  114.                 onmouseout="window.status='';  return true;"> <img src="images/Onderstrepen.gif" width="24" height="23" border="0" name="u" onMouseDown="klik('u','bfrm','bericht');" onMouseUp="foc('bfrm','bericht');" alt="Onderstrepen"></a></td>
  115.             <td width="23"><a href="#"
  116.                 onmouseover="window.status='Links uitlijnen';  return true;"
  117.                 onmouseout="window.status='';  return true;"> <img src="images/Links-uitlijnen.gif" width="23" height="23" border="0" name="l" onMouseDown="klik('l','bfrm','bericht');" onMouseUp="foc('bfrm','bericht');" alt="Links uitlijnen"></a></td>
  118.             <td width="22"><a href="#"
  119.                 onmouseover="window.status='Centreren';  return true;"
  120.                 onmouseout="window.status='';  return true;"> <img src="images/Centreren.gif" width="22" height="23" border="0" name="c" onMouseDown="klik('c','bfrm','bericht');" onMouseUp="foc('bfrm','bericht');" alt="Centreren"></a></td>
  121.             <td width="23"><a href="#"
  122.                 onmouseover="window.status='Rechts uitlijnen';  return true;"
  123.                 onmouseout="window.status='';  return true;"> <img src="images/Rechts-uitlijnen.gif" width="23" height="23" border="0" name="r" onMouseDown="klik('r','bfrm','bericht');" onMouseUp="foc('bfrm','bericht');" alt="Rechts uitlijnen"></a></td>
  124.             <td width="23"><a href="#"
  125.                 onmouseover="window.status='Uitvullen';  return true;"
  126.                 onmouseout="window.status='';  return true;"> <img src="images/Uitvullen.gif" width="23" height="23" border="0" name="uit" onMouseDown="klik('uit','bfrm','bericht');" onMouseUp="foc('bfrm','bericht');" alt="Uitvullen"></a></td>
  127.             <td><a href="#"
  128.                 onmouseover="window.status='Nummering';  return true;"
  129.                 onmouseout="window.status='';  return true;"> </a><a href="#"
  130.                 onmouseover="window.status='Opsommingstekens';  return true;"
  131.                 onmouseout="window.status='';  return true;"> </a><a href="#"
  132.                 onmouseover="window.status='Hyperlink invoegen';  return true;"
  133.                 onmouseout="window.status='';  return true;"> <img src="images/Hyperlink-invoegen.gif" width="28" height="23" border="0" name="h" onMouseDown="klik('h','bfrm','bericht');" onMouseUp="foc('bfrm','bericht');" alt="Hyperlink invoegen"></a></td>
  134.             <td width="49" bgcolor="EFEDDE">&nbsp;</td>
  135.           </tr>
  136.       </table></td>
  137.     </tr>
  138.     <tr>
  139.       <td><div align="center">
  140.           <textarea name="bericht" rows="20" tabindex="1" style="width: 99%;">
  141. <? echo $tekst; ?></textarea>
  142.       </div></td>
  143.     </tr>
  144.     <tr>
  145.       <td><div align="center">
  146.           <input type="submit" name="submitreactie" value="Pagina wijzigen" tabindex="2" />
  147.       </div></td>
  148.     </tr>
  149.   </table>
  150. </form>
  151.  
  152. </body>
  153. </html>
  154. <? 
  155. } 
  156. ?>


bericht bestaat wel...
Het is de bedoeling dat je met dit script tekst kan wijzigen, maar hij slaat het niet op? (ofwel schrijft het niet weg in de database)
Offline Rens - 20/09/2005 21:07
Avatar van Rens Gouden medaille

Crew algemeen		 Owh, op die manier $bericht.
Je moet hier $_POST['bericht'] gebruiken.
Offline roy - 20/09/2005 21:10
Avatar van roy PHP gevorderde dan zegt hij : unexpected T_VARIABLE
Offline brechtjeu - 20/09/2005 21:45
Avatar van brechtjeu Onbekend HEY,

mysql_query("UPDATE tekst SET tekst='".$bericht."' WHERE id='".$_SESSION['kop']."'") or die(mysql_error("Kon geen verbinding maken met MySQL"));

staat in je script
kan dat wel een tabel tekst met een kolom tekst? verwart hij dit niet
kweetniet hoor(niet sarcastisch bedoelt)
Offline Rens - 20/09/2005 21:50
Avatar van Rens Gouden medaille

Crew algemeen		 neej.
int engels is ut text...
Offline brechtjeu - 20/09/2005 21:52
Avatar van brechtjeu Onbekend mja, kdacht dat je geen dezelfde namen kon gebruiken zoals hij doet
tabelnaam: tekst, kolomnaam: tekst
Offline roy - 20/09/2005 21:57 (laatste wijziging 20/09/2005 22:38)
Avatar van roy PHP gevorderde ik heb de kolomnaam veranderd in inhoud....
maar dat maakt niks uit..

@rensjuh of iemand anders: zie jij niet wat k fout doe? Als ik in plaats van die variabel een regelnummer invul dan werkt het wel...

maar wil juist dat hij uit de hyperlink het regelnummer haalt. Dat werkt enigzins want hij haalt de tekst er wel uit, maar hij schrijft niks erin
Offline cracker - 21/09/2005 01:06
Avatar van cracker Onbekend
Citaat:
mysql_query("UPDATE tekst SET tekst='".$bericht."' WHERE id='".$_SESSION['kop']."'") or die(mysql_error("Kon geen verbinding maken met MySQL"));
Dit is niet fout qua code maar die die() geeft niet weer of er al dan niet een verbinding gemaakt werd !
Wel of de query uitgevoerd is 
Offline marten - 21/09/2005 09:25 (laatste wijziging 21/09/2005 09:32)
Avatar van marten Beheerder Heb je al geprobeerd om stukje bij beetje de code te gaan bekijken? Dus bijvoorbeeld eerst kijken of er een sessie geopend is en dat de cookie gezet is enzovoorts......

Hier kom je dan meestal wel tot een fout

EDIT
if(isset($_POST["submitreactie"])){
print $query2 = mysql_query("UPDATE tekst SET inhoud='".$bericht."' WHERE id='".$_SESSION['kop']."'") or die(mysql_error("Kon geen verbinding maken met MySQL"));
echo "De tekst is succesvol gewijzigd.";

}



?>
<?
$query="SELECT * FROM `tekst` WHERE id='".$_SESSION['kop']."'";
$sql=mysql_query($query) or die (mysql_error());

while ($obj=mysql_fetch_object($sql)) {

$tekst = $obj->tekst;


?>

Bij de bovenste gebruik je print voor de query en daaronder zonder print..... misschien dat je bij de bovenste query die print kan weghalen want die staat volgens mij aardig onnodig daar 
Offline roy - 21/09/2005 09:47
Avatar van roy PHP gevorderde ja, die is onnodig, maar dat was niet de oplossing:(
Offline Sc0tTy - 21/09/2005 10:29
Avatar van Sc0tTy PHP interesse wrom gebruikt iedereen $_GET['XXXX'] je kan toch ook gewoon $get doen ??
Offline Button - 21/09/2005 19:19 (laatste wijziging 21/09/2005 19:29)
Avatar van Button PHP ver gevorderde @scoty: het gaat hier om een superglobal niet om een variabele met de naam $get

@roy: natuurlijk dat er "De tekst is succesvol gewijzigd." komt te staan, dat is normaal want je test het eigenlijk niet. maak daar eens : (het is ook wat verduidelijkt):

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?
  2. $query2 = "UPDATE tekst SET tekst='".$bericht."' WHERE id='".$_SESSION['kop']."'" ;//dit is echt de query
  3. $result=mysql_query($query2) or die(mysql_error()); //dit is het $result=resultaat van de query...
  4.  
  5. if($result){//hier wordt getest of het wel degelijk gelukt is, eigenlijk zou als het niet lukt dit al duidelijk moeten worden met de or die() maar toch...
  6.      echo "De tekst is succesvol gewijzigd.";
  7. }
  8. ?>

edit:

php code - Bekijk de code zonder highlighting - Klap code in 
  1. ?>
  2.  
  3. <?


en waar slaagt dit op ? regel 6 en 8 (op het volledige script)
en je maakt toch degelijk connectie he?
en waarom gebruik je $_SESSION al ? die $_GET staat toch nog op die pagina?

(niet om je af te breken hoor, alleen om te helpen)
good luck

edit2: mysql_error(): kan volgens mij geen text bevatten -> khem mijn scriptje hierboven ook aangepast!
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.235s