From 12d5b90b6c05536dacef8aca3d214ba60bb68e1e Mon Sep 17 00:00:00 2001
From: Laurent Destailleur <eldy@users.sourceforge.net>
Date: Mon, 8 Sep 2008 23:48:01 +0000
Subject: [PATCH] Fix: Statut to close a proposal is required.

---
 htdocs/comm/propal.php | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

diff --git a/htdocs/comm/propal.php b/htdocs/comm/propal.php
index 509e8792c8f..b8718c5393b 100644
--- a/htdocs/comm/propal.php
+++ b/htdocs/comm/propal.php
@@ -293,12 +293,21 @@ if ($_REQUEST['action'] == 'setstatut' && $user->rights->propale->cloturer)
 {
 	if (! $_POST['cancel'])
 	{
-		$propal = new Propal($db);
-		$propal->fetch($_GET['propalid']);
-		// prevent browser refresh from closing proposal several times
-		if ($propal->statut==1)
+		if (empty($_REQUEST['statut']))
 		{
-			$propal->cloture($user, $_REQUEST['statut'], $_REQUEST['note']);
+			$mesg='<div class="error">'.$langs->trans("ErrorFieldRequired",$langs->transnoentities("CloseAs")).'</div>';
+			$_REQUEST['action']='statut';
+			$_GET['action']='statut';
+		}
+		else
+		{
+			$propal = new Propal($db);
+			$propal->fetch($_GET['propalid']);
+			// prevent browser refresh from closing proposal several times
+			if ($propal->statut==1)
+			{
+				$propal->cloture($user, $_REQUEST['statut'], $_REQUEST['note']);
+			}
 		}
 	}
 }
-- 
GitLab