Ticket #2132 (closed defect: fixed)
adresse email bidon
Reported by: | brol | Owned by: | JcDenis |
---|---|---|---|
Priority: | normal | Milestone: | 2.11 |
Component: | module:core | Version: | 2.8 |
Severity: | normal | Keywords: | |
Cc: |
Description
Pour info, quand on installe un nouveau dc, on crée le super admin. L'adresse email demandée alors n'est pas prise en compte dans le config.php (c'est celle générique n'existant pas dotclear@urladmin qui est inscrite).
define('DC_ADMIN_MAILFROM','dotclear@urladmin');
Me semblerait plus logique que l'adresse email soit celle du super admin créé.
Attachments
Change History
Changed 9 years ago by JcDenis
-
attachment
wizard_email_2132.patch
added
Une solution possible est de l'ajouter au wizard
comment:2 follow-up: ↓ 4 Changed 9 years ago by franck
En mettant
'dotclear@'.$_SERVER['HTTP_HOST']
Comme valeur par défaut, ça peut en effet.
Tu ?
comment:4 in reply to: ↑ 2 Changed 9 years ago by JcDenis
Replying to franck:
En mettant
'dotclear@'.$_SERVER['HTTP_HOST']Comme valeur par défaut, ça peut en effet.
Cette valeur était déjà celle codé en dur, là je rajoute un champ à l'installateur. (Et manquera les traductions)
comment:5 Changed 9 years ago by franck
Je sais, oui, mais utilise cette valeur comme valeur par défaut du nouveau champ, c'était mon conseil.
Dans ton patch :
$ADMINMAILFROM = !empty($_POST['ADMINMAILFROM']) ? $_POST['ADMINMAILFROM'] : '';
Deviendrait :
$ADMINMAILFROM = !empty($_POST['ADMINMAILFROM']) ? $_POST['ADMINMAILFROM'] : 'dotclear@'.$_SERVER['HTTP_HOST'];
comment:6 Changed 9 years ago by franck
Comme ça, ça fonctionnera comme avant si on a pas envie (ou la possibilité) de l'indiquer au moment de l'install.
comment:7 Changed 9 years ago by JcDenis
En fait je laisse le champs complétement vide, et si vide à l'enregistrement, je met celle par défaut, bref peu importe ! Je fais ça, restera les traductions car je ne sais toujours pas faire.
comment:9 Changed 9 years ago by Jean-Christian Denis
(In [2d629dd0e6e9]) Add field for master email to install wizard, addresses #2132
comment:10 Changed 9 years ago by franck <carnet.franck.paul@…>
- Status changed from new to closed
- Resolution set to fixed
(In [9b62d74f505c]) FR Locales, closes #2132
Plus logique, en effet, mais le config.php est créé avant la création du super-admin, du coup ça va singulièrement compliquer les choses.