Răsfoiți Sursa

Fix delete instance see BT#11414

jmontoyaa 8 ani în urmă
părinte
comite
97bc1f4eb6
1 a modificat fișierele cu 23 adăugiri și 25 ștergeri
  1. 23 25
      plugin/vchamilo/views/manage.controller.php

+ 23 - 25
plugin/vchamilo/views/manage.controller.php

@@ -64,39 +64,37 @@ switch ($action) {
             foreach ($todelete as $fooid => $instance) {
                 $slug = $instance['slug'];
 
-                if (empty($slug)) {
-                   continue;
-                }
-
-                // Remove all files and eventual symlinks
-                $absalternatecourse = Virtual::getConfig('vchamilo', 'course_real_root');
-                $coursedir = $absalternatecourse.$slug;
+                if (!empty($slug)) {
+                    // Remove all files and eventual symlinks
+                    $absalternatecourse = Virtual::getConfig('vchamilo', 'course_real_root');
+                    $coursedir = $absalternatecourse.$slug;
 
-                Display::addFlash(Display::return_message("Deleting $coursedir"));
+                    Display::addFlash(Display::return_message("Deleting $coursedir"));
 
-                removeDir($coursedir);
+                    removeDir($coursedir);
 
-                if ($absalternatehome = Virtual::getConfig('vchamilo', 'home_real_root')) {
-                    $homedir = $absalternatehome.'/'.$slug;
+                    if ($absalternatehome = Virtual::getConfig('vchamilo', 'home_real_root')) {
+                        $homedir = $absalternatehome.'/'.$slug;
 
-                    Display::addFlash(Display::return_message("Deleting $homedir"));
-                    removeDir($homedir);
-                }
+                        Display::addFlash(Display::return_message("Deleting $homedir"));
+                        removeDir($homedir);
+                    }
 
-                // delete archive
-                if ($absalternatearchive = Virtual::getConfig('vchamilo', 'archive_real_root')) {
-                    $archivedir = $absalternatearchive.'/'.$slug;
+                    // delete archive
+                    if ($absalternatearchive = Virtual::getConfig('vchamilo', 'archive_real_root')) {
+                        $archivedir = $absalternatearchive.'/'.$slug;
 
-                    Display::addFlash(Display::return_message("Deleting $archivedir"));
-                    removeDir($archivedir);
-                }
+                        Display::addFlash(Display::return_message("Deleting $archivedir"));
+                        removeDir($archivedir);
+                    }
 
-                // Delete upload
-                if ($dir = Virtual::getConfig('vchamilo', 'upload_real_root')) {
-                    $dir = $dir.'/'.$slug;
+                    // Delete upload
+                    if ($dir = Virtual::getConfig('vchamilo', 'upload_real_root')) {
+                        $dir = $dir.'/'.$slug;
 
-                    Display::addFlash(Display::return_message("Deleting $dir"));
-                    removeDir($dir);
+                        Display::addFlash(Display::return_message("Deleting $dir"));
+                        removeDir($dir);
+                    }
                 }
 
                 $sql = "DELETE FROM {$table} WHERE id = ".$instance['id'];