MDL-14589 file api improvements - converting more params to general $options array

This commit is contained in:
skodak 2009-05-18 19:04:07 +00:00
parent a19a06d09e
commit 1b7f3041f0

View File

@ -320,8 +320,9 @@ function file_get_unused_draft_itemid() {
* @param integer $contextid This parameter and the next two identify the file area to copy files from.
* @param string $filearea helps indentify the file area.
* @param integer $itemid helps identify the file area. Can be null if there are no files yet.
* @param array $options text and file options ('subdirs'=>false, 'forcehttps'=>false)
* @param boolean $subdirs allow directory structure within the file area.
* @param string $text some html content that needs to have embedded links rewritten to point to the draft area.
* @param boolean $forcehttps force https urls.
* @return string if $text was passed in, the rewritten $text is returned. Otherwise NULL.
*/
function file_prepare_draft_area(&$draftitemid, $contextid, $filearea, $itemid, array $options=null, $text=null) {
@ -331,7 +332,7 @@ function file_prepare_draft_area(&$draftitemid, $contextid, $filearea, $itemid,
if (!isset($options['subdirs'])) {
$options['subdirs'] = false;
}
if (!isset($options['forcehttps'])) {
if (!isset($options['maxfiles'])) {
$options['forcehttps'] = false;
}
@ -369,17 +370,12 @@ function file_prepare_draft_area(&$draftitemid, $contextid, $filearea, $itemid,
* @param integer $contextid This parameter and the next two identify the file area to use.
* @param string $filearea helps indentify the file area.
* @param integer $itemid helps identify the file area.
* @param array $options text and file options ('forcehttps'=>false)
* @param boot $forcehttps if we should output a https URL.
* @return string the processed text.
*/
function file_rewrite_pluginfile_urls($text, $file, $contextid, $filearea, $itemid, array $options=null) {
function file_rewrite_pluginfile_urls($text, $file, $contextid, $filearea, $itemid, $forcehttps=false) {
global $CFG;
$options = (array)$options;
if (!isset($options['forcehttps'])) {
$options['forcehttps'] = false;
}
if (!$CFG->slasharguments) {
$file = $file . '?file=';
}
@ -390,7 +386,7 @@ function file_rewrite_pluginfile_urls($text, $file, $contextid, $filearea, $item
$baseurl .= "$itemid/";
}
if ($options['forcehttps']) {
if ($forcehttps) {
$baseurl = str_replace('http://', 'https://', $baseurl);
}