check inbouwen in controller

Bedoel je zo ?

<?
<script>
$(document).ready(function(){


$("button").click(function(){
$.ajax({
type:'POST',
url: '/leren',
dataType:'json',
data:{"email": $('#email').val() , "order-hash": $('#order-hash').val(),},
success: function(data){

if ($gegevens['order-hash'] === '12345') {
$gegevens['status'] = true;
}
alert(data["email"] + " " + data["order-hash"]);

else{
alert(data["email"] + " " + "You fucked up");
}
console.log(data);
}});
});

});
</script>
?>

Variabelen in JavaScript zijn zonder dollar-tekens, en in je succes-event hadden we daar als data voor, waar die data in staat.

Verder klopt er weinig van de opbouw.
Is het niet zinniger om eerst eens te kijken hoe if-else precies werkt? Want je kanniet zomaar tussen de accolades iets neer plempen.

Ik snap ook bovendien totaal niet waarom je status opeens op true zet in je ajax()-functie. Je wilt het enkel alleen ophalen en op de juiste manier presenteren!

Zorg dat je weet wat je doet, en waarom het gebeurt.
Met random wat proberen schiet je niks mee op!

Ik ben aan het oefenen vandaar.

De variabelen zijn niet meer in dollar tekens.
Heb mijn code nu zo.

<?
<script>
$(document).ready(function(){


$("button").click(function(){
$.ajax({
type:'POST',
url: '/leren',
dataType:'json',
data:{"email": $('#email').val() , "order-hash": $('#order-hash').val(),},
success: function(data){

if (data['order-hash'] === '12345') {
data['status'] = true;

alert(data["email"] + " " + data["order-hash"]);

else
alert(data["email"] + " " + "You fucked up!");
console.log(data);
}});
});

});
</script>
?>

Je kan wel blijven klooien, maar er klopt nog steeds niks van.
Als ik opeens zomaar dit zie:
- Op lijn 15 opeens status op true zetten, terwijl je enkel dingen wilt ophalen.
- Accolades die ontbreken
- Verkeerde variabelen gebruiken

Dan heb ik echt het idee dat je je met dingen bezighoudt die veel hoofdstukken verder staan, dan waar jij nog geen kennis van hebt.

Begin eerst eens met wat simpels om met een if-else scriptje (zonder AJAX) bijvoorbeeld te bepalen of iemand 18 jaar of ouder is, en zelfstandig mag autorijden. En declareer die leeftijd in een variabele.
Begin dus eerst eens met simpele opdrachtjes i.p.v. je met AJAX bezig te houden.

Ik heb echt het idee dat je lukraak wat probeert zonder te weten wat er gebeurt en wat je doet. Dus doe hier liever eerst wat ervaring mee op in plaats van je op een groter project te richten waar je geen wijs uit wordt vanwege de omvang ervan.

<?php
$leeftijd= 18;
if($leeftijd >= 18) {
echo 'Je bent ouder dan 18 jaar je mag zelfstandig auto rijden.';
} else {
echo 'Je bent jonger dan 18 jaar je mag niet zelfstandig auto rijden';
}
?>

Dat is PHP, maar nu in JavaScript ;-)

<?

<script>
var leeftijd;
var text;
if ("leeftijd >18") {
text = "Je bent ouder dan 18 je mag zelfstandig auto rijden";
} else {
text = "Je bent jonger dan 18 je mag niet zelfstandig auto rijden";
}
</script>
?>

Waarom een string op lijn 6?

ik heb probleem al opgelost.

<?
<script>
$(document).ready(function(){


$("button").click(function(){
$.ajax({
type:'POST',
url: '/leren',
dataType:'json',
data:{"email": $('#email').val() , "order-hash": $('#order-hash').val(),},
success: function(data){
if(data.status){
alert(data["email"] + " " + data["order-hash"]);
}
else{
alert(data["email"] + " " + "You fucked up!");
}
console.log(data);
}});
});

});
</script>

?>

Nu moet ik controller aanpassen van help zodat deze json teruggeeft, en ook weer met een status true of false.

Dit is code van help.ctr.php

<?
if (isset($_POST['email'])){

$order = new Order();
$order->set_query("SELECT orders.* FROM orders INNER JOIN users ON orders.user_id = users.id WHERE email = :email AND order_hash = :order_hash", false);
$order->bind('email', $_POST['email']);
$order->bind('order_hash', $_POST['order-hash']);

if($order->fetch()){

EmailHelper::order_confirmation($order);


echo"Gevonden";
}else{

echo"Niet gevonden";
}
}
?>

Dit is code van leren.ctr.php

<?php
if (isset($_POST['email'])) {
$gegevens = array('email'=> $_POST['email'], 'order-hash'=> $_POST['order-hash'], 'status'=> false,);

if ($gegevens['order-hash'] === '12345') {
$gegevens['status'] = true;
}

echo json_encode($gegevens);
exit;
}
?>

Dit doe je op dezelfde wijze als in leren.ctr.php: je bouwt een array (bijvoorbeeld $gegevens) en stuurt deze naar output via echo json_encode($gegevens);.

Overigens, als je data ontvangt in JSON-formaat, dan kun je de variabelen hierin aanspreken via data.<naam_van_variabele> zoals je ook in het if-statement doet: data.status. Het is dan wel zaak dat deze informatie ook in $gegevens zat en wordt doorgegeven, anders kent JavaScript deze variabelen uiteraard niet. Zo geef je geen e-mail door, deze zit dan ook niet in data.email (dit zou dus data.email moeten zijn, en niet data["email"] waarschijnlijk).

Zorg ervoor dat je begrijpt hoe de JavaScript (client side) communiceert met de bijbehorende PHP-scripts (server side) en zorg er ook voor dat alle informatie die je nodig hebt en wilt gebruiken ook echt wordt doorgegeven anders gaat dit niet werken.