Contact script , Swift-Mailer

Door Jordi Kroon, 12 jaar geleden, 11.323x bekeken

Ik had even tijd en had voor jullie een contact script gemaakt.
Omdat er hier heel veel personen een mailing script hebben waarbij header injection mogelijk is

Ik hoop dat jullie hier iets mee kunnen

Verbeteringen en tips altijd welkom


Download swiftmailer:
Download link

Demo:
Demo

Gesponsorde koppelingen

PHP script bestanden

  1. contact.php

 

Er zijn 16 reacties op 'Contact script swiftmailer'

PHP hulp
PHP hulp
0 seconden vanaf nu
 

Gesponsorde koppelingen


12 jaar geleden
 
0 +1 -0 -1
Ga je eigen tutorial volgen.
Dit is onzin.
Jordi Kroon
Jordi Kroon
12 jaar geleden
 
0 +1 -0 -1
En , wat is er dan allemaal niet goed ?
Jordi Kroon
Jordi Kroon
12 jaar geleden
 
0 +1 -0 -1
Edit: Klein foutje eruit gehaald omdat bannen bij email niet werkte


12 jaar geleden
 
0 +1 -0 -1
Hmm, volgens mij is hij nu anders. Maar goed, ik zie het nut van dit script niet echt in aangezien dit ook wel bij swiftmailer staat uitgelegd.
Ook is het zo dat je niet goed kan checken met een regex of een e-mailadres geldig is.
Exit / die is onzin.
trim?
Waar komt $error weg?
Waarom $error op deze manier?
Foutieve html.
Als je html verstuurd moet je ook htmlentities gebruiken, want het is html.
Foutive logica.
Dubbelop zooi.
Tabellen horen niet gebruikt te worden voor forms.
Inline css + vage javascript zooi.
Jordi Kroon
Jordi Kroon
12 jaar geleden
 
0 +1 -0 -1
regex controleren: Of hij valid is kan hij inderdaad niet controleren maar hij kan wel controleren of er een goede logica in zit dus zo : naamvoor@domein.taal

Exit:
Nu je het zegt , Ja . Heb hem ook even gewijzigd

Waar komt $error weg?
Kromme zin

Htmlentities
Je moet html kunnen versturen
voor als je bijv iets dikgedrukt wil

Inline css
Om aantegeven of een veld niet ingevuld is

Logica.
Ik vind dit redelijk logisch hoor


12 jaar geleden
 
0 +1 -0 -1
Nee, die regex laat mogelijk goede e-mailadressen door. Zoals ik dus zei 'Ook is het zo dat je niet goed kan checken met een regex of een e-mailadres geldig is.'
Dus dat betekend dat je geen regex (niet op een zinnige manier) kan toepassen om te kijken of een e-mailadres geldig is.

De zin 'Waar komt $error weg?' is niet krom. Jij bent zelf krom door zomaar opeens $error te gaan gebruiken. Wat je wilt bereiken snap ik, maar kan veel logischer.

Van mij mogen gebruikers ook geen html mailen. Dus daarom zeg ik dat je ook htmlentities moet gebruiken. Of iets dik gedrukt moet worden hoeft de gebruiker niet te bepalen.

Inline css / javscript is dus echt niet net. Wat als ik nu wil dat mijn border groen moet worden als iets is ingevuld? Dat kan niet makkelijk veranderd worden met jou script.

Logica: Het is toch een onzinnig script dus ga uitleggen hoe je het helemaal moet aanpassen.
Jordi Kroon
Jordi Kroon
12 jaar geleden
 
0 +1 -0 -1
Dan maak je er toch groen van

Toch vind ik waar komt $error weg? een kromme zin
Vincent Huisman
Vincent Huisman
12 jaar geleden
 
0 +1 -0 -1
tabs gebruiken kan overigens ook


12 jaar geleden
 
0 +1 -0 -1
Inline rommel is rommel omdat het niet onderhoudsvriendelijk is.
Waar komt de variabele, die later tot array wordt omgetoverd, genaamd $error vandaan c.q. weg?
Waar is die gedefinieerd als zijnde array?
Waarom op deze onhandige manier?
Jordi Kroon
Jordi Kroon
12 jaar geleden
 
0 +1 -0 -1
Ik heb er zelf even $error = array(); aan toegevoegd , deze werkik morgen bij omdat ik nu opmijn ipod zit en geen rar kan openen

@karl ik denk dat je dit ermee bedoelde of niet
Chris -
Chris -
12 jaar geleden
 
0 +1 -0 -1
Gelieve geen download links toe te voegen, maar het script hier te plaatsen. Een link naar de Swift Mailer library is geen probleem, plaats hier dan gewoon de 'toepassing' van die library.

Doe je dit niet, verwijder ik dit script.
TJVB tvb
TJVB tvb
12 jaar geleden
 
0 +1 -0 -1
@Chris, Hoera

@Jordi, als je het script gaat plaatsen denk dan aan:
Instellingen, het to adres is staat midden in je code, vaak gebruik je bepaalde adressen op meerdere plekken, het zou handiger zijn om een setting gedeelte te hebben.
Je HTML is geheel invalid. Stop het in een variabele die je aan het einde terug geeft of maak je HTML valid.
Over inline css/javascript. Wat is je reden on het inline te doen. Extern is vele malen makkelijker bij te houden.
Jordi Kroon
Jordi Kroon
12 jaar geleden
 
0 +1 -0 -1
Ik heb nu het grootste gedeelte verbeterd :)
Ook karls htmlentries
Html validated voor het grootste gedeelte alleen 2 errors die ik niet opgelost krijg

Ook heb ik wat TJVN mij aanraadde de settings bovenaan de pagina gezet

Verder :
Ik gebruik inline css in onchange omdat ik dus wil dat als er een error is , Het vlak rood gekleurd word en als iemand erop klikt dat het weer normaal word
vandaar
TJVB tvb
TJVB tvb
12 jaar geleden
 
0 +1 -0 -1
Dan kun je met een onchange de class aanpassen en die vanuit je css stylen.
Chris -
Chris -
12 jaar geleden
 
0 +1 -0 -1
Waarom zijn de echo's met een hoofdletter? strtolower ;-)
PHP hulp
PHP hulp
0 seconden vanaf nu
 

Gesponsorde koppelingen
Jordi Kroon
Jordi Kroon
12 jaar geleden
 
0 +1 -0 -1
@chris , Automatisme :P

Om te reageren heb je een account nodig en je moet ingelogd zijn.

Inhoudsopgave

  1. contact.php

Labels

Navigatie

 
 

Om de gebruiksvriendelijkheid van onze website en diensten te optimaliseren maken wij gebruik van cookies. Deze cookies gebruiken wij voor functionaliteiten, analytische gegevens en marketing doeleinden. U vindt meer informatie in onze privacy statement.