undefined _observer in c-table disconnectedCallback
Nobody has claimed this yet.
Assessment
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Newbie friendliness
- 84/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Quiet
- Tech stack
- typescript
- Domain
- frontend
Research direction
Inspect c-table.tsx around line 114 and compare its teardown with the null-safe handling in c-data-table. Ensure disconnectedCallback does not throw when _observer was not initialized, then verify the affected teardown path no longer produces the reported TypeError.
Written by the indexing model from the issue text.
Description
Happens rarely, but still happens.
The component throws in disconnectedCallback because _observer is not always initialized before teardown.
Affected version
@cscfi/csc-ui 3.0.19
Error
TypeError: Cannot read properties of undefined (reading 'disconnect')
at Lt.disconnectedCallback (c-table.tsx:114:20)
this._observer.disconnect();
without a null guard.
The same package appears to use null-safe teardown in other components (for example c-data-table uses optional chaining), so this looks inconsistent.
- Dominant language
- Vue
- Stars
- 1
- Forks
- 0
- Avg merge
- 2h 56m
- Merged PRs (30d)
- 31
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Similar issues
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
gitbutlerapp/gitbutler#15998 · 1 comment ·
-
t/bug
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
bug ready-for-dev
Difficulty 1/5 1-3 hours Newbie friendliness 90/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Difficulty 1/5 Under an hour Newbie friendliness 92/100