platform.isAttached should return false if canvas is false-y

[DAcodedBEAT]

Expected behavior

isAttached() from platform.dom.js should only call _getParentNode() on valid a truth-y canvas value.

Current behavior

TypeError: Cannot read properties of null (reading 'parentNode')
  at _getParentNode(../../node_modules/chart.js/dist/chunks/helpers.segment.js:1820:24)
  at DomPlatform.isAttached(../../node_modules/chart.js/dist/chart.js:3434:23)
  at _a.bindResponsiveEvents(../../node_modules/chart.js/dist/chart.js:6266:18)
  at _a.bindEvents(../../node_modules/chart.js/dist/chart.js:6211:12)
  at _a._checkEventBindings(../../node_modules/chart.js/dist/chart.js:5907:12)
  at _a.update(../../node_modules/chart.js/dist/chart.js:5856:10)
  at this._doResize(../../node_modules/chart.js/dist/chart.js:5624:46)
  at sentryWrapped(../../node_modules/@sentry/browser/esm/helpers.js:37:17)

From sentry:

Reproducible sample

.

Optional extra steps/info to reproduce

No response

Context

• While encountering the prior contribution (Cannot read properties of null (reading 'addEventListener') #11677) seemed to be frequent, this particular issue appears to be an even more unique edge case. Nonetheless, I've implemented a fix to enhance the stability of chart.js and the applications using it.
• Alternatively, we could check for the canvas within _getParentNode itself. I will defer that decision to the @chartjs maintainers if they want this changed.

chart.js version

4.4.2

Browser name and version

Experienced on Chrome 122.0.0 according to my project's Sentry, but probably impacts others since this is missing a null check.

Link to your project

No response

[DAcodedBEAT]

Hello @etimberg / @LeeLenaleee 👋 Thanks again for the prompt review and merge. Do you have an ETA for the release of 4.5.0 ?

[DAcodedBEAT]

@chartjs When should I expect this to land? Should I run a fork in production in the interim?