Skip to content

WB Attachments: Possible crash if attachments column is missing #7419

@alesan99

Description

@alesan99

Describe the bug
Related to #7416
There is still an instance in the code that could possibly crash if there is no attachment column, specifically in attachment datasets.

const attachmentColumnIndex = hot.toVisualColumn(getAttachmentsColumn(dataset));

Realistically this shouldn't happen, but there should be a safe check to prevent the original issue from reoccuring.

To Reproduce
Haven't caused this, but hypothetically:

  • Import Attachments
  • Delete the attachments column
  • See that validation silently fails.

Expected behavior
Validation should not silently fail. The attachment column check should not crash if the column isn't there.

Screenshots
If applicable, add screenshots to help explain your problem.

Crash Report

See: #7416

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions