nextdom-core icon indicating copy to clipboard operation
nextdom-core copied to clipboard

additional check in case of null eqLogic

Open fensoft opened this issue 6 years ago • 10 comments

cf https://github.com/NextDom/nextdom-core/issues/1577

fensoft avatar Dec 11 '19 16:12 fensoft

@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 avatar Dec 12 '19 19:12 vinceg77

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 ???

vinceg77 avatar Dec 12 '19 19:12 vinceg77

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".

fensoft avatar Jan 03 '20 12:01 fensoft

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

Sylvaner avatar Jan 03 '20 22:01 Sylvaner

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)

fensoft avatar Jan 03 '20 22:01 fensoft

@fensoft j'obtiens ça à la place d'une page vide

image

Mais je n'ai pas ce que l'on devrait avoir.

Gis70 avatar Jan 19 '20 21:01 Gis70

C'est pas vraiment mieux. Un vrai fix tel que celui que j'ai mis est nécessaire

fensoft avatar Jan 19 '20 22:01 fensoft

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

Gis70 avatar Jan 19 '20 22:01 Gis70

J'imagine que le code a changé depuis la PR

fensoft avatar Jan 19 '20 23:01 fensoft

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

Sylvaner avatar Mar 19 '20 01:03 Sylvaner