Some legacy datasets pointing to external files have empty data worksheets: no data fields, just the header metadata fields and row numbers in Column A. At the moment, this correctly throws up error logs - because we no longer want to allow this to happen - but DataWorksheet.n_fields and DataWorksheet.n_empty_fields are currently typed but not defined which causes an Exception when printing the worksheet summary message.
This PR defines defaults of 0 for those two values. It also adds some extra FORMATTER.pops() into taxa.py to maintain correct indenting in failure modes for NCBITaxa and GBIFTaxa.
Some legacy datasets pointing to external files have empty data worksheets: no data fields, just the header metadata fields and row numbers in Column A. At the moment, this correctly throws up error logs - because we no longer want to allow this to happen - but
DataWorksheet.n_fields
andDataWorksheet.n_empty_fields
are currently typed but not defined which causes an Exception when printing the worksheet summary message.This PR defines defaults of 0 for those two values. It also adds some extra
FORMATTER.pops()
intotaxa.py
to maintain correct indenting in failure modes forNCBITaxa
andGBIFTaxa
.