Merge branch 'feature/backup-before-wmredownload' into develop

app/Console/Commands/ParseCachedWebMentions.php

@@ -41,6 +41,7 @@ class ParseCachedWebMentions extends Command
     {
         $HTMLfiles = $filesystem->allFiles(storage_path() . '/HTML');
         foreach ($HTMLfiles as $file) {
+            if ($file->getExtension() != 'backup') { //we don’t want to parse.backup files
                 $filepath = $file->getPathname();
                 $html = $filesystem->get($filepath);
                 $url = $this->URLFromFilename($filepath);
@@ -50,6 +51,7 @@ class ParseCachedWebMentions extends Command
                 $webmention->save();
             }
         }
+    }
 
     /**
      * Determine the source URL from a filename.

app/Jobs/DownloadWebMention.php

@@ -41,10 +41,21 @@ class DownloadWebMention implements ShouldQueue
         //Laravel should catch and retry these automatically.
         if ($response->getStatusCode() == '200') {
             $filesystem = \Illuminate\FileSystem\FileSystem();
+            $filename = $this->createFilenameFromURL($source);
+            //backup file first
+            $filenameBackup = $filename . '.' . date('Y-m-d') . '.backup';
+            if ($filesystem->exists($filename)) {
+                $filesystem->copy($filename, $filenameBackup);
+            }
+            //save new HTML
             $filesystem->put(
-                $this->createFilenameFromURL($source),
+                $filename,
                 (string) $response->getBody()
             );
+            //remove backup if the same
+            if ($filesystem->get($filename) == $filesystem->get($filenameBackup)) {
+                $filesystem->delete($filenameBackup);
+            }
         }
     }