TW-692: Implement UploadProfile in presentation layer

assets/l10n/intl_en.arb

@@ -1689,7 +1689,7 @@
         "type": "text",
         "placeholders": {}
     },
-    "search": "Search for people and channels",
+    "searchForPeopleAndChannels": "Search for people and channels",
     "@search": {
         "type": "text",
         "placeholders": {}
@@ -2647,7 +2647,6 @@
     "tapToAllowAccessToYourGallery": "Tap to allow access to your Gallery",
     "tapToAllowAccessToYourCamera": "You can enable camera access in the Settings app to make video calls in",
     "twake": "Twake",
-    "dismiss": "Dismiss",
     "permissionAccess": "Permission access",
     "allow": "Allow",
     "explainStoragePermission": "Twake need access to your storage to preview file",
@@ -2669,7 +2668,6 @@
         }
     },
     "keyboard": "Keyboard",
-    "tapToAllowAccessToYourGallery": "Tap to allow access to your Gallery",
     "changeChatAvatar": "Change the Chat avatar",
     "roomAvatarMaxFileSize": "The avatar size is too large",
     "@roomAvatarMaxFileSize": {},
@@ -2755,5 +2753,6 @@
     "editProfileDescriptions": "Update your profile with a new name, picture and a short introduction.",
     "workIdentitiesInfo": "WORK IDENTITIES INFO",
     "editWorkIdentitiesDescriptions": "Edit your work identity settings such as Matrix ID, email or company name.",
-    "copiedMatrixIdToClipboard": "Copied Matrix ID to clipboard."
+    "copiedMatrixIdToClipboard": "Copied Matrix ID to clipboard.",
+    "changeProfilePhoto": "Change profile photo"
 }

lib/config/go_routes/go_router.dart

@@ -248,9 +248,7 @@ abstract class AppRoutes {
               path: 'profile',
               pageBuilder: (context, state) => defaultPageBuilder(
                 context,
-                SettingsProfile(
-                  profile: state.extra as Profile?,
-                ),
+                const SettingsProfile(),
               ),
             ),
             GoRoute(

lib/di/global/get_it_initializer.dart

@@ -41,7 +41,7 @@ import 'package:fluffychat/domain/usecase/send_file_interactor.dart';
 import 'package:fluffychat/domain/usecase/send_file_on_web_interactor.dart';
 import 'package:fluffychat/domain/usecase/send_image_interactor.dart';
 import 'package:fluffychat/domain/usecase/send_images_interactor.dart';
-import 'package:fluffychat/domain/usecase/settings/upload_profile_interactor.dart';
+import 'package:fluffychat/domain/usecase/settings/update_profile_interactor.dart';
 import 'package:fluffychat/event/twake_event_dispatcher.dart';
 import 'package:fluffychat/utils/responsive/responsive_utils.dart';
 import 'package:get_it/get_it.dart';
@@ -178,8 +178,8 @@ class GetItInitializer {
     getIt.registerSingleton<TimelineSearchEventInteractor>(
       TimelineSearchEventInteractor(),
     );
-    getIt.registerSingleton<UploadProfileInteractor>(
-      UploadProfileInteractor(),
+    getIt.registerSingleton<UpdateProfileInteractor>(
+      UpdateProfileInteractor(),
     );
   }
 }

lib/domain/app_state/settings/upload_profile_failure.dart → lib/domain/app_state/settings/update_profile_failure.dart

@@ -1,9 +1,9 @@
 import 'package:fluffychat/app_state/failure.dart';
 
-class UploadProfileFailure extends Failure {
+class UpdateProfileFailure extends Failure {
   final dynamic exception;
 
-  const UploadProfileFailure(this.exception) : super();
+  const UpdateProfileFailure(this.exception) : super();
 
   @override
   List<Object?> get props => [exception];

lib/domain/app_state/settings/upload_profile_loading.dart → lib/domain/app_state/settings/update_profile_loading.dart

@@ -1,7 +1,7 @@
 import 'package:fluffychat/app_state/success.dart';
 
-class UploadProfileLoading extends Success {
-  const UploadProfileLoading();
+class UpdateProfileLoading extends Success {
+  const UpdateProfileLoading();
 
   @override
   List<Object?> get props => [];

lib/domain/app_state/settings/update_profile_success.dart

@@ -0,0 +1,21 @@
+import 'package:fluffychat/app_state/success.dart';
+
+class UpdateProfileInitial extends Success {
+  @override
+  List<Object?> get props => [];
+}
+
+class UpdateProfileSuccess extends Success {
+  final Uri? avatar;
+  final String? displayName;
+  final bool isDeleteAvatar;
+
+  const UpdateProfileSuccess({
+    this.avatar,
+    this.displayName,
+    this.isDeleteAvatar = false,
+  });
+
+  @override
+  List<Object?> get props => [avatar, displayName, isDeleteAvatar];
+}

lib/domain/usecase/settings/update_profile_interactor.dart

@@ -0,0 +1,47 @@
+import 'package:dartz/dartz.dart';
+import 'package:fluffychat/app_state/failure.dart';
+import 'package:fluffychat/app_state/success.dart';
+import 'package:fluffychat/domain/app_state/settings/update_profile_failure.dart';
+import 'package:fluffychat/domain/app_state/settings/update_profile_loading.dart';
+import 'package:fluffychat/domain/app_state/settings/update_profile_success.dart';
+import 'package:matrix/matrix.dart';
+
+class UpdateProfileInteractor {
+  Stream<Either<Failure, Success>> execute({
+    required Client client,
+    Uri? avatarUrl,
+    bool isDeleteAvatar = false,
+    String? displayName,
+  }) async* {
+    yield const Right(UpdateProfileLoading());
+    try {
+      Logs().d(
+        'UploadProfileInteractor::execute(): Uri - $avatarUrl - displayName - $displayName',
+      );
+      if (avatarUrl != null || isDeleteAvatar) {
+        await client.setAvatarUrl(
+          client.userID!,
+          avatarUrl ?? Uri.parse(''),
+        );
+      }
+      if (displayName != null) {
+        await client.setDisplayName(
+          client.userID!,
+          displayName,
+        );
+      }
+      yield Right(
+        UpdateProfileSuccess(
+          displayName: displayName,
+          avatar: avatarUrl,
+          isDeleteAvatar: isDeleteAvatar,
+        ),
+      );
+    } catch (e) {
+      Logs().d(
+        'UploadAvatarInteractor::execute(): Exception - $e}',
+      );
+      yield Left(UpdateProfileFailure(e));
+    }
+  }
+}

lib/pages/settings_dashboard/settings/settings.dart

@@ -33,9 +33,8 @@ class Settings extends StatefulWidget {
 }
 
 class SettingsController extends State<Settings> with ConnectPageMixin {
-  final ValueNotifier<Profile> profileNotifier = ValueNotifier(
-    Profile(userId: ''),
-  );
+  final ValueNotifier<Uri?> avatarUriNotifier = ValueNotifier(Uri());
+  final ValueNotifier<String?> displayNameNotifier = ValueNotifier('');
 
   StreamSubscription? onAccountDataSubscription;
 
@@ -53,7 +52,7 @@ class SettingsController extends State<Settings> with ConnectPageMixin {
   final ValueNotifier<SettingEnum?> optionsSelectNotifier = ValueNotifier(null);
 
   String get displayName =>
-      profileNotifier.value.displayName ??
+      displayNameNotifier.value ??
       client.mxid(context).localpart ??
       client.mxid(context);
 
@@ -94,7 +93,8 @@ class SettingsController extends State<Settings> with ConnectPageMixin {
     Logs().d(
       'Settings::_getCurrentProfile() - currentProfile: $profile',
     );
-    profileNotifier.value = profile;
+    avatarUriNotifier.value = profile.avatarUrl;
+    displayNameNotifier.value = profile.displayName;
   }
 
   void checkBootstrap() async {
@@ -135,12 +135,9 @@ class SettingsController extends State<Settings> with ConnectPageMixin {
     checkBootstrap();
   }
 
-  void goToSettingsProfile(Profile? profile) async {
+  void goToSettingsProfile() async {
     optionsSelectNotifier.value = SettingEnum.profile;
-    context.push(
-      '/rooms/profile',
-      extra: profile,
-    );
+    context.go('/rooms/profile');
   }
 
   void onClickToSettingsItem(SettingEnum settingEnum) {
@@ -179,7 +176,13 @@ class SettingsController extends State<Settings> with ConnectPageMixin {
   void _handleOnAccountDataSubscription() {
     onAccountDataSubscription = client.onAccountData.stream.listen((event) {
       if (event.type == TwakeInappEventTypes.uploadAvatarEvent) {
-        profileNotifier.value = Profile.fromJson(event.content);
+        final newProfile = Profile.fromJson(event.content);
+        if (newProfile.avatarUrl != avatarUriNotifier.value) {
+          avatarUriNotifier.value = newProfile.avatarUrl;
+        }
+        if (newProfile.displayName != displayNameNotifier.value) {
+          displayNameNotifier.value = newProfile.displayName;
+        }
       }
     });
   }
@@ -197,6 +200,8 @@ class SettingsController extends State<Settings> with ConnectPageMixin {
   @override
   void dispose() {
     onAccountDataSubscription?.cancel();
+    avatarUriNotifier.dispose();
+    displayNameNotifier.dispose();
     super.dispose();
   }