ghorwin / SIM-VICUS

Building and District Energy Simulation and more...
https://ghorwin.github.io/SIM-VICUS/
Other
33 stars 12 forks source link

Absturz Component Dialog #675

Open Labauke opened 11 months ago

Labauke commented 11 months ago

Absturz wenn man vom Komponentendialog eine Konstuktion auswählt die kein gültiges Material enthält. Das könnte passieren wenn das material der Konstruktion später gelöscht wird.

hirseboy commented 10 months ago

KAnn das geschlossen werden?

ghorwin commented 10 months ago

Es gab bisher keinen Bugfix dazu - weder im Dialog, noch in den verwendeten Klassen. Hier fehlt irgendwo noch eine Konsistenzprüfung, bzw. muss konzeptionell geklärt werden, ob man eine aktiv verwendete Definition (hier Material) löschen darf und falls ja, wie die Oberfläche mit einer nun ungültigen Definition umgehen sollte.

Ich denke, dass man ungültige/fehlende Unterdefinitionen/Verweise im Datenmodell nicht komplett verhindern kann (schon alleine, weil man beim Einlesen einer solchen Projektdatei mit teilweise fehlenden Definitionen keinen Datenverlust möchte), also sollte die Oberfläche immer von derartigen Fehlstellen ausgehen. Im Dialog müsste also explizit auf Gültigkeit der Definition (und aller benötigten Kinder) geprüft werden und entsprechende Infoboxen ausgegraut werden/leer gelassen werden (aber so, dass der User sieht, dass hier was ungültig/fehlend ist).

Da das ein allgemeines Problem ist (betrifft mehrere Dialoge), würde ich dieses Ticket schließen und eine neues, übergeordnetes QA Ticket aufmachen.