Probleme, die das Kodieren verhindern werden fälschlicherweise angezeigt

[Theresa-Reiche]

Fehlerbeschreibung
Im Reiter Kodierung wird vor Variablen eine Fehlermeldung angezeigt (rotes Quadrat: Probleme, die das Kodieren verhindern), obwohl es eigentlich nicht fehlerhaft angelegt wurde und auch scheinbar kein Problem das Kodieren verhindert.

Nachstellen
Beim Nachstellen ist es mir nicht gelungen das Problem zu reproduzieren. Ich habe auch einen StuMi dran gesetzt, leider auch ergebnislos. ("Hab jetzt 20 Minuten Aufgaben erstellt in verschiedenen Formaten und Sachen, Aufgaben doppelt angelegt und manche gelöscht um zu schauen, ob sich der Bug so nachstellen lässt, aber alles ohne Erfolg. [,,,] In 2 der [Pilo-]Aufgaben tritt er bei 01 in der Kodierung auf, die anderen Aufgaben sind jedoch nicht betroffen, selbst wenn 01 das selbe Aufgabenformat hat.")

Erwartetes Verhalten
Vor den Variablen erscheint kein Fehler.

Screenshots, Links, erschienene Fehlermeldungen
https://www.iqb-studio.de/#/a/1102/146015/schemer

Versionen

• Verwendete Player-  und Editorversion: 2.12.5, Schemer: 2.7.0
• Browser: firefox

[schumaki]

Das könnte mit den doppelten Variablennamen/IDs zusammenhängen.
Ich kann das Bsp. leider nicht öffnen, @jurei733 siehst Du da, woran das liegt?

[Theresa-Reiche]

Ihr habt nun beide Zugriff auf das Beispiel.

[jurei733]

Update nach Prüfung der aktuellen Stände:

Ich habe den Fall gegen den aktuellen Stand von coding-components und gegen iqb-berlin/responses geprüft.

In coding-components ist inzwischen PR #190 gemergt; die neue Varlist-Diagnose würde bei dieser Unit früher anschlagen. Allerdings ist die aktuelle Namensprüfung dabei zu streng für reale Player/Editor-Daten: IDs wie likert-row_4 kommen aktuell vor und sollten daher nicht pauschal als ungültig behandelt werden.

Die ursprünglich beobachteten roten Marker werden durch die responses-Validierung weiterhin reproduziert. Auch mit dem aktuellen responses-Master nach PR #89 (Fix derived alias shadowing, intern 5.1.1, noch nicht npm latest) liefert CMC weiterhin:

• INVALID_SOURCE für 01a
• INVALID_SOURCE für 01b

PR #89 behebt einen verwandten Spezialfall: Eine abgeleitete Variable darf per Alias ihre eigene Basisquelle überschreiben. Der CMC-Fall ist anders: Hier kollidieren Basisvariablen untereinander, weil likert-row_4 den Alias 01a hat, während eine andere Basisvariable die ID 01a hat; analog für 01b.

Fazit: Der Fall ist nicht durch responses behoben. In coding-components sollte die Varlist-Diagnose angepasst werden:

1. IDs mit Bindestrich zulassen, weil solche IDs real aus Player/Editor kommen.
2. Alias/ID-Kollisionen explizit erkennen und verständlich anzeigen.
3. Die roten INVALID_SOURCE-Marker möglichst nicht mehr als erstes/alleiniges Symptom zeigen, weil sie auf 01a/01b zeigen und damit die eigentliche Ursache verschleiern.

[jurei733]

Update: Der Fix aus PR #193 ist inzwischen in master gemergt.

Was sich geändert hat:

1. Hyphen in Variablen-IDs und Aliasen werden in der Varlist-Diagnose zugelassen.

   Die betroffene CMC-Unit enthält generierte IDs wie likert-row_4. Solche Werte kommen aktuell real aus Player/Editor-Daten und dürfen deshalb nicht pauschal als ungültig behandelt werden. Die Diagnose akzeptiert für IDs und Aliase jetzt Namen mit Buchstaben, Zahlen, Unterstrich und Bindestrich, solange mindestens zwei Zeichen vorhanden sind.

2. Alias/ID-Kollisionen werden jetzt explizit erkannt.

   Der eigentliche Konflikt in der CMC-Unit ist nicht likert-row_4 selbst, sondern diese Konstellation:

   • Variable likert-row_4 hat Alias 01a
   • eine andere Variable hat die ID 01a
   • Variable likert-row_5 hat Alias 01b
   • eine andere Variable hat die ID 01b

   Genau solche Fälle werden nun als eigene Problemklasse erkannt: „Alias entspricht anderer ID“. Dadurch wird die Ursache direkt sichtbar, statt erst indirekt über rote INVALID_SOURCE-Marker bei 01a/01b aufzutauchen.

3. Der Diagnose-Dialog wurde erweitert.

   Der Dialog heißt jetzt allgemeiner „Problematische Variablen-IDs/Aliase“ und listet neben doppelten IDs, doppelten Aliasen und ungültigen Namen auch Alias/ID-Kollisionen auf. Die betroffenen ID- und Alias-Felder werden zusätzlich markiert.

4. Tests wurden ergänzt.

   Abgedeckt sind jetzt insbesondere:

   • generierte IDs mit Bindestrich, z. B. likert-row_4
   • Aliase mit Bindestrich
   • Alias/ID-Kollisionen wie im CMC-Fall (01a, 01b)
   • die Anzeige der neuen Konfliktkategorie im Dialog

Was der Fix nicht ändert:

• Die responses-Validierung meldet für die ursprüngliche CMC-Konstellation weiterhin INVALID_SOURCE für 01a und 01b. Das ist erwartbar, weil dort weiterhin ein echter Namenskonflikt besteht.
• Der Fix sorgt aber dafür, dass coding-components diesen Konflikt früher und verständlicher als problematische Variablenliste diagnostiziert.
• Das Issue wird durch den Merge nicht automatisch geschlossen; ich lasse es bewusst offen, damit noch geprüft werden kann, ob die neue Diagnose im Studio/Player-Kontext wie gewünscht sichtbar ist.

Validierung im PR:

• gezielte Karma-Tests für Varlist-Diagnose und Dialog erfolgreich
• ESLint auf den geänderten Dateien erfolgreich