ProjektAdLer / Autorentool

Apache License 2.0
7 stars 2 forks source link

[Bug] Zwei Lernelemente mit gleichen Namen und Lerninhalt in einem Lernraum #342

Closed e01118 closed 1 year ago

e01118 commented 1 year ago

Beschreibung des Problems:

Es können zwei Lernelemente mit dem gleichen Namen und Lerninhalt erstellt werden, ohne dass eine Meldung des Autorentools dies verhindert.

Schritte zum Reproduzieren:

Wenn man einem Lernraum "Test1" ein Lernelement zuweist "Bla2", dann den Lernraum wechselt, erneut ein Lernelement "Bla2" erstellt, dieses dann in der Liste landet und man dann erneut auf Lernraum "Test1" geht, kann man das zweite Lernelement "Bla2" problemlos in den Grundriss ziehen.

Erwartetes Verhalten:

Es dürfen nicht zwei Lernelemente mit dem gleichen Lerninhalt und Namen (in diesem Fall "Bla2") in einen Lernraum abgelegt werden, geschweige denn überhaupt als solche existieren. Hier müsste eine Fehlermeldung kommen, dass "Bla2" als Lernelement bereits existiert.

Tatsächliches Verhalten:

Grundsätzlich wird dem Nutzer untersagt, innerhalb eines Lernraums (z.B. "Test2") direkt hintereinander zwei Lernelemente mit dem Namen "Bla2" anzulegen. Es erscheint eine Meldung, dass dies nicht geht. Sobald man aber ein Lernelement anlegt und in einen Lernraum zieht, den Lernraum dann wechselt und erneut ein Lernelement mit dem gleichen Namen anlegt, kommt keine Meldung und man kann das zweite identische Lernelement ebenfalls in den vorherigen Lernraum ziehen.

grafik.png

Ansprechpartner:

@e01118 @niklasstich

Ursache (erst bei Fix angeben):

LearningElementNamesProvider hat in GetLearningElements() nur die unplatzierten Elemente sowie die Elemente im momentan ausgewählten Lernraum berücksichtigt, nicht alle Elemente in der Lernwelt

Lösung (erst bei Fix angeben):

LearningElementNamesProvider verwendet nun stattdessen ILearningWorldViewModel.AllLearningElements als Basis

Commit, in dem das Problem gefixed wurde:

d8c2160