Conversation
|
Why do you log exception in "core"? this is weird... who will ever thought about checking this log? |
|
Not sure where to put this log, and 'core' is the default for log path Line 180 in 61c6ce2 what else do you suggest ? 'plugin' could be better ? |
5064fe3 to
4d35417
Compare
4d35417 to
63e7021
Compare
63e7021 to
9a648d7
Compare
| @@ -69,9 +69,9 @@ function jeedomAutoload($_classname) { | |||
| try { | |||
| include_file('core', $classname, 'class', $classname); | |||
| } catch (Exception $e) { | |||
There was a problem hiding this comment.
Comme on ne gère plus sous php7.4, je propose de remplacer ces double catch par des catch (Throwable $e).
Pas nécessaire de le faire partout, mais si ca peut se faire au fur et à mesure qu'on passe sur du code ca sera déjà pas mal.
There was a problem hiding this comment.
Oui aucun intérêt de faire 2 catch si le traitement est identique.
(le rapport avec php7.4 ? c'est déjà possible je pense dès php7 d'après la doc)
There was a problem hiding this comment.
C'était juste pour préciser que c'était pleinement géré sur cette version, rien de plus.
40a16cb to
854b259
Compare
854b259 to
58d3cf1
Compare
58d3cf1 to
c14311a
Compare
Description
add log in case of plugin error via the plugin autoloader, this is a great help for plugin developers :)
Suggested changelog entry
add log in case of plugin error via the plugin autoloader
Related issues/external references
Fixes #3017
Types of changes
PR checklist