Piwigo.org

Eric · 2011-02-05 23:18:01

Bonsoir à tous et toutes,

Je suis confronté à un petit problème technique que je n'arrive pas à résoudre dans un de mes plugins (UAM pour ne pas le citer ^^) : Je cherche à surcharger les informations envoyées par email lorsq'un utilisateur a oublié son mot de passe et demande une régénération. Le but n'est pas, évidemment, de supprimer les informations transmises (à savoir username et password) mais de pouvoir en ajouter d'autres personnalisables.

Le fichier de base qui exploite la génération et l'envoi d'un nouveau mot de passe est ./password.php. Je me suis donc basé sur celui-ci. Comme il ne contient pas de trigger, j'ai tenté ceci:

Code:

// Customization of lost password email
  if (script_basename() == 'password')
  {
    if (isset($_POST['submit']))
    {
      if (isset($_POST['mail_address']) and !empty($_POST['mail_address']) and isset($conf_UAM[29]) and $conf_UAM[29] == 'true')
      {
        $mail_address = pwg_db_real_escape_string($_POST['mail_address']);

        $query = '
SELECT '.$conf['user_fields']['id'].' AS id
     , '.$conf['user_fields']['username'].' AS username
     , '.$conf['user_fields']['email'].' AS email
FROM '.USERS_TABLE.' as u
  INNER JOIN '.USER_INFOS_TABLE.' AS ui
      ON u.'.$conf['user_fields']['id'].' = ui.user_id
WHERE '.$conf['user_fields']['email'].' = \''.$mail_address.'\'
  AND ui.status = \'normal\'
;';
        $result = pwg_query($query);

        if (pwg_db_num_rows($result) > 0)
        {
          while ($row = pwg_db_fetch_assoc($result))
          {
            $new_password = generate_key(6);

            $infos =
              $conf_UAM[30]
              ."\n"."\n".l10n('Username').': '.stripslashes($row['username'])
              ."\n".l10n('Password').': '.$new_password;
          }
        }
      }
    }
  }

Evidemment, cela ne fonctionne pas. J'ai essayé en "clonant" le code originel de password.php. Là, çà fonctionne... Trop ! En effet, comme je fais appel 2 fois à pwg_mail (une fois via password.php et une seconde par le plugin), l'utilisateur reçois 2 emails de régénération de mot de passe; et le dernier arrivé aura raison.

Je suis donc bloqué. D'où ma question : Quelqu'un aurait-il une idée brillante et géniale pour pouvoir surcharger le contenu la variable $infos de password.php ?

Gotcha · 2011-02-05 23:39:49

Pourquoi pas ajouter des triggers :-D
Profites-en avec la v2.2

;-)

Eric · 2011-02-06 02:32:50

Oui, ce serait l'occasion.

Avis des codeurs en chef ?

ddtddt · 2011-02-06 09:27:35

As tu regarder le chemin que prenais le plugin captcha ?
Il y a des chance que ce soit le même chemin ? non ?

Eric · 2011-02-06 12:49:13

Oui... Mais non ;-)

Le plugin Captcha ajoute un élément sur la page d'inscription (register.php) et non sur password.php. De plus, register.php comporte les triggers nécessaires et permet le pre/post filtering.
Avec password.php, autant on pourrait insérer un élément sur la page, autant il n'est pas possible de modifier le comportement de la fonction. Les utilisateurs ayant perdu leur mot de passe reçoivent un mail laconique ne comportant que leur login et mot de passe. Mon but est de permettre "d'embellir" ce contenu.

plg · 2011-02-09 15:04:36

Est-ce que cela t'intéresserait ?

Code:

$ svn diff password.php 
Index: password.php
===================================================================
--- password.php        (revision 9116)
+++ password.php        (working copy)
@@ -84,6 +84,8 @@
           ."\n".l10n('Password').': '.$new_password
           ;
 
+        $infos = trigger_event('render_lost_password_mail_content', $infos);
+
         if (pwg_mail($row['email'],
               array('subject' => l10n('password updated'), 'content' => $infos)))
         {

Eric · 2011-02-09 17:24:23

Absolument ! J'avais déjà procédé à un ajout de trigger sur ma galerie locale pour voir 1) si j'étais capable de faire 2) si cela correspondait au besoin. "render_lost_password_mail_content", c'est nickel.

Merci plg :-)

plg · 2011-02-09 17:31:08

OK, je vais m'en occuper. Comptes sur cet ajout.

Eric · 2011-02-09 17:39:18

plg a écrit:
OK, je vais m'en occuper. Comptes sur cet ajout.

Cool ! ^^

Je résous ce topic ;-)

LucMorizur · 2011-02-09 21:40:10

Eric a écrit:
Cool ! ^^

+1

plg · 2011-02-11 10:54:59

[Bugtracker] ticket 2187

Piwigo.org

Annonce

#1 2011-02-05 23:18:01

[Résolu] [entraide] Personnaliser l'email sur mot de passe perdu

Code:

#2 2011-02-05 23:39:49

Re: [Résolu] [entraide] Personnaliser l'email sur mot de passe perdu

#3 2011-02-06 02:32:50

Re: [Résolu] [entraide] Personnaliser l'email sur mot de passe perdu

#4 2011-02-06 09:27:35

Re: [Résolu] [entraide] Personnaliser l'email sur mot de passe perdu

#5 2011-02-06 12:49:13

Re: [Résolu] [entraide] Personnaliser l'email sur mot de passe perdu

#6 2011-02-09 15:04:36

Re: [Résolu] [entraide] Personnaliser l'email sur mot de passe perdu

Code:

#7 2011-02-09 17:24:23

Re: [Résolu] [entraide] Personnaliser l'email sur mot de passe perdu

#8 2011-02-09 17:31:08

Re: [Résolu] [entraide] Personnaliser l'email sur mot de passe perdu

#9 2011-02-09 17:39:18

Re: [Résolu] [entraide] Personnaliser l'email sur mot de passe perdu

plg a écrit:

#10 2011-02-09 21:40:10

Re: [Résolu] [entraide] Personnaliser l'email sur mot de passe perdu

Eric a écrit:

#11 2011-02-11 10:54:59

Re: [Résolu] [entraide] Personnaliser l'email sur mot de passe perdu

Pied de page des forums