]> git.ipfire.org Git - thirdparty/Chart.js.git/commit
Better error message when importing charts incorrectly. (#7790)
authorLAU <34402401+Lautarourtiaga@users.noreply.github.com>
Mon, 14 Sep 2020 13:57:50 +0000 (10:57 -0300)
committerGitHub <noreply@github.com>
Mon, 14 Sep 2020 13:57:50 +0000 (09:57 -0400)
commitc20360d08a010bec31c7da86926fc0024b500bc2
treec35643bd531bfc01b21ab6933fb74ec7af2fe3f0
parentb5853a46b49a19464b8840ab19ec3e9015f45cb7
Better error message when importing charts incorrectly. (#7790)

* Created a better error message when chartjs can't find an element in the DOM.
Also, prevented an exception when trying to find the canvas assuming is an array. When it could also be null.

* Exception is not thrown anymore in getCanvas. Now this pr only prevents the
item.lenght exception and logs a better error when a chart cant be created.

* Deleted error description because i forgot chartjs is used in varois other ways than importing, also used isArray from helpers.core

* isArray check deleted because the type of the dom element received in getCanvas could be not only an array.

Co-authored-by: Lautaro Alonso Urtiaga <lautaro.urtiaga@10pines.com>
src/core/core.controller.js