MDL-18111 improving file api comments and docs, fixing license header
authorskodak <skodak>
Thu, 21 May 2009 10:12:53 +0000 (10:12 +0000)
committerskodak <skodak>
Thu, 21 May 2009 10:12:53 +0000 (10:12 +0000)
files/draftfiles.php
files/index.php

index c6371d2f21139f9990466cd49688340b2f1d6ce5..61befdb279205489527bdae9fc49db9c412208be 100644 (file)
-<?php // $Id$
-
-    require('../config.php');
-    require_once($CFG->libdir.'/filelib.php');
-
-    $itemid     = required_param('itemid', PARAM_INT);
-    $filepath   = optional_param('filepath', '/', PARAM_PATH);
-    $newdirname = optional_param('newdirname', '', PARAM_FILE);
-    $delete     = optional_param('delete', '', PARAM_PATH);
-    $subdirs    = optional_param('subdirs', 0, PARAM_BOOL);
-    $maxbytes   = optional_param('maxbytes', 0, PARAM_INT);
-
-    require_login();
-    if (isguestuser()) {
-        print_error('noguest');
-    }
-
-    if (!$context = get_context_instance(CONTEXT_USER, $USER->id)) {
-        print_error('invalidcontext');
-    }
-
-    $notice = '';
-
-    $contextid = $context->id;
-    $filearea  = 'user_draft';
-
-    $browser = get_file_browser();
-    $fs      = get_file_storage();
-
-    if (!$subdirs) {
-        $filepath = '/';
-    }
-
-    if (!$directory = $fs->get_file($context->id, 'user_draft', $itemid, $filepath, '.')) {
-        $directory = new virtual_root_file($context->id, 'user_draft', $itemid);
-        $filepath = $directory->get_filepath();
-    }
-    $files = $fs->get_directory_files($context->id, 'user_draft', $itemid, $directory->get_filepath());
-    $parent = $directory->get_parent_directory();
-
-    $totalbytes = 0;
-    foreach ($files as $hash=>$file) {
-        if (!$subdirs and $file->get_filepath() !== '/') {
-            unset($files[$hash]);
-            continue;
-        }
-        $totalbytes += $file->get_filesize();
+<?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/>.
+
+/**
+ * Draft files management script used when javascript not available.
+ *
+ * @package    moodlecore
+ * @subpackage file
+ * @copyright  2008 Petr Skoda (http://skodak.org)
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
+require('../config.php');
+require_once($CFG->libdir.'/filelib.php');
+
+$itemid     = required_param('itemid', PARAM_INT);
+$filepath   = optional_param('filepath', '/', PARAM_PATH);
+$newdirname = optional_param('newdirname', '', PARAM_FILE);
+$delete     = optional_param('delete', '', PARAM_PATH);
+$subdirs    = optional_param('subdirs', 0, PARAM_BOOL);
+$maxbytes   = optional_param('maxbytes', 0, PARAM_INT);
+
+require_login();
+if (isguestuser()) {
+    print_error('noguest');
+}
+
+if (!$context = get_context_instance(CONTEXT_USER, $USER->id)) {
+    print_error('invalidcontext');
+}
+
+$notice = '';
+
+$contextid = $context->id;
+$filearea  = 'user_draft';
+
+$browser = get_file_browser();
+$fs      = get_file_storage();
+
+if (!$subdirs) {
+    $filepath = '/';
+}
+
+if (!$directory = $fs->get_file($context->id, 'user_draft', $itemid, $filepath, '.')) {
+    $directory = new virtual_root_file($context->id, 'user_draft', $itemid);
+    $filepath = $directory->get_filepath();
+}
+$files = $fs->get_directory_files($context->id, 'user_draft', $itemid, $directory->get_filepath());
+$parent = $directory->get_parent_directory();
+
+$totalbytes = 0;
+foreach ($files as $hash=>$file) {
+    if (!$subdirs and $file->get_filepath() !== '/') {
+        unset($files[$hash]);
+        continue;
     }
+    $totalbytes += $file->get_filesize();
+}
 
 /// process actions
-    if ($newdirname !== '' and data_submitted() and confirm_sesskey()) {
-        $newdirname = $directory->get_filepath().$newdirname.'/';
-        $fs->create_directory($contextid, $filearea, $itemid, $newdirname, $USER->id);
-        redirect('draftfiles.php?itemid='.$itemid.'&amp;filepath='.rawurlencode($newdirname).'&amp;subdirs='.$subdirs.'&amp;maxbytes='.$maxbytes);
-    }
-
-    if (isset($_FILES['newfile']) and data_submitted() and confirm_sesskey()) {
-        if (!empty($_FILES['newfile']['error'])) {
-            $notice = file_get_upload_error($_FILES['newfile']['error']);
-        } else {
-            $file = $_FILES['newfile'];
-            $newfilename = clean_param($file['name'], PARAM_FILE);
-            if (is_uploaded_file($_FILES['newfile']['tmp_name'])) {
-                if ($existingfile = $fs->get_file($contextid, $filearea, $itemid, $filepath, $newfilename)) {
-                    $existingfile->delete();
-                }
-                $filerecord = array('contextid'=>$contextid, 'filearea'=>$filearea, 'itemid'=>$itemid, 'filepath'=>$filepath,
-                                    'filename'=>$newfilename, 'userid'=>$USER->id);
-                $newfile = $fs->create_file_from_pathname($filerecord, $_FILES['newfile']['tmp_name']);
-                redirect('draftfiles.php?itemid='.$itemid.'&amp;filepath='.rawurlencode($filepath).'&amp;subdirs='.$subdirs.'&amp;maxbytes='.$maxbytes);
+if ($newdirname !== '' and data_submitted() and confirm_sesskey()) {
+    $newdirname = $directory->get_filepath().$newdirname.'/';
+    $fs->create_directory($contextid, $filearea, $itemid, $newdirname, $USER->id);
+    redirect('draftfiles.php?itemid='.$itemid.'&amp;filepath='.rawurlencode($newdirname).'&amp;subdirs='.$subdirs.'&amp;maxbytes='.$maxbytes);
+}
+
+if (isset($_FILES['newfile']) and data_submitted() and confirm_sesskey()) {
+    if (!empty($_FILES['newfile']['error'])) {
+        $notice = file_get_upload_error($_FILES['newfile']['error']);
+    } else {
+        $file = $_FILES['newfile'];
+        $newfilename = clean_param($file['name'], PARAM_FILE);
+        if (is_uploaded_file($_FILES['newfile']['tmp_name'])) {
+            if ($existingfile = $fs->get_file($contextid, $filearea, $itemid, $filepath, $newfilename)) {
+                $existingfile->delete();
             }
+            $filerecord = array('contextid'=>$contextid, 'filearea'=>$filearea, 'itemid'=>$itemid, 'filepath'=>$filepath,
+                                'filename'=>$newfilename, 'userid'=>$USER->id);
+            $newfile = $fs->create_file_from_pathname($filerecord, $_FILES['newfile']['tmp_name']);
+            redirect('draftfiles.php?itemid='.$itemid.'&amp;filepath='.rawurlencode($filepath).'&amp;subdirs='.$subdirs.'&amp;maxbytes='.$maxbytes);
         }
     }
+}
+
+if ($delete !== '' and $file = $fs->get_file($contextid, $filearea, $itemid, $filepath, $delete)) {
+    if (!data_submitted() or !confirm_sesskey()) {
+        print_header();
+        notify(get_string('deletecheckwarning').': '.s($file->get_filepath().$file->get_filename()));
+        $optionsno  = array('itemid'=>$itemid, 'filepath'=>$filepath, 'subdirs'=>$subdirs);
+        $optionsyes = array('itemid'=>$itemid, 'filepath'=>$filepath, 'delete'=>$delete, 'sesskey'=>sesskey(), 'subdirs'=>$subdirs);
+        notice_yesno (get_string('deletecheckfiles'), 'draftfiles.php', 'draftfiles.php', $optionsyes, $optionsno, 'post', 'get');
+        print_footer('empty');
+        die;
 
-    if ($delete !== '' and $file = $fs->get_file($contextid, $filearea, $itemid, $filepath, $delete)) {
-        if (!data_submitted() or !confirm_sesskey()) {
-            print_header();
-            notify(get_string('deletecheckwarning').': '.s($file->get_filepath().$file->get_filename()));
-            $optionsno  = array('itemid'=>$itemid, 'filepath'=>$filepath, 'subdirs'=>$subdirs);
-            $optionsyes = array('itemid'=>$itemid, 'filepath'=>$filepath, 'delete'=>$delete, 'sesskey'=>sesskey(), 'subdirs'=>$subdirs);
-            notice_yesno (get_string('deletecheckfiles'), 'draftfiles.php', 'draftfiles.php', $optionsyes, $optionsno, 'post', 'get');
-            print_footer('empty');
-            die;
-
+    } else {
+        $isdir = $file->is_directory();
+        $file->delete();
+        if ($isdir) {
+            redirect('draftfiles.php?itemid='.$itemid.'&amp;filepath='.rawurlencode($parent->get_filepath()).'&amp;subdirs='.$subdirs.'&amp;maxbytes='.$maxbytes);
         } else {
-            $isdir = $file->is_directory();
-            $file->delete();
-            if ($isdir) {
-                redirect('draftfiles.php?itemid='.$itemid.'&amp;filepath='.rawurlencode($parent->get_filepath()).'&amp;subdirs='.$subdirs.'&amp;maxbytes='.$maxbytes);
-            } else {
-                redirect('draftfiles.php?itemid='.$itemid.'&amp;filepath='.rawurlencode($filepath).'&amp;subdirs='.$subdirs.'&amp;maxbytes='.$maxbytes);
-            }
+            redirect('draftfiles.php?itemid='.$itemid.'&amp;filepath='.rawurlencode($filepath).'&amp;subdirs='.$subdirs.'&amp;maxbytes='.$maxbytes);
         }
     }
+}
 
-    print_header();
-
-    if ($notice !== '') {
-        notify($notice);
-    }
+print_header();
 
-    echo '<div class="areafiles">';
+if ($notice !== '') {
+    notify($notice);
+}
 
-    $strfolder   = get_string('folder');
-    $strfile     = get_string('file');
-    $strdownload = get_string('download');
-    $strdelete   = get_string('delete');
-
-    if ($parent) {
-        echo '<div class="folder">';
-        echo '<a href="draftfiles.php?itemid='.$itemid.'&amp;filepath='.$parent->get_filepath().'&amp;subdirs='.$subdirs.'&amp;maxbytes='.$maxbytes.'"><img src="'.$CFG->pixpath.'/f/parent.gif" class="icon" alt="" />&nbsp;'.get_string('parentfolder').'</a>';
-        echo '</div>';
-    }
+echo '<div class="areafiles">';
 
-    foreach ($files as $file) {
-        $filename    = $file->get_filename();
-        $filenameurl = rawurlencode($filename);
-        $filepath    = $file->get_filepath();
-        $filesize    = $file->get_filesize();
-        $filesize    = $filesize ? display_size($filesize) : '';
-
-        $mimetype = $file->get_mimetype();
-
-        if ($file->is_directory()) {
-            if ($subdirs) {
-                $dirname = explode('/', trim($filepath, '/'));
-                $dirname = array_pop($dirname);
-                echo '<div class="folder">';
-                echo "<a href=\"draftfiles.php?itemid=$itemid&amp;filepath=$filepath&amp;subdirs=$subdirs&amp;maxbytes=$maxbytes\"><img src=\"$CFG->pixpath/f/folder.gif\" class=\"icon\" alt=\"$strfolder\" />&nbsp;".s($dirname)."</a> ";
-                echo "<a href=\"draftfiles.php?itemid=$itemid&amp;filepath=$filepath&amp;delete=$filenameurl&amp;subdirs=$subdirs&amp;maxbytes=$maxbytes\"><img src=\"$CFG->pixpath/t/delete.gif\" class=\"iconsmall\" alt=\"$strdelete\" /></a>";
-                echo '</div>';
-            }
+$strfolder   = get_string('folder');
+$strfile     = get_string('file');
+$strdownload = get_string('download');
+$strdelete   = get_string('delete');
 
-        } else {
-            $icon    = mimeinfo_from_type('icon', $mimetype);
-            $viewurl = file_encode_url("$CFG->wwwroot/draftfile.php", "/$contextid/user_draft/$itemid".$filepath.$filename, false, false);
-            echo '<div class="file">';
-            echo "<a href=\"$viewurl\"><img src=\"$CFG->pixpath/f/$icon\" class=\"icon\" alt=\"$strfile\" />&nbsp;".s($filename)." ($filesize)</a> ";
-            echo "<a href=\"draftfiles.php?itemid=$itemid&amp;filepath=$filepath&amp;delete=$filenameurl&amp;subdirs=$subdirs&amp;maxbytes=$maxbytes\"><img src=\"$CFG->pixpath/t/delete.gif\" class=\"iconsmall\" alt=\"$strdelete\" /></a>";;
+if ($parent) {
+    echo '<div class="folder">';
+    echo '<a href="draftfiles.php?itemid='.$itemid.'&amp;filepath='.$parent->get_filepath().'&amp;subdirs='.$subdirs.'&amp;maxbytes='.$maxbytes.'"><img src="'.$CFG->pixpath.'/f/parent.gif" class="icon" alt="" />&nbsp;'.get_string('parentfolder').'</a>';
+    echo '</div>';
+}
+
+foreach ($files as $file) {
+    $filename    = $file->get_filename();
+    $filenameurl = rawurlencode($filename);
+    $filepath    = $file->get_filepath();
+    $filesize    = $file->get_filesize();
+    $filesize    = $filesize ? display_size($filesize) : '';
+
+    $mimetype = $file->get_mimetype();
+
+    if ($file->is_directory()) {
+        if ($subdirs) {
+            $dirname = explode('/', trim($filepath, '/'));
+            $dirname = array_pop($dirname);
+            echo '<div class="folder">';
+            echo "<a href=\"draftfiles.php?itemid=$itemid&amp;filepath=$filepath&amp;subdirs=$subdirs&amp;maxbytes=$maxbytes\"><img src=\"$CFG->pixpath/f/folder.gif\" class=\"icon\" alt=\"$strfolder\" />&nbsp;".s($dirname)."</a> ";
+            echo "<a href=\"draftfiles.php?itemid=$itemid&amp;filepath=$filepath&amp;delete=$filenameurl&amp;subdirs=$subdirs&amp;maxbytes=$maxbytes\"><img src=\"$CFG->pixpath/t/delete.gif\" class=\"iconsmall\" alt=\"$strdelete\" /></a>";
             echo '</div>';
         }
-    }
 
-    echo '</div>';
-
-    if ($maxbytes == 0 or $maxbytes > $totalbytes) {
-        echo '<form enctype="multipart/form-data" method="post" action="draftfiles.php"><div>';
-        if ($maxbytes) {
-            echo '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxbytes-$totalbytes).'" />';
-        }
-        echo '<input type="hidden" name="itemid" value="'.$itemid.'" />';
-        echo '<input type="hidden" name="filepath" value="'.s($filepath).'" />';
-        echo '<input type="hidden" name="subdirs" value="'.$subdirs.'" />';
-        echo '<input type="hidden" name="maxbytes" value="'.$maxbytes.'" />';
-        echo '<input type="hidden" name="sesskey" value="'.sesskey().'" />';
-        echo '<input name="newfile" type="file" />';
-        echo '<input type="submit" value="'.get_string('uploadafile').'" />';
-        if ($maxbytes) {
-            echo ' ('.get_string('maxsize', '', display_size(get_max_upload_file_size($CFG->maxbytes, $maxbytes-$totalbytes))).')';
-        } else {
-            echo ' ('.get_string('maxsize', '', display_size(get_max_upload_file_size($CFG->maxbytes))).')';
-        }
-        echo '</div></form>';
     } else {
-        //TODO: notify upload limit reached here
-        echo get_string('maxsize', '', display_size(get_max_upload_file_size($CFG->maxbytes, $maxbytes)));
+        $icon    = mimeinfo_from_type('icon', $mimetype);
+        $viewurl = file_encode_url("$CFG->wwwroot/draftfile.php", "/$contextid/user_draft/$itemid".$filepath.$filename, false, false);
+        echo '<div class="file">';
+        echo "<a href=\"$viewurl\"><img src=\"$CFG->pixpath/f/$icon\" class=\"icon\" alt=\"$strfile\" />&nbsp;".s($filename)." ($filesize)</a> ";
+        echo "<a href=\"draftfiles.php?itemid=$itemid&amp;filepath=$filepath&amp;delete=$filenameurl&amp;subdirs=$subdirs&amp;maxbytes=$maxbytes\"><img src=\"$CFG->pixpath/t/delete.gif\" class=\"iconsmall\" alt=\"$strdelete\" /></a>";;
+        echo '</div>';
     }
+}
+
+echo '</div>';
 
-    if ($subdirs) {
-        echo '<form action="draftfiles.php" method="post"><div>';
-        echo '<input type="hidden" name="itemid" value="'.$itemid.'" />';
-        echo '<input type="hidden" name="filepath" value="'.s($filepath).'" />';
-        echo '<input type="hidden" name="subdirs" value="'.$subdirs.'" />';
-        echo '<input type="hidden" name="maxbytes" value="'.$maxbytes.'" />';
-        echo '<input type="hidden" name="sesskey" value="'.sesskey().'" />';
-        echo '<input type="text" name="newdirname" value="" />';
-        echo '<input type="submit" value="'.get_string('makeafolder').'" />';
-        echo '</div></form>';
+if ($maxbytes == 0 or $maxbytes > $totalbytes) {
+    echo '<form enctype="multipart/form-data" method="post" action="draftfiles.php"><div>';
+    if ($maxbytes) {
+        echo '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxbytes-$totalbytes).'" />';
+    }
+    echo '<input type="hidden" name="itemid" value="'.$itemid.'" />';
+    echo '<input type="hidden" name="filepath" value="'.s($filepath).'" />';
+    echo '<input type="hidden" name="subdirs" value="'.$subdirs.'" />';
+    echo '<input type="hidden" name="maxbytes" value="'.$maxbytes.'" />';
+    echo '<input type="hidden" name="sesskey" value="'.sesskey().'" />';
+    echo '<input name="newfile" type="file" />';
+    echo '<input type="submit" value="'.get_string('uploadafile').'" />';
+    if ($maxbytes) {
+        echo ' ('.get_string('maxsize', '', display_size(get_max_upload_file_size($CFG->maxbytes, $maxbytes-$totalbytes))).')';
+    } else {
+        echo ' ('.get_string('maxsize', '', display_size(get_max_upload_file_size($CFG->maxbytes))).')';
     }
+    echo '</div></form>';
+} else {
+    //TODO: notify upload limit reached here
+    echo get_string('maxsize', '', display_size(get_max_upload_file_size($CFG->maxbytes, $maxbytes)));
+}
+
+if ($subdirs) {
+    echo '<form action="draftfiles.php" method="post"><div>';
+    echo '<input type="hidden" name="itemid" value="'.$itemid.'" />';
+    echo '<input type="hidden" name="filepath" value="'.s($filepath).'" />';
+    echo '<input type="hidden" name="subdirs" value="'.$subdirs.'" />';
+    echo '<input type="hidden" name="maxbytes" value="'.$maxbytes.'" />';
+    echo '<input type="hidden" name="sesskey" value="'.sesskey().'" />';
+    echo '<input type="text" name="newdirname" value="" />';
+    echo '<input type="submit" value="'.get_string('makeafolder').'" />';
+    echo '</div></form>';
+}
+
+print_footer('empty');
 
-    print_footer('empty');
 
-?>
index a7c65290fc39456e80ed9673263c7efc9988600c..d4006cd756ee943ef9c7b5374271bdc3cd1c1170 100644 (file)
-<?php // $Id$
-
-    require('../config.php');
-    require_once($CFG->libdir.'/filelib.php');
-    require_once($CFG->libdir.'/adminlib.php');
-
-    $courseid   = optional_param('id', 0, PARAM_INT);
-
-    $contextid  = optional_param('contextid', SYSCONTEXTID, PARAM_INT);
-    $filearea   = optional_param('filearea', '', PARAM_ALPHAEXT);
-    $itemid     = optional_param('itemid', -1, PARAM_INT);
-    $filepath   = optional_param('filepath', '', PARAM_PATH);
-    $filename   = optional_param('filename', '', PARAM_FILE);
-
-    $newdirname = optional_param('newdirname', '', PARAM_FILE);
-    $delete     = optional_param('delete', 0, PARAM_BOOL);
-
-    if ($courseid) {
-        if (!$course = $DB->get_record('course', array('id'=>$courseid))) {
-            print_error('invalidcourseid');
-        }
-        if (!$context = get_context_instance(CONTEXT_COURSE, $course->id)) {
-            print_error('invalidcontext');
-        }
-        redirect('index.php?contextid='.$context->id.'&amp;itemid=0&amp;filearea=course_content');
+<?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/>.
+
+/**
+ * Temporary file manager for all moodle files. To be replaced by something much better.
+ *
+ * @package    moodlecore
+ * @subpackage file
+ * @copyright  1999 onwards Martin Dougiamas (http://dougiamas.com)
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
+require('../config.php');
+require_once($CFG->libdir.'/filelib.php');
+require_once($CFG->libdir.'/adminlib.php');
+
+$courseid   = optional_param('id', 0, PARAM_INT);
+
+$contextid  = optional_param('contextid', SYSCONTEXTID, PARAM_INT);
+$filearea   = optional_param('filearea', '', PARAM_ALPHAEXT);
+$itemid     = optional_param('itemid', -1, PARAM_INT);
+$filepath   = optional_param('filepath', '', PARAM_PATH);
+$filename   = optional_param('filename', '', PARAM_FILE);
+
+$newdirname = optional_param('newdirname', '', PARAM_FILE);
+$delete     = optional_param('delete', 0, PARAM_BOOL);
+
+if ($courseid) {
+    if (!$course = $DB->get_record('course', array('id'=>$courseid))) {
+        print_error('invalidcourseid');
     }
-
-    if (!$context = get_context_instance_by_id($contextid)) {
+    if (!$context = get_context_instance(CONTEXT_COURSE, $course->id)) {
         print_error('invalidcontext');
     }
+    redirect('index.php?contextid='.$context->id.'&amp;itemid=0&amp;filearea=course_content');
+}
+
+if (!$context = get_context_instance_by_id($contextid)) {
+    print_error('invalidcontext');
+}
 
-    require_login();
-    require_capability('moodle/course:managefiles', $context);
+require_login();
+require_capability('moodle/course:managefiles', $context);
 
-    if ($filearea === '') {
-        $filearea = null;
-    }
+if ($filearea === '') {
+    $filearea = null;
+}
 
-    if ($itemid < 0) {
-        $itemid = null;
-    }
+if ($itemid < 0) {
+    $itemid = null;
+}
 
-    if ($filepath === '') {
-        $filepath = null;
-    }
+if ($filepath === '') {
+    $filepath = null;
+}
 
-    if ($filename === '') {
-        $filename = null;
-    }
+if ($filename === '') {
+    $filename = null;
+}
 
-    $error = '';
+$error = '';
 
-    $browser = get_file_browser();
+$browser = get_file_browser();
 
-    $file_info = $browser->get_file_info($context, $filearea, $itemid, $filepath, $filename);
+$file_info = $browser->get_file_info($context, $filearea, $itemid, $filepath, $filename);
 
 /// process actions
-    if ($file_info and $file_info->is_directory() and $file_info->is_writable() and $newdirname !== '' and data_submitted() and confirm_sesskey()) {
-        if ($newdir_info = $file_info->create_directory($newdirname, $USER->id)) {
-            $params = $newdir_info->get_params_rawencoded();
-            $params = implode('&amp;', $params);
-            redirect("index.php?$params");
-        } else {
-            $error = "Could not create new dir"; // TODO: localise
-        }
+if ($file_info and $file_info->is_directory() and $file_info->is_writable() and $newdirname !== '' and data_submitted() and confirm_sesskey()) {
+    if ($newdir_info = $file_info->create_directory($newdirname, $USER->id)) {
+        $params = $newdir_info->get_params_rawencoded();
+        $params = implode('&amp;', $params);
+        redirect("index.php?$params");
+    } else {
+        $error = "Could not create new dir"; // TODO: localise
     }
+}
 
-    if ($file_info and $file_info->is_directory() and $file_info->is_writable() and isset($_FILES['newfile']) and data_submitted() and confirm_sesskey()) {
-        $file = $_FILES['newfile'];
-        $newfilename = clean_param($file['name'], PARAM_FILE);
-        if (is_uploaded_file($_FILES['newfile']['tmp_name'])) {
-            try {
-                if ($newfile = $file_info->create_file_from_pathname($newfilename, $_FILES['newfile']['tmp_name'], $USER->id)) {
-                    $params = $file_info->get_params_rawencoded();
-                    $params = implode('&amp;', $params);
-                    redirect("index.php?$params");
+if ($file_info and $file_info->is_directory() and $file_info->is_writable() and isset($_FILES['newfile']) and data_submitted() and confirm_sesskey()) {
+    $file = $_FILES['newfile'];
+    $newfilename = clean_param($file['name'], PARAM_FILE);
+    if (is_uploaded_file($_FILES['newfile']['tmp_name'])) {
+        try {
+            if ($newfile = $file_info->create_file_from_pathname($newfilename, $_FILES['newfile']['tmp_name'], $USER->id)) {
+                $params = $file_info->get_params_rawencoded();
+                $params = implode('&amp;', $params);
+                redirect("index.php?$params");
 
-                } else {
-                    $error = "Could not create upload file"; // TODO: localise
-                }
-            } catch (file_exception $e) {
-                $error = "Exception: Could not create upload file"; // TODO: localise
+            } else {
+                $error = "Could not create upload file"; // TODO: localise
             }
+        } catch (file_exception $e) {
+            $error = "Exception: Could not create upload file"; // TODO: localise
         }
     }
+}
 
-    if ($file_info and $delete) {
-        if (!data_submitted() or !confirm_sesskey()) {
-            print_header();
-            notify(get_string('deletecheckwarning').': '.$file_info->get_visible_name());
-            $parent_info = $file_info->get_parent();
+if ($file_info and $delete) {
+    if (!data_submitted() or !confirm_sesskey()) {
+        print_header();
+        notify(get_string('deletecheckwarning').': '.$file_info->get_visible_name());
+        $parent_info = $file_info->get_parent();
 
-            $optionsno  = $parent_info->get_params();
-            $optionsyes = $file_info->get_params();
-            $optionsyes['delete'] = 1;
-            $optionsyes['sesskey'] = sesskey();
+        $optionsno  = $parent_info->get_params();
+        $optionsyes = $file_info->get_params();
+        $optionsyes['delete'] = 1;
+        $optionsyes['sesskey'] = sesskey();
 
-            notice_yesno (get_string('deletecheckfiles'), 'index.php', 'index.php', $optionsyes, $optionsno, 'post', 'get');
-            print_footer();
-            die;
-        }
+        notice_yesno (get_string('deletecheckfiles'), 'index.php', 'index.php', $optionsyes, $optionsno, 'post', 'get');
+        print_footer();
+        die;
+    }
 
-        if ($parent_info = $file_info->get_parent() and $parent_info->is_writable()) {
-            if (!$file_info->delete()) {
-                $error = "Could not delete file!"; // TODO: localise
-            }
-            $params = $parent_info->get_params_rawencoded();
-            $params = implode('&amp;', $params);
-            redirect("index.php?$params", $error);
+    if ($parent_info = $file_info->get_parent() and $parent_info->is_writable()) {
+        if (!$file_info->delete()) {
+            $error = "Could not delete file!"; // TODO: localise
         }
+        $params = $parent_info->get_params_rawencoded();
+        $params = implode('&amp;', $params);
+        redirect("index.php?$params", $error);
     }
+}
 
 
 /// print dir listing
-    html_header($context, $file_info);
+html_header($context, $file_info);
 
-    if ($error !== '') {
-        notify($error);
-    }
+if ($error !== '') {
+    notify($error);
+}
 
-    displaydir($file_info);
-
-    if ($file_info and $file_info->is_directory() and $file_info->is_writable()) {
-        echo '<br />';
-
-        echo '<form action="index.php" method="post"><div>';
-        echo '<input type="hidden" name="contextid" value="'.$contextid.'" />';
-        echo '<input type="hidden" name="filearea" value="'.$filearea.'" />';
-        echo '<input type="hidden" name="itemid" value="'.$itemid.'" />';
-        echo '<input type="hidden" name="filepath" value="'.s($filepath).'" />';
-        echo '<input type="hidden" name="filename" value="'.s($filename).'" />';
-        echo '<input type="hidden" name="sesskey" value="'.sesskey().'" />';
-        echo '<input type="text" name="newdirname" value="" />';
-        echo '<input type="submit" value="'.get_string('makeafolder').'" />';
-        echo '</div></form>';
-
-        echo '<br />';
-
-        echo '<form enctype="multipart/form-data" method="post" action="index.php"><div>';
-        echo '<input type="hidden" name="contextid" value="'.$contextid.'" />';
-        echo '<input type="hidden" name="filearea" value="'.$filearea.'" />';
-        echo '<input type="hidden" name="itemid" value="'.$itemid.'" />';
-        echo '<input type="hidden" name="filepath" value="'.s($filepath).'" />';
-        echo '<input type="hidden" name="filename" value="'.s($filename).'" />';
-        echo '<input type="hidden" name="sesskey" value="'.sesskey().'" />';
-        echo '<input name="newfile" type="file" />';
-        echo '<input type="submit" value="'.get_string('uploadafile').'" />';
-        echo '</div></form>';
-    }
+displaydir($file_info);
+
+if ($file_info and $file_info->is_directory() and $file_info->is_writable()) {
+    echo '<br />';
+
+    echo '<form action="index.php" method="post"><div>';
+    echo '<input type="hidden" name="contextid" value="'.$contextid.'" />';
+    echo '<input type="hidden" name="filearea" value="'.$filearea.'" />';
+    echo '<input type="hidden" name="itemid" value="'.$itemid.'" />';
+    echo '<input type="hidden" name="filepath" value="'.s($filepath).'" />';
+    echo '<input type="hidden" name="filename" value="'.s($filename).'" />';
+    echo '<input type="hidden" name="sesskey" value="'.sesskey().'" />';
+    echo '<input type="text" name="newdirname" value="" />';
+    echo '<input type="submit" value="'.get_string('makeafolder').'" />';
+    echo '</div></form>';
+
+    echo '<br />';
+
+    echo '<form enctype="multipart/form-data" method="post" action="index.php"><div>';
+    echo '<input type="hidden" name="contextid" value="'.$contextid.'" />';
+    echo '<input type="hidden" name="filearea" value="'.$filearea.'" />';
+    echo '<input type="hidden" name="itemid" value="'.$itemid.'" />';
+    echo '<input type="hidden" name="filepath" value="'.s($filepath).'" />';
+    echo '<input type="hidden" name="filename" value="'.s($filename).'" />';
+    echo '<input type="hidden" name="sesskey" value="'.sesskey().'" />';
+    echo '<input name="newfile" type="file" />';
+    echo '<input type="submit" value="'.get_string('uploadafile').'" />';
+    echo '</div></form>';
+}
 
-    html_footer();
+html_footer();
 
 /// UI functions /////////////////////////
 
@@ -309,4 +333,4 @@ function displaydir($file_info) {
 
 }
 
-?>
+