Changeset 112651 in spip-zone

Timestamp:

Dec 6, 2018, 12:02:59 PM (5 months ago)

Author:

maieul@…

Message:

L'option forcer_from fait deux choses:

• changer l'adresse d'envoi si domaine différent de celle du webmestre
• et, si on change l'adresse d'envoi, changer aussi le nom de l'expediteur

Le premier point permet de fonctionner avec des serveurs SMTP stricts,
qui vérifient le domaine.

Toutefois, on peut vouloir garder le nom expediteur, même si on change
l'adresse expeditrice (ce que fait, par ex, Formidable lorsqu'on ne
force pas le champ From).

Une nouvelle option permet cela.

Location:

_plugins_/facteur/trunk

Files:

• classes/facteur.php (modified) (3 diffs)
• formulaires/configurer_facteur.html (modified) (1 diff)
• formulaires/configurer_facteur.php (modified) (2 diffs)
• lang/facteur_fr.php (modified) (1 diff)
• paquet.xml (modified) (1 diff)

_plugins_/facteur/trunk/classes/facteur.php

@@ -55 +55 @@
                 $defaut = array();
                 foreach (array(
-                        'adresse_envoi', 'adresse_envoi_email', 'adresse_envoi_nom', 'forcer_from',
+                        'adresse_envoi', 'adresse_envoi_email', 'adresse_envoi_nom', 'forcer_from', 'garder_fromname',
                         'cc', 'bcc',
                         'smtp', 'smtp_host', 'smtp_port', 'smtp_auth',
@@ -107 +107 @@
                 // si From n'est pas dans le meme domaine
                 // (utiliser le facteur avec un service externe qui necessite la validation des domaines d'envoi)
+                // On peut avoir une sous-option garder_fromname pour garder le FromName passé
                 if ($options['forcer_from']=='oui'){
                         $this->ForceFrom = $this->From;
-                        $this->ForceFromName = $this->FromName;
+                        if ($options['garder_fromname']!='oui') {
+                                $this->ForceFromName = $this->FromName;
+                        }
                 }
 
@@ -438 +441 @@
                                 // on force le From
                                 $this->From = $this->ForceFrom;
-                                $this->FromName = $this->ForceFromName;
+                                if ($this->ForceFromName) {//null si jamais garder_fromname était activé
+                                        $this->FromName = $this->ForceFromName;
+                                }
                         }
                 }

_plugins_/facteur/trunk/formulaires/configurer_facteur.html

@@ -44 +44 @@
                                                                 <input type="checkbox" name="#GET{name}" class="checkbox" id="#GET{name}_#GET{val}" value="#GET{val}"[(#ENV{#GET{name},#GET{defaut}}|=={#GET{val}}|oui)checked="checked"] />
                                                                 <label for="#GET{name}_#GET{val}"><:facteur:label_facteur_forcer_from:></label>
+                                                        </div>
+                                                </li>
+                                                #SET{name,facteur_garder_fromname}#SET{obli,''}#SET{defaut,''}#SET{erreurs,#ENV**{erreurs}|table_valeur{#GET{name}}}
+                                                <li class="editer editer_[(#GET{name})][ (#GET{obli})][ (#GET{erreurs}|oui)erreur]">[
+                                                        <span class='erreur_message'>(#GET{erreurs})</span>
+                                                        ]
+                                                        <div class="choix">
+                                                                #SET{val,non}
+                                                                <input type="hidden" name="#GET{name}" value="#GET{val}" />
+                                                                #SET{val,oui}
+                                                                <input type="checkbox" name="#GET{name}" class="checkbox" id="#GET{name}_#GET{val}" value="#GET{val}"[(#ENV{#GET{name},#GET{defaut}}|=={#GET{val}}|oui)checked="checked"] />
+                                                                <label for="#GET{name}_#GET{val}"><:facteur:label_facteur_garder_fromname:></label>
                                                         </div>
                                                 </li>

_plugins_/facteur/trunk/formulaires/configurer_facteur.php

@@ -18 +18 @@
                 'facteur_adresse_envoi_email'       => lire_config('facteur_adresse_envoi_email'),
                 'facteur_forcer_from'               => lire_config('facteur_forcer_from'),
+                'facteur_garder_fromname'           => lire_config('facteur_garder_fromname'),
                 'facteur_smtp'                      => lire_config('facteur_smtp'),
                 'facteur_smtp_host'                 => lire_config('facteur_smtp_host'),
@@ -132 +133 @@
         ecrire_meta('facteur_forcer_from', ($facteur_forcer_from=='oui')?'oui':'non');
 
+        $facteur_garder_fromname = _request('facteur_garder_fromname');
+        ecrire_meta('facteur_garder_fromname', ($facteur_garder_fromname=='oui')?'oui':'non');
+
         $facteur_smtp = _request('facteur_smtp');
         ecrire_meta('facteur_smtp', ($facteur_smtp=='oui')?'oui':'non');

_plugins_/facteur/trunk/lang/facteur_fr.php

@@ -59 +59 @@
         // L
         'label_facteur_forcer_from' => 'Forcer cette adresse d’envoi quand le <tt>From</tt> n’est pas sur le même domaine',
+        'label_facteur_garder_fromname' => 'Garder le nom de l\'expéditeur, même lorsqu\'on force l\'adresse d\'envoi',
 
         // M

_plugins_/facteur/trunk/paquet.xml

@@ -2 +2 @@
         prefix="facteur"
         categorie="communication"
-        version="3.5.2"
+        version="3.6.0"
         etat="stable"
         compatibilite="[3.0.0;3.2.*]"