Admin/ocean-cleanup
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: correct EXIF GPS data extraction using exifr.gps()

Browse Source 
The previous implementation was using exifr.parse() with pick option,
which doesn't work for GPS coordinates since latitude/longitude are
computed values, not direct EXIF fields.

Changed to use exifr.gps() method which properly extracts and computes
GPS coordinates from EXIF GPSLatitude/GPSLongitude fields.

This fixes the issue where photos with GPS EXIF data were showing
'No GPS data' in the UI.
This commit is contained in:
Paul Bailey
2025-11-16 16:22:17 +00:00
parent e18997ed95
commit 3864d3aa37
1 changed files with 8 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
src/components/report/PhotoUpload.tsx
View File

@@ -32,18 +32,17 @@ export function PhotoUpload({
const extractEXIF = async (file: File): Promise<Partial<PhotoMetadata>> => { const extractEXIF = async (file: File): Promise<Partial<PhotoMetadata>> => {
try { try {
// Parse GPS data separately to get computed latitude/longitude
const gpsData = await exifr.gps(file);
// Parse other EXIF data for timestamp
const exifData = await exifr.parse(file, { const exifData = await exifr.parse(file, {
gps: true, pick: ['DateTimeOriginal', 'CreateDate', 'DateTime']
pick: ['latitude', 'longitude', 'DateTimeOriginal', 'CreateDate']
}); });
if (!exifData) { const latitude = gpsData?.latitude;
return { hasGPS: false }; const longitude = gpsData?.longitude;
} const timestamp = exifData?.DateTimeOriginal || exifData?.CreateDate || exifData?.DateTime;
const latitude = exifData.latitude;
const longitude = exifData.longitude;
const timestamp = exifData.DateTimeOriginal || exifData.CreateDate;
return { return {
latitude: latitude !== undefined ? latitude : undefined, latitude: latitude !== undefined ? latitude : undefined,