From 802c37c75c23b70aad41bc75582080e2082028f5 Mon Sep 17 00:00:00 2001 From: Laurent Destailleur Date: Wed, 28 Nov 2007 22:53:21 +0000 Subject: [PATCH] Suppression des TODO/FIXME --- htdocs/lib/CMailFile.class.php | 27 ++++++++------------------- htdocs/projet/tasks/fiche.php | 9 +++------ scripts/mailing/mailing-send.php | 29 ++++++++++++++--------------- 3 files changed, 25 insertions(+), 40 deletions(-) diff --git a/htdocs/lib/CMailFile.class.php b/htdocs/lib/CMailFile.class.php index 1808acf3188..66ec234aaa9 100644 --- a/htdocs/lib/CMailFile.class.php +++ b/htdocs/lib/CMailFile.class.php @@ -198,7 +198,7 @@ class CMailFile if (isset($_SERVER["WINDIR"])) { if (empty($this->addr_from)) $this->addr_from = 'robot@mydomain.com'; - if ($this->addr_from) @ini_set('sendmail_from',getValidAddress($this->addr_from,2)); + @ini_set('sendmail_from',getValidAddress($this->addr_from,2)); } // Forcage parametres @@ -213,19 +213,9 @@ class CMailFile } else { - /* Le errors_to doit se gerer dans en-tete http et non par option -f - if ($this->errors_to) - { - // \TODO Tester que le safe_mode est inactif car fonction mail avec ces param non dispo en safe_mode - dolibarr_syslog("CMailFile::sendfile: mail start SMTP=".ini_get('SMTP').", PORT=".ini_get('smtp_port').", with errorsto : ".getValidAddress($this->errors_to,1)); - $res = mail($dest,$this->subject,stripslashes($this->message),$this->headers,"-f".getValidAddress($this->errors_to,2)); - } - else - {*/ - dolibarr_syslog("CMailFile::sendfile: mail start SMTP=".ini_get('SMTP').", PORT=".ini_get('smtp_port')); - //dolibarr_syslog("to=".getValidAddress($this->addr_to,2).", subject=".$this->subject.", message=".stripslashes($this->message).", header=".$this->headers); - $res = mail($dest,$this->subject,stripslashes($this->message),$this->headers); - /* } */ + dolibarr_syslog("CMailFile::sendfile: mail start SMTP=".ini_get('SMTP').", PORT=".ini_get('smtp_port')); + //dolibarr_syslog("to=".getValidAddress($this->addr_to,2).", subject=".$this->subject.", message=".stripslashes($this->message).", header=".$this->headers); + $res = mail($dest,$this->subject,stripslashes($this->message),$this->headers); if (! $res) { $this->error="Failed to send mail to SMTP=".ini_get('SMTP').", PORT=".ini_get('smtp_port')."
Check your server logs and your firewalls setup"; @@ -240,12 +230,11 @@ class CMailFile if (isset($_SERVER["WINDIR"])) { @ini_restore('sendmail_from'); - - // Forcage parametres - // \TODO A mettre pout tout OS ? - if ($conf->global->MAIN_MAIL_SMTP_SERVER) ini_restore('SMTP'); - if ($conf->global->MAIN_MAIL_SMTP_PORT) ini_restore('smtp_port'); } + + // Forcage parametres + if (! empty($conf->global->MAIN_MAIL_SMTP_SERVER)) ini_restore('SMTP'); + if (! empty($conf->global->MAIN_MAIL_SMTP_PORT)) ini_restore('smtp_port'); } else { diff --git a/htdocs/projet/tasks/fiche.php b/htdocs/projet/tasks/fiche.php index 9b215bf226f..d4b17ca08ba 100644 --- a/htdocs/projet/tasks/fiche.php +++ b/htdocs/projet/tasks/fiche.php @@ -18,13 +18,12 @@ * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. * * $Id$ - * $Source$ */ /** \file htdocs/projet/tasks/fiche.php \ingroup projet - \brief Fiche tâches d'un projet + \brief Fiche taches d'un projet \version $Revision$ */ @@ -36,7 +35,7 @@ $user->getrights('projet'); if (!$user->rights->projet->lire) accessforbidden(); /* - * Sécurité accés client + * S�curit� acc�s client */ $projetid=''; if ($_GET["id"]) { $projetid=$_GET["id"]; } @@ -95,8 +94,6 @@ Function PLines(&$inc, $parent, $lines, &$level, $tasksrole) print ''.$heure." h ".$minutes."\n"; - // TODO améliorer le test - if ($tasksrole[$lines[$i]->id] == 'admin') { print ''; @@ -251,7 +248,7 @@ if ($_GET["action"] == 'create' && $user->rights->projet->creer) $tasksarray=$projet->getTasksArray(); - /* Nouvelle tâche */ + /* Nouvelle t�che */ print ''.$langs->trans("NewTask").''; print ' '; if ($tasksarray) diff --git a/scripts/mailing/mailing-send.php b/scripts/mailing/mailing-send.php index 53ff2aed657..c2e313773ba 100644 --- a/scripts/mailing/mailing-send.php +++ b/scripts/mailing/mailing-send.php @@ -23,25 +23,25 @@ * * !!! Envoi mailing !!! * - * L'utilisation d'adresses de courriers électroniques dans les opérations - * de prospection commerciale est subordonnée au recueil du consentement - * préalable des personnes concernées. + * L'utilisation d'adresses de courriers �lectroniques dans les op�rations + * de prospection commerciale est subordonn�e au recueil du consentement + * pr�alable des personnes concern�es. * - * Le dispositif juridique applicable a été introduit par l'article 22 de - * la loi du 21 juin 2004 pour la confiance dans l'économie numérique. + * Le dispositif juridique applicable a �t� introduit par l'article 22 de + * la loi du 21 juin 2004 pour la confiance dans l'�conomie num�rique. * - * Les dispositions applicables sont définies par les articles L. 34-5 du - * code des postes et des télécommunications et L. 121-20-5 du code de la - * consommation. L'application du principe du consentement préalable en - * droit français résulte de la transposition de l'article 13 de la Directive - * européenne du 12 juillet 2002 « Vie privée et communications électroniques ». + * Les dispositions applicables sont d�finies par les articles L. 34-5 du + * code des postes et des t�l�communications et L. 121-20-5 du code de la + * consommation. L'application du principe du consentement pr�alable en + * droit fran�ais r�sulte de la transposition de l'article 13 de la Directive + * europ�enne du 12 juillet 2002 � Vie priv�e et communications �lectroniques �. */ /** \file scripts/mailing/mailing-send.php \ingroup mailing - \brief Script d'envoi d'un mailing préparé et validé + \brief Script d'envoi d'un mailing pr�par� et valid� */ @@ -69,7 +69,7 @@ require_once (DOL_DOCUMENT_ROOT."/lib/CMailFile.class.php"); $error = 0; -// On récupère données du mail +// On r�cup�re donn�es du mail $sql = "SELECT m.rowid, m.titre, m.sujet, m.body"; $sql .= " , m.email_from, m.email_replyto, m.email_errorsto"; $sql .= " FROM ".MAIN_DB_PREFIX."mailing as m"; @@ -107,8 +107,8 @@ if ($resql) $nbok=0; $nbko=0; -// On choisit les mails non déjà envoyés pour ce mailing (statut=0) -// ou envoyés en erreur (statut=-1) +// On choisit les mails non d�j� envoy�s pour ce mailing (statut=0) +// ou envoy�s en erreur (statut=-1) $sql = "SELECT mc.rowid, mc.nom, mc.prenom, mc.email"; $sql .= " FROM ".MAIN_DB_PREFIX."mailing_cibles as mc"; $sql .= " WHERE mc.statut < 1 AND mc.fk_mailing = ".$id; @@ -152,7 +152,6 @@ if ($resql) $substitutionisok=true; $subject=make_substitutions($subject,$substitutionarray); $message=make_substitutions($message,$substitutionarray); - // TODO Set substitutionisok to flase if errors // Fabrication du mail $mail = new CMailFile($subject, $sendto, $from, $message,