#38: Made a few tweaks to the analysis function that doesn't delete the photo if it was previously analysed (i.e. it has a metadata version). Also if the original image contained Exif data (e.g. camera make), we no longer remove it if the re-analysed image doesn't (see #39)

app/Http/Controllers/Admin/PhotoController.php

@@ -59,9 +59,12 @@ class PhotoController extends Controller
             $result['is_successful'] = false;
             $result['message'] = $ex->getMessage();
 
-            // Remove the photo if it cannot be analysed
+            // Remove the photo if it cannot be analysed (only if there isn't currently a version of metadata)
+            if (is_null($photo->metadata_version))
+            {
                 $photo->delete();
             }
+        }
 
         return response()->json($result);
     }

app/Services/PhotoService.php

@@ -108,10 +108,10 @@ class PhotoService
             if ($isExifDataFound)
             {
                 $this->photo->metadata_version = self::METADATA_VERSION;
-                $this->photo->taken_at = $this->metadataDateTime($exifData);
-                $this->photo->camera_make = $this->metadataCameraMake($exifData);
-                $this->photo->camera_model = $this->metadataCameraModel($exifData);
-                $this->photo->camera_software = $this->metadataCameraSoftware($exifData);
+                $this->photo->taken_at = $this->metadataDateTime($exifData, $this->photo->taken_at);
+                $this->photo->camera_make = $this->metadataCameraMake($exifData, $this->photo->camera_make);
+                $this->photo->camera_model = $this->metadataCameraModel($exifData, $this->photo->camera_model);
+                $this->photo->camera_software = $this->metadataCameraSoftware($exifData, $this->photo->camera_software);
             }
 
             $this->photo->is_analysed = true;
@@ -301,37 +301,37 @@ class PhotoService
         return $photoPath;
     }
 
-    private function metadataCameraMake(array $exifData)
+    private function metadataCameraMake(array $exifData, $originalValue = null)
     {
         if (isset($exifData['Make']))
         {
             return $exifData['Make'];
         }
 
-        return null;
+        return $originalValue;
     }
 
-    private function metadataCameraModel(array $exifData)
+    private function metadataCameraModel(array $exifData, $originalValue = null)
     {
         if (isset($exifData['Model']))
         {
             return $exifData['Model'];
         }
 
-        return null;
+        return $originalValue;
     }
 
-    private function metadataCameraSoftware(array $exifData)
+    private function metadataCameraSoftware(array $exifData, $originalValue = null)
     {
         if (isset($exifData['Software']))
         {
             return $exifData['Software'];
         }
 
-        return null;
+        return $originalValue;
     }
 
-    private function metadataDateTime(array $exifData)
+    private function metadataDateTime(array $exifData, $originalValue = null)
     {
         $dateTime = null;
         if (isset($exifData['DateTimeOriginal']))
@@ -343,11 +343,11 @@ class PhotoService
             $dateTime = $exifData['DateTime'];
         }
 
-        if (!is_null($dateTime))
+        if (is_null($dateTime))
         {
-            $dateTime = preg_replace('/^([\d]{4}):([\d]{2}):([\d]{2})/', '$1-$2-$3', $dateTime);
+            return $originalValue;
         }
 
-        return $dateTime;
+        return preg_replace('/^([\d]{4}):([\d]{2}):([\d]{2})/', '$1-$2-$3', $dateTime);
     }
 }

database/migrations/2017_09_15_141905_add_photo_exif_data_column.php

@@ -0,0 +1,34 @@
+<?php
+
+use Illuminate\Support\Facades\Schema;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Database\Migrations\Migration;
+
+class AddPhotoExifDataColumn extends Migration
+{
+    /**
+     * Run the migrations.
+     *
+     * @return void
+     */
+    public function up()
+    {
+        Schema::table('photos', function (Blueprint $table)
+        {
+            $table->longText('raw_exif_data');
+        });
+    }
+
+    /**
+     * Reverse the migrations.
+     *
+     * @return void
+     */
+    public function down()
+    {
+        Schema::table('photos', function (Blueprint $table)
+        {
+            $table->dropColumn('raw_exif_data');
+        });
+    }
+}

resources/views/themes/base/admin/album_metadata.blade.php

@@ -32,3 +32,13 @@
             </div>
         </div>
 @endsection
+
+@push('scripts')
+    <script type="text/javascript">
+        $(document).ready(function() {
+            $('form').submit(function() {
+                $('button[type=submit]').prop('disabled', true);
+            });
+        });
+    </script>
+@endpush