From 8a738be5834b2bfc42c2a51166cf348a180b9cfe Mon Sep 17 00:00:00 2001
From: Eliah REBSTOCK <eliah.rebstock@ensiie.fr>
Date: Sat, 7 May 2016 18:50:24 +0200
Subject: [PATCH] ajout de * et correction de message d'erreurs

---
 app/controller/JeuController.php | 11 +++++++----
 app/view/Jeu/displayFormAdd.html | 10 +++++-----
 2 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/app/controller/JeuController.php b/app/controller/JeuController.php
index a4e7e5e..7566109 100644
--- a/app/controller/JeuController.php
+++ b/app/controller/JeuController.php
@@ -90,7 +90,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,7 +103,7 @@ 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']))
+                    if(!is_numeric($data['max_joueurs']))
                         $_SESSION["addRequestErrCode"] = 8;
                     else
                         $_SESSION["addRequestErrCode"] = 9; // on stocke le code d'erreur
@@ -126,7 +126,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 +137,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"] = 8;
+						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/view/Jeu/displayFormAdd.html b/app/view/Jeu/displayFormAdd.html
index f40d5ca..9b704e7 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">
-- 
GitLab