Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No se renderizaba el contenido del email si no se encontraba idioma #167

Merged
merged 5 commits into from
Aug 9, 2024

Conversation

lcbautista
Copy link
Member

@lcbautista lcbautista commented Aug 8, 2024

Comportamiento antiguo

Si al evaluar el registro no se encontraba idioma (False), no se podía renderizar el cuerpo del correo (tampoco los destinatarios)

Comportamiento nuevo

Aunque no se encuentre el idioma según el registro, se renderiza el correo en función del idioma por defecto del ERP

También se ha añadido un campo en el asistente que visualiza el idioma, para detectar que idioma se está obteniendo:

imagen

Origen del error

#163

Descripción del error

La nueva clase Localize necesita obligatoriamente un idioma para poder funcionar. Cuando se evalúa el idioma y el registro en cuestión no tiene (False), el resultado del idioma de la plantilla es "False" (string). Este "idioma" se pasa al resto de variables a evaluar (body, cc, bcc, etc) el cual no es un idioma existente por lo que la clase Localize falla al instanciarse.

@lcbautista lcbautista temporarily deployed to frankenergie-pre August 8, 2024 09:13 Inactive
@lcbautista lcbautista temporarily deployed to frankenergie-pre August 8, 2024 09:13 Inactive
@lcbautista lcbautista force-pushed the 61462_fix_localize_requires_language branch from 21efdb0 to edc9948 Compare August 8, 2024 09:15
@lcbautista lcbautista temporarily deployed to frankenergie-pre August 8, 2024 09:17 Inactive
@lcbautista lcbautista temporarily deployed to frankenergie-pre August 8, 2024 09:21 Inactive
@lcbautista lcbautista changed the title 61462 fix localize requires language No se renderizaba el contenido del email si no se encontraba idioma Aug 8, 2024
@lcbautista lcbautista self-assigned this Aug 8, 2024
Copy link

github-actions bot commented Aug 8, 2024

PY2TestsResults

76 tests   76 ✅  1m 45s ⏱️
 2 suites   0 💤
 2 files     0 ❌

Results for commit 84fa7ad.

Copy link

github-actions bot commented Aug 8, 2024

PY3TestsResults

76 tests   76 ✅  1m 40s ⏱️
 2 suites   0 💤
 2 files     0 ❌

Results for commit 84fa7ad.

@lcbautista lcbautista temporarily deployed to gisceerpprod August 8, 2024 12:08 Inactive
Copy link
Contributor

@AdriaGual AdriaGual left a comment

Choose a reason for hiding this comment

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

LGTM

@lcbautista lcbautista merged commit a06f88e into v5_backport Aug 9, 2024
7 checks passed
@lcbautista lcbautista deleted the 61462_fix_localize_requires_language branch August 9, 2024 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants