fix: sync is_favorite from native (#20412)

* feat: sync is_favorite from native

* handle favorite during upload

* Update mobile/ios/Runner/Sync/MessagesImpl.swift

Co-authored-by: Alex <[email protected]>

---------

Co-authored-by: shenlong-tanwen <[email protected]>
Co-authored-by: Alex <[email protected]>

Author: shenlong-tanwen

mobile/android/app/src/main/kotlin/app/alextran/immich/sync/Messages.g.kt

@@ -88,7 +88,8 @@ data class PlatformAsset (
   val width: Long? = null,
   val height: Long? = null,
   val durationInSeconds: Long,
-  val orientation: Long
+  val orientation: Long,
+  val isFavorite: Boolean
 )
  {
   companion object {
@@ -102,7 +103,8 @@ data class PlatformAsset (
       val height = pigeonVar_list[6] as Long?
       val durationInSeconds = pigeonVar_list[7] as Long
       val orientation = pigeonVar_list[8] as Long
-      return PlatformAsset(id, name, type, createdAt, updatedAt, width, height, durationInSeconds, orientation)
+      val isFavorite = pigeonVar_list[9] as Boolean
+      return PlatformAsset(id, name, type, createdAt, updatedAt, width, height, durationInSeconds, orientation, isFavorite)
     }
   }
   fun toList(): List<Any?> {
@@ -116,6 +118,7 @@ data class PlatformAsset (
       height,
       durationInSeconds,
       orientation,
+      isFavorite,
     )
   }
   override fun equals(other: Any?): Boolean {

mobile/android/app/src/main/kotlin/app/alextran/immich/sync/MessagesImplBase.kt

@@ -42,6 +42,7 @@ open class NativeSyncApiImplBase(context: Context) {
       MediaStore.MediaColumns.HEIGHT,
       MediaStore.MediaColumns.DURATION,
       MediaStore.MediaColumns.ORIENTATION,
+      MediaStore.MediaColumns.IS_FAVORITE,
     )
 
     const val HASH_BUFFER_SIZE = 2 * 1024 * 1024
@@ -77,6 +78,7 @@ open class NativeSyncApiImplBase(context: Context) {
         val durationColumn = c.getColumnIndexOrThrow(MediaStore.MediaColumns.DURATION)
         val orientationColumn =
           c.getColumnIndexOrThrow(MediaStore.MediaColumns.ORIENTATION)
+        val favoriteColumn = c.getColumnIndexOrThrow(MediaStore.MediaColumns.IS_FAVORITE)
 
         while (c.moveToNext()) {
           val id = c.getLong(idColumn).toString()
@@ -105,6 +107,7 @@ open class NativeSyncApiImplBase(context: Context) {
           else c.getLong(durationColumn) / 1000
           val bucketId = c.getString(bucketIdColumn)
           val orientation = c.getInt(orientationColumn)
+          val isFavorite = c.getInt(favoriteColumn) != 0;
 
           val asset = PlatformAsset(
             id,
@@ -116,6 +119,7 @@ open class NativeSyncApiImplBase(context: Context) {
             height,
             duration,
             orientation.toLong(),
+            isFavorite,
           )
           yield(AssetResult.ValidAsset(asset, bucketId))
         }

mobile/ios/Runner/Sync/Messages.g.swift

@@ -139,6 +139,7 @@ struct PlatformAsset: Hashable {
   var height: Int64? = nil
   var durationInSeconds: Int64
   var orientation: Int64
+  var isFavorite: Bool
 
 
   // swift-format-ignore: AlwaysUseLowerCamelCase
@@ -152,6 +153,7 @@ struct PlatformAsset: Hashable {
     let height: Int64? = nilOrValue(pigeonVar_list[6])
     let durationInSeconds = pigeonVar_list[7] as! Int64
     let orientation = pigeonVar_list[8] as! Int64
+    let isFavorite = pigeonVar_list[9] as! Bool
 
     return PlatformAsset(
       id: id,
@@ -162,7 +164,8 @@ struct PlatformAsset: Hashable {
       width: width,
       height: height,
       durationInSeconds: durationInSeconds,
-      orientation: orientation
+      orientation: orientation,
+      isFavorite: isFavorite
     )
   }
   func toList() -> [Any?] {
@@ -176,6 +179,7 @@ struct PlatformAsset: Hashable {
       height,
       durationInSeconds,
       orientation,
+      isFavorite,
     ]
   }
   static func == (lhs: PlatformAsset, rhs: PlatformAsset) -> Bool {

mobile/ios/Runner/Sync/MessagesImpl.swift

@@ -28,7 +28,8 @@ extension PHAsset {
       width: Int64(pixelWidth),
       height: Int64(pixelHeight),
       durationInSeconds: Int64(duration),
-      orientation: 0
+      orientation: 0,
+      isFavorite: isFavorite
     )
   }
 }

mobile/lib/domain/services/local_sync.service.dart

@@ -312,6 +312,7 @@ extension on Iterable<PlatformAsset> {
         height: e.height,
         durationInSeconds: e.durationInSeconds,
         orientation: e.orientation,
+        isFavorite: e.isFavorite,
       ),
     ).toList();
   }

mobile/lib/infrastructure/repositories/local_album.repository.dart

@@ -236,6 +236,7 @@ class DriftLocalAlbumRepository extends DriftDatabaseRepository {
           id: asset.id,
           orientation: Value(asset.orientation),
           checksum: const Value(null),
+          isFavorite: Value(asset.isFavorite),
         );
         batch.insert<$LocalAssetEntityTable, LocalAssetEntityData>(
           _db.localAssetEntity,

mobile/lib/platform/native_sync_api.g.dart

@@ -247,6 +247,7 @@ class UploadService {
       metadata: metadata,
       group: group,
       priority: priority,
+      isFavorite: asset.isFavorite,
     );
   }
 
@@ -270,6 +271,7 @@ class UploadService {
       fields: fields,
       group: kBackupLivePhotoGroup,
       priority: 0, // Highest priority to get upload immediately
+      isFavorite: asset.isFavorite,
     );
   }
 
@@ -281,6 +283,7 @@ class UploadService {
     String? deviceAssetId,
     String? metadata,
     int? priority,
+    bool? isFavorite,
   }) async {
     final serverEndpoint = Store.get(StoreKey.serverEndpoint);
     final url = Uri.parse('$serverEndpoint/assets').toString();
@@ -297,7 +300,7 @@ class UploadService {
       'deviceId': deviceId,
       'fileCreatedAt': fileCreatedAt.toUtc().toIso8601String(),
       'fileModifiedAt': fileModifiedAt.toUtc().toIso8601String(),
-      'isFavorite': 'false',
+      'isFavorite': isFavorite?.toString() ?? 'false',
       'duration': '0',
       if (fields != null) ...fields,
     };

mobile/lib/services/upload.service.dart

@@ -5,8 +5,7 @@ import 'package:pigeon/pigeon.dart';
     dartOut: 'lib/platform/native_sync_api.g.dart',
     swiftOut: 'ios/Runner/Sync/Messages.g.swift',
     swiftOptions: SwiftOptions(),
-    kotlinOut:
-        'android/app/src/main/kotlin/app/alextran/immich/sync/Messages.g.kt',
+    kotlinOut: 'android/app/src/main/kotlin/app/alextran/immich/sync/Messages.g.kt',
     kotlinOptions: KotlinOptions(package: 'app.alextran.immich.sync'),
     dartOptions: DartOptions(),
     dartPackageName: 'immich_mobile',
@@ -24,6 +23,7 @@ class PlatformAsset {
   final int? height;
   final int durationInSeconds;
   final int orientation;
+  final bool isFavorite;
 
   const PlatformAsset({
     required this.id,
@@ -35,6 +35,7 @@ class PlatformAsset {
     this.height,
     this.durationInSeconds = 0,
     this.orientation = 0,
+    this.isFavorite = false,
   });
 }