numberToSimpleDate returns day 0 for 31 December of some years when leapYear1900 is true
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 85/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- backend
Línea de trabajo
Localiza la función numberToSimpleDate en el codebase de HyperFormula. El error está en la comprobación de año, donde compara contra {year: year - 1, …} en lugar de {year, …}. Corrige esa comparación, luego ejecuta las especificaciones de date, EDATE, EOMONTH y DAY para verificar que la corrección resuelve los 32 años afectados sin romper el comportamiento existente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
With leapYear1900: true, numberToSimpleDate decomposes 31 December of 32 years between 1900 and 2200 (for example 2036 and 2040) as day 0 of January of the next year. =EOMONTH(DATE(2036,12,31),0) returns 31 January 2037, and TEXT of that serial shows 2037-01-00. It happens with and without nullDate 1899-12-31.
The year check in that function compares against {year: year - 1, …} where {year, …} looks intended. Changing it brings the 32 affected years to 0 in both leap configurations, leaves the default configuration unchanged, and the 80 date, EDATE, EOMONTH and DAY specs I ran still pass.
Found while reviewing #1781. That PR doesn't cause it.
- Lenguaje dominante
- TypeScript
- Estrellas
- 2.8k
- Forks
- 175
- Merge medio
- 1 d 20 h
- PR fusionados (30 d)
- 17
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de handsontable/hyperformula
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
handsontable/hyperformula#1803 ·
Los mantenedores suelen responder en 1 día
-
Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
handsontable/hyperformula#1745 ·
Los mantenedores suelen responder en 1 día
-
Bug Impact: Low
Dificultad 1/5 1-3 horas Aptitud para principiantes 68/100
handsontable/hyperformula#1641 ·
Los mantenedores suelen responder en 1 día
-
Bug
Dificultad 5/5 Más de una semana Aptitud para principiantes 22/100
handsontable/hyperformula#1825 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 58/100
handsontable/hyperformula#1785 ·
Los mantenedores suelen responder en 1 día
Todos los issues de handsontable/hyperformula
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
Los mantenedores suelen responder en 2 días
-
reproduced
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
anomalyco/opencode#62052 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
strapi/design-system#2084 ·
Los mantenedores suelen responder en 3 días
-
Report: &TV InternationalAbiertocheck:passed streams:remove
Dificultad 1/5 Menos de una hora Aptitud para principiantes 67/100
Los mantenedores suelen responder en 1 día
-
OAuth documentation for GET /api/v2/users/me/ appears inconsistent with required global read scopeAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día