Open fensoft opened 4 years ago
vinceg77
commented
4 years ago @TesteursFous : Pour tester cette PR, il faut supprimer manuellement le répertoire d'un plugin de votre choix. Il faut ensuite se rendre sur la page Diagnostics/Equipement et s'assurer que cette page n'est pas vide ou blanche
vinceg77
commented
4 years ago J'avoue que je ne comprends pas bien le sens du fix de cette PR ... Qui sont les utilisateurs qui - pour désinstaller un plugin dont il n'ont plus besoin - vont aller manuellement supprimer le repertoire de ce plugin, alors qu'il existe un bouton Supprimer dans la GUI de Nextdom ???
fensoft
commented
4 years ago Qui sont les utilisateurs qui - pour désinstaller un plugin dont il n'ont plus besoin - vont aller manuellement supprimer le repertoire de ce plugin, alors qu'il existe un bouton Supprimer dans la GUI de Nextdom ???
Quand tu as un plugin qui plante tout ND c'est le derniers recours "simple".
Sylvaner
commented
4 years ago Le soucis, c'est que cette solution ne fonctionne que sur 1 page ou 2, mais si ton nextdom est planté, ça cassera sur les autres pages. Je pense pas que ce soit la meilleur des méthodes
fensoft
commented
4 years ago Le plugin incriminé est toujours présent sur mon installation. C'est la seule page qui fait ça et uniquement sur nextdom (c'est ok sur jeedom)
Gis70
commented
4 years ago @fensoft j'obtiens ça à la place d'une page vide
Mais je n'ai pas ce que l'on devrait avoir.
fensoft
commented
4 years ago C'est pas vraiment mieux. Un vrai fix tel que celui que j'ai mis est nécessaire
Gis70
commented
4 years ago C'est pas vraiment mieux. Un vrai fix tel que celui que j'ai mis est nécessaire
J'ai mis ton code en fait
fensoft
commented
4 years ago J'imagine que le code a changé depuis la PR
Sylvaner
commented
4 years ago Le soucis de cette correction, c'est qu'elle fix 1 affichage mais pas le reste des autres problèmes. Donc au pire le dashboard s'affichera, mais tout pourra crasher plus loin
cf https://github.com/NextDom/nextdom-core/issues/1577