Dabra720 / educom-webshop-database

0 stars 0 forks source link

De check 'rep_password" is hardcoded aan het veld 'password' maak dit flexibeler #12

Open JeroenHeemskerk opened 1 year ago

JeroenHeemskerk commented 1 year ago

https://github.com/Dabra720/educom-webshop-database/blob/646063c414cf0f50b2ad79098f3c95709f78cc53/validation.php#L101-L114 Het veld wat wordt gecontroleerd wordt altijd gecontroleerd tegen het veld password, dit kan je flexibeler maken door bijv in de naam van de check ook het veld te zetten, bijv achter een dubbele punt: $data = validateField($data, 'pass_rep', 'pass_rep:password');

function validateField($array, $value, $check){
  checkFields = explode(":", $check);
  switch($checkFields[0]){
  /* ... */
  case 'pass_rep': 
   if(empty($_POST[$value])){ 
     $array['errors'][$value] = "Repeat the password"; 
   } else { 
     if(!strcmp($_POST[$value], $array['values'][$checkFields[1]])){ 
       $array['values'][$value] = $_POST[$value]; 
     } else{ 
       $array['errors'][$value] = "Passwords don't match"; 
       $array['errors'][$checkFields[1]] = "Passwords don't match"; 
       $array['values'][$value] = ""; 
       $array['values'][$checkFields[1]] = ""; 
     } 
   } 
   break;