Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

additional check in case of null eqLogic #1598

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

fensoft
Copy link

@fensoft fensoft commented Dec 11, 2019

cf #1577

@vinceg77 vinceg77 self-requested a review December 12, 2019 15:05
@vinceg77
Copy link
Contributor

vinceg77 commented Dec 12, 2019

@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
Copy link
Contributor

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

Copy link
Contributor

@vinceg77 vinceg77 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Après avoir supprimé le répertoire d'un plugin (XiaomiHome en l'occurence) et être allé dans la page Diagnostiques/Equipements, je constate que cette page est vide...

image

@fensoft
Copy link
Author

fensoft commented Jan 3, 2020

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
Copy link
Contributor

Sylvaner commented Jan 3, 2020

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
Copy link
Author

fensoft commented Jan 3, 2020

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
Copy link

Gis70 commented Jan 19, 2020

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

image

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

@fensoft
Copy link
Author

fensoft commented Jan 19, 2020

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

@Gis70
Copy link

Gis70 commented Jan 19, 2020

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
Copy link
Author

fensoft commented Jan 19, 2020

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

@Sylvaner
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants