Fixed issue: ComfortUpdate not working when one of the files to back …

…up has 0 size
LimeSurvey · Aug 11, 2021 · aa37aa8 · aa37aa8
1 parent bb73ab5
commit aa37aa8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/application/config/updater_version.php b/application/config/updater_version.php
@@ -15,7 +15,7 @@
 
     // THIS FILE SHOULD NOT BE CHANGED MANUALLY !
 
-$config['updaterversion'] = 9;
+$config['updaterversion'] = 10;
 $config['comfort_update_server_url'] = 'comfortupdate.limesurvey.org/';
 //$config['comfort_update_server_url'] = 'web.comfortupdate.org/';
 $config['comfort_update_server_ssl'] = 0;

diff --git a/application/models/UpdateForm.php b/application/models/UpdateForm.php
@@ -471,8 +471,8 @@ public function backupFiles($updateinfos)
             // To block the access to subdirectories
             $sFileToZip = str_replace("..", "", $file['file']);
 
-            if (is_file($this->publicdir . $sFileToZip) === true && basename($sFileToZip) != 'config.php') {
-                $filestozip[] = $this->publicdir . $sFileToZip;
+            if (is_file($this->publicdir.$sFileToZip) === true && basename($sFileToZip) != 'config.php' && filesize($this->publicdir.$sFileToZip) > 0 ) {
+                $filestozip[] = $this->publicdir.$sFileToZip;
             }
         }