diff --git a/files/renderer.php b/files/renderer.php
index 98c84311bb394..c9d640e481737 100644
--- a/files/renderer.php
+++ b/files/renderer.php
@@ -454,14 +454,21 @@ public function filemanager_js_templates() {
*/
private function fm_print_restrictions($fm) {
$maxbytes = display_size($fm->options->maxbytes);
- if (empty($fm->options->maxfiles) || $fm->options->maxfiles == -1) {
- $maxsize = get_string('maxfilesize', 'moodle', $maxbytes);
- } else {
- $strparam = (object)array('size' => $maxbytes, 'attachments' => $fm->options->maxfiles);
+ $strparam = (object) array('size' => $maxbytes, 'attachments' => $fm->options->maxfiles,
+ 'areasize' => display_size($fm->options->areamaxbytes));
+ $hasmaxfiles = !empty($fm->options->maxfiles) && $fm->options->maxfiles > 0;
+ $hasarealimit = !empty($fm->options->areamaxbytes) && $fm->options->areamaxbytes != -1;
+ if ($hasmaxfiles && $hasarealimit) {
+ $maxsize = get_string('maxsizeandattachmentsandareasize', 'moodle', $strparam);
+ } else if ($hasmaxfiles) {
$maxsize = get_string('maxsizeandattachments', 'moodle', $strparam);
+ } else if ($hasarealimit) {
+ $maxsize = get_string('maxsizeandareasize', 'moodle', $strparam);
+ } else {
+ $maxsize = get_string('maxfilesize', 'moodle', $maxbytes);
}
// TODO MDL-32020 also should say about 'File types accepted'
- return ''. $maxsize. '';
+ return ''. $maxsize . '';
}
/**
diff --git a/lang/en/error.php b/lang/en/error.php
index b9e57071480cb..bf9e294fe0720 100644
--- a/lang/en/error.php
+++ b/lang/en/error.php
@@ -346,6 +346,7 @@
$string['loginasnoenrol'] = 'You cannot use enrol or unenrol when in course "Login as" session';
$string['loginasonecourse'] = 'You cannot enter this course.
You have to terminate the "Login as" session before entering any other course.';
$string['maxbytes'] = 'This file is bigger than the maximum size';
+$string['maxareabytes'] = 'Not enough available space to store this file';
$string['messagingdisable'] = 'Messaging is disabled on this site';
$string['mimetexisnotexist'] = 'Your system is not configured to run mimeTeX. You need to download the appropriate executable for you PHP_OS platform from http://moodle.org/download/mimetex/, or obtain the C source from http://www.forkosh.com/mimetex.zip, compile it and put the executable into your moodle/filter/tex/ directory.';
$string['mimetexnotexecutable'] = 'Custom mimetex is not executable!';
diff --git a/lang/en/moodle.php b/lang/en/moodle.php
index 39ab8077ba60e..456071873aeaa 100644
--- a/lang/en/moodle.php
+++ b/lang/en/moodle.php
@@ -985,6 +985,8 @@
$string['markthistopic'] = 'Highlight this topic as the current topic';
$string['matchingsearchandrole'] = 'Matching \'{$a->search}\' and {$a->role}';
$string['maxsizeandattachments'] = 'Maximum size for new files: {$a->size}, maximum attachments: {$a->attachments}';
+$string['maxsizeandattachmentsandareasize'] = 'Maximum size for new files: {$a->size}, maximum attachments: {$a->attachments}, overall limit: {$a->areasize}';
+$string['maxsizeandareasize'] = 'Maximum size for new files: {$a->size}, overall limit: {$a->areasize}';
$string['maxfilesreached'] = 'You are allowed to attach a maximum of {$a} file(s) to this item';
$string['maximumgrade'] = 'Maximum grade';
$string['maximumgradex'] = 'Maximum grade: {$a}';
diff --git a/lib/filelib.php b/lib/filelib.php
index a9a49d9b7d5f8..bf9d9cf87016d 100644
--- a/lib/filelib.php
+++ b/lib/filelib.php
@@ -481,6 +481,25 @@ function file_get_draft_area_info($draftitemid) {
return $results;
}
+/**
+ * Returns whether a draft area has exceeded/will exceed its size limit.
+ *
+ * @param int $draftitemid the draft area item id.
+ * @param int $areamaxbytes the maximum size allowed in this draft area.
+ * @param int $newfilesize the size that would be added to the current area.
+ * @return bool true if the area will/has exceeded its limit.
+ * @since 2.4
+ */
+function file_is_draft_area_limit_reached($draftitemid, $areamaxbytes, $newfilesize = 0) {
+ if ($areamaxbytes != -1) {
+ $draftinfo = file_get_draft_area_info($draftitemid);
+ if ($draftinfo['filesize'] + $newfilesize > $areamaxbytes) {
+ return true;
+ }
+ }
+ return false;
+}
+
/**
* Get used space of files
* @global moodle_database $DB
diff --git a/lib/form/dndupload.js b/lib/form/dndupload.js
index bbadd5c03f69f..e8ed86eaeeb68 100644
--- a/lib/form/dndupload.js
+++ b/lib/form/dndupload.js
@@ -38,6 +38,8 @@ M.form_dndupload.init = function(Y, options) {
acceptedtypes: [],
// maximum size of files allowed in this form
maxbytes: 0,
+ // maximum combined size of files allowed in this form
+ areamaxbytes: 0,
// unqiue id of this form field used for html elements
clientid: '',
// upload repository id, used for upload
@@ -63,6 +65,7 @@ M.form_dndupload.init = function(Y, options) {
* acceptdtypes: accepted filetypes by this form
* maxfiles: maximum number of files this form allows
* maxbytes: maximum size of files allowed in this form
+ * areamaxbytes: maximum combined size of files allowed in this form
* clientid: unqiue id of this form field used for html elements
* containerid: htmlid of container
* repositories: array of repository objects passed from filepicker
@@ -90,6 +93,7 @@ M.form_dndupload.init = function(Y, options) {
this.acceptedtypes = options.acceptedtypes;
this.clientid = options.clientid;
this.maxbytes = options.maxbytes;
+ this.areamaxbytes = options.areamaxbytes;
this.itemid = options.itemid;
this.author = options.author;
this.container = this.Y.one('#'+options.containerid);
@@ -406,10 +410,14 @@ M.form_dndupload.init = function(Y, options) {
currentfiles: null,
// Total number of files already uploaded (to check for exceeding limits).
currentfilecount: 0,
+ // Total size of the files present in the area.
+ currentareasize: 0,
// The list of files to upload.
uploadqueue: [],
// This list of files with name clashes.
renamequeue: [],
+ // Size of the current queue.
+ queuesize: 0,
// Set to true if the user has clicked on 'overwrite all'.
overwriteall: false,
// Set to true if the user has clicked on 'rename all'.
@@ -432,6 +440,12 @@ M.form_dndupload.init = function(Y, options) {
this.callback = params.callback;
this.currentfiles = params.currentfiles;
this.currentfilecount = params.currentfilecount;
+ this.currentareasize = 0;
+
+ // Retrieve the current size of the area.
+ for (var i = 0; i < this.currentfiles.length; i++) {
+ this.currentareasize += this.currentfiles[i].size;
+ };
this.initialise_queue(params.files);
},
@@ -489,8 +503,9 @@ M.form_dndupload.init = function(Y, options) {
initialise_queue: function(files) {
this.uploadqueue = [];
this.renamequeue = [];
+ this.queuesize = 0;
- // Loop through the files and find any name clashes with existing files
+ // Loop through the files and find any name clashes with existing files.
var i;
for (i=0; i 0 && files[i].size > this.options.maxbytes) {
@@ -507,6 +522,7 @@ M.form_dndupload.init = function(Y, options) {
return;
}
}
+ this.queuesize += files[i].size;
}
},
@@ -528,6 +544,16 @@ M.form_dndupload.init = function(Y, options) {
this.print_msg(M.util.get_string('maxfilesreached', 'moodle', this.options.maxfiles), 'error');
return false;
}
+ // The new file will cause the area to reach its limit, we cancel the upload of all files.
+ if (this.options.areamaxbytes > -1) {
+ var sizereached = this.currentareasize + this.queuesize + file.size;
+ if (sizereached > this.options.areamaxbytes) {
+ this.uploadqueue = [];
+ this.renamequeue = [];
+ this.print_msg(M.util.get_string('uploadformlimit', 'moodle', file.name), 'error');
+ return false;
+ }
+ }
this.uploadqueue.push({file:file, filename:filename, overwrite:overwrite});
return true;
},
diff --git a/lib/form/filemanager.js b/lib/form/filemanager.js
index cbd528a71d5a2..c5bd2b6bd6afb 100644
--- a/lib/form/filemanager.js
+++ b/lib/form/filemanager.js
@@ -26,6 +26,7 @@
* this.filecount, how many files in this filemanager
* this.maxfiles
* this.maxbytes
+ * this.areamaxbytes, the maximum size of the area
* this.filemanager, contains reference to filemanager Node
* this.selectnode, contains referenct to select-file Node
* this.selectui, YUI Panel to select the file
@@ -68,6 +69,7 @@ M.form_filemanager.init = function(Y, options) {
this.currentpath = '/';
this.maxfiles = options.maxfiles;
this.maxbytes = options.maxbytes;
+ this.areamaxbytes = options.areamaxbytes;
this.emptycallback = null; // Used by drag and drop upload
this.filepicker_options = options.filepicker?options.filepicker:{};
@@ -75,6 +77,7 @@ M.form_filemanager.init = function(Y, options) {
this.filepicker_options.context = options.context;
this.filepicker_options.maxfiles = this.maxfiles;
this.filepicker_options.maxbytes = this.maxbytes;
+ this.filepicker_options.areamaxbytes = this.areamaxbytes;
this.filepicker_options.env = 'filemanager';
this.filepicker_options.itemid = options.itemid;
@@ -1003,6 +1006,7 @@ M.form_filemanager.init = function(Y, options) {
author: options.author,
maxfiles: options.maxfiles,
maxbytes: options.maxbytes,
+ areamaxbytes: options.areamaxbytes,
itemid: options.itemid,
repositories: manager.filepicker_options.repositories,
containerid: manager.dndcontainer.get('id')
diff --git a/lib/form/filemanager.php b/lib/form/filemanager.php
index c5daa9ec0c7d3..166794031d6a6 100644
--- a/lib/form/filemanager.php
+++ b/lib/form/filemanager.php
@@ -48,7 +48,7 @@ class MoodleQuickForm_filemanager extends HTML_QuickForm_element {
// PHP doesn't support 'key' => $value1 | $value2 in class definition
// We cannot do $_options = array('return_types'=> FILE_INTERNAL | FILE_REFERENCE);
// So I have to set null here, and do it in constructor
- protected $_options = array('mainfile'=>'', 'subdirs'=>1, 'maxbytes'=>-1, 'maxfiles'=>-1, 'accepted_types'=>'*', 'return_types'=> null);
+ protected $_options = array('mainfile'=>'', 'subdirs'=>1, 'maxbytes'=>-1, 'maxfiles'=>-1, 'accepted_types'=>'*', 'return_types'=> null, 'areamaxbytes' => -1);
/**
* Constructor
@@ -133,6 +133,24 @@ function setMaxbytes($maxbytes) {
$this->_options['maxbytes'] = get_user_max_upload_file_size($PAGE->context, $CFG->maxbytes, $maxbytes);
}
+ /**
+ * Returns the maximum size of the area.
+ *
+ * @return int
+ */
+ function getAreamaxbytes() {
+ return $this->_options['areamaxbytes'];
+ }
+
+ /**
+ * Sets the maximum size of the area.
+ *
+ * @param int $areamaxbytes size limit
+ */
+ function setAreamaxbytes($areamaxbytes) {
+ $this->_options['areamaxbytes'] = $areamaxbytes;
+ }
+
/**
* Returns true if subdirectoy can be created, else false
*
@@ -237,6 +255,7 @@ function toHtml() {
$options->accepted_types = $accepted_types;
$options->return_types = $this->_options['return_types'];
$options->context = $PAGE->context;
+ $options->areamaxbytes = $this->_options['areamaxbytes'];
$html = $this->_getTabs();
$fm = new form_filemanager($options);
diff --git a/repository/filepicker.js b/repository/filepicker.js
index f51cb059f09e9..05ceb3be9416a 100644
--- a/repository/filepicker.js
+++ b/repository/filepicker.js
@@ -544,6 +544,7 @@ M.core_filepicker.init = function(Y, options) {
params['client_id'] = args.client_id;
params['itemid'] = this.options.itemid?this.options.itemid:0;
params['maxbytes'] = this.options.maxbytes?this.options.maxbytes:-1;
+ params['areamaxbytes'] = this.options.areamaxbytes ? this.options.areamaxbytes : -1;
if (this.options.context && this.options.context.id) {
params['ctx_id'] = this.options.context.id;
}
diff --git a/repository/repository_ajax.php b/repository/repository_ajax.php
index 7e5d99d662d82..ad21a4f4d1379 100644
--- a/repository/repository_ajax.php
+++ b/repository/repository_ajax.php
@@ -46,6 +46,7 @@
$req_path = optional_param('p', '', PARAM_RAW); // Path
$accepted_types = optional_param_array('accepted_types', '*', PARAM_RAW);
$saveas_filename = optional_param('title', '', PARAM_FILE); // save as file name
+$areamaxbytes = optional_param('areamaxbytes', -1, PARAM_INT); // Area max bytes
$saveas_path = optional_param('savepath', '/', PARAM_PATH); // save as file path
$search_text = optional_param('s', '', PARAM_CLEANHTML);
$linkexternal = optional_param('linkexternal', '', PARAM_ALPHA);
@@ -301,6 +302,11 @@
die(json_encode($err));
}
+ // Check if we exceed the max bytes of the area.
+ if (file_is_draft_area_limit_reached($itemid, $areamaxbytes, filesize($downloadedfile['path']))) {
+ throw new file_exception('maxareabytes');
+ }
+
// Check if exceed maxbytes.
if ($maxbytes != -1 && filesize($downloadedfile['path']) > $maxbytes) {
throw new file_exception('maxbytes');
diff --git a/repository/upload/lib.php b/repository/upload/lib.php
index e7c4ae4bc8022..f411d0161ef4c 100644
--- a/repository/upload/lib.php
+++ b/repository/upload/lib.php
@@ -57,9 +57,10 @@ public function upload($saveas_filename, $maxbytes) {
$itemid = optional_param('itemid', 0, PARAM_INT);
$license = optional_param('license', $CFG->sitedefaultlicense, PARAM_TEXT);
$author = optional_param('author', '', PARAM_TEXT);
+ $areamaxbytes = optional_param('areamaxbytes', -1, PARAM_INT);
$overwriteexisting = optional_param('overwrite', false, PARAM_BOOL);
- return $this->process_upload($saveas_filename, $maxbytes, $types, $savepath, $itemid, $license, $author, $overwriteexisting);
+ return $this->process_upload($saveas_filename, $maxbytes, $types, $savepath, $itemid, $license, $author, $overwriteexisting, $areamaxbytes);
}
/**
@@ -74,7 +75,8 @@ public function upload($saveas_filename, $maxbytes) {
* @param bool $overwriteexisting optional user has asked to overwrite the existing file
* @return object containing details of the file uploaded
*/
- public function process_upload($saveas_filename, $maxbytes, $types = '*', $savepath = '/', $itemid = 0, $license = null, $author = '', $overwriteexisting = false) {
+ public function process_upload($saveas_filename, $maxbytes, $types = '*', $savepath = '/', $itemid = 0,
+ $license = null, $author = '', $overwriteexisting = false, $areamaxbytes = -1) {
global $USER, $CFG;
if ((is_array($types) and in_array('*', $types)) or $types == '*') {
@@ -192,6 +194,10 @@ public function process_upload($saveas_filename, $maxbytes, $types = '*', $savep
$record->itemid = 0;
}
+ if (file_is_draft_area_limit_reached($record->itemid, $areamaxbytes, filesize($_FILES[$elname]['tmp_name']))) {
+ throw new file_exception('maxareabytes');
+ }
+
if (($maxbytes!==-1) && (filesize($_FILES[$elname]['tmp_name']) > $maxbytes)) {
throw new file_exception('maxbytes');
}
diff --git a/user/files.php b/user/files.php
index 973bbf5773c16..be694b5de9f82 100644
--- a/user/files.php
+++ b/user/files.php
@@ -53,7 +53,8 @@
$data = new stdClass();
$data->returnurl = $returnurl;
-$options = array('subdirs'=>1, 'maxbytes'=>$CFG->userquota, 'maxfiles'=>-1, 'accepted_types'=>'*');
+$options = array('subdirs' => 1, 'maxbytes' => $CFG->userquota, 'maxfiles' => -1, 'accepted_types' => '*',
+ 'areamaxbytes' => $CFG->userquota);
file_prepare_standard_filemanager($data, 'files', $options, $context, 'user', 'private', 0);
$mform = new user_files_form(null, array('data'=>$data, 'options'=>$options));
diff --git a/user/files_form.php b/user/files_form.php
index 55451daa8ce35..1cb3715a7e8f2 100644
--- a/user/files_form.php
+++ b/user/files_form.php
@@ -46,8 +46,7 @@ function validation($data, $files) {
$errors = array();
$draftitemid = $data['files_filemanager'];
- $fileinfo = file_get_draft_area_info($draftitemid);
- if ($fileinfo['filesize'] > $CFG->userquota) {
+ if (file_is_draft_area_limit_reached($draftitemid, $this->_customdata['options']['areamaxbytes'])) {
$errors['files_filemanager'] = get_string('userquotalimit', 'error');
}