De PHPhulp kroeg
Contact Script
Door Hans , 21 jaar geleden, 6.198x bekeken
Maak een nieuwe File : contact.php
Bewerk alleen Die dingen onder //Configureer//
Gesponsorde koppelingen
PHP script bestanden
Er zijn 20 reacties op 'Contact script'
0 
-0 
-0
Form controle klopt niet: If(!empty($_POST['verstuur'])) -> Dat moet met request method.
Dit is zo geweldig voor mail injections: $header = "From: ". $_POST['name'] . "\n"; //Door wie die is verstuurd
Waar zijn de <html><body> en <head> tags? En het is niet echt netjes om een form in een tabel te stoppen, en eventueel zou je die html uit die error kunnen halen. Die zou ik ook eerder bovenaan de pagina zetten.... Dat is netter dan dat de gebruiker terug moet gaan (en als je dan nog verder wilt kan je de data die de gebruiker ingevuld had ook nog weer eens in het form stoppen, zodat die niet weer alles opnieuw hoeft te doen).
Voor de rest is het wel oke. (Maar pas dus goed op met die mail injections.)
Dit is zo geweldig voor mail injections: $header = "From: ". $_POST['name'] . "\n"; //Door wie die is verstuurd
Waar zijn de <html><body> en <head> tags? En het is niet echt netjes om een form in een tabel te stoppen, en eventueel zou je die html uit die error kunnen halen. Die zou ik ook eerder bovenaan de pagina zetten.... Dat is netter dan dat de gebruiker terug moet gaan (en als je dan nog verder wilt kan je de data die de gebruiker ingevuld had ook nog weer eens in het form stoppen, zodat die niet weer alles opnieuw hoeft te doen).
Voor de rest is het wel oke. (Maar pas dus goed op met die mail injections.)
0 -0
-0
Één van ze zoveel contact scripts in de code lib. van PHPHulp, en dan is het ook nog niet eens netjes en veilig gescript.
0 -0
-0
En weer zeiken we een script af, laten we hem nou gewoon aardig helpen? Iedereen heeft zijn eerste script!
0 -0
-0
Quote:
hans schreef op 09.09.2008 16:36
het is me eerste script
het is me eerste script
Ja, niet de mijne.
Quote:
Han schreef op 09.09.2008 16:51
En weer zeiken we een script af, laten we hem nou gewoon aardig helpen? Iedereen heeft zijn eerste script!
En weer zeiken we een script af, laten we hem nou gewoon aardig helpen? Iedereen heeft zijn eerste script!
Wil jij me eerste tekening hebben soms? Misschien moet je effe bijhouden met de topics, zie dit topic...
0 -0
-0
Van je script klopt niets?
Waarom geef je in hemelsnaam $voornaam/$achternaam hier al op? Vrij nutteloos, aangezien er meerdere zijn die stukken beter is.. Wellicht dat je daar even naar kan kijken?
En Han, het is niet het afkraken van een script, maar de doelloosheid van het toevoegen van dit soort scripts. Als het je eerste script ever made is, laat hem dan eerst even zien op het forum om te beoordelen...
Code (php)
1
2
3
4
5
6
7
8
2
3
4
5
6
7
8
<?
//Configureer//
$websitenaam = "phphulp"; // Naam Van je Website ( waar ze contact hebben genomen!)
$subject = "**phphulp**"; // Onderwerp (Als iemand contact neemt krijgt hij meteen bericht terug!)
$voornaam = "phphulp"; // Je Voornaam
$achternaam = "phphulp"; // Deze HOEF je niet te doen mag wel!
$recipient = "Beste ".$voornaam." , Uw bericht is Verstuurd en zal Zo spoedig mogelijk Beantwoord Worden, MvG Admin";
?>
//Configureer//
$websitenaam = "phphulp"; // Naam Van je Website ( waar ze contact hebben genomen!)
$subject = "**phphulp**"; // Onderwerp (Als iemand contact neemt krijgt hij meteen bericht terug!)
$voornaam = "phphulp"; // Je Voornaam
$achternaam = "phphulp"; // Deze HOEF je niet te doen mag wel!
$recipient = "Beste ".$voornaam." , Uw bericht is Verstuurd en zal Zo spoedig mogelijk Beantwoord Worden, MvG Admin";
?>
Waarom geef je in hemelsnaam $voornaam/$achternaam hier al op? Vrij nutteloos, aangezien er meerdere zijn die stukken beter is.. Wellicht dat je daar even naar kan kijken?
En Han, het is niet het afkraken van een script, maar de doelloosheid van het toevoegen van dit soort scripts. Als het je eerste script ever made is, laat hem dan eerst even zien op het forum om te beoordelen...
0 -0
-0
@karl: Tuurlijk, ik snap het ook wel, en dit is allang ten sprake geweest. Je gaat direct in de aanval als het een slecht script is, en natuurlijk moet dat anders, maar wat ik bedoelde te zeggen was, dat iedereen voor het eerst zijn script post. En natuurlijk kan het slecht zijn, maar waarom direct afzijken? Ik ben het met je eens, dat er iets anders voor moet komen, maar ik vind niet dat je als iemand een nieuw script post, en het is slecht, direct de persoon/script moet afzijken. (en ja, dit is aan afgezaagd onderwerp, en ja, om dit op te lossen zullen we een andere mod systeem oid moeten hebben, en ja, dit is al eerder gezegd ;-) )
0 -0
-0
Quote:
Han schreef op 09.09.2008 19:20
@karl: Tuurlijk, ik snap het ook wel, en dit is allang ten sprake geweest. Je gaat direct in de aanval als het een slecht script is, en natuurlijk moet dat anders, maar wat ik bedoelde te zeggen was, dat iedereen voor het eerst zijn script post. En natuurlijk kan het slecht zijn, maar waarom direct afzeiken? Ik ben het met je eens, dat er iets anders voor moet komen, maar ik vind niet dat je als iemand een nieuw script post, en het is slecht, direct de persoon/script moet afzijken. (en ja, dit is aan afgezaagd onderwerp, en ja, om dit op te lossen zullen we een andere mod systeem oid moeten hebben, en ja, dit is al eerder gezegd ;-) )
@karl: Tuurlijk, ik snap het ook wel, en dit is allang ten sprake geweest. Je gaat direct in de aanval als het een slecht script is, en natuurlijk moet dat anders, maar wat ik bedoelde te zeggen was, dat iedereen voor het eerst zijn script post. En natuurlijk kan het slecht zijn, maar waarom direct afzeiken? Ik ben het met je eens, dat er iets anders voor moet komen, maar ik vind niet dat je als iemand een nieuw script post, en het is slecht, direct de persoon/script moet afzijken. (en ja, dit is aan afgezaagd onderwerp, en ja, om dit op te lossen zullen we een andere mod systeem oid moeten hebben, en ja, dit is al eerder gezegd ;-) )
Oke, dus alles wat je daar zegt is dat ik direct in de aanval ga en het script afzeik?
Hmm... Zou je mij dan willen quote'en want ik zie echt niet waar dat is. Verder ben ik ook niet direct in de aanval gegaan, misschien vind je het niet fijn dat ik direct zeg wat er niet klopt, maar op het end zeg ik wel dat het wel een aardig script is verder (dat van Chris moet d'r ook bij). Dat is gewoon mijn manier, eerst de fouten punten, dan pas de goede.
Voor de rest, als je een soort kruistocht ofzo tegen mij probeert te voeren verwijs ik je naar het admin gedeelte ofzo.
0 -0
-0
Quote:
En weer zeiken we een script af, laten we hem nou gewoon aardig helpen? Iedereen heeft zijn eerste script!
Ik snap dat het leuk is om je eerste script met andere te delen, maar door dit soort dingen al een aantal jaar lang, staan er een heleboel krek scripts/tutorials online die het verkeerd leren aan andere beginners. Daarom heb ik liever alleen goeie scripts online. Maar dat kan aan mij liggen...
0 -0
-0
@hans, kun je volgende keer eerst kijken of er al gelijke scripts in de lib staan? Het is wel een aardig script, maar het heeft geen toegevoegde waarde ;)
@discussie over het afkraken, als we daar nou een topic voor openen? :D en misschien is het een goed idee dat mensen die nog nooit iets online hebben gezet hier eerst een topic moeten openen over dat script? of dat bij iedereen het eerste script/tut door een admin moet worden gecontroleerd?
@discussie over het afkraken, als we daar nou een topic voor openen? :D en misschien is het een goed idee dat mensen die nog nooit iets online hebben gezet hier eerst een topic moeten openen over dat script? of dat bij iedereen het eerste script/tut door een admin moet worden gecontroleerd?
0 -0
-0
@karl: Sorry, dat bedoelde ik nou ook weer niet.
Oké, je zijkt niet letterlijk af, maar je kan nou niet zeggen dat je toon super aardig is ;-)
Maar goed, maakt mij ook niet uit.
Quote:
Form controle klopt niet: If(!empty($_POST['verstuur'])) -> Dat moet met request method.
Dit is zo geweldig voor mail injections: $header = "From: ". $_POST['name'] . "\n"; //Door wie die is verstuurd
Dit is zo geweldig voor mail injections: $header = "From: ". $_POST['name'] . "\n"; //Door wie die is verstuurd
Oké, je zijkt niet letterlijk af, maar je kan nou niet zeggen dat je toon super aardig is ;-)
Maar goed, maakt mij ook niet uit.
0 -0
-0
er zit een foutje in de scripts,,
jullie zijn deze vergeten,,,
maar heb hem verbetert,,
want jullie zijn deze stukje vergeten: $recipient_mail = "[email protected]";
maar hij staat er nu in;)
want anders krijgen mensen geen email,, want als mensen contact met je willen, vullen ze dan voor niks dat forum in, want je kreeg geen email!
maar heb hem verandert,, dus deze stukje is goed;)
jullie zijn deze vergeten,,,
maar heb hem verbetert,,
want jullie zijn deze stukje vergeten: $recipient_mail = "[email protected]";
maar hij staat er nu in;)
want anders krijgen mensen geen email,, want als mensen contact met je willen, vullen ze dan voor niks dat forum in, want je kreeg geen email!
maar heb hem verandert,, dus deze stukje is goed;)
Code (php)
1
2
3
4
5
6
7
8
9
2
3
4
5
6
7
8
9
<?
//Configureer//
$websitenaam = "phphulp"; // Naam Van je Website ( waar ze contact hebben genomen!)
$subject = "**phphulp**"; // Onderwerp (Als iemand contact neemt krijgt hij meteen bericht terug!)
$voornaam = "phphulp"; // Je Voornaam
$achternaam = "phphulp"; // Deze HOEF je niet te doen mag wel!
$recipient_mail = "je [email protected]"; //je email!!
$recipient = "Beste ".$voornaam." , Uw bericht is Verstuurd en zal Zo spoedig mogelijk Beantwoord Worden, MvG Admin";
?>
//Configureer//
$websitenaam = "phphulp"; // Naam Van je Website ( waar ze contact hebben genomen!)
$subject = "**phphulp**"; // Onderwerp (Als iemand contact neemt krijgt hij meteen bericht terug!)
$voornaam = "phphulp"; // Je Voornaam
$achternaam = "phphulp"; // Deze HOEF je niet te doen mag wel!
$recipient_mail = "je [email protected]"; //je email!!
$recipient = "Beste ".$voornaam." , Uw bericht is Verstuurd en zal Zo spoedig mogelijk Beantwoord Worden, MvG Admin";
?>
0 -0
-0
Code (php)
1
echo "<br>Klik Alstublieft op <a href=javascript:history.back(1)>ga terug</a> en vull alle velden in<br><br>";
Klein spellingsfoutje, vul alle velden in.
Staat iets netter ;)
0 -0
-0
wellicht een idee om op deze website een waarderingssysteem te maken als die er al niet is? Des te hoger de waardering des te hoger je niveau van scripten. Die waardering krijg je van leden en niet van bezoekers.
Bezoekers (en leden) zien dan diverse niveaumappen, zoals Beginneling of PRO.
Beginneling heeft al een map
in dit forum
Bijv. 1-250 punten Beginneling (rubriek = dan Beginneling zodat iedereen snapt dat dit scripts van beginnelingen zijn en je zo daar rekening mee kunt houden !)
251-500 punten = etc etc
(binnen die niveaumappen tref je dan diverse scriptmappen aan, zoals Contactscripts, Classscripts, etc)
Als je een nieuw script hebt komt deze éérst in een NOG TE WAARDEREN map. Is dit script minimaal 1 keer gewaardeerd komt het in de Beginneling map etc.... Je kunt een waardering uitspreken van bijv. 1 punt of maximaal 10. Je kunt zelfs nog instellen dat een PRO meer (of minder) punten mag geven dan een Beginneling....
Zoiets? Ik ben maar even aan het brainstormen...
Uiteraard is een waardering subjectief, maar je moet toch iets bedenken...
Bezoekers (en leden) zien dan diverse niveaumappen, zoals Beginneling of PRO.
Beginneling heeft al een map
Code (php)
1
http://www.phphulp.nl/php/scripts/
in dit forum
Bijv. 1-250 punten Beginneling (rubriek = dan Beginneling zodat iedereen snapt dat dit scripts van beginnelingen zijn en je zo daar rekening mee kunt houden !)
251-500 punten = etc etc
(binnen die niveaumappen tref je dan diverse scriptmappen aan, zoals Contactscripts, Classscripts, etc)
Als je een nieuw script hebt komt deze éérst in een NOG TE WAARDEREN map. Is dit script minimaal 1 keer gewaardeerd komt het in de Beginneling map etc.... Je kunt een waardering uitspreken van bijv. 1 punt of maximaal 10. Je kunt zelfs nog instellen dat een PRO meer (of minder) punten mag geven dan een Beginneling....
Zoiets? Ik ben maar even aan het brainstormen...
Uiteraard is een waardering subjectief, maar je moet toch iets bedenken...
0 -0
-0
@ sunadumari : heel constructief!!! Ik vind het persoonlijk ver buiten alle fatsoensnorm gaan wanneer iemand zijn 1e script post veelal vol trots om dan vervolgens hier zo te worden afgez*k*n. Waarom toch? Zijn we allemaal vergeten hoe trots we zelf waren toen we eindelijk na veel geploeter een script in elkaar hadden gezet? Maar goed ik zal wel ouderwets en dom en vooral super blond zijn om deze motivatie te kunnen begrijpen.........sterker nog ik wil het niet eens meer begrijpen.
0 -0
-0
Ik ga helemaal met je mee Lissy ! Je moet ALTIJD verbeterpunten aangeven en mensen in de juiste richting sturen zodat ze er wat van leren en gemotiveerd raken of nog meer gemotiveerd raken! MOTIVATIE werkt.
Gr. Sunadumari
Gr. Sunadumari
0 -0
-0
@lissy helemaal mee eens. Ik heb nog niet eens een script geschreven, maar je zou het bijna niet eens hier neerzetten vanwege sommige negatieve reacties. Als je dan toch zo nodig (negatief) commentaar moet geven, geef dan ook aan wat je er fout aan vindt en hoe het beter kan.
0 -0
-0
Staat dit serieus in een boek? Anyway, check reactie 1. Daar staat allemaal in wat er verbeterd moet worden.
Toch raad ik iedereen aan om swiftmailer te gebruiken om te mailen.
Toch raad ik iedereen aan om swiftmailer te gebruiken om te mailen.
Om te reageren heb je een account nodig en je moet ingelogd zijn.
PHP hulp
0 seconden vanaf nu