From 930f2962af93081b40dc32aec3d1f59d8fd52cdd Mon Sep 17 00:00:00 2001
From: skodak <skodak>
Date: Sat, 16 Aug 2008 18:24:03 +0000
Subject: [PATCH] MDL-15904 simplified moddata cleanup

---
 mod/assignment/db/upgrade.php | 26 +++-----------------------
 1 file changed, 3 insertions(+), 23 deletions(-)

diff --git a/mod/assignment/db/upgrade.php b/mod/assignment/db/upgrade.php
index ebfa5aee4ba..f10fe77c84c 100644
--- a/mod/assignment/db/upgrade.php
+++ b/mod/assignment/db/upgrade.php
@@ -51,9 +51,6 @@ function xmldb_assignment_upgrade($oldversion) {
 
         $count = $DB->count_records_sql("SELECT 'x' $sqlfrom"); 
 
-        $lastcourse     = 0;
-        $lastassignment = 0;
-
         if ($rs = $DB->get_recordset_sql("SELECT s.id, s.userid, s.teacher, s.assignment, a.course, cm.id AS cmid $sqlfrom")) {
 
             $pbar = new progress_bar('migrateassignmentfiles', 500, true);
@@ -126,31 +123,14 @@ function xmldb_assignment_upgrade($oldversion) {
                     @rmdir("$CFG->dataroot/$submission->course/$CFG->moddata/assignment/$submission->assignment/$submission->userid/responses");
                 }
 
+                // remove dirs if empty
                 @rmdir("$CFG->dataroot/$submission->course/$CFG->moddata/assignment/$submission->assignment/$submission->userid");
-
-                if ($lastassignment and $lastassignment != $submission->assignment) {
-                    @rmdir("$CFG->dataroot/$lastcourse/$CFG->moddata/assignment/$lastassignment");
-                }
-
-                if ($lastcourse and $lastcourse != $submission->course) {
-                    @rmdir("$CFG->dataroot/$lastcourse/$CFG->moddata/assignment");
-                    @rmdir("$CFG->dataroot/$lastcourse/$CFG->moddata");
-                    @rmdir("$CFG->dataroot/$lastcourse");
-                }
-                $lastsubmission = $submission->assignment;
-                $lastcourse     = $submission->course;
-
+                @rmdir("$CFG->dataroot/$submission->course/$CFG->moddata/assignment/$submission->assignment");
+                @rmdir("$CFG->dataroot/$submission->course/$CFG->moddata/assignment");
             }
             $DB->set_debug($olddebug); // reset debug level
             $rs->close();
 
-            // cleanup after the last submission
-            if ($lastcourse) {
-                @rmdir("$CFG->dataroot/$lastcourse/$CFG->moddata/assignment/$lastassignment");
-                @rmdir("$CFG->dataroot/$lastcourse/$CFG->moddata/assignment");
-                @rmdir("$CFG->dataroot/$lastcourse/$CFG->moddata");
-                @rmdir("$CFG->dataroot/$lastcourse");
-            }
         }
 
         upgrade_mod_savepoint($result, 2008073000, 'assignment');