From a4727b81f6324d29ba9f5f5c8b44477d63a4bd37 Mon Sep 17 00:00:00 2001 From: Juan Leyva Date: Thu, 27 Apr 2017 15:48:10 +0200 Subject: [PATCH] MDL-58741 webservice: Add missing params to webservice/pluginfile.php MDL-58334 and MDL-32471 introduced new parameters in pluginfile.php but they are missing from webservice/pluginfile.php We need the offline one in the Mobile app (this new parameter was designed thinking in the mobile app) --- webservice/pluginfile.php | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/webservice/pluginfile.php b/webservice/pluginfile.php index ba95f722012..f90aaf5d866 100644 --- a/webservice/pluginfile.php +++ b/webservice/pluginfile.php @@ -41,17 +41,23 @@ require_once($CFG->dirroot . '/webservice/lib.php'); // Allow CORS requests. header('Access-Control-Allow-Origin: *'); -//authenticate the user +// Authenticate the user. $token = required_param('token', PARAM_ALPHANUM); +// Use preview in order to display the preview of the file (e.g. "thumb" for a thumbnail). +$preview = optional_param('preview', null, PARAM_ALPHANUM); +// Offline means download the file from the repository and serve it, even if it was an external link. +// The repository may have to export the file to an offline format. +$offline = optional_param('offline', 0, PARAM_BOOL); + $webservicelib = new webservice(); $authenticationinfo = $webservicelib->authenticate_user($token); -//check the service allows file download +// Check the service allows file download. $enabledfiledownload = (int) ($authenticationinfo['service']->downloadfiles); if (empty($enabledfiledownload)) { throw new webservice_access_exception('Web service file downloading must be enabled in external service settings'); } -//finally we can serve the file :) +// Finally we can serve the file :). $relativepath = get_file_argument(); -file_pluginfile($relativepath, 0); +file_pluginfile($relativepath, 0, $preview, $offline);