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.

Sign up for GitHub

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

Jump to bottom

Hotfix - Administración - Valores por defecto de DateTime traducidos y valor "first day of next month" corregido #488

Merged
merged 6 commits into from
Dec 19, 2024
Merged

Hotfix - Administración - Valores por defecto de DateTime traducidos y valor "first day of next month" corregido #488

merged 6 commits into from
Dec 19, 2024

Conversation

ainaraRT
Copy link
Collaborator

Description

Este PR resuelve dos incidencias.

  1. La primera, como se describe en el issue, se detectaba que en la vista de creación/edición de campos de Estudio, para campos de tipo DateTime, los valores por defecto a seleccionar no se muestran en los diferentes idiomas, si no que se muestran siempre en inglés.
  2. En la segunda, se detecta que si se selecciona el Valor por defecto "first day of next month" para un campo de tipo "DateTime", de cualquier módulo, la instancia da un error que la deja inutilizable.

Motivation and Context

  1. Se ha añadido la función del constructor junto a $app_strings donde se recoge las etiquetas del desplegable. Generando dicho desplegable.
  2. Se ha corregido el valor "first of next month" que estaba generando problema, por "first day of next month".

How To Test This

  1. Ir a Estudio, a un módulo cualquiera y añadir un campo.
  2. Crear un campo de tipo Fecha y hora (DateTime) y comprobar que se muestra la lista de los valores por defecto en el idioma que el usuario ha seleccionado en el login o tiene la instancia por defecto.
  3. Además, comprobar que al seleccionar "primer día del próximo mes" (first day of next month), se puede guardar el campo y se visualiza correctamente sin generar un error que impide acceder a "Administración".

@ainaraRT ainaraRT self-assigned this Nov 26, 2024
jalbaiges
jalbaiges requested changes Nov 26, 2024
View reviewed changes
Copy link

@jalbaiges jalbaiges left a comment

Choose a reason for hiding this comment

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

Cadenas revisadas. Ver comentario sobre comentarios STIC-Custom.

include/language/ca_ES.lang.php Outdated
@@ -2052,6 +2052,10 @@
'LBL_DESCRIPTION' => 'Descripció',

'LBL_YESTERDAY' => 'ahir',
// STIC-Custom 20241126 ART - Translated Default Datetime Values
Copy link

Choose a reason for hiding this comment

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

En los cambios de los ficheros de idiomas que no sean el inglés no hace falta poner los comentarios // STIC-Custom porque no son ficheros que se incluyan en PR al core de SuiteCRM.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

He visto en el fichero que hay más STIC-Custom. Si aún así, queremos quitarlo, lo quito

Copy link

Choose a reason for hiding this comment

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

Cierto, hay otros. Como norma general no debería ser necesario que estuvieran, aunque podemos distinguir dos casos: aquellos en los que se introducen cambios que acabarán llegando al core de SuiteCRM y aquellos que quedarán en el contexto de SinergiaCRM. Los primeros no hay necesidad de etiquetarlos, en tanto que en los últimos opcionalmente puede considerarse. En el caso que nos ocupa estaríamos en el primer escenario.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Leyendo tu cometario, tienes razón ¡Comentarios quitados!

jalbaiges
jalbaiges previously approved these changes Nov 26, 2024
View reviewed changes
Copy link

@jalbaiges jalbaiges left a comment

Choose a reason for hiding this comment

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

Aprobado a efectos de cadenas y a valorar el tema los comentarios // STIC-Custom.

@ainaraRT ainaraRT added General bug Something isn't working labels Dec 3, 2024
enricsinergia
enricsinergia requested changes Dec 4, 2024
View reviewed changes
Copy link
Collaborator

@enricsinergia enricsinergia left a comment

Choose a reason for hiding this comment

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

Funciona correctamente, incluso los literales con apóstrofe :)
Sin embargo, aplicaría elpequeño cambio comentado sobre mayúsculas y minúsculas

include/language/es_ES.lang.php Outdated
@@ -2055,6 +2055,7 @@
'LBL_DESCRIPTION' => 'Descripción',

'LBL_YESTERDAY' => 'Ayer',
Copy link
Collaborator

Choose a reason for hiding this comment

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

Una pequeñísima tontería: Si el resto de opciones empiezan en minúscula, esta también debería hacerlo.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Corregido!

include/language/gl_ES.lang.php Outdated
@@ -2055,6 +2055,7 @@
'LBL_DESCRIPTION' => 'Descrición',

'LBL_YESTERDAY' => 'Onte',
Copy link
Collaborator

Choose a reason for hiding this comment

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

ídem que en español

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Corregido!

enricsinergia
enricsinergia approved these changes Dec 16, 2024
View reviewed changes
Copy link
Collaborator

@enricsinergia enricsinergia left a comment

Choose a reason for hiding this comment

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

(A)provado

juanSTIC
juanSTIC approved these changes Dec 18, 2024
View reviewed changes
Copy link
Collaborator

@juanSTIC juanSTIC left a comment

Choose a reason for hiding this comment

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

Visto y probado

@AlbertoSTIC AlbertoSTIC merged commit 9e2787a into develop Dec 19, 2024
1 check failed
@AlbertoSTIC AlbertoSTIC deleted the bug/translatedDefaultDatetimeValues branch December 19, 2024 08:21
@github-actions GitHub Actions
Copy link

Actions executed at: 2024-12-19 08:22:03.

@SinergiaCRM
Copy link
Collaborator

PR en SA: salesagility#10617

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juanSTIC juanSTIC juanSTIC approved these changes

@enricsinergia enricsinergia enricsinergia approved these changes

@jalbaiges jalbaiges jalbaiges left review comments

Assignees

@ainaraRT ainaraRT

Labels
Administración del CRM bug Something isn't working General
Projects
None yet
Milestone
No milestone
6 participants
@ainaraRT @SinergiaCRM @jalbaiges @juanSTIC @enricsinergia @AlbertoSTIC