[php] Probleem met webshop

[Verwijderd]

Beste tweakers....

Ik ben nog niet zo goed in PHP, dus heb ik een webshop script van het net gehaald (Gratis). Nu heb ik deze geinstalleerd en het werkt allemaal prima. Toch mistte ik een onderdeel: Hoe de betaling zal geschieden.

Ik heb dus samen met een vriend een onderdeel toegevoegd, en het werkt gedeeltelijk.

Het is dus nu zo: klant zoekt product op, klikt op bestellen, voert NAW gegevens in en krijgt een pagina met het ingevulde. Hier staat dus ook de betalingsmethode bij.

Ik krijg vervolgens een mail met dezelfde tekst als de gebruiker ziet, maar ik zie de betalingsmethode niet!

Als jullie de shop zelf willen zien: http://mencomshop.mencom.nl

code:

<?
include("header.inc");
$_SESSION['winkelwagen']['activate'] = 0;

function valid_mail($email){
$email_host = explode("@", $email); 
$email_host = $email_host[1]; 
$email_resolved = gethostbyname($email_host); 
if ($email_resolved == $email_host) {
return false;
}
else{
return true;
}
}
function showcart()
{
//Code om de winkelwagen te laten zien
$totaal = "";
foreach($_SESSION['winkelwagen'] as $prodID => $at){
if (file_exists("WSFILES/" . $prodID . "_n")){
$totaal = $totaal + $at;
}
}
echo "<h1>Winkelwagen</h1>";
echo '<a href="index.php">[img]"pr.bmp"[/img]</a>';
if ($totaal == "1"){
$hv = "artikel";
}
else{
$hv = "artikelen";
}
echo "<p>U heeft op dit moment " . $totaal . " " . $hv . " in uw winkelwagen.";
echo "<br><br>";
if (!$totaal == "0"){
//Laat de tabel met artikelen zien...
?>
<table border="0" width="100%">
<tr bgcolor="#6487DC">
<td valign="top"><p><b>Procuct</b></p>
<td valign="top" colspan="2"><p><b>Prijs per stuk</b></p>
<td valign="top"><p><b>Aantal</b></p>
<td valign="top" colspan="2"><p><b>Bedrag</b></p></tr>
<?
$prijs = "";
foreach($_SESSION['winkelwagen'] as $prodID => $aantal){
if((!$aantal == "" || !$aantal == "0") && file_exists("WSFILES/" . $prodID . "_n")){
$file = fopen("WSFILES/" . $prodID . "_n", "r");
$q = "";
while (!feof($file)){
$q .= fgets($file, 1024);
}
fclose($file);
echo "<form name=bestelform action=cart.php method=GET>";
echo "<input type=hidden name=q value=add>";
echo "<input type=hidden name=PHPSESSID value=" . session_id() . ">";
echo "<input type=hidden name=prodID value=" . $prodID . ">";
echo "<tr><td valign=top><p>" . $q . "</p>";

$file = fopen("WSFILES/" . $prodID . "_p", "r");
$q = "";
while (!feof($file)){
$q .= fgets($file, 1024);
}
fclose($file);
$prijs = $prijs + ($aantal * $q);
echo "<td valign=top><p>&euro;<td valign=top>";
echo "<div align=right><p>" . number_format($q, 2, ",", ".") . "&nbsp;&nbsp;&nbsp;</p></div>";
echo "<td valign=top><input type=text size=3 maxlength=2 name=aantal value=" . $aantal . ">";
echo "&nbsp;&nbsp;<input type=submit value=Bijwerken>";
echo "&nbsp;&nbsp;&nbsp;";
echo "<a href=cart.php?PHPSESSID=" . session_id() . "&q=del&prodID=" . $prodID . ">";
echo "[img]verwijderen.bmp[/img]</a>";
echo "<td valign=top><p>&euro;<td valign=top>";
echo "<div align=right><p>" . number_format(($aantal * $q), 2, ",", ".");
echo "</p></div></p></td></tr></form>";
}
}
echo "<tr><td colspan=4 valign=top><p>Verzendkosten binnen Nederland</p>";
echo "<td valign=top><p>&euro;</p><td valign=top><div align=right>";
$file = fopen("config.php", "r");
$string = fgets($file, 1024);
list($q, $webshopnaam, $mail, $verzend, $gebruiker, $wachtwoord, $q) = split(",", $string);
fclose($file);
echo "<p>" . number_format($verzend, 2, ",", ".") . "</p></div></td></tr>";
$prijs = $prijs + $verzend;
echo "<tr><td valign=top><p><b>Totaal</b></p>";
echo "<td valign=top>";
echo "<td valign=top>";
echo "<td valign=top>";
echo "<td valign=top><p><b>&euro;</b></p><td valign=top><div align=right>";
echo "<p><b>" . number_format($prijs, 2, ",", ".") . "</b></p></div></td></tr>";
echo "</table>";
?><br>

<table border="0"><tr><td>
<a href="cart.php?PHPSESSID=<? echo session_id(); ?>&q=bfr">
[img]"bes.bmp"[/img]</a><td>
<a href="cart.php?PHPSESSID=<? echo session_id(); ?>&q=bfr">Bestelling afhandelen</a>
<td>

<a href="cart.php?PHPSESSID=<? echo session_id(); ?>&q=emp">
[img]"emp.bmp"[/img]</a><td>
<a href="cart.php?PHPSESSID=<? echo session_id(); ?>&q=emp">Winkelwagen leegmaken</a>
<td>

<a href="javascript:window.print()">
[img]"printer.bmp"[/img]</a><td>
<a href="javascript:window.print()">Pagina afdrukken</a>
</td></tr></table>
<?
}}

