Menu

Solicitud de mejora al nuevo formateador de fecha

JOSE ELIAS
2024-05-14
2024-07-26
  • JOSE ELIAS

    JOSE ELIAS - 2024-05-14

    Hoa JP, están encantados los usuarios con el formateo de fecha escribiendo únicamente el día, se autocompleta mes actual y año actual.

    Me solicitan la mejora de escribir ddMM y que rellene el año en curso.
    ¿Lo puedo modificar yo?

     
    • Chungyen Tsai

      Chungyen Tsai - 2024-05-15

      Buen día Jose,

      Si, siéntete libre de contribuir para el framework.
      De todos modos hay 3 opciones,
      * Añadir un request de la funcionalidad, así lo tengamos en cuenta para las siguientes versiones menores(7.4, 7.5 ...).
      * Modificar la versión de OX que estás usando para tu proyecto, esto es más fácil por si se trata de una funcionalidad pequeña. Solamente debes añadir y probar para tus clientes. La diferencia con hacer una contribución será explicado abajo.
      * Contribuir para la comunidad OX, como sabes, en OX lo usan en muchos países diferentes del mundo y justo estamos tocando fechas, donde cada país tiene su formato de fecha. Entonces debemos contemplar diferentes casos. Luego de hacer las modificaciones, en openxavatest, debes añadir tu test que pruebe los distintos casos, lanzar la suite completa y verificar que no rompa nada.

      Tips que sirve para la segunda opción:
      1. Para el formateo del día se ha añadido una función dateCalendarEditor.formatTwoDigitDate en dateCalendarEditor.js y se ha hecho una verificación más arriba usándolo.

      Tips que sirve para la tercera opción:
      1. Por lo menos ahora se me ocurren 2 casos posibles a contemplar, cuando ingresan datos los usuarios pueden poner día mes y mes día.
      2. Quizás podamos poner ciertas limitaciones, como que el usuario debe ingresar 2 dígitos tanto para mes como para día, de esta forma evitamos estar adivinando si 11 es
      * 1/1/año actual
      * 11/mes actual/año actual entrando como 2 dígitos para el formateador de 2.
      también evitar adivinar si 101 es
      * 10/1/año actual
      * 1/1/año actual tomando 1 para dia y 01 para el mes
      3. La buena noticia es que ya pasamos todas las pruebas manuales del calendario pop up a automático y no tendrás que estar 30min probando manualmente si todo ha salido bien.
      4. La mala noticia es que deberás añadir una prueba en Selenium para la funcionalidad contemplando los casos posibles en la clase DateCalendarTest.java de src/test/java/org.openxava.test.tests.byfeature.
      5. La buena noticia es que no hace falta que lances la suite completa, al tratarse de tema de fechas si no tocas otro código aparte del dateCalendarEditor.js, con que pase la prueba de DateCalendarTest, ya está bien.

      Saludos.

       
  • JOSE ELIAS

    JOSE ELIAS - 2024-05-17

    Prefiero hacer la solicitud y esperar a que puedas implementarlo. Seguro que va a ir mejor que si lo hago yo.
    Mientras lo intentaré.

     
    • Chungyen Tsai

      Chungyen Tsai - 2024-05-17

      Hola Jose,

      Cero drama, crea la solicitud y lo sumaremos en las siguientes versiones menores.

      Saludos.

       
  • JOSE ELIAS

    JOSE ELIAS - 2024-05-18
     
    • Chungyen Tsai

      Chungyen Tsai - 2024-07-26

      Hola Jose,

      El formateador estará en la version 7.4 que se lanza en septiembre.

      Saludos.

       

Log in to post a comment.