From 22060485bf91aff23ac094b5a42d1e69ec1525bb Mon Sep 17 00:00:00 2001
From: moodler <moodler>
Date: Wed, 12 Apr 2006 08:53:24 +0000
Subject: [PATCH] FIx notices and logic when deleting files and checking
 resources.   bug 5156

---
 files/index.php | 48 +++++++++++++++++++++++++++---------------------
 1 file changed, 27 insertions(+), 21 deletions(-)

diff --git a/files/index.php b/files/index.php
index 7e36b89f7fa..1eebf667044 100644
--- a/files/index.php
+++ b/files/index.php
@@ -204,10 +204,12 @@
         case "delete":
             if (!empty($confirm) and confirm_sesskey()) {
                 html_header($course, $wdir);
-                foreach ($USER->filelist as $file) {
-                    $fullfile = $basedir.$file;
-                    if (! fulldelete($fullfile)) {
-                        echo "<br />Error: Could not delete: $fullfile";
+                if (!empty($USER->filelist)) {
+                    foreach ($USER->filelist as $file) {
+                        $fullfile = $basedir.$file;
+                        if (! fulldelete($fullfile)) {
+                            echo "<br />Error: Could not delete: $fullfile";
+                        }
                     }
                 }
                 clearfilelist();
@@ -217,27 +219,31 @@
             } else {
                 html_header($course, $wdir);
 
-                echo '<p align=\"center\">'.get_string('deletecheckwarning').':</p>';
-
-                print_simple_box_start("center");
-                printfilelist($USER->filelist);
-                print_simple_box_end();
-                echo "<br />";
-                print_simple_box_start("center");
-
                 if (setfilelist($_POST)) {
-                    foreach ($USER->filelist as $file) {
-                        //if file is part of resource then update resource table as well
-                        $clean_name = substr($file, 1);
-                        if (record_exists('resource', 'reference', $clean_name)) {
-                            $resource_id = files_get_cm_from_resource_name($clean_name);
-                            echo '<p align=\"center\"><b>'.get_string('warningdeleteresource', '', $file)."</align><a href='$CFG->wwwroot/course/mod.php?update=$resource_id&sesskey=$USER->sesskey'> Update</a></b>";
-                        }
-                    }
-
+                    notify(get_string('deletecheckwarning').':');
+                    print_simple_box_start("center");
+                    printfilelist($USER->filelist);
                     print_simple_box_end();
                     echo "<br />";
 
+                    $resourcelist = false;
+                    foreach ($USER->filelist as $file) {
+                        // If file is specified in a resource, then delete that too.
+                        $clean_name = substr($file, 1);
+                        if (record_exists('resource', 'reference', $clean_name)) {
+                            if (!$resourcelist) {
+                                print_simple_box_start("center");
+                                $resourcelist = true;
+                            }
+                            $resource_id = files_get_cm_from_resource_name($clean_name);
+                            echo '<p>'.get_string('warningdeleteresource', '', $file)." <a href='$CFG->wwwroot/course/mod.php?update=$resource_id&sesskey=$USER->sesskey'>".get_string('update')."</a></p>";
+                        }
+                    }
+                    if ($resourcelist) {
+                        print_simple_box_end();
+                        echo "<br />";
+                    }
+
                     notice_yesno (get_string("deletecheckfiles"), 
                                 "index.php?id=$id&amp;wdir=$wdir&amp;action=delete&amp;confirm=1&amp;sesskey=$USER->sesskey&amp;choose=$choose",
                                 "index.php?id=$id&amp;wdir=$wdir&amp;action=cancel&amp;choose=$choose");