Door
Rob Peres
op 13-02-2012 15:39
gewijzigd op 13-02-2012 16:40
5.311 views
Ik heb een formulier op de site. Na invulling worden de gegevens in een database gelade. Tot zover werkt het. Nu wil ik graag dat deze gegevens (dus eigenlijk het formulier) niet alleen in de DB wordt geladen, maar dat de gegevens ook per email woprden verzonden naar één of meerdere adressen.
*ik heb dit niet zelf gemaakt en dit gaat dus ver boven mijn pet.
Als iemand mij op weg kan helpen, graag!
is dit alles? niet meer php? want dit doet helemaal niets, zelfs niet in de database zetten. wat heb je nog meer?
verder zijn er een aantal aandachtspuntjes:
- geen foutafhandeling met or die(mysql_error())
- niet gebruik maken van isset, gebruik hier empty voor
- <?=$_SERVER['PHP_SELF']?> vervang dit door de pagina die wordt gebruikt bij verwerking, dit is onveilig
- voor meerdere <br> is de <p> tag uitgevonden
- geen opmaak met tabellen
Dit is een zeer slechte check:
<?php
!empty($_POST))
{
$aFouten = array();
if($_POST['Naam'] == '')
?>
Ten eerste gebruik je empty waar je isset zou moeten gebruiken (zoals santhe al zei), daarnaast check je helemaal niets. Checken of $_POST bestaat heb je helemaal nul komma niets aan. Het zegt namelijk nog niets over of $_POST['Naam'] bestaat (en alle andere die je wilt hebben). Je zal elke key afzonderlijk moeten checken of die key bestaat of niet, met isset() dus.
Hey,
Ik stel ALLE opbouwende kritieken op prijs, maar zou graag één probleempje tegelijk aanpakken. G hier net mee beginnen en zal met dus in de !empty en !isset gaan verdiepen en zelf proberen. Als ik hier niet uitkom, vind je me weer terug hier.
Maar is er iemend die mij eerst op weg kan helpen met het verwerken in de database (gelukt) èn verzenden per email please.
Heb 4 van dit soort formulieren.
[size=xsmall]Toevoeging op 13/02/2012 16:31:16:[/size]
Oeps, iets te snel
[size=xsmall]Toevoeging op 13/02/2012 16:37:09:[/size]
Allen bedankt. Ga met een mailclass aan de gang. Heb geen zin het wiel opnieuw uit te vinden en neem jullie advies ter harte. Zal via phphulp de tutorials goed bekijken en kijken of dit lukt. Als ik dat voor elkaar heb ga ik eerst aan de veligheid van de site werken (heel hard nodig)
Dat je één probleem tegelijk aan wilt pakken is te begrijpen, maar begin dan wel vooraan. Bovenstaand script is gewoon niet goed. Het is zelfs lek wat betreft de beveiliging van de database. Je controles zijn zeer onvoldoende.
Ik kom er dus niet uit. Heb hier gewoon te weinig verstand van en de basis ontbreekt. Degene die dit voor mij in elkaar heeft gezet (nu blijkt niet erg goed gedaan) heeft afgehaakt en ik zit nu met een site waar ik niks mee kan.
Is er iemand die mij (eventueel tegen betaling) kan helpen?
*formulier moet beveiligd worden
*gegevens moeten na invulling in de database per email worden verstuurd naar leden
Ik heb dagenlang zitten klooien maar dit is gewoon te moeilijk voor mij. Heb heeeel veel forums gevolgd, tutorials gelezen, SwiftMailer geinstalleerd en gepuzzeld: kom er niet uit.
[size=xsmall]Toevoeging op 17/03/2012 14:50:29:[/size]
Voor de duidelijkheid:
De gegevens worden netjes geladen in de database. De bedoeling is dat de ingevulde gegevens ook worden gemaild naar alle leden zodat iedereen weet dat er een aanvraag is en er op kan worden gereageerd.
Ik weet dus echt niet hoe.
Ik heb inmiddels ook begrepen dat dit formulier zo lek is als een mandje dus dat moet ook aangepast worden.