fix for issue #131: GPS lat/lon refs

photostructure · Feb 13, 2023 · 174b25f · 174b25f
1 parent cd897a7
commit 174b25f
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -114,7 +114,7 @@
     "luxon": "^3.2.1"
   },
   "optionalDependencies": {
-    "exiftool-vendored.exe": "12.50.0",
-    "exiftool-vendored.pl": "12.50.0"
+    "exiftool-vendored.exe": "12.56.0",
+    "exiftool-vendored.pl": "12.56.0"
   }
 }
diff --git a/src/WriteTask.ts b/src/WriteTask.ts
@@ -92,6 +92,17 @@ export class WriteTask extends ExifToolTask<void> {
       return new VersionTask() as any
     }
 
+    // Special handling for GPSLatitude and GPSLongitude (due to differences
+    // in EXIF, XMP, and MIE encodings). See
+    // https://exiftool.org/forum/index.php?topic=14488.0 and
+    // https://github.com/photostructure/exiftool-vendored.js/issues/131
+    if (isNumber(tags.GPSLatitude)) {
+      tags.GPSLatitudeRef ??= tags.GPSLatitude >= 0 ? "N" : "S"
+    }
+    if (isNumber(tags.GPSLongitude)) {
+      tags.GPSLongitudeRef ??= tags.GPSLongitude >= 0 ? "E" : "W"
+    }
+
     for (const key of keys(tags)) {
       const val = tags[key]
       args.push(`-${key}=${enc(val)}`)