mirror_php/moodle
1
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-01-18 14:03:52 +01:00
Code Issues Projects Releases Wiki Activity

MDL-79073 question: detect missing context when deleting question.

Browse Source
This commit is contained in:
Paul Holden 2023-08-22 09:03:57 +01:00
parent f12e1b82a0
commit 08344a8c4b
No known key found for this signature in database
GPG Key ID: A81A96D6045F6164
2 changed files with 29 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/questionlib.php
View File

@ -352,11 +352,12 @@ function question_delete_question($questionid): void {
qv.version,
qbe.id as entryid,
qc.id as categoryid,
qc.contextid as contextid
ctx.id as contextid
FROM {question} q
LEFT JOIN {question_versions} qv ON qv.questionid = q.id
LEFT JOIN {question_bank_entries} qbe ON qbe.id = qv.questionbankentryid
LEFT JOIN {question_categories} qc ON qc.id = qbe.questioncategoryid
LEFT JOIN {context} ctx ON ctx.id = qc.contextid
WHERE q.id = ?';
$questiondata = $DB->get_record_sql($sql, [$question->id]);

27
lib/tests/questionlib_test.php
View File

@ -322,6 +322,33 @@ class questionlib_test extends \advanced_testcase {
$this->assertFalse($DB->record_exists('question', ['id' => $q1->id]));
}
/**
* Test deleting a broken question whose category refers to a missing context
*/
public function test_question_delete_question_missing_context() {
global $DB;
$coursecategory = $this->getDataGenerator()->create_category();
$context = $coursecategory->get_context();
/** @var \core_question_generator $generator */
$generator = $this->getDataGenerator()->get_plugin_generator('core_question');
$questioncategory = $generator->create_question_category(['contextid' => $context->id]);
$question = $generator->create_question('shortanswer', null, ['category' => $questioncategory->id]);
// Now delete the context, to simulate what happens in old sites where
// referential integrity has failed.
$DB->delete_records('context', ['id' => $context->id]);
question_delete_question($question->id);
$this->assertDebuggingCalled('Deleting question ' . $question->id .
' which is no longer linked to a context. Assuming system context ' .
'to avoid errors, but this may mean that some data like ' .
'files, tags, are not cleaned up.');
$this->assertFalse($DB->record_exists('question', ['id' => $question->id]));
}
/**
* This function tests the question_category_delete_safe function.
*/