if($_GET['q'] == "add"){
//Product toevoegen óf wijzigen...
if(!file_exists("WSFILES/" . $_GET['prodID'] . "_n")){
echo "<h1>Winkelwagen</h1>";
echo "<p class=w>Er is een fout opgetreden, er bestaat geen product met artikelnummer ";
echo $_GET['prodID'];
echo ".</p>";
}
else{
$_SESSION['winkelwagen'][$_GET['prodID']] = $_GET['aantal'];
showcart();
}
}

if($_GET['q'] == "del"){
$_SESSION['winkelwagen'][$_GET['prodID']] = 0;
showcart();
}

if($_GET['q'] == "emp"){
//Winkelwagen leegmaken...
foreach($_SESSION['winkelwagen'] as $prod => $aantal){ 
$_SESSION['winkelwagen'][$prod] = ""; 
}
showcart();
}

if($_GET['q'] == "shw"){
showcart();
}

if($_GET['q'] == "bfr"){
echo "<h1>Bestelling afhandelen</h1>";
echo '<a href="cart.php?PHPSESSID=' . session_id() . '&q=shw">';
echo '[img]"pr.bmp"[/img]</a><br>';
?>
<form name="bestelform" action="cart.php" method="GET">
<input type="hidden" name="PHPSESSID" value="<? echo session_id(); ?>">
<input type="hidden" name="q" value="cfr">
Uw naam:&nbsp;&nbsp;<br>
<input type="text" name="naam">
<br>
Uw e-mailadres:&nbsp;&nbsp;<br>
<input type="text" name="mail">
<br>
Uw adres:&nbsp;&nbsp;<br>
<input type="text" name="adres">
<br>
Uw postcode:&nbsp;&nbsp;<br>
<input type="text" name="post">
<br>
Uw woonplaats:&nbsp;&nbsp;<br>
<input type="text" name="wnpl">

<br><br>Betaling: 
<SELECT NAME="betalingmeth">
<option VALUE=a SELECTED>Op rekening vooruit</option>
<option VALUE=b>Contant vooruit</option>
<option VALUE=c>Rembours</option>
<option VALUE=d>Op afspraak</option>
</SELECT>

<br><br><input type="submit" value="Bestelling controleren">
</form>
<script>
document.bestelform.naam.focus();
</script>
<?
}

