Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 4a8573a7 authored by Rohit Sekhar's avatar Rohit Sekhar
Browse files

Merge branch '7480-master-FP5_longitude_zero' into 'master' 

ImageSaver: Update GPS EXIF data if longitude is not set or zero

See merge request !72
parents 5992f11f 9faf17d6

app/src/main/java/net/sourceforge/opencamera/ImageSaver.java

+2 −1
Original line number Diff line number Diff line
@@ -3950,9 +3950,10 @@ public class ImageSaver extends Thread { 
        }

        setCustomExif(exif, custom_tag_artist, custom_tag_copyright);



        if( store_location && ( !exif.hasAttribute(ExifInterface.TAG_GPS_LATITUDE) || !exif.hasAttribute(ExifInterface.TAG_GPS_LATITUDE) ) ) {

        if( store_location && ( !exif.hasAttribute(ExifInterface.TAG_GPS_LATITUDE) || !exif.hasAttribute(ExifInterface.TAG_GPS_LONGITUDE) || exif.getLatLong()[1] == 0.0d) ) {

            // We need this when using camera extensions (since Camera API doesn't support location for camera extensions).

            // But some devices (e.g., Pixel 6 Pro with Camera2 API) seem to not store location data, so we always check if we need to add it.

            // Also for FP5, wherein longitude randomly is 0.0d, even when properly passed via JPEG_GPS_LOCATION

            // fine to ignore request.remove_device_exif, as this is a separate user option

            if( MyDebug.LOG )

                Log.d(TAG, "store location"); // don't log location for privacy reasons!