feat: complete update profile bloc

aman-singh7 · aman-singh7 · commit cf7956fb8941 · 2022-05-20T08:10:30.000+05:30
Signed-off-by: Aman &lt;aman2@me.iitr.ac.in&gt;
diff --git a/lib/domain/models/handle.dart b/lib/domain/models/handle.dart
@@ -6,11 +6,11 @@ part 'handle.g.dart';
 @freezed
 class Handle with _$Handle {
   factory Handle({
-    @JsonKey(name: 'handle.codechef') String? codechef,
-    @JsonKey(name: 'handle.codeforces') String? codeforces,
-    @JsonKey(name: 'handle.hackerrank') String? hackerrank,
-    @JsonKey(name: 'handle.spoj') String? spoj,
-    @JsonKey(name: 'handle.leetcode') String? leetcode,
+    @JsonKey(name: 'codechef') String? codechef,
+    @JsonKey(name: 'codeforces') String? codeforces,
+    @JsonKey(name: 'hackerrank') String? hackerrank,
+    @JsonKey(name: 'spoj') String? spoj,
+    @JsonKey(name: 'leetcode') String? leetcode,
   }) = _Handle;
 
   factory Handle.fromJson(Map<String, dynamic> json) => _$HandleFromJson(json);
diff --git a/lib/domain/repositories/user_repository.dart b/lib/domain/repositories/user_repository.dart
@@ -193,6 +193,8 @@ class UserRepository {
     );
 
     if (response['status_code'] == 200) {
+      if (response['data'] == 'null') return <Following>[];
+
       return List<Following>.from(
         response['data'].map((e) => Following.fromJson(e)),
       );
@@ -204,9 +206,7 @@ class UserRepository {
   static Future<bool> verifyHandle(String site, String handle) async {
     final endpoint = 'user/verify/$site?handle=$handle';
 
-    final response = await ApiService.post(
-      endpoint,
-    );
+    final response = await ApiService.get(endpoint);
 
     return response['status_code'] == 200;
   }
@@ -279,6 +279,8 @@ class UserRepository {
     );
 
     if (response['status_code'] == 200) {
+      if (response['data'] == 'null') return <ActivityDetails>[];
+
       return List<ActivityDetails>.from(
         response['data'].map((e) => ActivityDetails.fromJson(e)),
       );
@@ -298,7 +300,7 @@ class UserRepository {
     final response = await ApiService.post(
       endpoint,
       headers: headers,
-      data: <String, String>{
+      data: {
         'new_password': newPassword,
         'old_password': oldPassword,
       },
@@ -319,7 +321,7 @@ class UserRepository {
     final headers = <String, dynamic>{};
 
     ApiService.addTokenToHeaders(headers);
-    final response = await ApiService.post(
+    final response = await ApiService.put(
       endpoint,
       headers: headers,
       data: data,
diff --git a/lib/presentation/profile/widgets/profile_header.dart b/lib/presentation/profile/widgets/profile_header.dart
@@ -7,7 +7,10 @@ import 'package:get/get.dart';
 import '../../../data/constants/assets.dart';
 import '../../../data/constants/colors.dart';
 import '../../../data/constants/routes.dart';
+import '../../../data/constants/strings.dart';
 import '../../../data/constants/styles.dart';
+import '../../../domain/repositories/user_repository.dart';
+import '../../../utils/snackbar.dart';
 import '../bloc/profile_bloc.dart';
 import '../components/follow_button.dart';
 import '../components/following_button.dart';
@@ -85,7 +88,15 @@ class ProfileHeader extends StatelessWidget {
                                 horizontal: 12.w,
                                 vertical: 4.h,
                               ),
-                              onTap: () {},
+                              onTap: () async {
+                                final res = await UserRepository.logout();
+                                if (res) {
+                                  Get.offNamed(AppRoutes.login);
+                                  return;
+                                }
+
+                                showSnackBar(message: AppStrings.genericError);
+                              },
                               child: Text(
                                 'Logout',
                                 style: AppStyles.h6.copyWith(
diff --git a/lib/presentation/update_profile/bloc/update_profile_bloc.dart b/lib/presentation/update_profile/bloc/update_profile_bloc.dart
@@ -5,7 +5,9 @@ import 'package:equatable/equatable.dart';
 import 'package:flutter/widgets.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:freezed_annotation/freezed_annotation.dart';
+import 'package:get/get.dart';
 
+import '../../../data/constants/strings.dart';
 import '../../../data/services/local/image_service.dart';
 import '../../../data/services/local/storage_service.dart';
 import '../../../domain/models/status.dart';
@@ -27,6 +29,7 @@ class UpdateProfileBloc extends Bloc<UpdateProfileEvent, UpdateProfileState> {
     on<UpdateFocusField>(_onUpdateFocusField);
     on<ToggleObscure>(_onToggleObscure);
     on<UpdatePassword>(_onUpdatePassword);
+    on<UpdateUserDetails>(_onUpdateUserDetails);
   }
 
   static List<String> institutes = <String>[
@@ -37,27 +40,43 @@ class UpdateProfileBloc extends Bloc<UpdateProfileEvent, UpdateProfileState> {
     'Indian Institute of Technology Bombay',
   ];
 
+  static User get user => StorageService.user!;
+
+  static Map<String, TextEditingController?> controllers =
+      <String, TextEditingController?>{
+    'name': TextEditingController(),
+    'username': TextEditingController(),
+    'codechef': TextEditingController(),
+    'codeforces': TextEditingController(),
+    'hackerrank': TextEditingController(),
+    'spoj': TextEditingController(),
+    'leetcode': TextEditingController(),
+    'old_pass': TextEditingController(),
+    'new_pass': TextEditingController(),
+    're_enter': TextEditingController(),
+  };
+
+  void _initializeTextField() {
+    controllers['name']?.text = _currentUser.fullname;
+    controllers['username']?.text = _currentUser.username ?? '';
+    controllers['codechef']?.text = _currentUser.handle?.codechef ?? '';
+    controllers['codeforces']?.text = _currentUser.handle?.codeforces ?? '';
+    controllers['hackerrank']?.text = _currentUser.handle?.hackerrank ?? '';
+    controllers['spoj']?.text = _currentUser.handle?.spoj ?? '';
+    controllers['leetcode']?.text = _currentUser.handle?.leetcode ?? '';
+  }
+
   void _onInitialize(Initialize event, Emitter<UpdateProfileState> emit) async {
     final _instituteList = await UserRepository.getInstituteList();
     if (_instituteList.isNotEmpty) {
       institutes = _instituteList;
     }
 
-    final _controllers = <String, TextEditingController?>{
-      'codechef': TextEditingController(),
-      'codeforces': TextEditingController(),
-      'hackerrank': TextEditingController(),
-      'spoj': TextEditingController(),
-      'leetcode': TextEditingController(),
-      'old_pass': TextEditingController(),
-      'new_pass': TextEditingController(),
-      're_enter': TextEditingController(),
-    };
+    _initializeTextField();
 
     emit(state.copyWith(
       status: const Status(),
       user: _currentUser,
-      controllers: _controllers,
     ));
   }
 
@@ -75,14 +94,13 @@ class UpdateProfileBloc extends Bloc<UpdateProfileEvent, UpdateProfileState> {
 
   void _onSwitchView(SwitchView event, Emitter<UpdateProfileState> emit) {
     final currentState = state.showChangePasswordView;
-    final _controllers = state.controllers;
-    _controllers['old_pass']?.text = '';
-    _controllers['new_pass']?.text = '';
-    _controllers['re_enter']?.text = '';
+    _initializeTextField();
+    controllers['old_pass']?.text = '';
+    controllers['new_pass']?.text = '';
+    controllers['re_enter']?.text = '';
     emit(state.copyWith(
       showChangePasswordView: !currentState,
       activePasswordTextField: -1,
-      controllers: _controllers,
     ));
   }
 
@@ -108,8 +126,8 @@ class UpdateProfileBloc extends Bloc<UpdateProfileEvent, UpdateProfileState> {
       UpdatePassword event, Emitter<UpdateProfileState> emit) async {
     emit(state.copyWith(isUpdating: true));
     try {
-      await UserRepository.updatePassword(state.controllers['old_pass']!.text,
-          state.controllers['new_pass']!.text);
+      await UserRepository.updatePassword(controllers['old_pass']!.text.trim(),
+          controllers['new_pass']!.text.trim());
       add(const SwitchView());
       showSnackBar(message: 'Password Changed');
     } on Failure catch (err) {
@@ -118,6 +136,108 @@ class UpdateProfileBloc extends Bloc<UpdateProfileEvent, UpdateProfileState> {
     emit(state.copyWith(isUpdating: false));
   }
 
+  void _onUpdateUserDetails(
+      UpdateUserDetails event, Emitter<UpdateProfileState> emit) async {
+    isChanged = false;
+    emit(state.copyWith(isUpdating: true));
+
+    final _errors = <String, dynamic>{};
+
+    /// Name Field Validation
+    isChanged |= controllers['name']!.text.trim() != state.user?.fullname;
+    if (controllers['name']!.text.isEmpty) {
+      _errors['name'] = 'Required Field';
+    }
+
+    /// Username Field Validation
+    if (controllers['username']!.text.isEmpty) {
+      _errors['username'] = 'Required Field';
+    } else if (controllers['username']?.text.trim() != _currentUser.username) {
+      isChanged = true;
+      final res = await UserRepository.isUsernameAvailable(
+          controllers['username']!.text.trim());
+
+      if (!res) _errors['username'] = 'Already Taken';
+    }
+
+    /// Handles Validation
+    final platforms = [
+      'codechef',
+      'codeforces',
+      'hackerrank',
+      'spoj',
+      'leetcode'
+    ];
+    for (final platform in platforms) {
+      final _handle = controllers[platform]!.text.trim();
+
+      if (_handle.isEmpty) continue;
+
+      if (!compareHandles(platform, _handle)) {
+        isChanged |= true;
+        final res = await UserRepository.verifyHandle(platform, _handle);
+
+        if (!res) _errors[platform] = 'Invalid Handle';
+      }
+    }
+
+    /// Returns if errors are not empty
+    if (_errors.keys.isNotEmpty || !isChanged) {
+      emit(state.copyWith(isUpdating: false, errors: _errors));
+      return;
+    }
+
+    final _updatedData = <String, dynamic>{};
+    _updatedData['fullname'] = controllers['name']?.text.trim();
+    _updatedData['username'] = controllers['username']?.text.trim();
+    _updatedData['institute'] = state.user!.institute;
+    _updatedData['handle.codechef'] = controllers['codechef']?.text.trim();
+    _updatedData['handle.codeforces'] = controllers['codeforces']?.text.trim();
+    _updatedData['handle.hackerrank'] = controllers['hackerrank']?.text.trim();
+    _updatedData['handle.spoj'] = controllers['spoj']?.text.trim();
+    _updatedData['handle.leetcode'] = controllers['leetcode']?.text.trim();
+
+    final statusCode = await UserRepository.updateUserDetails(_updatedData);
+
+    if (state.image != null) {
+      await UserRepository.uploadProfilePicture(state.image!);
+    }
+
+    if (statusCode == 202) {
+      StorageService.user = await UserRepository.fetchUserDetails();
+      showSnackBar(message: 'Updated Sucessfully');
+      Get.back(result: true);
+      return;
+    }
+
+    emit(state.copyWith(isUpdating: false));
+    showSnackBar(message: AppStrings.genericError);
+  }
+
+  /// Compares the value from [TextEditingController] and [User]
+  /// model of that handle. Returns true if they are same
+  bool compareHandles(String platform, String value) {
+    switch (platform) {
+      case 'codechef':
+        return value == state.user?.handle?.codechef;
+
+      case 'codeforces':
+        return value == state.user?.handle?.codeforces;
+
+      case 'hackerrank':
+        return value == state.user?.handle?.hackerrank;
+
+      case 'spoj':
+        return value == state.user?.handle?.spoj;
+
+      case 'leetcode':
+        return value == state.user?.handle?.leetcode;
+
+      default:
+        return false;
+    }
+  }
+
   Future<bool> onWillPop() async {
     if (state.showChangePasswordView) {
       add(const SwitchView());
@@ -129,4 +249,5 @@ class UpdateProfileBloc extends Bloc<UpdateProfileEvent, UpdateProfileState> {
 
   final User _currentUser = StorageService.user!;
   User _updatedUser = StorageService.user!;
+  bool isChanged = false;
 }
diff --git a/lib/presentation/update_profile/bloc/update_profile_event.dart b/lib/presentation/update_profile/bloc/update_profile_event.dart
@@ -58,3 +58,10 @@ class UpdatePassword extends UpdateProfileEvent {
   @override
   List<Object?> get props => [];
 }
+
+class UpdateUserDetails extends UpdateProfileEvent {
+  const UpdateUserDetails();
+
+  @override
+  List<Object?> get props => [];
+}
diff --git a/lib/presentation/update_profile/bloc/update_profile_state.dart b/lib/presentation/update_profile/bloc/update_profile_state.dart
@@ -9,7 +9,7 @@ class UpdateProfileState with _$UpdateProfileState {
     User? user,
     @Default(-1) int activePasswordTextField,
     @Default([true, true, true]) List<bool> passwordFieldObscureState,
-    @Default({}) Map<String, TextEditingController?> controllers,
+    @Default({}) Map<String, dynamic> errors,
     @Default(false) bool isUpdating,
   }) = _UpdateProfileState;
 
diff --git a/lib/presentation/update_profile/update_profile.dart b/lib/presentation/update_profile/update_profile.dart
@@ -41,7 +41,12 @@ class UpdateProfile extends StatelessWidget {
                     SizedBox(height: 30.h),
                     PrimaryButton(
                       label: 'Save Changes',
-                      onPressed: () {},
+                      onPressed: () {
+                        if (state.isUpdating) return;
+                        context
+                            .read<UpdateProfileBloc>()
+                            .add(const UpdateUserDetails());
+                      },
                     ),
                     SizedBox(height: 30.h),
                   ],
diff --git a/lib/presentation/update_profile/widgets/change_password.dart b/lib/presentation/update_profile/widgets/change_password.dart
@@ -41,7 +41,7 @@ class ChangePassword extends StatelessWidget {
                         index: 0,
                         activeIndex: state.activePasswordTextField,
                         obscure: state.passwordFieldObscureState[0],
-                        controller: state.controllers['old_pass'],
+                        controller: UpdateProfileBloc.controllers['old_pass'],
                       ),
                       SizedBox(height: 24.h),
                       _buildTextField(
@@ -50,7 +50,7 @@ class ChangePassword extends StatelessWidget {
                         index: 1,
                         activeIndex: state.activePasswordTextField,
                         obscure: state.passwordFieldObscureState[1],
-                        controller: state.controllers['new_pass'],
+                        controller: UpdateProfileBloc.controllers['new_pass'],
                       ),
                       SizedBox(height: 24.h),
                       _buildTextField(
@@ -59,8 +59,9 @@ class ChangePassword extends StatelessWidget {
                         index: 2,
                         activeIndex: state.activePasswordTextField,
                         obscure: state.passwordFieldObscureState[2],
-                        controller: state.controllers['re_enter'],
-                        compareText: state.controllers['new_pass']?.text,
+                        controller: UpdateProfileBloc.controllers['re_enter'],
+                        compareText:
+                            UpdateProfileBloc.controllers['new_pass']?.text,
                       ),
                     ],
                   ),
@@ -70,6 +71,7 @@ class ChangePassword extends StatelessWidget {
             PrimaryButton(
               label: 'Update Password',
               onPressed: () {
+                if (state.isUpdating) return;
                 if (_key.currentState!.validate()) {
                   context.read<UpdateProfileBloc>().add(const UpdatePassword());
                 }
diff --git a/lib/presentation/update_profile/widgets/profile_details.dart b/lib/presentation/update_profile/widgets/profile_details.dart
diff --git a/lib/presentation/update_profile/widgets/update_handles.dart b/lib/presentation/update_profile/widgets/update_handles.dart
diff --git a/lib/utils/platform_util.dart b/lib/utils/platform_util.dart
