sheet_to_html does not handle blank sheets gracefully #2835
Labels
No Label
DBF
Dates
Defined Names
Features
Formula
HTML
Images
Infrastructure
Integration
International
ODS
Operations
Performance
PivotTables
Pro
Protection
Read Bug
SSF
SYLK
Style
Write Bug
good first issue
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: sheetjs/sheetjs#2835
Loading…
Reference in New Issue
No description provided.
Delete Branch "%!s(<nil>)"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Currently,
sheet_to_html
fails when it attempts to parse a blank sheet because thedecode_range
helper function does not find the!ref
key within thews
object (because it is empty).Because there is not a
book_to_html
function, I'm using thesheet_to_html
function to parse multiple sheets in a workbook, but some may be empty. Can we update thesheet_to_html
function to do a null check on the sheet and return early?Sure, and other methods should also check null object or missing worksheet range.
In the case of
sheet_to_html
, is it preferable to throw an error or generate an empty table?Thanks! I think generating an empty table makes sense.