Mejorar formato email recuperar contraseña

[nosolored]

El cuerpo del mensajes que llega a gestores de correo como Thunderbird y Outlook aparece en una sola línea ignorando el salto de línea definido con \n. Al sustituirlos por código html <br>, funciona correctamente.

[ywarnier]

Se supone que la función nl2br (new line to break row HTML) debe hacer esto ya...
https://github.com/chamilo/chamilo-lms/pull/3565/files#diff-097fa9c0d967bef4c4ca1d6a938f8f205a82a0aee52d4277848fed3fc03844a6R184 (https://php.bet/nl2br)
Me parece super raro que no funcione así...
No puedo aceptar este cambio sin entender porqué no está funcionando el nl2br.

[AngelFQC]

Es cierto lo que dice @ywarnier. Pero hay un lugar, en login.lib.php:95 (Login::send_password_to_user), en que no se usa nl2br para parsear los \n de la función Login::get_user_account_list

Entonces, los cambios del PR para reemplazar forzosamente a usar
en el mensaje no van.
Sino que se necesita agregar el nl2br en Login::send_password_to_user

diff --git a/main/inc/lib/login.lib.php b/main/inc/lib/login.lib.php
index 60a558739f..1569be7c63 100755
--- a/main/inc/lib/login.lib.php
+++ b/main/inc/lib/login.lib.php
@@ -121,6 +121,7 @@ class Login
             null,
             PERSON_NAME_EMAIL_ADDRESS
         );
+        $email_body = nl2br($email_body);
 
         $email_admin = api_get_setting('emailAdministrator');
         $result = api_mail_html('', $email_to, $email_subject, $email_body, $sender_name, $email_admin);

[ywarnier]

Esto me suena mejor aun a lo que conversé a parte con @nosolored
Solo necesitaríamos una prueba de @nosolored para confirmar que funciona.

[nosolored]

Ok, cambios realizados. Lo hemos probado en una demo y funciona correctamente.

[FlintCIBot]

Lint errors were found. A patch is also available.

Please see the report: https://flintci.io/repositories/1013/analyses/49453

This comment was posted by FlintCI. It can be disabled in the repository settings.