Merge branch 'master' of https://github.com/moodle/moodle
authorPaul Charsley <p.charsley@massey.ac.nz>
Tue, 14 Aug 2012 02:00:51 +0000 (14:00 +1200)
committerPaul Charsley <p.charsley@massey.ac.nz>
Tue, 14 Aug 2012 02:00:51 +0000 (14:00 +1200)
files/externallib.php

index f6001d2..68ec099 100644 (file)
@@ -1,33 +1,33 @@
-<?php
-// This file is part of Moodle - http://moodle.org/
-//
-// Moodle is free software: you can redistribute it and/or modify
-// it under the terms of the GNU General Public License as published by
-// the Free Software Foundation, either version 3 of the License, or
-// (at your option) any later version.
-//
-// Moodle is distributed in the hope that it will be useful,
-// but WITHOUT ANY WARRANTY; without even the implied warranty of
-// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-// GNU General Public License for more details.
-//
-// You should have received a copy of the GNU General Public License
-// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
-
-
-/**
- * External files API
- *
+<?php\r
+// This file is part of Moodle - http://moodle.org/\r
+//\r
+// Moodle is free software: you can redistribute it and/or modify\r
+// it under the terms of the GNU General Public License as published by\r
+// the Free Software Foundation, either version 3 of the License, or\r
+// (at your option) any later version.\r
+//\r
+// Moodle is distributed in the hope that it will be useful,\r
+// but WITHOUT ANY WARRANTY; without even the implied warranty of\r
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the\r
+// GNU General Public License for more details.\r
+//\r
+// You should have received a copy of the GNU General Public License\r
+// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.\r
+\r
+
+/**\r
+ * External files API\r
+ *\r
  * @package    core_files
  * @category   external
  * @copyright  2010 Dongsheng Cai
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-require_once("$CFG->libdir/externallib.php");
-require_once("$CFG->libdir/filelib.php");
-
-/**
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later\r
+ */\r
+\r
+require_once("$CFG->libdir/externallib.php");\r
+require_once("$CFG->libdir/filelib.php");\r
+\r
+/**\r
  * Files external functions
  *
  * @package    core_files
@@ -35,31 +35,31 @@ require_once("$CFG->libdir/filelib.php");
  * @copyright  2011 Jerome Mouneyrac
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  * @since Moodle 2.2
- */
-class core_files_external extends external_api {
-
-    /**
-     * Returns description of get_files parameters
+ */\r
+class core_files_external extends external_api {\r
+\r
+    /**\r
+     * Returns description of get_files parameters\r
      *
-     * @return external_function_parameters
+     * @return external_function_parameters\r
      * @since Moodle 2.2
-     */
-    public static function get_files_parameters() {
-        return new external_function_parameters(
-            array(
-                'contextid' => new external_value(PARAM_INT, 'context id'),
-                'component' => new external_value(PARAM_TEXT, 'component'),
-                'filearea'  => new external_value(PARAM_TEXT, 'file area'),
-                'itemid'    => new external_value(PARAM_INT, 'associated id'),
-                'filepath'  => new external_value(PARAM_PATH, 'file path'),
+     */\r
+    public static function get_files_parameters() {\r
+        return new external_function_parameters(\r
+            array(\r
+                'contextid' => new external_value(PARAM_INT, 'context id'),\r
+                'component' => new external_value(PARAM_TEXT, 'component'),\r
+                'filearea'  => new external_value(PARAM_TEXT, 'file area'),\r
+                'itemid'    => new external_value(PARAM_INT, 'associated id'),\r
+                'filepath'  => new external_value(PARAM_PATH, 'file path'),\r
                 'filename'  => new external_value(PARAM_FILE, 'file name'),
                 'modified' => new external_value(PARAM_INT, 'timestamp to return files changed after this time.', VALUE_DEFAULT, null)
-            )
-        );
-    }
-
-    /**
-     * Return moodle files listing
+            )\r
+        );\r
+    }\r
+\r
+    /**\r
+     * Return moodle files listing\r
      *
      * @param int $contextid context id
      * @param int $component component
@@ -68,59 +68,59 @@ class core_files_external extends external_api {
      * @param string $filepath file path
      * @param string $filename file name
      * @param int $modified timestamp to return files changed after this time.
-     * @return array
+     * @return array\r
      * @since Moodle 2.2
-     */
+     */\r
     public static function get_files($contextid, $component, $filearea, $itemid, $filepath, $filename, $modified = null) {
-        global $CFG, $USER, $OUTPUT;
+        global $CFG, $USER, $OUTPUT;\r
         $fileinfo = self::validate_parameters(self::get_files_parameters(), array(
                     'contextid'=>$contextid, 'component'=>$component, 'filearea'=>$filearea,
                     'itemid'=>$itemid, 'filepath'=>$filepath, 'filename'=>$filename, 'modified'=>$modified));
-
-        $browser = get_file_browser();
-
-        if (empty($fileinfo['contextid'])) {
-            $context  = get_system_context();
-        } else {
-            $context  = get_context_instance_by_id($fileinfo['contextid']);
-        }
-        if (empty($fileinfo['component'])) {
-            $fileinfo['component'] = null;
-        }
-        if (empty($fileinfo['filearea'])) {
-            $fileinfo['filearea'] = null;
-        }
-        if (empty($fileinfo['itemid'])) {
-            $fileinfo['itemid'] = null;
-        }
-        if (empty($fileinfo['filename'])) {
-            $fileinfo['filename'] = null;
-        }
-        if (empty($fileinfo['filepath'])) {
-            $fileinfo['filepath'] = null;
-        }
-
-        $return = array();
-        $return['parents'] = array();
-        $return['files'] = array();
+\r
+        $browser = get_file_browser();\r
+\r
+        if (empty($fileinfo['contextid'])) {\r
+            $context  = get_system_context();\r
+        } else {\r
+            $context  = get_context_instance_by_id($fileinfo['contextid']);\r
+        }\r
+        if (empty($fileinfo['component'])) {\r
+            $fileinfo['component'] = null;\r
+        }\r
+        if (empty($fileinfo['filearea'])) {\r
+            $fileinfo['filearea'] = null;\r
+        }\r
+        if (empty($fileinfo['itemid'])) {\r
+            $fileinfo['itemid'] = null;\r
+        }\r
+        if (empty($fileinfo['filename'])) {\r
+            $fileinfo['filename'] = null;\r
+        }\r
+        if (empty($fileinfo['filepath'])) {\r
+            $fileinfo['filepath'] = null;\r
+        }\r
+\r
+        $return = array();\r
+        $return['parents'] = array();\r
+        $return['files'] = array();\r
         $list = array();
         if ($file = $browser->get_file_info(
             $context, $fileinfo['component'], $fileinfo['filearea'], $fileinfo['itemid'],
                 $fileinfo['filepath'], $fileinfo['filename'])) {
-            $level = $file->get_parent();
-            while ($level) {
-                $params = $level->get_params();
-                $params['filename'] = $level->get_visible_name();
-                array_unshift($return['parents'], $params);
-                $level = $level->get_parent();
-            }
-            $children = $file->get_children();
-            foreach ($children as $child) {
-
-                $params = $child->get_params();
+            $level = $file->get_parent();\r
+            while ($level) {\r
+                $params = $level->get_params();\r
+                $params['filename'] = $level->get_visible_name();\r
+                array_unshift($return['parents'], $params);\r
+                $level = $level->get_parent();\r
+            }\r
+            $children = $file->get_children();\r
+            foreach ($children as $child) {\r
+\r
+                $params = $child->get_params();\r
                 $timemodified = $child->get_timemodified();
-
-                if ($child->is_directory()) {
+\r
+                if ($child->is_directory()) {\r
                     if ((is_null($modified)) or ($modified < $timemodified)) {
                         $node = array(
                             'contextid' => $params['contextid'],
@@ -135,7 +135,7 @@ class core_files_external extends external_api {
                            );
                            $list[] = $node;
                     }
-                } else {
+                } else {\r
                     if ((is_null($modified)) or ($modified < $timemodified)) {
                         $node = array(
                             'contextid' => $params['contextid'],
@@ -150,76 +150,76 @@ class core_files_external extends external_api {
                         );
                            $list[] = $node;
                     }
-                }
-            }
-        }
-        $return['files'] = $list;
-        return $return;
-    }
-
-    /**
-     * Returns description of get_files returns
+                }\r
+            }\r
+        }\r
+        $return['files'] = $list;\r
+        return $return;\r
+    }\r
+\r
+    /**\r
+     * Returns description of get_files returns\r
      *
      * @return external_single_structure
      * @since Moodle 2.2
-     */
-    public static function get_files_returns() {
-        return new external_single_structure(
-            array(
-                'parents' => new external_multiple_structure(
-                    new external_single_structure(
-                        array(
-                            'contextid' => new external_value(PARAM_INT, ''),
-                            'component' => new external_value(PARAM_COMPONENT, ''),
-                            'filearea'  => new external_value(PARAM_AREA, ''),
-                            'itemid'    => new external_value(PARAM_INT, ''),
-                            'filepath'  => new external_value(PARAM_TEXT, ''),
-                            'filename'  => new external_value(PARAM_TEXT, ''),
-                        )
-                    )
-                ),
-                'files' => new external_multiple_structure(
-                    new external_single_structure(
-                        array(
-                            'contextid' => new external_value(PARAM_INT, ''),
-                            'component' => new external_value(PARAM_COMPONENT, ''),
-                            'filearea'  => new external_value(PARAM_AREA, ''),
-                            'itemid'   => new external_value(PARAM_INT, ''),
-                            'filepath' => new external_value(PARAM_TEXT, ''),
-                            'filename' => new external_value(PARAM_FILE, ''),
-                            'isdir'    => new external_value(PARAM_BOOL, ''),
-                            'url'      => new external_value(PARAM_TEXT, ''),
+     */\r
+    public static function get_files_returns() {\r
+        return new external_single_structure(\r
+            array(\r
+                'parents' => new external_multiple_structure(\r
+                    new external_single_structure(\r
+                        array(\r
+                            'contextid' => new external_value(PARAM_INT, ''),\r
+                            'component' => new external_value(PARAM_COMPONENT, ''),\r
+                            'filearea'  => new external_value(PARAM_AREA, ''),\r
+                            'itemid'    => new external_value(PARAM_INT, ''),\r
+                            'filepath'  => new external_value(PARAM_TEXT, ''),\r
+                            'filename'  => new external_value(PARAM_TEXT, ''),\r
+                        )\r
+                    )\r
+                ),\r
+                'files' => new external_multiple_structure(\r
+                    new external_single_structure(\r
+                        array(\r
+                            'contextid' => new external_value(PARAM_INT, ''),\r
+                            'component' => new external_value(PARAM_COMPONENT, ''),\r
+                            'filearea'  => new external_value(PARAM_AREA, ''),\r
+                            'itemid'   => new external_value(PARAM_INT, ''),\r
+                            'filepath' => new external_value(PARAM_TEXT, ''),\r
+                            'filename' => new external_value(PARAM_FILE, ''),\r
+                            'isdir'    => new external_value(PARAM_BOOL, ''),\r
+                            'url'      => new external_value(PARAM_TEXT, ''),\r
                             'timemodified' => new external_value(PARAM_INT, ''),
-                        )
-                    )
-                )
-            )
-        );
-    }
-
-    /**
-     * Returns description of upload parameters
+                        )\r
+                    )\r
+                )\r
+            )\r
+        );\r
+    }\r
+\r
+    /**\r
+     * Returns description of upload parameters\r
      *
-     * @return external_function_parameters
+     * @return external_function_parameters\r
      * @since Moodle 2.2
-     */
-    public static function upload_parameters() {
-        return new external_function_parameters(
-            array(
-                'contextid' => new external_value(PARAM_INT, 'context id'),
-                'component' => new external_value(PARAM_COMPONENT, 'component'),
-                'filearea'  => new external_value(PARAM_AREA, 'file area'),
-                'itemid'    => new external_value(PARAM_INT, 'associated id'),
-                'filepath'  => new external_value(PARAM_PATH, 'file path'),
-                'filename'  => new external_value(PARAM_FILE, 'file name'),
-                'filecontent' => new external_value(PARAM_TEXT, 'file content')
-            )
-        );
-    }
-
-    /**
-     * Uploading a file to moodle
-     *
+     */\r
+    public static function upload_parameters() {\r
+        return new external_function_parameters(\r
+            array(\r
+                'contextid' => new external_value(PARAM_INT, 'context id'),\r
+                'component' => new external_value(PARAM_COMPONENT, 'component'),\r
+                'filearea'  => new external_value(PARAM_AREA, 'file area'),\r
+                'itemid'    => new external_value(PARAM_INT, 'associated id'),\r
+                'filepath'  => new external_value(PARAM_PATH, 'file path'),\r
+                'filename'  => new external_value(PARAM_FILE, 'file name'),\r
+                'filecontent' => new external_value(PARAM_TEXT, 'file content')\r
+            )\r
+        );\r
+    }\r
+\r
+    /**\r
+     * Uploading a file to moodle\r
+     *\r
      * @param int $contextid context id
      * @param string $component component
      * @param string $filearea file aera
@@ -227,112 +227,112 @@ class core_files_external extends external_api {
      * @param string $filepath file path
      * @param string $filename file name
      * @param string $filecontent file content
-     * @return array
+     * @return array\r
      * @since Moodle 2.2
-     */
-    public static function upload($contextid, $component, $filearea, $itemid, $filepath, $filename, $filecontent) {
-        global $USER, $CFG;
-
+     */\r
+    public static function upload($contextid, $component, $filearea, $itemid, $filepath, $filename, $filecontent) {\r
+        global $USER, $CFG;\r
+\r
         $fileinfo = self::validate_parameters(self::upload_parameters(), array(
             'contextid'=>$contextid, 'component'=>$component, 'filearea'=>$filearea, 'itemid'=>$itemid,
             'filepath'=>$filepath, 'filename'=>$filename, 'filecontent'=>$filecontent));
-
-        if (!isset($fileinfo['filecontent'])) {
-            throw new moodle_exception('nofile');
-        }
+\r
+        if (!isset($fileinfo['filecontent'])) {\r
+            throw new moodle_exception('nofile');\r
+        }\r
         // Saving file.
-        $dir = make_temp_directory('wsupload');
-
-        if (empty($fileinfo['filename'])) {
-            $filename = uniqid('wsupload', true).'_'.time().'.tmp';
-        } else {
-            $filename = $fileinfo['filename'];
-        }
-
-        if (file_exists($dir.$filename)) {
-            $savedfilepath = $dir.uniqid('m').$filename;
-        } else {
-            $savedfilepath = $dir.$filename;
-        }
-
-        file_put_contents($savedfilepath, base64_decode($fileinfo['filecontent']));
-        unset($fileinfo['filecontent']);
-
-        if (!empty($fileinfo['filepath'])) {
-            $filepath = $fileinfo['filepath'];
-        } else {
-            $filepath = '/';
-        }
-
-        if (isset($fileinfo['itemid'])) {
+        $dir = make_temp_directory('wsupload');\r
+\r
+        if (empty($fileinfo['filename'])) {\r
+            $filename = uniqid('wsupload', true).'_'.time().'.tmp';\r
+        } else {\r
+            $filename = $fileinfo['filename'];\r
+        }\r
+\r
+        if (file_exists($dir.$filename)) {\r
+            $savedfilepath = $dir.uniqid('m').$filename;\r
+        } else {\r
+            $savedfilepath = $dir.$filename;\r
+        }\r
+\r
+        file_put_contents($savedfilepath, base64_decode($fileinfo['filecontent']));\r
+        unset($fileinfo['filecontent']);\r
+\r
+        if (!empty($fileinfo['filepath'])) {\r
+            $filepath = $fileinfo['filepath'];\r
+        } else {\r
+            $filepath = '/';\r
+        }\r
+\r
+        if (isset($fileinfo['itemid'])) {\r
             // TODO MDL-31116 in user private area, itemid is always 0.
-            $itemid = 0;
-        } else {
-            throw new coding_exception('itemid cannot be empty');
-        }
-
-        if (!empty($fileinfo['contextid'])) {
-            $context = get_context_instance_by_id($fileinfo['contextid']);
-        } else {
-            $context = get_system_context();
-        }
-
-        if (!($fileinfo['component'] == 'user' and $fileinfo['filearea'] == 'private')) {
-            throw new coding_exception('File can be uploaded to user private area only');
-        } else {
+            $itemid = 0;\r
+        } else {\r
+            throw new coding_exception('itemid cannot be empty');\r
+        }\r
+\r
+        if (!empty($fileinfo['contextid'])) {\r
+            $context = get_context_instance_by_id($fileinfo['contextid']);\r
+        } else {\r
+            $context = get_system_context();\r
+        }\r
+\r
+        if (!($fileinfo['component'] == 'user' and $fileinfo['filearea'] == 'private')) {\r
+            throw new coding_exception('File can be uploaded to user private area only');\r
+        } else {\r
             // TODO MDL-31116 hard-coded to use user_private area.
-            $component = 'user';
-            $filearea = 'private';
-        }
-
-        $browser = get_file_browser();
-
+            $component = 'user';\r
+            $filearea = 'private';\r
+        }\r
+\r
+        $browser = get_file_browser();\r
+\r
         // Check existing file.
-        if ($file = $browser->get_file_info($context, $component, $filearea, $itemid, $filepath, $filename)) {
-            throw new moodle_exception('fileexist');
-        }
-
+        if ($file = $browser->get_file_info($context, $component, $filearea, $itemid, $filepath, $filename)) {\r
+            throw new moodle_exception('fileexist');\r
+        }\r
+\r
         // Move file to filepool.
-        if ($dir = $browser->get_file_info($context, $component, $filearea, $itemid, $filepath, '.')) {
-            $info = $dir->create_file_from_pathname($filename, $savedfilepath);
-            $params = $info->get_params();
-            unlink($savedfilepath);
-            return array(
-                'contextid'=>$params['contextid'],
-                'component'=>$params['component'],
-                'filearea'=>$params['filearea'],
-                'itemid'=>$params['itemid'],
-                'filepath'=>$params['filepath'],
-                'filename'=>$params['filename'],
-                'url'=>$info->get_url()
-                );
-        } else {
-            throw new moodle_exception('nofile');
-        }
-    }
-
-    /**
-     * Returns description of upload returns
+        if ($dir = $browser->get_file_info($context, $component, $filearea, $itemid, $filepath, '.')) {\r
+            $info = $dir->create_file_from_pathname($filename, $savedfilepath);\r
+            $params = $info->get_params();\r
+            unlink($savedfilepath);\r
+            return array(\r
+                'contextid'=>$params['contextid'],\r
+                'component'=>$params['component'],\r
+                'filearea'=>$params['filearea'],\r
+                'itemid'=>$params['itemid'],\r
+                'filepath'=>$params['filepath'],\r
+                'filename'=>$params['filename'],\r
+                'url'=>$info->get_url()\r
+                );\r
+        } else {\r
+            throw new moodle_exception('nofile');\r
+        }\r
+    }\r
+\r
+    /**\r
+     * Returns description of upload returns\r
      *
      * @return external_single_structure
      * @since Moodle 2.2
-     */
-    public static function upload_returns() {
-        return new external_single_structure(
-             array(
-                 'contextid' => new external_value(PARAM_INT, ''),
-                 'component' => new external_value(PARAM_COMPONENT, ''),
-                 'filearea'  => new external_value(PARAM_AREA, ''),
-                 'itemid'   => new external_value(PARAM_INT, ''),
-                 'filepath' => new external_value(PARAM_TEXT, ''),
-                 'filename' => new external_value(PARAM_FILE, ''),
-                 'url'      => new external_value(PARAM_TEXT, ''),
-             )
-        );
-    }
-}
-
-/**
+     */\r
+    public static function upload_returns() {\r
+        return new external_single_structure(\r
+             array(\r
+                 'contextid' => new external_value(PARAM_INT, ''),\r
+                 'component' => new external_value(PARAM_COMPONENT, ''),\r
+                 'filearea'  => new external_value(PARAM_AREA, ''),\r
+                 'itemid'   => new external_value(PARAM_INT, ''),\r
+                 'filepath' => new external_value(PARAM_TEXT, ''),\r
+                 'filename' => new external_value(PARAM_FILE, ''),\r
+                 'url'      => new external_value(PARAM_TEXT, ''),\r
+             )\r
+        );\r
+    }\r
+}\r
+\r
+/**\r
  * Deprecated files external functions
  *
  * @package    core_files
@@ -342,97 +342,97 @@ class core_files_external extends external_api {
  * @deprecated Moodle 2.2 MDL-29106 - Please do not use this class any more.
  * @todo MDL-31194 This will be deleted in Moodle 2.5.
  * @see core_files_external
- */
-class moodle_file_external extends external_api {
-
-    /**
-     * Returns description of get_files parameters
+ */\r
+class moodle_file_external extends external_api {\r
+\r
+    /**\r
+     * Returns description of get_files parameters\r
      *
-     * @return external_function_parameters
+     * @return external_function_parameters\r
      * @since Moodle 2.0
      * @deprecated Moodle 2.2 MDL-29106 - Please do not call this function any more.
      * @todo MDL-31194 This will be deleted in Moodle 2.5.
      * @see core_files_external::get_files_parameters()
-     */
-    public static function get_files_parameters() {
-        return core_files_external::get_files_parameters();
-    }
-
-    /**
-     * Return moodle files listing
+     */\r
+    public static function get_files_parameters() {\r
+        return core_files_external::get_files_parameters();\r
+    }\r
+\r
+    /**\r
+     * Return moodle files listing\r
      *
-     * @param int $contextid
-     * @param int $component
-     * @param int $filearea
-     * @param int $itemid
-     * @param string $filepath
-     * @param string $filename
-     * @return array
+     * @param int $contextid\r
+     * @param int $component\r
+     * @param int $filearea\r
+     * @param int $itemid\r
+     * @param string $filepath\r
+     * @param string $filename\r
+     * @return array\r
      * @since Moodle 2.0
      * @deprecated Moodle 2.2 MDL-29106 - Please do not call this function any more.
      * @todo MDL-31194 This will be deleted in Moodle 2.5.
      * @see core_files_external::get_files()
-     */
-    public static function get_files($contextid, $component, $filearea, $itemid, $filepath, $filename) {
-        return core_files_external::get_files($contextid, $component, $filearea, $itemid, $filepath, $filename);
-    }
-
-    /**
-     * Returns description of get_files returns
+     */\r
+    public static function get_files($contextid, $component, $filearea, $itemid, $filepath, $filename) {\r
+        return core_files_external::get_files($contextid, $component, $filearea, $itemid, $filepath, $filename);\r
+    }\r
+\r
+    /**\r
+     * Returns description of get_files returns\r
      *
      * @return external_single_structure
      * @since Moodle 2.0
      * @deprecated Moodle 2.2 MDL-29106 - Please do not call this function any more.
      * @todo MDL-31194 This will be deleted in Moodle 2.5.
      * @see core_files_external::get_files_returns()
-     */
-    public static function get_files_returns() {
-        return core_files_external::get_files_returns();
-    }
-
-    /**
-     * Returns description of upload parameters
+     */\r
+    public static function get_files_returns() {\r
+        return core_files_external::get_files_returns();\r
+    }\r
+\r
+    /**\r
+     * Returns description of upload parameters\r
      *
-     * @return external_function_parameters
+     * @return external_function_parameters\r
      * @since Moodle 2.0
      * @deprecated Moodle 2.2 MDL-29106 - Please do not call this function any more.
      * @todo MDL-31194 This will be deleted in Moodle 2.5.
      * @see core_files_external::upload_parameters()
-     */
-    public static function upload_parameters() {
-        return core_files_external::upload_parameters();
-    }
-
-    /**
-     * Uploading a file to moodle
+     */\r
+    public static function upload_parameters() {\r
+        return core_files_external::upload_parameters();\r
+    }\r
+\r
+    /**\r
+     * Uploading a file to moodle\r
      *
-     * @param int $contextid
-     * @param string $component
-     * @param string $filearea
-     * @param int $itemid
-     * @param string $filepath
-     * @param string $filename
-     * @param string $filecontent
-     * @return array
+     * @param int $contextid\r
+     * @param string $component\r
+     * @param string $filearea\r
+     * @param int $itemid\r
+     * @param string $filepath\r
+     * @param string $filename\r
+     * @param string $filecontent\r
+     * @return array\r
      * @since Moodle 2.0
      * @deprecated Moodle 2.2 MDL-29106 - Please do not call this function any more.
      * @todo MDL-31194 This will be deleted in Moodle 2.5.
      * @see core_files_external::upload()
-     */
-    public static function upload($contextid, $component, $filearea, $itemid, $filepath, $filename, $filecontent) {
-        return core_files_external::upload($contextid, $component, $filearea, $itemid, $filepath, $filename, $filecontent);
-    }
-
-    /**
-     * Returns description of upload returns
+     */\r
+    public static function upload($contextid, $component, $filearea, $itemid, $filepath, $filename, $filecontent) {\r
+        return core_files_external::upload($contextid, $component, $filearea, $itemid, $filepath, $filename, $filecontent);\r
+    }\r
+\r
+    /**\r
+     * Returns description of upload returns\r
      *
      * @return external_single_structure
      * @since Moodle 2.0
      * @deprecated Moodle 2.2 MDL-29106 - Please do not call this function any more.
      * @todo MDL-31194 This will be deleted in Moodle 2.5.
      * @see core_files_external::upload_returns()
-     */
-    public static function upload_returns() {
-        return core_files_external::upload_returns();
-    }
-}
+     */\r
+    public static function upload_returns() {\r
+        return core_files_external::upload_returns();\r
+    }\r
+}\r