diff --git a/mobile/openapi/doc/ExifResponseDto.md b/mobile/openapi/doc/ExifResponseDto.md index 5ae1099c0b..7c40bc742a 100644 Binary files a/mobile/openapi/doc/ExifResponseDto.md and b/mobile/openapi/doc/ExifResponseDto.md differ diff --git a/mobile/openapi/lib/model/exif_response_dto.dart b/mobile/openapi/lib/model/exif_response_dto.dart index 1cb298c355..071d8caf70 100644 Binary files a/mobile/openapi/lib/model/exif_response_dto.dart and b/mobile/openapi/lib/model/exif_response_dto.dart differ diff --git a/mobile/openapi/test/exif_response_dto_test.dart b/mobile/openapi/test/exif_response_dto_test.dart index 173a37b3a3..63623cd4fe 100644 Binary files a/mobile/openapi/test/exif_response_dto_test.dart and b/mobile/openapi/test/exif_response_dto_test.dart differ diff --git a/server/apps/microservices/src/processors/metadata-extraction.processor.ts b/server/apps/microservices/src/processors/metadata-extraction.processor.ts index bd308a7fd4..a6b6d11e70 100644 --- a/server/apps/microservices/src/processors/metadata-extraction.processor.ts +++ b/server/apps/microservices/src/processors/metadata-extraction.processor.ts @@ -17,6 +17,7 @@ import { ConfigService } from '@nestjs/config'; import { InjectRepository } from '@nestjs/typeorm'; import { Job } from 'bull'; import { ExifDateTime, exiftool, Tags } from 'exiftool-vendored'; +import tz_lookup from '@photostructure/tz-lookup'; import ffmpeg, { FfprobeData } from 'fluent-ffmpeg'; import { getName } from 'i18n-iso-countries'; import geocoder, { InitOptions } from 'local-reverse-geocoder'; @@ -190,6 +191,17 @@ export class MetadataExtractionProcessor { return exifDate.toDate(); }; + const exifTimeZone = (exifDate: string | ExifDateTime | undefined) => { + if (!exifDate) return null; + + if (typeof exifDate === 'string') { + return null; + } + + return exifDate.zone ?? null; + }; + + const timeZone = exifTimeZone(exifData?.DateTimeOriginal ?? exifData?.CreateDate ?? asset.fileCreatedAt); const fileCreatedAt = exifToDate(exifData?.DateTimeOriginal ?? exifData?.CreateDate ?? asset.fileCreatedAt); const fileModifiedAt = exifToDate(exifData?.ModifyDate ?? asset.fileModifiedAt); const fileStats = fs.statSync(asset.originalPath); @@ -207,6 +219,7 @@ export class MetadataExtractionProcessor { newExif.orientation = exifData?.Orientation?.toString() || null; newExif.dateTimeOriginal = fileCreatedAt; newExif.modifyDate = fileModifiedAt; + newExif.timeZone = timeZone; newExif.lensModel = exifData?.LensModel || null; newExif.fNumber = exifData?.FNumber || null; newExif.focalLength = exifData?.FocalLength ? parseFloat(exifData.FocalLength) : null; @@ -308,6 +321,7 @@ export class MetadataExtractionProcessor { newExif.fileSizeInByte = data.format.size || null; newExif.dateTimeOriginal = fileCreatedAt ? new Date(fileCreatedAt) : null; newExif.modifyDate = null; + newExif.timeZone = null; newExif.latitude = null; newExif.longitude = null; newExif.city = null; @@ -345,6 +359,14 @@ export class MetadataExtractionProcessor { } } + if (newExif.longitude && newExif.latitude) { + try { + newExif.timeZone = tz_lookup(newExif.latitude, newExif.longitude); + } catch (error: any) { + this.logger.warn(`Error while calculating timezone from gps coordinates: ${error}`, error?.stack); + } + } + // Reverse GeoCoding if (this.isGeocodeInitialized && newExif.longitude && newExif.latitude) { const { country, state, city } = await this.reverseGeocodeExif(newExif.latitude, newExif.longitude); diff --git a/server/immich-openapi-specs.json b/server/immich-openapi-specs.json index dd6e71ac5f..70b11afa94 100644 --- a/server/immich-openapi-specs.json +++ b/server/immich-openapi-specs.json @@ -3537,6 +3537,11 @@ "nullable": true, "default": null }, + "timeZone": { + "type": "string", + "nullable": true, + "default": null + }, "lensModel": { "type": "string", "nullable": true, diff --git a/server/libs/domain/src/asset/response-dto/exif-response.dto.ts b/server/libs/domain/src/asset/response-dto/exif-response.dto.ts index 036e7e64ab..83961a1960 100644 --- a/server/libs/domain/src/asset/response-dto/exif-response.dto.ts +++ b/server/libs/domain/src/asset/response-dto/exif-response.dto.ts @@ -13,6 +13,7 @@ export class ExifResponseDto { orientation?: string | null = null; dateTimeOriginal?: Date | null = null; modifyDate?: Date | null = null; + timeZone?: string | null = null; lensModel?: string | null = null; fNumber?: number | null = null; focalLength?: number | null = null; @@ -36,6 +37,7 @@ export function mapExif(entity: ExifEntity): ExifResponseDto { orientation: entity.orientation, dateTimeOriginal: entity.dateTimeOriginal, modifyDate: entity.modifyDate, + timeZone: entity.timeZone, lensModel: entity.lensModel, fNumber: entity.fNumber, focalLength: entity.focalLength, diff --git a/server/libs/domain/test/fixtures.ts b/server/libs/domain/test/fixtures.ts index ea092d44e7..e5383703f2 100644 --- a/server/libs/domain/test/fixtures.ts +++ b/server/libs/domain/test/fixtures.ts @@ -323,6 +323,7 @@ const assetInfo: ExifResponseDto = { orientation: 'orientation', dateTimeOriginal: today, modifyDate: today, + timeZone: 'America/Los_Angeles', lensModel: 'fancy', fNumber: 100, focalLength: 100, @@ -607,6 +608,7 @@ export const sharedLinkStub = { orientation: 'orientation', dateTimeOriginal: today, modifyDate: today, + timeZone: 'America/Los_Angeles', latitude: 100, longitude: 100, city: 'city', diff --git a/server/libs/infra/src/entities/exif.entity.ts b/server/libs/infra/src/entities/exif.entity.ts index 8375bdbfca..00b57f9e28 100644 --- a/server/libs/infra/src/entities/exif.entity.ts +++ b/server/libs/infra/src/entities/exif.entity.ts @@ -34,6 +34,9 @@ export class ExifEntity { @Column({ type: 'timestamptz', nullable: true }) modifyDate!: Date | null; + @Column({ type: 'varchar', nullable: true }) + timeZone!: string | null; + @Column({ type: 'float', nullable: true }) latitude!: number | null; diff --git a/server/libs/infra/src/migrations/1677497925328-AddExifTimeZone.ts b/server/libs/infra/src/migrations/1677497925328-AddExifTimeZone.ts new file mode 100644 index 0000000000..33f958336e --- /dev/null +++ b/server/libs/infra/src/migrations/1677497925328-AddExifTimeZone.ts @@ -0,0 +1,14 @@ +import { MigrationInterface, QueryRunner } from "typeorm"; + +export class AddExifTimeZone1677497925328 implements MigrationInterface { + name = 'AddExifTimeZone1677497925328' + + public async up(queryRunner: QueryRunner): Promise { + await queryRunner.query(`ALTER TABLE "exif" ADD "timeZone" character varying`); + } + + public async down(queryRunner: QueryRunner): Promise { + await queryRunner.query(`ALTER TABLE "exif" DROP COLUMN "timeZone"`); + } + +} diff --git a/web/src/api/open-api/api.ts b/web/src/api/open-api/api.ts index 2313411d99..3853e55758 100644 --- a/web/src/api/open-api/api.ts +++ b/web/src/api/open-api/api.ts @@ -1126,6 +1126,12 @@ export interface ExifResponseDto { * @memberof ExifResponseDto */ 'modifyDate'?: string | null; + /** + * + * @type {string} + * @memberof ExifResponseDto + */ + 'timeZone'?: string | null; /** * * @type {string} diff --git a/web/src/lib/components/asset-viewer/detail-panel.svelte b/web/src/lib/components/asset-viewer/detail-panel.svelte index 05575035e0..9ab4d99a1b 100644 --- a/web/src/lib/components/asset-viewer/detail-panel.svelte +++ b/web/src/lib/components/asset-viewer/detail-panel.svelte @@ -8,6 +8,7 @@ import { AssetResponseDto, AlbumResponseDto } from '@api'; import { asByteUnitString } from '../../utils/byte-units'; import { locale } from '$lib/stores/preferences.store'; + import { DateTime } from 'luxon'; import type { LatLngTuple } from 'leaflet'; export let asset: AssetResponseDto; @@ -55,7 +56,9 @@ {/if} {#if asset.exifInfo?.dateTimeOriginal} - {@const assetDateTimeOriginal = new Date(asset.exifInfo.dateTimeOriginal)} + {@const assetDateTimeOriginal = DateTime.fromISO(asset.exifInfo.dateTimeOriginal, { + zone: asset.exifInfo.timeZone ?? undefined + })}
@@ -63,20 +66,26 @@

- {assetDateTimeOriginal.toLocaleDateString($locale, { - month: 'short', - day: 'numeric', - year: 'numeric' - })} + {assetDateTimeOriginal.toLocaleString( + { + month: 'short', + day: 'numeric', + year: 'numeric' + }, + { locale: $locale } + )}

- {assetDateTimeOriginal.toLocaleString($locale, { - weekday: 'short', - hour: 'numeric', - minute: '2-digit', - timeZoneName: 'longOffset' - })} + {assetDateTimeOriginal.toLocaleString( + { + weekday: 'short', + hour: 'numeric', + minute: '2-digit', + timeZoneName: 'longOffset' + }, + { locale: $locale } + )}