if($_GET['q'] == "cfr"){
echo "<h1>Bestelling controleren</h1>";
echo '<a href="cart.php?PHPSESSID=' . session_id() . '&q=shw">';
echo '[img]"pr.bmp"[/img]</a>';
if (!$_GET['naam'] || !valid_mail($_GET['mail']) || !$_GET['adres'] || !$_GET['post'] || !$_GET['wnpl']){
//Formulier fout ingevuld!
echo "<p class=w>Het formuler is niet correct ingevuld!</p>";
}else{
//Formulier goed ingevuld!
echo "<p>Controleer de onderstaande gegevens, zijn deze correct? Klik dan op 'Bestellen'.</p>";
echo "<p>";
echo "<b>Uw naam: </b>" . $_GET['naam'] . "<br>";
echo "<b>Uw e-mailadres: </b>" . $_GET['mail'] . "<br>";
echo "<b>Uw adres: </b>" . $_GET['adres'] . "<br>";
echo "<b>Uw postcode: </b>" . $_GET['post'] . "<br>";
echo "<b>Uw woonplaats: </b>" . $_GET['wnpl'] . "<br><br>";
echo "<b>Uw bestelling:</b><br>";

$selected_state = $_GET['betalingmeth'];

if($selected_state=="a") {  
   echo "<br>Betaling: Op rekening vooruit"; 
}
if($selected_state=="b") {  
   echo "<br>Betaling: Contant vooruit"; 
}
if($selected_state=="c") {  
   echo "<br>Betaling: Rembours"; 
}
if($selected_state=="d") {  
   echo "<br>Betaling: Op afspraak"; 
}

echo "<br>";

$totaal = "";
foreach($_SESSION['winkelwagen'] as $prodID => $aantal){
if (!$aantal == "0"){
$file = fopen("WSFILES/" . $prodID . "_n", "r");
$q = "";
while (!feof($file)){
$q .= fgets($file, 1024);
}
fclose($file);
$file = fopen("WSFILES/" . $prodID . "_p", "r");
$p = "";
while (!feof($file)){
$p .= fgets($file, 1024);
}
fclose($file);
$p = $aantal * $p;
$totaal = $totaal + $p;
echo $aantal . " x " . $q . " = &euro; " . number_format($p, 2, ",", ".") . "<br>";
}}

echo "Verzendkosten = ";
echo "&euro; ";
$file = fopen("config.php", "r");
$string = fgets($file, 1024);
list($q, $webshopnaam, $mail, $verzend, $gebruiker, $wachtwoord, $q) = split(",", $string);
fclose($file);
echo number_format($verzend, 2, ",", ".") . "<br>";
$totaal = $totaal + $verzend;
echo "<b>Totaal: &euro; " . number_format($totaal, 2, ",", ".") . "</b>";
echo "<form name=f action=cart.php?q=abe&PHPSESSID=" . session_id() . " method=GET>";
echo '<input type="hidden" name="PHPSESSID" value="' . session_id() . '">';
echo '<input type="hidden" name="q" value="abe">';
echo '<input type="hidden" name="naam" value="' . $_GET['naam'] . '">';
echo '<input type="hidden" name="mail" value="' . $_GET['mail'] . '">';
echo '<input type="hidden" name="adres" value="' . $_GET['adres'] . '">';
echo '<input type="hidden" name="post" value="' . $_GET['post'] . '">';
echo '<input type="hidden" name="wnpl" value="' . $_GET['wnpl'] . '">';
echo "<input type=submit value=Bestellen>";
echo "</form>";
}
}

if($_GET['q'] == "abe"){
$bericht = "";
$bericht .= "Naam:\n" . $_GET['naam'] . "\n";
$bericht .= "E-mailadres:\n" . $_GET['mail'] . "\n";
$bericht .= "Adres:\n" . $_GET['adres'] . "\n";
$bericht .= "Postcode:\n" . $_GET['post'] . "\n";
$bericht .= "Woonplaats:\n" . $_GET['wnpl'] . "\n\n";
$bericht .= "Bestelling:\n\n";
foreach($_SESSION['winkelwagen'] as $prodID => $aantal){
if (!$aantal == "0"){
$file = fopen("WSFILES/" . $prodID . "_n", "r");
$q = "";
while (!feof($file)){
$q .= fgets($file, 1024);
}
fclose($file);
$file = fopen("WSFILES/" . $prodID . "_p", "r");
$p = "";
while (!feof($file)){
$p .= fgets($file, 1024);
}
fclose($file);
$p = $aantal * $p;
$totaal = $totaal + $p;
$bericht .= $aantal . " x " . $q . " = € " . number_format($p, 2, ",", ".") . "\n";
}}

$bericht .= "Verzendkosten = ";
$bericht .= "€ ";
$file = fopen("config.php", "r");
$string = fgets($file, 1024);
list($q, $webshopnaam, $mail, $verzend, $gebruiker, $wachtwoord, $q) = split(",", $string);
fclose($file);
$bericht .= number_format($verzend, 2, ",", ".");

if($selected_state=="a") {  
   $bericht .= "\n\nBetaling: Op rekening vooruit"; 
}
if($selected_state=="b") {  
   $bericht .= "\n\nBetaling: Contant vooruit"; 
}
if($selected_state=="c") {  
   $bericht .= "\n\nBetaling: Rembours"; 
}
if($selected_state=="d") {  
   $bericht .= "\n\nBetaling: Op afspraak"; 
}

$totaal = $totaal + $verzend;
$bericht .= "\n\nTotaal: € " . number_format($totaal, 2, ",", ".");

if (!mail($_GET['mail'], "Uw bestelling", "De volgende bestelling is ingevoerd bij " . $webshopnaam . "\n\n" . $bericht) || !mail($mail, "Bestelling van " . $webshopnaam, "De volgende bestelling is ingevoerd bij " . $webshopnaam . "\n\n" . $bericht)){
//Bestelling niet verstuurd...
echo "<h1>Bestelling afhandelen</h1>";
echo '<a href="cart.php?PHPSESSID=' . session_id() . '&q=shw">';
echo '[img]"pr.bmp"[/img]</a>';
echo "<p class=w>De bestelling kon niet worden verzonden!</p>";
}
else{
foreach($_SESSION['winkelwagen'] as $x => $q){
$_SESSION['winkelwagen'][$x] = "0";
}
echo "<h1>Bestelling afhandelen</h1>";
echo '<a href="cart.php?PHPSESSID=' . session_id() . '&q=shw">';
echo '[img]"pr.bmp"[/img]</a>';
echo "<p>Uw bestelling is met succes verstuurd!";
echo "<br><br>Uw winkelwagen is automatisch leeg gemaakt!</p>";
}
}

