Skip to content

Commit

Permalink
Ensure correct context.
Browse files Browse the repository at this point in the history
  • Loading branch information
gjb2048 committed Aug 28, 2024
1 parent e3364e9 commit d2bdd9e
Showing 1 changed file with 22 additions and 18 deletions.
40 changes: 22 additions & 18 deletions lib.php
Original file line number Diff line number Diff line change
Expand Up @@ -801,24 +801,28 @@ function format_vsf_pluginfile($course, $cm, $context, $filearea, $args, $forced
send_file_not_found();
}

// Leave this line out if you set the itemid to null in make_pluginfile_url (set $itemid to 0 instead).
$itemid = array_shift($args); // The first item in the $args array.
// Use the itemid to retrieve any relevant data records and perform any security checks to see if the
// user really does have access to the file in question.
// Extract the filename / filepath from the $args array.
$filename = array_pop($args); // The last item in the $args array.
if (!$args) {
$filepath = '/';
} else {
$filepath = '/' . implode('/', $args) . '/';
}
if ($context->contextlevel == CONTEXT_MODULE) {
// Leave this line out if you set the itemid to null in make_pluginfile_url (set $itemid to 0 instead).
$itemid = array_shift($args); // The first item in the $args array.
// Use the itemid to retrieve any relevant data records and perform any security checks to see if the
// user really does have access to the file in question.
// Extract the filename / filepath from the $args array.
$filename = array_pop($args); // The last item in the $args array.
if (!$args) {
$filepath = '/';
} else {
$filepath = '/' . implode('/', $args) . '/';
}

// Retrieve the file from the Files API.
$fs = get_file_storage();
$file = $fs->get_file($context->id, 'format_vsf', $filearea, $itemid, $filepath, $filename);
if (!$file) {
send_file_not_found(); // The file does not exist.
}
// Retrieve the file from the Files API.
$fs = get_file_storage();
$file = $fs->get_file($context->id, 'format_vsf', $filearea, $itemid, $filepath, $filename);
if (!$file) {
send_file_not_found(); // The file does not exist.
}

send_stored_file($file, null, 0, $forcedownload, $options);
send_stored_file($file, null, 0, $forcedownload, $options);
} else {
send_file_not_found();
}
}

0 comments on commit d2bdd9e

Please sign in to comment.