From d581065f831f749ff26b55077989fa0d42a9449e Mon Sep 17 00:00:00 2001
From: Laurent Destailleur <eldy@users.sourceforge.net>
Date: Sun, 11 Jul 2010 12:16:04 +0000
Subject: [PATCH] Fix: Message more clear.

---
 htdocs/comm/mailing/fiche.php | 31 ++++++++++++++++---------------
 htdocs/langs/en_US/mails.lang |  2 +-
 htdocs/langs/fr_FR/mails.lang |  2 +-
 3 files changed, 18 insertions(+), 17 deletions(-)

diff --git a/htdocs/comm/mailing/fiche.php b/htdocs/comm/mailing/fiche.php
index 27da892b75a..77a92667090 100644
--- a/htdocs/comm/mailing/fiche.php
+++ b/htdocs/comm/mailing/fiche.php
@@ -490,14 +490,14 @@ if ($_POST["action"] == 'update' && empty($_POST["removedfile"]) && empty($_POST
 }
 
 // Action confirmation validation
-if ($_POST["action"] == 'confirm_valide')
+if ($_REQUEST["action"] == 'confirm_valide')
 {
 
-	if ($_POST["confirm"] == 'yes')
+	if ($_REQUEST["confirm"] == 'yes')
 	{
 		$mil = new Mailing($db);
 
-		if ($mil->fetch($_GET["id"]) >= 0)
+		if ($mil->fetch($_REQUEST["id"]) >= 0)
 		{
 			$mil->valid($user);
 
@@ -511,19 +511,19 @@ if ($_POST["action"] == 'confirm_valide')
 	}
 	else
 	{
-		Header("Location: fiche.php?id=".$_GET["id"]);
+		Header("Location: fiche.php?id=".$_REQUEST["id"]);
 		exit;
 	}
 }
 
 // Resend
-if ($_POST["action"] == 'confirm_reset')
+if ($_REQUEST["action"] == 'confirm_reset')
 {
-	if ($_POST["confirm"] == 'yes')
+	if ($_REQUEST["confirm"] == 'yes')
 	{
 		$mil = new Mailing($db);
 
-		if ($mil->fetch($_GET["id"]) >= 0)
+		if ($mil->fetch($_REQUEST["id"]) >= 0)
 		{
 			$db->begin();
 
@@ -552,22 +552,23 @@ if ($_POST["action"] == 'confirm_reset')
 	}
 	else
 	{
-		Header("Location: fiche.php?id=".$_GET["id"]);
+		Header("Location: fiche.php?id=".$_REQUEST["id"]);
 		exit;
 	}
 }
 
 // Action confirmation suppression
-if ($_POST["action"] == 'confirm_delete')
+if ($_REQUEST["action"] == 'confirm_delete')
 {
-	if ($_POST["confirm"] == 'yes')
+	if ($_REQUEST["confirm"] == 'yes')
 	{
 		$mil = new Mailing($db);
-		$mil->id = $_GET["id"];
+		$mil->id = $_REQUEST["id"];
 
 		if ($mil->delete($mil->id))
 		{
-			Header("Location: index.php");
+			Header("Location: ".DOL_URL_ROOT."/comm/mailing/liste.php");
+			exit;
 		}
 	}
 }
@@ -654,21 +655,21 @@ else
 		// Confirmation de la validation du mailing
 		if ($_GET["action"] == 'valide')
 		{
-			$ret=$html->form_confirm($_SERVER["PHP_SELF"]."?id=".$mil->id,$langs->trans("ValidMailing"),$langs->trans("ConfirmValidMailing"),"confirm_valide");
+			$ret=$html->form_confirm($_SERVER["PHP_SELF"]."?id=".$mil->id,$langs->trans("ValidMailing"),$langs->trans("ConfirmValidMailing"),"confirm_valide",'','',2);
 			if ($ret == 'html') print '<br>';
 		}
 
 		// Confirm reset
 		if ($_GET["action"] == 'reset')
 		{
-			$ret=$html->form_confirm($_SERVER["PHP_SELF"]."?id=".$mil->id,$langs->trans("ResetMailing"),$langs->trans("ConfirmResetMailing",$mil->ref),"confirm_reset");
+			$ret=$html->form_confirm($_SERVER["PHP_SELF"]."?id=".$mil->id,$langs->trans("ResetMailing"),$langs->trans("ConfirmResetMailing",$mil->ref),"confirm_reset",'','',2);
 			if ($ret == 'html') print '<br>';
 		}
 
 		// Confirm delete
 		if ($_GET["action"] == 'delete')
 		{
-			$ret=$html->form_confirm($_SERVER["PHP_SELF"]."?id=".$mil->id,$langs->trans("DeleteAMailing"),$langs->trans("ConfirmDeleteMailing"),"confirm_delete");
+			$ret=$html->form_confirm($_SERVER["PHP_SELF"]."?id=".$mil->id,$langs->trans("DeleteAMailing"),$langs->trans("ConfirmDeleteMailing"),"confirm_delete",'','',2);
 			if ($ret == 'html') print '<br>';
 		}
 
diff --git a/htdocs/langs/en_US/mails.lang b/htdocs/langs/en_US/mails.lang
index 6de888e8aaa..4ba4a26488b 100644
--- a/htdocs/langs/en_US/mails.lang
+++ b/htdocs/langs/en_US/mails.lang
@@ -91,7 +91,7 @@ SendMailing=Send emailing
 SendMail=Send email
 SentBy=Sent by
 MailingNeedCommand=For securities reason, sending an emailing should be performed from command line. Ask your administrator to launch the following command to send the emailing to all recipients:
-MailingNeedCommand2=You can however send them online by adding parameter MAILING_LIMIT_SENDBYWEB with value of max number of emails you want to send by session.
+MailingNeedCommand2=You can however send them online by adding parameter MAILING_LIMIT_SENDBYWEB with value of max number of emails you want to send by session. For this, go on Home - Setup - Other. 
 ConfirmSendingEmailing=Are you sure you want to send mailing ?
 LimitSendingEmailing=On line sending of emailings are limited for security and timeout reasons to <b>%s</b> recipients by sending session.
 TargetsReset=Clear list
diff --git a/htdocs/langs/fr_FR/mails.lang b/htdocs/langs/fr_FR/mails.lang
index 21d7191e76e..f2ed0938ccf 100644
--- a/htdocs/langs/fr_FR/mails.lang
+++ b/htdocs/langs/fr_FR/mails.lang
@@ -91,7 +91,7 @@ SendMailing=Envoi emailing
 SendMail=Envoi mail
 SentBy=Envoyé par
 MailingNeedCommand=Pour des raisons de sécurité, il est recommandé de faire les envois d'un mailing de masse depuis une ligne de commande. Demandez à votre administrateur de lancer la commande suivante pour envoyer le mailing à tous les destinataires :
-MailingNeedCommand2=Vous pouvez toutefois quand même les envoyer en ligne en ajoutant le paramètre MAILING_LIMIT_SENDBYWEB avec la valeur du nombre max de mails envoyés par session d'envoi.
+MailingNeedCommand2=Vous pouvez toutefois quand même les envoyer en ligne en ajoutant le paramètre MAILING_LIMIT_SENDBYWEB avec la valeur du nombre max de mails envoyés par session d'envoi. Pour cela, aller dans Accueil - Configuration - Divers.
 ConfirmSendingEmailing=Confirmez-vous l'envoi de l'emailing ?
 LimitSendingEmailing=L'envoi d'un emailing depuis les écrans est limité pour raisons de sécurité et de timeout à <b>%s</b> destinataires par session d'envoi.
 TargetsReset=Vider liste