AchrafReyani / educom-webshop-basis

0 stars 0 forks source link

Kan je `showFormField` een TYPE meegeven en dan op basis daarvan deze drie functies integreren #26

Open JeroenHeemskerk opened 7 months ago

JeroenHeemskerk commented 7 months ago

Onderstaande drie functies https://github.com/AchrafReyani/educom-webshop-basis/blob/13b8a9813927a2c263c279a1dc64feaf71d15ef7/contact.php#L153-L203 zou je ook in 1 functie kunnen stoppen als je twee extra zaken meegeeft: een $type en een lijst met $options

function showFormField($fieldName, $label, $type, $data, $options = NULL) { 

     echo " 
     <div> 
     <label for=\"$fieldName\">$label:</label>";
     switch($type) {
        default: 
           echo "<input type=\"$type\" name=\"$fieldName\" value=\"". $data[$fieldName]."\">";
           break;

        case "select":
           echo "<select name=\"$fieldName\">";
           foreach($options as $key => $value) {
              echo "<option value=\"$key\">$value</option>";
           }
           break;

        case "radio": 
           // ...
           break;

        case "textarea":
           // ...
           break;
     } 
     echo "<span>* " . $data[$fieldName . "Error"]  . "</span> 
     </div>";
  } 
JeroenHeemskerk commented 7 months ago

Deze code is een duplicaat van bovenstaande functies https://github.com/AchrafReyani/educom-webshop-basis/blob/13b8a9813927a2c263c279a1dc64feaf71d15ef7/login.php#L3-L26 Zet bovenstaande functies in een eigen php bestand en include deze in contact.php, register.php en login.php

JeroenHeemskerk commented 6 months ago

Idem for changePassword https://github.com/AchrafReyani/educom-webshop-database/blob/1fc395f04b97733702f4f371c5af27e6d8e323f4/changepassword.php#L72-L95