Skip to content

Corrige manejo de direcciones duplicadas en envío de correos#1906

Merged
NeoRazorX merged 1 commit intoNeoRazorX:masterfrom
daniel89fg:bugfix-email-bcc
Mar 11, 2026
Merged

Corrige manejo de direcciones duplicadas en envío de correos#1906
NeoRazorX merged 1 commit intoNeoRazorX:masterfrom
daniel89fg:bugfix-email-bcc

Conversation

@daniel89fg
Copy link
Contributor

Se limpia la lista de CC y BCC antes de mover direcciones a TO para evitar conflictos por duplicados. Se añade verificación para cancelar el envío si no hay destinatarios válidos.

Cuando se quiere mandar un email a una dirección y la misma dirección esta como copia oculta, entonces da error de envío porque no hay nada en el campo To, ya que evita enviar a direcciones duplicadas, con esta corrección corregimos el problema para poder enviar.

¿Cómo has probado los cambios?

Toda modificación debe haber sido mínimamente probada. Marca o describe las pruebas que has realizado:

  • He revisado mi código antes de enviarlo.
  • He probado que funciona correctamente en mi PC.
  • He probado que funciona correctamente con una base de datos vacía.
  • He ejecutado los tests unitarios.

Se limpia la lista de CC y BCC antes de mover direcciones a TO para evitar conflictos por duplicados. Se añade verificación para cancelar el envío si no hay destinatarios válidos.
Copilot AI review requested due to automatic review settings March 11, 2026 08:38
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Este PR ajusta el flujo de destinatarios en NewMail::send() para evitar errores de PHPMailer cuando una dirección aparece duplicada entre TO/CC/BCC, promoviendo un destinatario desde CC/BCC a TO y cancelando el envío si finalmente no hay destinatarios válidos.

Changes:

  • Limpia CC/BCC antes de promover un destinatario a TO para reducir conflictos por duplicados.
  • Re-agrega los CC/BCC restantes tras la promoción.
  • Añade una verificación final que cancela el envío si TO sigue vacío tras intentar la promoción.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@NeoRazorX NeoRazorX merged commit efde8c6 into NeoRazorX:master Mar 11, 2026
25 of 26 checks passed
@daniel89fg daniel89fg deleted the bugfix-email-bcc branch March 12, 2026 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants