AchrafReyani / educom-webshop-basis

0 stars 0 forks source link

Pas deze functie aan dat in het geval van een POST request, de data niet uit de GET wordt gehaald maar uit de POST data #20

Open JeroenHeemskerk opened 7 months ago

JeroenHeemskerk commented 7 months ago

De opdracht was om GEEN urlparameter te gebruiken:

Dit zie ik niet terug in de code hieronder https://github.com/AchrafReyani/educom-webshop-basis/blob/c0d27255493e0c696902ee526f99974169c62981/index.php#L36-L43

Sterker nog het hele formulier https://github.com/AchrafReyani/educom-webshop-basis/blob/c0d27255493e0c696902ee526f99974169c62981/contact.php#L68-L82 gaat niet door de 'single-door' van index.php maar is een eigen file - ook dat was niet de bedoeling van de opdracht.

Deze code zou iets moeten zijn als:

  function showContactForm($data) {
      echo '<form action="index.php" method="POST">'; 
      echo '  <div>'
      echo '    <label for="pronouns">Pronouns:</label>';
      echo '    <select name="pronouns" value="' . $data['pronouns'] . '"> 
                  <option value="">Please select a pronoun</option> 
                  <option value="he/him">He/him</option> 
                  <option value="she/her">She/her</option> 
                  <option value="they/them">They/them</option> 
                  <option value="other">Other</option> 
                  <option value="prefer not to say">Prefer not to say</option> 
                </select>;'
      echo '    <span>* ' . $data['pronounError']. '</span>'; 
      echo '  </div>;'

      /* ... */
  }