Dabra720 / educom-webshop-database

0 stars 0 forks source link

Maak gebruik van de `getPostVar('field')` functie in plaats van `$_POST['field']` #11

Open JeroenHeemskerk opened 1 year ago

JeroenHeemskerk commented 1 year ago

Probeer het gebruik van de super-global $_POST te beperken, je zou gebruik kunnen maken van de getPostVar functie https://github.com/Dabra720/educom-webshop-database/blob/646063c414cf0f50b2ad79098f3c95709f78cc53/validation.php#L70-L78 zou je dan kunnen opschrijven als:

function validateField($array, $value, $check){ 
   $array['values'][$value] = test_input(getPostVar($value)); 
   switch($check){ 
     case 'isEmpty': 
       if(empty($array['values'][$value])){ 
         $array['errors'][$value] = $value . " is required"; 
       } 
       break; 

Voordeel is dat nu als het veld afwezig is dat er dan een lege string aan de value wordt toegekend, maar ook als het veld alleen maar uit spaties bestaat wordt het nu ook als leeg gemarkeerd.