ProjektAdLer / Autorentool

Apache License 2.0
7 stars 2 forks source link

[Bug] Fehlermeldung bei Verschieben von Lernelementen in der Liste #371

Closed e01118 closed 11 months ago

e01118 commented 11 months ago

Beschreibung des Problems:

Es tritt ein Problem auf, sobald erstellte Lernelemente, die zunächst in der Liste abgelegt werden, bewegt werden.

Schritte zum Reproduzieren:

Man kann mit dem neuen Stand des Autorentools nun Lernelemente erstellen, ohne zuvor Lernräume erstellt zu haben. Wenn man eine Lernwelt erstellt, danach direkt ein Lernelement erzeugt (ohne davor einen Lernraum zu erstellen) und dann in der Liste das Lernelement bewegt, stürzt das Autorentool ab bzw. erscheint der rote Balken mit der Bitte um neues Laden.

Erwartetes Verhalten:

Das erwartete Verhalten wäre, dass das Bewegen, Ziehen des Lernelements zu keinem Absturz führt. Da noch kein Lernraum erzeugt wurde, wäre es vielleicht sinnvoll, überhaupt kein Verschieben des Lernelements zuzulassen, sondern dies erst mit dem Erstellen eines Lernraums und erscheinen eines Grundrisses zu erlauben.

Tatsächliches Verhalten:

Lernelemente lassen sich bewegen.

Ansprechpartner:

@e01118 @niklasstich

Ursache (erst bei Fix angeben):

In LearningElementDropZoneHelper.DragItemToUnplaced müssen wir checken, ob der ausgewählte LearningSpace null ist, falls das Element aus einem LearningSpace heraus zu unplaced gezogen wird. Dieser Check wurde jedoch immer ausgeführt, auch wenn das Lernelement zuvor gar nicht in einem Space war. Bei negativem Check wurde eine Exception geworfen

Lösung (erst bei Fix angeben):

Check nur ausführen, wenn das Lernelement aus einem Space heraus gezogen wird.

Commit, in dem das Problem gefixed wurde:

09701d9

e01118 commented 11 months ago

grafik