shuffleanswers aus Quiz-Einstellungen berücksichtigen
#103
Conversation
alexanderschmitz
force-pushed
the
feat/allowshuffle
branch
2 times, most recently
from
b9b210a
to
61e73df
Compare
alexanderschmitz
force-pushed
the
feat/allowshuffle
branch
3 times, most recently
from
59b0e5e
to
7f1e1eb
Compare
- possibility to set allowshuffle in the admin settings - overwrite allowshuffle with the quiz setting
alexanderschmitz
force-pushed
the
feat/allowshuffle
branch
from
7f1e1eb
to
a33b3c8
Compare
| @@ -92,6 +94,7 @@ public function start_attempt(question_attempt_step $step, $variant): void { | |||
| $this->scoringstate = null; | |||
|
|
|||
| $this->ui = new question_ui_renderer($attempt->ui->content, $attempt->ui->placeholders); | |||
| $this->ui->shuffleanswers = $this->shuffleanswers; | |||
There was a problem hiding this comment.
Bitte im Konstruktor übergeben.
|
|
||
| // Check if we are in a quiz context. | ||
| if ($this->page->context->contextlevel == CONTEXT_MODULE && $this->page->cm->modname == 'quiz') { | ||
| $quiz = $DB->get_record('quiz', ['id' => $this->page->cm->instance]); |
There was a problem hiding this comment.
Die Information steckt doch bereits in $question->shuffleanswers!?
There was a problem hiding this comment.
Wird das $question->shuffleanswers denn automatisch überschrieben, wenn man in den Quiz Settings > Question Behaviour > Shuffle within questions ändert?
There was a problem hiding this comment.
Wenn ich mir https://github.com/moodle/moodle/blob/main/question/engine/bank.php#L278 ansehe, dann scheint das ja gemacht zu werden. Da der Wert aber nur im Falle von "kein shuffle" gesetzt/überschrieben wird, müsstest du vermutlich vorher mittels isset prüfen, ob das Attribut überhaupt existiert.
| // Whether the content of a qpy:shuffle-contents can be randomly shuffled. | ||
| $mform->addElement('hidden', 'shuffleanswers', get_config('qtype_questionpy', 'shuffleanswers')); | ||
| $mform->setType('shuffleanswers', PARAM_BOOL); | ||
|
|
There was a problem hiding this comment.
Bitte keine neuen Einstellungen einführen. Ob so eine Einstellung für das jeweilige Paket sinnvoll ist, müssen die Pakete selber wissen und in ihren Options definieren.
| */ | ||
| protected function initialise_question_instance(question_definition $question, $questiondata) { | ||
| parent::initialise_question_instance($question, $questiondata); | ||
| $question->shuffleanswers = $questiondata->shuffleanswers; |
There was a problem hiding this comment.
Greifst du hier auf das richtige Attribut zu? Moodle schreibt ja in $questiondata->options->shuffleanswers.
|
Also das Moodle-Feature riecht für mich ganz stark nach einem schlecht durchdachten Nebengedanken. Datenbankzugriff oder Schema-Änderungen sind dann gar nicht notwendig, auch keine Änderungen an der options form. |
|
Was die Nützlichkeit einer Plugin-weiten Einstellung betrifft bin ich noch etwas skeptisch. Hast du mal geschaut, wie sich das mit der Standard-Quiz-Einstellung (Quiz → General settings → Shuffle within questions) vergleicht? |
Ich hab das so ähnlich umgesetzt wie bei dem Multichoice Fragetypen. D.h. die
shuffleanswersFlag wird in den Admin settings gesetzt und über die edit form in der Datenbank gespeichert.Ich dachte, wenn ich mir schon Gedanken zu dem Thema mache, hat man später die Möglichkeit hier global diese Option zu setzen oder eventuell in der Options Form die Möglichkeit bieten.
Um das
shuffleanswersaus dem Quiz Kontext zu laden, wird imrenderer::formulation_and_controls()das Quiz abgefragt.Closes #90