Completed
Push — master ( 1af806...a2f2c9 )
by Angel Fernando Quiroz
73:48 queued 35:36
created
src/Chamilo/CourseBundle/Component/CourseCopy/CourseArchiver.php 1 patch
Spacing   +31 added lines, -31 removed lines patch added patch discarded remove patch
@@ -71,49 +71,49 @@  discard block
 block discarded – undo
71 71
         $backupDirectory = self::getBackupDir();
72 72
 
73 73
         // Create a temp directory
74
-        $backup_dir = $backupDirectory . 'CourseArchiver_' . api_get_unique_id() . '/';
74
+        $backup_dir = $backupDirectory.'CourseArchiver_'.api_get_unique_id().'/';
75 75
 
76 76
         // All course-information will be stored in course_info.dat
77
-        $course_info_file = $backup_dir . 'course_info.dat';
77
+        $course_info_file = $backup_dir.'course_info.dat';
78 78
 
79 79
         $user = api_get_user_info();
80 80
         $date = new \DateTime(api_get_local_time());
81
-        $zipFileName = $user['user_id'] . '_' . $course->code . '_' . $date->format('Ymd-His') . '.zip';
82
-        $zipFilePath = $backupDirectory. $zipFileName;
81
+        $zipFileName = $user['user_id'].'_'.$course->code.'_'.$date->format('Ymd-His').'.zip';
82
+        $zipFilePath = $backupDirectory.$zipFileName;
83 83
 
84 84
         $php_errormsg = '';
85 85
         $res = @mkdir($backup_dir, $perm_dirs);
86 86
         if ($res === false) {
87 87
             //TODO set and handle an error message telling the user to review the permissions on the archive directory
88
-            error_log(__FILE__ . ' line ' . __LINE__ . ': ' . (ini_get('track_errors') != false ? $php_errormsg : 'error not recorded because track_errors is off in your php.ini') . ' - This error, occuring because your archive directory will not let this script write data into it, will prevent courses backups to be created', 0);
88
+            error_log(__FILE__.' line '.__LINE__.': '.(ini_get('track_errors') != false ? $php_errormsg : 'error not recorded because track_errors is off in your php.ini').' - This error, occuring because your archive directory will not let this script write data into it, will prevent courses backups to be created', 0);
89 89
         }
90 90
         // Write the course-object to the file
91 91
         $fp = @fopen($course_info_file, 'w');
92 92
         if ($fp === false) {
93
-            error_log(__FILE__ . ' line ' . __LINE__ . ': ' . (ini_get('track_errors') != false ? $php_errormsg : 'error not recorded because track_errors is off in your php.ini'), 0);
93
+            error_log(__FILE__.' line '.__LINE__.': '.(ini_get('track_errors') != false ? $php_errormsg : 'error not recorded because track_errors is off in your php.ini'), 0);
94 94
         }
95 95
 
96 96
         $res = @fwrite($fp, base64_encode(serialize($course)));
97 97
         if ($res === false) {
98
-            error_log(__FILE__ . ' line ' . __LINE__ . ': ' . (ini_get('track_errors') != false ? $php_errormsg : 'error not recorded because track_errors is off in your php.ini'), 0);
98
+            error_log(__FILE__.' line '.__LINE__.': '.(ini_get('track_errors') != false ? $php_errormsg : 'error not recorded because track_errors is off in your php.ini'), 0);
99 99
         }
100 100
 
101 101
         $res = @fclose($fp);
102 102
         if ($res === false) {
103
-            error_log(__FILE__ . ' line ' . __LINE__ . ': ' . (ini_get('track_errors') != false ? $php_errormsg : 'error not recorded because track_errors is off in your php.ini'), 0);
103
+            error_log(__FILE__.' line '.__LINE__.': '.(ini_get('track_errors') != false ? $php_errormsg : 'error not recorded because track_errors is off in your php.ini'), 0);
104 104
         }
