[16.0][FIX] l10n_es_aeat_mod347: fix on partner_records' states #3310
Conversation
ACheung-FactorLibre
force-pushed
the
16.0-fix-l10n_es_aeat_mod347
branch
from
60589ce to
4e7529b
Compare
ACheung-FactorLibre
force-pushed
the
16.0-fix-l10n_es_aeat_mod347
branch
from
4e7529b to
edffcde
Compare
There was a problem hiding this comment.
LGTM
Corregido el envío individual en el que cuando el contacto no tiene correo, si descartas sin ponerlo, el registro no pasa a enviado
Corregido el envío masivo no poniendo todos como enviado y dejando en pendiente aquellos registros que su contacto no tiene correo
|
This PR has the |
l10n_es_aeat_mod347/models/mod347.py
Outdated
| self.write({"state": "sent"}) | ||
| mail_id = template.send_mail(record.id) | ||
| if mail_id: | ||
| record.write({"state": "sent"}) |
There was a problem hiding this comment.
Acumula todos para escribirlos de un golpe.
There was a problem hiding this comment.
Listo, gracias!
| active_id = self._context.get("active_id") | ||
| record = self.env["l10n.es.aeat.mod347.partner_record"].browse(active_id) | ||
| if result_message.mail_ids: | ||
| record.write({"state": "sent"}) |
There was a problem hiding this comment.
Esto está repetido según veo.
There was a problem hiding this comment.
No, no está repetido, ya que este es el que controla el envío de correo individual, mientras que el otro método es para el envío de correos en masivo (para el botón de enviar correos arriba a la izquierda).
There was a problem hiding this comment.
Ya, pues habría que unificar código y evitar tantas líneas para una función tan pequeña. Intenta por favor darle una vuelta a ver si ambas cosas pueden funcionar con el mismo código o que no se requiera algo tan abigarrado.
There was a problem hiding this comment.
Hay dos vias por los que se está enviando el mail.
- Envía el mail directamente mediante
send_maildel template, y ahí se puede comprobar si se ha enviado el correo, y marcar a sent. - Abre el wizard del composer y se envía desde ahí el correo. Por eso se está tocando la función
_action_send_maildemail.compose.message, porque no vemos otra manera de comprobar si se ha enviado el mail correctamente.
Para unificar los dos casos lo que se nos ocurre es hacer que se envíe el mail siempre directamente sin el wizard de composer. Lo único que esto no permitiría modificar el mensaje que se va a enviar.
A priori no vemos otra forma de comprobar el envío de mail si se envía de manera diferente.
No se que opinas @pedrobaeza
There was a problem hiding this comment.
Yo estaba pensando en el mecanismo que utiliza por ejemplo el módulo sale para cambiar de estado al sale.order:
así se unifica venga de dónde venga.
También sería conveniente que el mensaje no se auto-elimine para dejar el rastro y saber cuándo y qué se ha enviado.
ACheung-FactorLibre
force-pushed
the
16.0-fix-l10n_es_aeat_mod347
branch
from
edffcde to
10b60b9
Compare
ACheung-FactorLibre
force-pushed
the
16.0-fix-l10n_es_aeat_mod347
branch
from
10b60b9 to
570bcd7
Compare
| @@ -175,7 +176,7 @@ def button_confirm(self): | |||
|
|
|||
| def button_send_mails(self): | |||
| self.partner_record_ids.filtered( | |||
| lambda x: x.state == "pending" | |||
| lambda x: x.state == "pending" and x.partner_id.email | |||
There was a problem hiding this comment.
Esto no sería correcto, ya que se utiliza la dirección de facturación cuando está disponible.
| active_id = self._context.get("active_id") | ||
| record = self.env["l10n.es.aeat.mod347.partner_record"].browse(active_id) | ||
| if result_message.mail_ids: | ||
| record.write({"state": "sent"}) |
There was a problem hiding this comment.
Yo estaba pensando en el mecanismo que utiliza por ejemplo el módulo sale para cambiar de estado al sale.order:
así se unifica venga de dónde venga.
También sería conveniente que el mensaje no se auto-elimine para dejar el rastro y saber cuándo y qué se ha enviado.
aritzolea
force-pushed
the
16.0-fix-l10n_es_aeat_mod347
branch
from
570bcd7 to
383d411
Compare
ljsalvatierra-factorlibre
force-pushed
the
16.0-fix-l10n_es_aeat_mod347
branch
from
383d411 to
600ac90
Compare
danielduqma
force-pushed
the
16.0-fix-l10n_es_aeat_mod347
branch
from
600ac90 to
71db155
Compare
aritzolea
force-pushed
the
16.0-fix-l10n_es_aeat_mod347
branch
from
71db155 to
54779b6
Compare
aritzolea
force-pushed
the
16.0-fix-l10n_es_aeat_mod347
branch
from
54779b6 to
2922707
Compare
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
bot
added
the
stale
l10n.es.aeat.mod347.partner_recordwhere being marked assent, when the partner_id related did not have an email configured.