XLSX ingest fails when first worksheet row has no spans attribute
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 74/100
Línea de trabajo
Empieza por localizar XLSXFileReader y el código que lee el atributo spans de la primera fila de la hoja de cálculo. Comprueba cómo gestiona el lector el atributo cuando no está presente y, después, encuentra y ejecuta las pruebas pertinentes de ingestión de XLSX. El objetivo se cumple cuando se ingiere correctamente un XLSX válido en los demás aspectos que no tenga spans y las pruebas siguen cubriendo el comportamiento existente de ingestión de XLSX.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Dataverse 6.11 fails to ingest an otherwise valid XLSX file when the first worksheet row does not contain the optional spans attribute (file may have been exported?).
The original XLSX fails to ingest. Opening the same file in Microsoft Excel and saving it as a new XLSX causes it to ingest successfully. Saving it as CSV also works. The column names and data do not appear to be the problem.
Error
Null r attribute in the first row element!
Null spans attribute in the first row element!
Ingest failure (IO Exception): Could not parse Excel/XLSX spreadsheet.
Cannot invoke "String.indexOf(int)" because "spansAttribute" is null.
Cause
The original worksheet XML contains a first row like:
<row r="1">
It does not contain a spans attribute such as:
<row r="1" spans="1:12">
The spans attribute is optional in XLSX/SpreadsheetML. XLSXFileReader checks whether it is null but then calls spansAttribute.indexOf(':'), resulting in the exception.
Steps to reproduce
- Create or upload an XLSX whose first worksheet row omits the spans attribute.
- Upload it to a dataset and allow tabular ingest to run.
- Observe that ingest fails with the exception above.
- Open the same file in Microsoft Excel, save it as a new XLSX, and upload it again.
- Observe that the rewritten XLSX ingests successfully.
Environment
Dataverse 6.11
I did look through open issues and I don't think this one is mentioned yet, but I could have missed it.
- Lenguaje dominante
- Java
- Estrellas
- 1.1k
- Forks
- 571
- Merge medio
- 1 d 18 h
- PR fusionados (30 d)
- 8
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 IQSS/dataverse
-
API `/api/dataverses/{identifier}/dataset/linkingDataverses?alreadyLinking=true` returns at most 1 collectionPosiblemente ocupada @vera la tomó hace 3 días. AbiertoType: Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
Return to Author unreachable when a dataset in review has incomplete metadataPosiblemente ocupada @ErykKul la tomó hace 16 días. AbiertoType: Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
Type: Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
IQSS/dataverse#12837 · 2 comentarios · 2 reacciones ·
Los mantenedores suelen responder en 1 día
-
Language controlled vocabulary displays 'Logudorese Sardinian, Croatian' instead of 'Croatian'Posiblemente ocupada @dpavlin la tomó hace 19 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
IQSS/dataverse#12818 · 1 comentario · 1 reacción · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Release Size: 30
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
Todos los issues de IQSS/dataverse
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
commons-app/apps-android-commons#7275 ·
Los mantenedores suelen responder en 1 día
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
liberation-data/drivine4j#682 ·
Los mantenedores suelen responder en 1 día
-
Document and enforce pull request review guidelinesPosiblemente ocupada @dashorst la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
`Time.at(t, in: nil)` returns UTC instead of the local time (or the time zone of `t`), unlike CRubyAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
hermes-queue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
chrisalunlloyd2-sudo/mindpalace#349 ·
Los mantenedores suelen responder en 1 día