 Ouwe rakker |
|
Er zijn enkele dingen die verkeerd gaan in je script, tevens heb ik een vraagje.
Wat er verkeerd gaat is dat je de variabele $Bericht maakt aan de hand van de $_POST['Bericht']. Op regel 44 van je code plak je de aanhef voor het bericht. Wanneer het bericht dus 'Je hebt gewonnen' zou zijn, dan wordt het:
Citaat: <aanhef> <gebruikersnaam>,
Je hebt gewonnen
Het bericht is op dat moment dus die tekst. Wanneer je bij de tweede record aankomt, dus de tweede persoon die je gaat mailen, plakt hij weer voor $Bericht de aanhef. Het bericht wordt dan dus:
Citaat: <aanhef> <gebruikersnaam2>,
<aanhef> <gebruikersnaam>,
Je hebt gewonnen
Je moet dus gewoon even een aparte variabele maken waar je het bericht voor de e-mail in voorbereid. Die kan je dan veilig gebruiken. Ik raad je trouwens ook aan om niet alle POST variabelen om te zetten naar lokale ($iets) zonder er een bewerking op uit te voeren. Het is wat onzinnig om extra variabelen aan te maken met dezelfde identieke informatie.
Het is tevens niet nodig om bij elke rij de PHP.net: htmlentities toe te passen. Eénmaal op het bericht en daarna op de waardes uit de database is voldoende.
Als je trouwens ook kijkt naar je code in de highlighter dan zal je zien dat je bepaalde stukken tekst niet op de juiste manier escaped en dat daarom php het ook niet zal pakken. Wanneer je de echo begint met een ' dan dien je ook alle ' te escapen in de zin om te voorkomen dan php zegt dat je het maar lekker zelf kan uitzoeken. Hieronder zie je het voorbeeld met een plaatsnaam, in de eerste regel staat het goed, in de tweede niet.
<?php
echo 'ik woon in \'s-Hertogenbosch';
echo 'ik woon in 's-Hertogenbosch';
<?php echo 'ik woon in \'s -Hertogenbosch '; echo 'ik woon in 's-Hertogenbosch';
Waarschijnlijk zie je nu het verschil zelf ook wel. 
Wat ik me zelf nog afvroeg is hoe jij de aanhef bijvoorbeeld invult in je formulier? Ik neem aan dat het 'Beste ' is oid. Want aangezien jij maar één maal een aanhef kan kiezen, maar iemand in de database van het vrouwelijke of het mannelijke geslacht is, dien jij dan dus een aanhef te kiezen die universeel is.
Ik heb in ieder geval even je script opnieuw gemaakt, zoals ik het zou doen. Hoop dat je er wat van opsteekt
http://www.pastebin.be/12980
ps. Ik heb het zelf niet geprobeerd, zit namelijk op de camping dit te tikken. Het is voor de rest toch maar bedoeld als informatie. |
Wat is jouw favoriete javascript framework? (S: 18, R: 2)
Gesponsorde links
Actiefste leden van de maand
