diff --git a/app/controller/JeuController.php b/app/controller/JeuController.php index a4e7e5eed04e9505ed2113f88f4ff9252099def2..e3ceeea5afd591244fc43291649f4784461b6219 100644 --- a/app/controller/JeuController.php +++ b/app/controller/JeuController.php @@ -51,6 +51,9 @@ class JeuController extends Controller { } elseif ($errCode == 9) { $errMsg = "<h3>Les valeurs pour les joueurs et/ou les équipes sont incohérentes.</h3>"; } + elseif ($errCode == 10) { + $errMsg = "<h3>Les joueurs/équipes max ne sont pas remplis.</h3><p><em>Si vous ne voulez pas les remplir, cochez la case infini, ou changez de type de jeu.</em></p>"; + } else{ $errMsg = "<h3>Exception non gérée.</h3>"; } @@ -90,7 +93,7 @@ class JeuController extends Controller { die(); } - if(!is_numeric($data['max_joueurs'])) { + if(!is_numeric($data['min_joueurs'])) { $_SESSION["addRequestErrCode"] = 8; // on stocke le code d'erreur header('Location:../addjeu'); // on redirige vers la page de formulaire (il faut faire un retour dans le dossier parent pour que ça fonctionne) die(); @@ -103,8 +106,8 @@ class JeuController extends Controller { } if(!isset($data['max_joueurs_inf']) && ($data['max_joueurs'] < 0 || $data['max_joueurs'] < $data['min_joueurs'])) { - if(!is_numeric($data['min_joueurs'])) - $_SESSION["addRequestErrCode"] = 8; + if(!is_numeric($data['max_joueurs'])) + $_SESSION["addRequestErrCode"] = 10; else $_SESSION["addRequestErrCode"] = 9; // on stocke le code d'erreur header('Location:../addjeu'); // on redirige vers la page de formulaire (il faut faire un retour dans le dossier parent pour que ça fonctionne) @@ -126,7 +129,7 @@ class JeuController extends Controller { echo "<h1>Erreur : variables absentes</h1>"; die(); } - if(!is_numeric($data['max_equipes']) || !is_numeric($data['min_equipes'])) { + if(!is_numeric($data['min_equipes'])) { $_SESSION["addRequestErrCode"] = 8; // on stocke le code d'erreur header('Location:../addjeu'); // on redirige vers la page de formulaire (il faut faire un retour dans le dossier parent pour que ça fonctionne) die(); @@ -137,7 +140,10 @@ class JeuController extends Controller { die(); } if(!isset($data['max_equipes_inf']) && ($data['max_equipes'] < 0 || $data['max_equipes'] < $data['min_equipes'])) { - $_SESSION["addRequestErrCode"] = 9; // on stocke le code d'erreur + if(!is_numeric($data['max_equipes'])) + $_SESSION["addRequestErrCode"] = 10; + else + $_SESSION["addRequestErrCode"] = 9; // on stocke le code d'erreur header('Location:../addjeu'); // on redirige vers la page de formulaire (il faut faire un retour dans le dossier parent pour que ça fonctionne) die(); } diff --git a/app/model/Jeu.php b/app/model/Jeu.php index 672342e38d2dd9899d2572ca13682bbf4175d36b..743c804838190916c709bba7a189e11cb65ac41b 100644 --- a/app/model/Jeu.php +++ b/app/model/Jeu.php @@ -79,16 +79,6 @@ class Jeu extends Model { $db = Database::getInstance(); $sql = "INSERT INTO R_jeu (nom, description, slug, est_valide, min_joueurs, max_joueurs, min_equipes, max_equipes, type, type_scores) VALUES (:nom, :description, :slug, :est_valide, :min_joueurs, :max_joueurs, :min_equipes, :max_equipes, :type, :type_scores);"; - print($nom_jeu); - print($description); - print($slug_jeu); - print($est_valide); - print($type); - print($type_scores); - print($min_joueurs); - print($max_joueurs); - print($min_equipes); - print($max_equipes); $stmt = $db->prepare($sql); if($stmt->execute(array(":nom" => $nom_jeu, ":description" => $description, ":slug" => $slug_jeu, ":est_valide" => $est_valide, ":type" => $type, ":type_scores" => $type_scores, ":min_joueurs" => $min_joueurs, ":max_joueurs" => $max_joueurs, ":min_equipes" => $min_equipes, ":max_equipes" => $max_equipes))) { //ok diff --git a/app/view/Jeu/displayFormAdd.html b/app/view/Jeu/displayFormAdd.html index f40d5ca9eec09c8c450d42b5fc7ded51c4795fc5..9b704e76bdea692dfa9dc5c7747fd9e5137fbf09 100644 --- a/app/view/Jeu/displayFormAdd.html +++ b/app/view/Jeu/displayFormAdd.html @@ -16,16 +16,16 @@ <h2>Formulaire de demande d'ajout de jeu</h2> <div class="addjeu-form"> <form method="post" action="addjeu/addrequest" enctype="multipart/form-data"> - <label for="nom">Nom du jeu :</label> <input type="text" name="nom" required="" id="nom"/> <br/> + <label for="nom">Nom du jeu * :</label> <input type="text" name="nom" required="" id="nom"/> <br/> <label for="description">Description :</label> <textarea name="description" id="description"></textarea> <br/> - <label for="type">Type de jeu :</label> + <label for="type">Type de jeu * :</label> <select name="type" required="" id="type"> <option>Équipes</option> <option>Chacun pour soi</option> </select><br/> - <label for="type_scores">Type du résultat :</label> + <label for="type_scores">Type du résultat * :</label> <select name="type_scores" required="" id="type_scores"> <option>Scores</option> <option>Classement</option> @@ -33,11 +33,11 @@ <label for="max_joueurs">Joueurs max : </label><input type="text" name="max_joueurs" id="max_joueurs"/><br/> <label for="max_joueurs_inf">Joueurs max infini ?</label><input type="checkbox" name="max_joueurs_inf" id="max_joueurs_inf" value="1"> - <label for="min_joueurs">Joueurs min : </label><input type="text" name="min_joueurs" required="" id="min_joueurs"/><br/> + <label for="min_joueurs">Joueurs min * : </label><input type="text" name="min_joueurs" required="" id="min_joueurs"/><br/> <label for="min_equipes">Equipes max : </label><input type="text" name="min_equipes" id="min_equipes"/><br/> <label for="max_equipes">Equipes min : </label><input type="text" name="max_equipes" id="max_equipes"/><br/> <label for="max_equipes_inf">Equipes max infini ?</label><input type="checkbox" name="max_equipes_inf" id="max_equipes_inf" value="1"> - <label for="image">Image du jeu: </label> <input type="file" required="" name="image" id="image"/><br/> + <label for="image">Image du jeu * (max 200x200px) : </label> <input type="file" required="" name="image" id="image"/><br/> <div class="submit"> <input type="reset" name="reset" value="Réinitialiser"> <input type="submit" name="submit" value="Demander l'ajout">