include("footer.inc");
?>

En hier de code.

Ziet een van jullie het probleem??

[ Voor 31% gewijzigd door Verwijderd op 11-10-2005 13:20 ]

[sjroorda]

Kan je iets geselecteerder code posten? Hier is echt geen wijs uit te worden... Ook indenting is geen overbodige luxe. Wat heb je al geprobeerd aan standaard debugging? print_r($_GET) en zo...

[TeeDee]

bwaap... ik reageer nooit meer in php topics...

Sorry PhoeniX-: zal het niet meer doen.
Op welk gedeelte van mijn post doelde je? Het stukje over het $_GET gedeelte?

[ Voor 138% gewijzigd door TeeDee op 11-10-2005 13:33 ]

[djexplo]

Met kant en klare script's helpen wij niet op GoT zie FAQ...
Als je nu alleen de relevante code laat zien, is het ook veel makkelijker om te helpen dan bij 332 regels tekst.
Het best kun je stap voor stap het probleem oplossen, Voeg extra echo variables toe, geef zo de verschillende variablen op het scherrm weer die voor de e-mail worden gebruikt. Dan kan je stap voor stap het probleem opzoeken...

[CodeCaster]

soort van offtopic & NOFI
Ik heb vorige week mijn eerste php script (echo 'hello world';) geschreven, maar zou jij daar niet mee beginnen?

Als ik bijvoorbeeld dit zie:

PHP:

if($selected_state=="a") {  
   $bericht .= "\n\nBetaling: Op rekening vooruit"; 
}
if($selected_state=="b") {  
   $bericht .= "\n\nBetaling: Contant vooruit"; 
}
if($selected_state=="c") {  
   $bericht .= "\n\nBetaling: Rembours"; 
}
if($selected_state=="d") {  
   $bericht .= "\n\nBetaling: Op afspraak"; 
}

Kan dat niet met een switch?

Ik zal even proberen de rest van je code te analyseren...

[ Voor 12% gewijzigd door CodeCaster op 11-10-2005 13:28 ]

[PhoeniX-]

Wat een mess die code, brr.

Verder dan wat TeeDee zegt kom ik inderdaad ook niet.

edit:

Ik bedoel: Verder dan wat TeeDee voor zijn edit zei kom ik ook niet.

[ Voor 35% gewijzigd door PhoeniX- op 11-10-2005 13:29 ]

[Verwijderd]

djexplo schreef op dinsdag 11 oktober 2005 @ 13:27:
Met kant en klare script's helpen wij niet op GoT zie FAQ...
Als je nu alleen de relevante code laat zien, is het ook veel makkelijker om te helpen dan bij 332 regels tekst.
Het best kun je stap voor stap het probleem oplossen, Voeg extra echo variables toe, geef zo de verschillende variablen op het scherrm weer die voor de e-mail worden gebruikt. Dan kan je stap voor stap het probleem opzoeken...

Waarom bestaat dit forum eigenlijk nog?? Bij alles wat ik open krijg ik gezeik van modder eters: Met dit en dit helpen we je niet, zoek eerst zelf maar! OMFG! Alsof ik dat niet doe!

Laat maar, de anderen, bedankt voor de reactie

(Slote, delete account: bpm)

[CodeCaster]

