#138: is_same_response

[MHajoha]

Hier wird auch ersichtlich, dass wir die Input-Namen der Frage entweder einschränken oder manglen müssen, um Präfixe wie _ und - zu verhindern. Bin ich aber noch nicht angegangen.

Closes: #138

[MartinGauk]

Hier wird auch ersichtlich, dass wir die Input-Namen der Frage entweder einschränken oder manglen müssen, um Präfixe wie _ und - zu verhindern. Bin ich aber noch nicht angegangen.

Was wäre dazu dein Vorschlag?
In der questionattemptstep.php findet sich der Hinweis "The remainder of the name was supposed to match the regex [a-z][a-z0-9]* - but this has never been enforced.". Wir könnten den Regex übernehmen, den wir in der QPPE für FormName haben.

[MHajoha]

Hier wird auch ersichtlich, dass wir die Input-Namen der Frage entweder einschränken oder manglen müssen, um Präfixe wie _ und - zu verhindern. Bin ich aber noch nicht angegangen.

Was wäre dazu dein Vorschlag? In der questionattemptstep.php findet sich der Hinweis "The remainder of the name was supposed to match the regex [a-z][a-z0-9]* - but this has never been enforced.". Wir könnten den Regex übernehmen, den wir in der QPPE für FormName haben.

FormName ist ^[a-zA-Z_][a-zA-Z0-9_]*$. Zumindest dass ein Name mit _ müssen wir verhindern, aber ansonsten finde ich das passend 👍. Das müssten wir dann vermutlich im Renderer enforcen. Ich denke wir sollten Inputs mit invaliden Namen dann komplett entfernen, und nicht den Namen ändern. Wie siehst du das?

[MHajoha]

Mir fällt auch diese sehr unschöne Stelle auf: https://github.com/moodle/moodle/blob/main/question/engine/questionattempt.php#L1276-L1286

Feld-Namen dürfen effektiv nicht bf_ enthalten :/

[MHajoha]

Für das Namen-Problem habe ich ein Issue erstellt: #142