105 105
 
106 106
         // Copy all documents to the temp-dir
107 107
         if (isset($course->resources[RESOURCE_DOCUMENT]) && is_array($course->resources[RESOURCE_DOCUMENT])) {
108 108
             foreach ($course->resources[RESOURCE_DOCUMENT] as $document) {
109 109
                 if ($document->file_type == DOCUMENT) {
110
-                    $doc_dir = $backup_dir . $document->path;
110
+                    $doc_dir = $backup_dir.$document->path;
111 111
                     @mkdir(dirname($doc_dir), $perm_dirs, true);
112
-                    if (file_exists($course->path . $document->path)) {
113
-                        copy($course->path . $document->path, $doc_dir);
112
+                    if (file_exists($course->path.$document->path)) {
113
+                        copy($course->path.$document->path, $doc_dir);
114 114
                     }
115 115
                 } else {
116
-                    @mkdir($backup_dir . $document->path, $perm_dirs, true);
116
+                    @mkdir($backup_dir.$document->path, $perm_dirs, true);
117 117
                 }
118 118
             }
119 119
         }
@@ -121,59 +121,59 @@  discard block
 block discarded – undo
121 121
         // Copy all scorm documents to the temp-dir
122 122
         if (isset($course->resources[RESOURCE_SCORM]) && is_array($course->resources[RESOURCE_SCORM])) {
123 123
             foreach ($course->resources[RESOURCE_SCORM] as $document) {
124
-                $doc_dir = dirname($backup_dir . $document->path);
124
+                $doc_dir = dirname($backup_dir.$document->path);
125 125
                 @mkdir($doc_dir, $perm_dirs, true);
126
-                copyDirTo($course->path . $document->path, $doc_dir, false);
126
+                copyDirTo($course->path.$document->path, $doc_dir, false);
127 127
             }
128 128
         }
129 129
 
130 130
         // Copy calendar attachments.
131 131
 
132 132
         if (isset($course->resources[RESOURCE_EVENT]) && is_array($course->resources[RESOURCE_EVENT])) {
133
-            $doc_dir = dirname($backup_dir . '/upload/calendar/');
133
+            $doc_dir = dirname($backup_dir.'/upload/calendar/');
134 134
             @mkdir($doc_dir, $perm_dirs, true);
135
-            copyDirTo($course->path . 'upload/calendar/', $doc_dir, false);
135
+            copyDirTo($course->path.'upload/calendar/', $doc_dir, false);
136 136
         }
137 137
 
138 138
         // Copy Learning path author image.
139 139
         if (isset($course->resources[RESOURCE_LEARNPATH]) && is_array($course->resources[RESOURCE_LEARNPATH])) {
140
-            $doc_dir = dirname($backup_dir . '/upload/learning_path/');
140
+            $doc_dir = dirname($backup_dir.'/upload/learning_path/');
141 141
             @mkdir($doc_dir, $perm_dirs, true);
142
-            copyDirTo($course->path . 'upload/learning_path/', $doc_dir, false);
142
+            copyDirTo($course->path.'upload/learning_path/', $doc_dir, false);
143 143
         }
144 144
 
145 145
         // Copy announcements attachments.
146 146
         if (isset($course->resources[RESOURCE_ANNOUNCEMENT]) && is_array($course->resources[RESOURCE_ANNOUNCEMENT])) {
147
-            $doc_dir = dirname($backup_dir . '/upload/announcements/');
147
+            $doc_dir = dirname($backup_dir.'/upload/announcements/');
148 148
             @mkdir($doc_dir, $perm_dirs, true);
149
-            copyDirTo($course->path . 'upload/announcements/', $doc_dir, false);
149
+            copyDirTo($course->path.'upload/announcements/', $doc_dir, false);
150 150
         }
151 151
 
152 152
         // Copy work folders (only folders)
153 153
         if (isset($course->resources[RESOURCE_WORK]) && is_array($course->resources[RESOURCE_WORK])) {
154
-            $doc_dir = dirname($backup_dir . '/upload/work/');
154
+            $doc_dir = dirname($backup_dir.'/upload/work/');
155 155
             @mkdir($doc_dir, $perm_dirs, true);
156 156
             // @todo: adjust to only create subdirs, but not copy files
157
-            copyDirTo($course->path . 'upload/work/', $doc_dir, false);
157
+            copyDirTo($course->path.'upload/work/', $doc_dir, false);
158 158
         }
159 159
 
160 160
         if (isset($course->resources[RESOURCE_ASSET]) && is_array($course->resources[RESOURCE_ASSET])) {
161 161
             /** @var Asset $asset */
162 162
             foreach ($course->resources[RESOURCE_ASSET] as $asset) {
163
-                $doc_dir = $backup_dir . $asset->path;
163
+                $doc_dir = $backup_dir.$asset->path;
164 164
                 @mkdir(dirname($doc_dir), $perm_dirs, true);
165
-                $assetPath = $course->path . $asset->path;
165
+                $assetPath = $course->path.$asset->path;
166 166
 
167 167
                 if (!file_exists($assetPath)) {
168 168
                     continue;
169 169
                 }
170 170
 
171
-                if (is_dir($course->path . $asset->path)) {
172
-                    copyDirTo($course->path . $asset->path, $doc_dir);
171
+                if (is_dir($course->path.$asset->path)) {
172
+                    copyDirTo($course->path.$asset->path, $doc_dir);
173 173
                     continue;
174 174
                 }
175 175
 
176
-                copy($course->path . $asset->path, $doc_dir);
176
+                copy($course->path.$asset->path, $doc_dir);
177 177
             }
178 178
         }
179 179
 
@@ -209,7 +209,7 @@  discard block
 block discarded – undo
209 209
                     $date = $file_parts[0];
210 210
                     $ext = isset($file_parts[1]) ? $file_parts[1] : null;
211 211
                     if ($ext == 'zip' && ($user_id != null && $owner_id == $user_id || $user_id == null)) {
212
-                        $date = substr($date, 0, 4) . '-' . substr($date, 4, 2) . '-' . substr($date, 6, 2) . ' ' . substr($date, 9, 2) . ':' . substr($date, 11, 2) . ':' . substr($date, 13, 2);
212
+                        $date = substr($date, 0, 4).'-'.substr($date, 4, 2).'-'.substr($date, 6, 2).' '.substr($date, 9, 2).':'.substr($date, 11, 2).':'.substr($date, 13, 2);
213 213
                         $backup_files[] = array(
214 214
                             'file' => $file,
215 215
                             'date' => $date,
@@ -230,7 +230,7 @@  discard block
 block discarded – undo
230 230
      */
231 231
     public static function importUploadedFile($file)
232 232
     {
233
-        $new_filename = uniqid('') . '.zip';
233
+        $new_filename = uniqid('').'.zip';
234 234
         $new_dir = self::getBackupDir();
235 235
         if (!is_dir($new_dir)) {
236 236
             $fs = new Filesystem();
@@ -264,11 +264,11 @@  discard block
 block discarded – undo
264 264
         @mkdir($unzip_dir, api_get_permissions_for_new_directories(), true);
265 265
         @copy(
266 266
             $filePath,
267
-            $unzip_dir . '/backup.zip'
267
+            $unzip_dir.'/backup.zip'
268 268
         );
269 269
 
270 270
         // unzip the archive
271
-        $zip = new \PclZip($unzip_dir . '/backup.zip');
271
+        $zip = new \PclZip($unzip_dir.'/backup.zip');
272 272
         @chdir($unzip_dir);
273 273
         $zip->extract(PCLZIP_OPT_TEMP_FILE_ON);
274 274
         // remove the archive-file
Please login to merge, or discard this patch.