Ok, ik heb even gekeken en bij regel 239-257 laat je de gebruiker zijn invoer controleren, terwijl je de eerder ingevoerde waardes allemaal in hidden input-controls zet.

Allemaal, behalve de betalingswijze.

Hopelijk kun je hiermee verder?

[TwoR]

Verwijderd schreef op dinsdag 11 oktober 2005 @ 13:34:
[...]


Waarom bestaat dit forum eigenlijk nog?? Bij alles wat ik open krijg ik gezeik van modder eters: Met dit en dit helpen we je niet, zoek eerst zelf maar! OMFG! Alsof ik dat niet doe!

Laat maar, de anderen, bedankt voor de reactie

(Slote, delete account: bpm)

Dit forum is voor scripts die je zelf gemaakt hebt niet voor het betere jat werk. En als je zoals jij het zegt dit zelf gemaakt hebt hoef je toch geen 322 regels code hierneer te plempen? Je weet dan toch zelf ongeveer waar het fout.

offtopic:
Lang niet al de door jou geopende topics zijn gesloten hoor

[ Voor 28% gewijzigd door TwoR op 11-10-2005 13:41 ]

[TeeDee]

Verwijderd schreef op dinsdag 11 oktober 2005 @ 13:34:
[...]


Waarom bestaat dit forum eigenlijk nog?? Bij alles wat ik open krijg ik gezeik van modder eters: Met dit en dit helpen we je niet, zoek eerst zelf maar! OMFG! Alsof ik dat niet doe!

Laat maar, de anderen, bedankt voor de reactie

(Slote, delete account: bpm)

Bij het eerste gedeelte van djexplo's reactie is er nog ruimte voor een discussie (aanpassing op een stuk script), maar de rest van zijn reactie is wel degelijk behulpzaam.

Debuggen enzo. PW heeft een prachtige FAQ waar een hoop tips/tricks in staan. Neem dat eens door voor de grap, voordat je hier een beetje overdreven loopt te bwaapen dat je account verwijderd kan worden.

[Creepy]

Verwijderd schreef op dinsdag 11 oktober 2005 @ 13:34:
[...]
Waarom bestaat dit forum eigenlijk nog?? Bij alles wat ik open krijg ik gezeik van modder eters: Met dit en dit helpen we je niet, zoek eerst zelf maar! OMFG! Alsof ik dat niet doe!

Het zou ontzettend schelen als je P&W FAQ - De "quickstart" eens goed door zou lezen. Je geeft namelijk in je startpost helemaal niet de zaken aan die je nu zelf al hebt geprobeerd. Daarnaast post je een gigantische lap code. Als je zelf al bezig bent geweest om het probleem op te sporen dan heb je vaak al een beter idee van waar de fout in de code kan zitten zodat je in staat bent om alleen de echte relevante code te posten.

Nu lijkt het erop dat je je code dumpt met de vraag "wie lost het voor me op". En zo werkt het hier niet. Het is een discussie forum en geen helpdesk waar je ff snel een antwoord voor je probleem kan vinden. Het magische woord hier is "inzet". Dit lijkt bij jou te missen. Met behulp van de al eerder genoemde quickstart kan je gewoon laten zien dat je wel degelijk zelf bent bezig geweest en ook echt een idee hebt waar de fout zou kunnen in zitten. In dat geval zijn er genoeg personen die je gewoon willen helpen zonder allerlei extra commentaar te geven.

Voor diegenen die offtopic commentaar hebben gepost: Zie FAQ, reageren. Doe dat de volgende keer gewoon via een TR en laat het topic verder links liggen als je toch niks nuttigs toe te voegen hebt. Thanx

[ Voor 14% gewijzigd door Creepy op 11-10-2005 13:47 ]

[Janoz]

Verwijderd schreef op dinsdag 11 oktober 2005 @ 13:34:
Waarom bestaat dit forum eigenlijk nog?? Bij alles wat ik open krijg ik gezeik van modder eters: Met dit en dit helpen we je niet, zoek eerst zelf maar! OMFG! Alsof ik dat niet doe!

Laat maar, de anderen, bedankt voor de reactie

(Slote, delete account: bpm)

Dit forum bestaat nog omdat het niet afgezakt is tot een verbasterd 'dump hier je project en een set nerds bouwen de door jou gewenste functionaliteit wel in'. Dat hebben we kunnen voorkomen om topics zoals deze waarin enkel een wens met vervolgens een enorme lap code staat niet toe te staan.

Ik raad je aan om, mocht je nog eens een topic willen openen, voor de grap gewoon eens het beleid van Programming & Webscripting door te nemen.