From c61a99456e84a61938696108fc96f03fc7a2aa13 Mon Sep 17 00:00:00 2001
From: vbochet <vbochet@gmail.com>
Date: Sat, 7 May 2016 22:22:18 +0200
Subject: [PATCH] =?UTF-8?q?D=C3=A9placement=20des=20messages=20d'erreur=20?=
 =?UTF-8?q?du=20formulaire=20d'=C3=A9dition=20de=20profil=20dans=20le=20co?=
 =?UTF-8?q?ntr=C3=B4leur?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/controller/ProfilController.php | 33 +++++++++++++++++++++++++++++
 app/view/Profil/displayEdit.html    | 26 ++++++-----------------
 2 files changed, 40 insertions(+), 19 deletions(-)

diff --git a/app/controller/ProfilController.php b/app/controller/ProfilController.php
index 57b635c..4208b9d 100644
--- a/app/controller/ProfilController.php
+++ b/app/controller/ProfilController.php
@@ -18,6 +18,39 @@ class ProfilController extends Controller { // classe de contrôleur pour l'affi
 		$slug = $this->route["params"]["slug"]; // on récupère le "slug", en gros tout le reste de l'URL qui n'a pas encore servi
 		// je précise que dans ce cas précis, le $slug ne contient que le pseudo du joueur sosu forme normalisée. Donc je pourrais renommer la variable $slug_joueur...
 		$this->view->profil = Profil::getFromSlug($slug); // on utilise le modèle pour aller chercher en BDD le profil qui nous intéresse
+		
+		session_start();
+		
+		
+		if(isset($_SESSION['registerErrCode'])) { // si message d'erreur
+			$errCode = $_SESSION['registerErrCode'];
+			if($errCode == 2) { // on détermine le message
+				$errMsg = '<h3>Pseudo déjà utilisé !</h3>';
+			} 
+			elseif($errCode == 3) {
+				$errMsg = '<h3>Adresse e-mail déjà utilisée !</h3>';
+			} 
+			elseif($errCode == 4) {
+				$errMsg = '<h3>Mots de passe différents</h3>';
+			} 
+			elseif($errCode == 5) {
+				$errMsg = '<h3>Problème sur l\'avatar</h3>';
+			} 
+			elseif($errCode == 6) {
+				$errMsg = '<h3>Le fichier n\'est pas une image</h3>';
+			} 
+			elseif($errCode == 7) {
+				$errMsg = '<h3>L\'image est trop grande</h3>';
+			}
+            else{
+                $errMsg = "<h3>Exception non gérée.</h3>";
+            }
+			
+			unset($_SESSION['registerErrCode']); // on retire la variable de session
+			
+            $this->view->errMsg = $errMsg;
+		}
+		
 		$this->view->display(); // on affiche ce profil
 	}
 
diff --git a/app/view/Profil/displayEdit.html b/app/view/Profil/displayEdit.html
index 7c6cfe2..b4b801b 100644
--- a/app/view/Profil/displayEdit.html
+++ b/app/view/Profil/displayEdit.html
@@ -1,30 +1,18 @@
 <?php
-	session_start();
+	// session déjà lancée dans le contrôleur
 
 	$page_title = "Édition du profil de ".$this->profil->pseudo;
 	include(ROOT."/app/view/Includes/header.include.html");
 ?>
 
 <?php
-		if(isset($_SESSION['registerErrCode'])) { // si message d'erreur
-			$errCode = $_SESSION['registerErrCode'];
-			if($errCode == 2) { // on détermine le message
-				$errMsg = '<h3>Pseudo déjà utilisé !</h3>';
-} elseif($errCode == 3) {
-$errMsg = '<h3>Adresse e-mail déjà utilisée !</h3>';
-} elseif($errCode == 4) {
-$errMsg = '<h3>Mots de passe différents</h3>';
-} elseif($errCode == 5) {
-$errMsg = '<h3>Problème sur l\'avatar</h3>';
-} elseif($errCode == 6) {
-$errMsg = '<h3>Le fichier n\'est pas une image</h3>';
-} elseif($errCode == 7) {
-$errMsg = '<h3>L\'image est trop grande</h3>';
-}
-echo '<article class="msg-erreur">'.$errMsg.'</article>'; // on affiche le message
-unset($_SESSION['registerErrCode']); // on retire la variable de session
-}
+if(!is_null($this->errMsg)) { 
 ?>
+	<article class="msg-erreur"><?php echo $this->errMsg;?></article>
+<?php 
+} 
+?>
+
 
 <?php if(isset($_SESSION['connected']) && $_SESSION['connected'] == true &&
     isset($_SESSION['user']) && $_SESSION['user']->slug == $this->profil->slug) { ?>
-- 
GitLab