Skip to content

Conversation

MartinGauk
Copy link
Contributor

Es wird geprüft, dass man auf die Frage zugreifen darf und ob diese Frage im selben Kurs wie die Aktivität ist. Seit Moodle 5.0 kann man zwar eigentlich auch Fragen aus anderen Kursen benutzen, aber ich befürchte, dass man dafür noch weitere Überprüfungen eigentlich einbauen müsste. Ich halte die Einschränkung für akzeptabel und eine Vereinfachung für uns.

Zudem wird geprüft, dass man als Bewertung Punkte ausgewählt hat. Aktuell werden zwar die erreichten Punkte aus der Frage gar nicht übernommen, aber das kann ja noch kommen.

@MartinGauk MartinGauk requested a review from janbritz October 9, 2025 16:29
Copy link
Contributor

@janbritz janbritz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wie besprochen: Da settings_validation erst ab Moodle 5.0 aufgerufen wird, sollten wir die minimal unterstützte Moodle-Version auf 5.0 setzen (per Anpassung der $plugin->requires in der version.php.

@janbritz janbritz self-requested a review October 13, 2025 10:08
@MartinGauk MartinGauk merged commit cbcc3cf into dev Oct 13, 2025
20 checks passed
@MartinGauk MartinGauk deleted the feat/check-permission branch October 13, 2025 10:15
@MartinGauk
Copy link
Contributor Author

Gut dass du das entdeckt hast. Kannst du die Anpassung von $plugin->requires bitte übernehmen? Du kannst direkt in dev pushen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants