Merge pull request 'data from MyAppointmentViewmodel and MyAppointmentRepo migrated to bookappointment viewmodel and repo' (#40) from feature/search_by_region into master

Reviewed-on: #40

lib/features/book_appointments/book_appointments_repo.dart

@@ -7,6 +7,7 @@ import 'package:hmg_patient_app_new/core/utils/date_util.dart';
 import 'package:hmg_patient_app_new/features/book_appointments/models/resp_models/doctor_profile_response_model.dart';
 import 'package:hmg_patient_app_new/features/book_appointments/models/resp_models/doctors_list_response_model.dart';
 import 'package:hmg_patient_app_new/features/book_appointments/models/resp_models/get_clinic_list_response_model.dart';
+import 'package:hmg_patient_app_new/features/my_appointments/models/resp_models/hospital_model.dart';
 import 'package:hmg_patient_app_new/features/my_appointments/models/resp_models/patient_appointment_history_response_model.dart';
 import 'package:hmg_patient_app_new/services/logger_service.dart';
 
@@ -40,6 +41,9 @@ abstract class BookAppointmentsRepo {
       String? invoiceNoVP,
       Function(dynamic)? onSuccess,
       Function(String)? onError});
+
+  Future<Either<Failure, GenericApiModel<List<HospitalsModel>>>>
+  getProjectList();
 }
 
 class BookAppointmentsRepoImp implements BookAppointmentsRepo {
@@ -362,4 +366,47 @@ class BookAppointmentsRepoImp implements BookAppointmentsRepo {
       return Left(UnknownFailure(e.toString()));
     }
   }
+
+  @override
+  Future<Either<Failure, GenericApiModel<List<HospitalsModel>>>>
+  getProjectList() async {
+    Map<String, dynamic> request = {};
+
+    try {
+      GenericApiModel<List<HospitalsModel>>? apiResponse;
+      Failure? failure;
+      await apiClient.post(
+        GET_PROJECT_LIST,
+        body: request,
+        onFailure: (error, statusCode, {messageStatus, failureType}) {
+          failure = failureType;
+        },
+        onSuccess: (response, statusCode, {messageStatus, errorMessage}) {
+          try {
+            final list = response['ListProject'];
+
+            final appointmentsList = list
+                .map((item) =>
+                HospitalsModel.fromJson(item as Map<String, dynamic>))
+                .toList()
+                .cast<HospitalsModel>();
+
+            apiResponse = GenericApiModel<List<HospitalsModel>>(
+              messageStatus: messageStatus,
+              statusCode: statusCode,
+              errorMessage: null,
+              data: appointmentsList,
+            );
+          } catch (e) {
+            failure = DataParsingFailure(e.toString());
+          }
+        },
+      );
+      if (failure != null) return Left(failure!);
+      if (apiResponse == null) return Left(ServerFailure("Unknown error"));
+      return Right(apiResponse!);
+    } catch (e) {
+      return Left(UnknownFailure(e.toString()));
+    }
+  }
 }

lib/features/book_appointments/book_appointments_view_model.dart

@@ -1,8 +1,10 @@
 import 'package:easy_localization/easy_localization.dart';
 import 'package:flutter/material.dart';
 import 'package:hmg_patient_app_new/core/app_state.dart';
+import 'package:hmg_patient_app_new/core/cache_consts.dart';
 import 'package:hmg_patient_app_new/core/dependencies.dart';
 import 'package:hmg_patient_app_new/core/utils/date_util.dart';
+import 'package:hmg_patient_app_new/core/utils/doctor_response_mapper.dart';
 import 'package:hmg_patient_app_new/core/utils/loading_utils.dart';
 import 'package:hmg_patient_app_new/core/utils/size_utils.dart';
 import 'package:hmg_patient_app_new/core/utils/utils.dart';
@@ -13,6 +15,8 @@ import 'package:hmg_patient_app_new/features/book_appointments/models/resp_model
 import 'package:hmg_patient_app_new/features/book_appointments/models/resp_models/doctors_list_response_model.dart';
 import 'package:hmg_patient_app_new/features/book_appointments/models/resp_models/get_clinic_list_response_model.dart';
 import 'package:hmg_patient_app_new/features/book_appointments/models/timeslots.dart';
+import 'package:hmg_patient_app_new/features/my_appointments/models/facility_selection.dart';
+import 'package:hmg_patient_app_new/features/my_appointments/models/resp_models/doctor_list_api_response.dart';
 import 'package:hmg_patient_app_new/features/my_appointments/models/resp_models/patient_appointment_history_response_model.dart';
 import 'package:hmg_patient_app_new/features/my_appointments/my_appointments_view_model.dart';
 import 'package:hmg_patient_app_new/generated/locale_keys.g.dart';
@@ -21,6 +25,7 @@ import 'package:hmg_patient_app_new/services/error_handler_service.dart';
 import 'package:hmg_patient_app_new/services/navigation_service.dart';
 import 'package:hmg_patient_app_new/widgets/common_bottom_sheet.dart';
 import 'package:hmg_patient_app_new/widgets/transitions/fade_page.dart';
+import 'package:location/location.dart' show Location;
 
 class BookAppointmentsViewModel extends ChangeNotifier {
   int selectedTabIndex = 0;
@@ -60,6 +65,10 @@ class BookAppointmentsViewModel extends ChangeNotifier {
   MyAppointmentsViewModel myAppointmentsViewModel;
 
   late AppState _appState;
+  RegionList? hospitalList;
+  RegionList? filteredHospitalList;
+  FacilitySelection currentlySelectedFacility = FacilitySelection.ALL;
+  bool isRegionListLoading = false;
 
   BookAppointmentsViewModel({required this.bookAppointmentsRepo, required this.errorHandlerService, required this.navigationService, required this.myAppointmentsViewModel});
 
@@ -380,4 +389,84 @@ class BookAppointmentsViewModel extends ChangeNotifier {
       },
     );
   }
+
+  Future<void> getRegionMappedProjectList() async {
+    if(hospitalList != null && hospitalList!.registeredDoctorMap != null && hospitalList!.registeredDoctorMap!.isNotEmpty){
+      filteredHospitalList = hospitalList;
+      return;
+    }
+    isRegionListLoading = true;
+    notifyListeners();
+    final result = await bookAppointmentsRepo.getProjectList();
+
+    result.fold(
+          (failure) async =>
+      await errorHandlerService.handleError(failure: failure),
+          (apiResponse) async {
+        if (apiResponse.messageStatus == 2) {
+          // dialogService.showErrorDialog(message: apiResponse.errorMessage!, onOkPressed: () {});
+        } else if (apiResponse.messageStatus == 1) {
+          var projectList = apiResponse.data!;
+          hospitalList = await DoctorMapper.getMappedHospitals(projectList,
+              isArabic: false);
+          var lat = await Utils.getNumFromPrefs(CacheConst.userLat);
+
+          var lng = await Utils.getNumFromPrefs(CacheConst.userLong);
+          var isLocationEnabled = (lat != 0) && (lng != 0);
+          hospitalList =
+          await DoctorMapper.sortList(isLocationEnabled, hospitalList!);
+
+          isRegionListLoading = false;
+          filteredHospitalList = hospitalList;
+          notifyListeners();
+        }
+      },
+    );
+  }
+
+  void setSelectedFacility(FacilitySelection selection) {
+    currentlySelectedFacility = selection;
+    notifyListeners();
+  }
+
+  void filterHospitalListByString(String? value, String? selectedRegionId, bool isHMG) {
+    if(value ==null || value.isEmpty){
+      filteredHospitalList = hospitalList;
+    } else {
+      filteredHospitalList = RegionList();
+
+      var list = isHMG
+          ? hospitalList?.registeredDoctorMap![selectedRegionId]!.hmgDoctorList
+          : hospitalList?.registeredDoctorMap![selectedRegionId]!.hmcDoctorList;
+
+      if(list != null && list.isEmpty){ notifyListeners(); return;}
+
+      var filteredList = list!.where((element) =>
+          element.filterName!.toLowerCase().contains(value.toLowerCase())
+      ).toList();
+      var regionData = PatientDoctorAppointmentListByRegion();
+      if(isHMG){
+        regionData.hmgDoctorList = filteredList;
+        regionData.hmgSize = filteredList.length;
+      } else {
+        regionData.hmcDoctorList = filteredList;
+        regionData.hmcSize = filteredList.length;
+      }
+
+      filteredHospitalList?.registeredDoctorMap = {
+        selectedRegionId! : regionData
+      };
+    }
+    notifyListeners();
+  }
+
+  Future<bool> isLocationEnabled() async{
+    return await Location().serviceEnabled();
+  }
+
+  bool getLocationStatus() {
+    bool isLocationAvaiable =  false;
+    isLocationEnabled().then((value) => isLocationAvaiable = value);
+    return isLocationAvaiable;
+  }
 }

lib/features/my_appointments/my_appointments_repo.dart

@@ -39,8 +39,7 @@ abstract class MyAppointmentsRepo {
 
   Future<Either<Failure, GenericApiModel<List<PatientAppointmentHistoryResponseModel>>>> getPatientDoctorsList();
 
-  Future<Either<Failure, GenericApiModel<List<HospitalsModel>>>>
-      getProjectList();
+
 }
 
 class MyAppointmentsRepoImp implements MyAppointmentsRepo {
@@ -496,47 +495,4 @@ class MyAppointmentsRepoImp implements MyAppointmentsRepo {
       return Left(UnknownFailure(e.toString()));
     }
   }
-
-  @override
-  Future<Either<Failure, GenericApiModel<List<HospitalsModel>>>>
-      getProjectList() async {
-    Map<String, dynamic> request = {};
-
-    try {
-      GenericApiModel<List<HospitalsModel>>? apiResponse;
-      Failure? failure;
-      await apiClient.post(
-        GET_PROJECT_LIST,
-        body: request,
-        onFailure: (error, statusCode, {messageStatus, failureType}) {
-          failure = failureType;
-        },
-        onSuccess: (response, statusCode, {messageStatus, errorMessage}) {
-          try {
-            final list = response['ListProject'];
-
-            final appointmentsList = list
-                .map((item) =>
-                    HospitalsModel.fromJson(item as Map<String, dynamic>))
-                .toList()
-            .cast<HospitalsModel>();
-
-            apiResponse = GenericApiModel<List<HospitalsModel>>(
-              messageStatus: messageStatus,
-              statusCode: statusCode,
-              errorMessage: null,
-              data: appointmentsList,
-            );
-          } catch (e) {
-            failure = DataParsingFailure(e.toString());
-          }
-        },
-      );
-      if (failure != null) return Left(failure!);
-      if (apiResponse == null) return Left(ServerFailure("Unknown error"));
-      return Right(apiResponse!);
-    } catch (e) {
-      return Left(UnknownFailure(e.toString()));
-    }
-  }
 }

lib/features/my_appointments/my_appointments_view_model.dart

@@ -41,11 +41,6 @@ class MyAppointmentsViewModel extends ChangeNotifier {
 
   PatientAppointmentShareResponseModel? patientAppointmentShareResponseModel;
 
-  RegionList? hospitalList;
-  RegionList? filteredHospitalList;
-  FacilitySelection currentlySelectedFacility = FacilitySelection.ALL;
-  bool isRegionListLoading = false;
-
   MyAppointmentsViewModel(
       {required this.myAppointmentsRepo, required this.errorHandlerService,required this.appState});
 
@@ -65,7 +60,6 @@ class MyAppointmentsViewModel extends ChangeNotifier {
     isAppointmentPatientShareLoading = true;
     isTimeLineAppointmentsLoading = true;
     isPatientMyDoctorsLoading = true;
-    isRegionListLoading = true;
     notifyListeners();
   }
 
@@ -373,84 +367,4 @@ class MyAppointmentsViewModel extends ChangeNotifier {
       },
     );
   }
-
-  Future<void> getRegionMappedProjectList() async {
-    if(hospitalList != null && hospitalList!.registeredDoctorMap != null && hospitalList!.registeredDoctorMap!.isNotEmpty){
-      filteredHospitalList = hospitalList;
-      return;
-    }
-    isRegionListLoading = true;
-    notifyListeners();
-    final result = await myAppointmentsRepo.getProjectList();
-
-    result.fold(
-      (failure) async =>
-          await errorHandlerService.handleError(failure: failure),
-      (apiResponse) async {
-        if (apiResponse.messageStatus == 2) {
-          // dialogService.showErrorDialog(message: apiResponse.errorMessage!, onOkPressed: () {});
-        } else if (apiResponse.messageStatus == 1) {
-          var projectList = apiResponse.data!;
-          hospitalList = await DoctorMapper.getMappedHospitals(projectList,
-              isArabic: false);
-          var lat = await Utils.getNumFromPrefs(CacheConst.userLat);
-
-          var lng = await Utils.getNumFromPrefs(CacheConst.userLong);
-          var isLocationEnabled = (lat != 0) && (lng != 0);
-          hospitalList =
-              await DoctorMapper.sortList(isLocationEnabled, hospitalList!);
-
-          isRegionListLoading = false;
-          filteredHospitalList = hospitalList;
-          notifyListeners();
-        }
-      },
-    );
-  }
-
-  void setSelectedFacility(FacilitySelection selection) {
-    currentlySelectedFacility = selection;
-    notifyListeners();
-  }
-
-  void filterHospitalListByString(String? value, String? selectedRegionId, bool isHMG) {
-    if(value ==null || value.isEmpty){
-      filteredHospitalList = hospitalList;
-    } else {
-      filteredHospitalList = RegionList();
-
-      var list = isHMG
-          ? hospitalList?.registeredDoctorMap![selectedRegionId]!.hmgDoctorList
-          : hospitalList?.registeredDoctorMap![selectedRegionId]!.hmcDoctorList;
-
-      if(list != null && list.isEmpty){ notifyListeners(); return;}
-
-      var filteredList = list!.where((element) =>
-          element.filterName!.toLowerCase().contains(value.toLowerCase())
-         ).toList();
-      var regionData = PatientDoctorAppointmentListByRegion();
-      if(isHMG){
-        regionData.hmgDoctorList = filteredList;
-        regionData.hmgSize = filteredList.length;
-      } else {
-        regionData.hmcDoctorList = filteredList;
-        regionData.hmcSize = filteredList.length;
-      }
-
-      filteredHospitalList?.registeredDoctorMap = {
-        selectedRegionId! : regionData
-      };
-    }
-    notifyListeners();
-  }
-
-  Future<bool> isLocationEnabled() async{
-     return await Location().serviceEnabled();
-  }
-
-  bool getLocationStatus() {
-    bool isLocationAvaiable =  false;
-    isLocationEnabled().then((value) => isLocationAvaiable = value);
-    return isLocationAvaiable;
-  }
 }

lib/presentation/appointments/widgets/faculity_selection/facility_type_selection_widget.dart

@@ -5,6 +5,7 @@ import 'package:hmg_patient_app_new/core/app_assets.dart';
 import 'package:hmg_patient_app_new/core/utils/size_utils.dart';
 import 'package:hmg_patient_app_new/extensions/string_extensions.dart';
 import 'package:hmg_patient_app_new/extensions/widget_extensions.dart';
+import 'package:hmg_patient_app_new/features/book_appointments/book_appointments_view_model.dart';
 import 'package:hmg_patient_app_new/features/my_appointments/appointment_via_region_viewmodel.dart';
 import 'package:hmg_patient_app_new/features/my_appointments/models/facility_selection.dart';
 import 'package:hmg_patient_app_new/features/my_appointments/my_appointments_view_model.dart'
@@ -15,7 +16,7 @@ import 'package:hmg_patient_app_new/theme/colors.dart';
 import 'package:provider/provider.dart' show Provider;
 
 class FacilityTypeSelectionWidget extends StatelessWidget {
-  late MyAppointmentsViewModel myAppointmentsViewModel;
+  late BookAppointmentsViewModel bookAppointmentViewModel;
   late AppointmentViaRegionViewmodel regionalViewModel;
   final String selectedRegion;
 
@@ -23,7 +24,7 @@ class FacilityTypeSelectionWidget extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
-    myAppointmentsViewModel = Provider.of<MyAppointmentsViewModel>(context);
+    bookAppointmentViewModel = Provider.of<BookAppointmentsViewModel>(context);
     regionalViewModel = Provider.of<AppointmentViaRegionViewmodel>(context);
     return Column(
       crossAxisAlignment: CrossAxisAlignment.start,
@@ -50,7 +51,7 @@ class FacilityTypeSelectionWidget extends StatelessWidget {
           title: "HMG".needTranslation,
           subTitle: LocaleKeys.hospitalsWithCount.tr(namedArgs: {
             'count':
-                "${myAppointmentsViewModel.hospitalList?.registeredDoctorMap?[selectedRegion]?.hmgSize ?? 0}"
+                "${bookAppointmentViewModel.hospitalList?.registeredDoctorMap?[selectedRegion]?.hmgSize ?? 0}"
           }),
         ).onPress(
           () {
@@ -65,7 +66,7 @@ class FacilityTypeSelectionWidget extends StatelessWidget {
             title: "HMC".needTranslation,
             subTitle: LocaleKeys.medicalCentersWithCount.tr(namedArgs: {
               'count':
-                  "${myAppointmentsViewModel.hospitalList?.registeredDoctorMap?[selectedRegion]?.hmcSize ?? 0}"
+                  "${bookAppointmentViewModel.hospitalList?.registeredDoctorMap?[selectedRegion]?.hmcSize ?? 0}"
             })).onPress(
           () {
             regionalViewModel.setFacility(FacilitySelection.HMC.name);

lib/presentation/appointments/widgets/hospital_bottom_sheet/hospital_bottom_sheet_body.dart

@@ -4,6 +4,7 @@ import 'package:flutter/material.dart';
 import 'package:hmg_patient_app_new/core/enums.dart';
 import 'package:hmg_patient_app_new/core/utils/size_utils.dart';
 import 'package:hmg_patient_app_new/extensions/string_extensions.dart';
+import 'package:hmg_patient_app_new/features/book_appointments/book_appointments_view_model.dart';
 import 'package:hmg_patient_app_new/features/my_appointments/appointment_via_region_viewmodel.dart';
 import 'package:hmg_patient_app_new/features/my_appointments/models/facility_selection.dart';
 import 'package:hmg_patient_app_new/features/my_appointments/my_appointments_view_model.dart';
@@ -16,7 +17,7 @@ import 'package:hmg_patient_app_new/widgets/input_widget.dart';
 import 'package:provider/provider.dart';
 
 class HospitalBottomSheetBody extends StatelessWidget {
-  late MyAppointmentsViewModel appointmentsViewModel;
+  late BookAppointmentsViewModel appointmentsViewModel;
   late AppointmentViaRegionViewmodel regionalViewModel;
   final TextEditingController searchText = TextEditingController();
 
@@ -24,7 +25,7 @@ class HospitalBottomSheetBody extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
-    appointmentsViewModel = Provider.of<MyAppointmentsViewModel>(context);
+    appointmentsViewModel = Provider.of<BookAppointmentsViewModel>(context);
     regionalViewModel = Provider.of<AppointmentViaRegionViewmodel>(context);
     return Column(
       crossAxisAlignment: CrossAxisAlignment.start,

lib/presentation/appointments/widgets/hospital_bottom_sheet/type_selection_widget.dart

@@ -2,6 +2,7 @@ import 'package:flutter/material.dart';
 import 'package:hmg_patient_app_new/core/app_assets.dart';
 import 'package:hmg_patient_app_new/extensions/string_extensions.dart';
 import 'package:hmg_patient_app_new/extensions/widget_extensions.dart';
+import 'package:hmg_patient_app_new/features/book_appointments/book_appointments_view_model.dart';
 import 'package:hmg_patient_app_new/features/my_appointments/models/facility_selection.dart';
 import 'package:hmg_patient_app_new/features/my_appointments/my_appointments_view_model.dart';
 import 'package:hmg_patient_app_new/theme/colors.dart';
@@ -17,7 +18,7 @@ class TypeSelectionWidget extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
-    return Consumer<MyAppointmentsViewModel>(
+    return Consumer<BookAppointmentsViewModel>(
       builder: (_, data, __) => Row(
         spacing: 8,
         mainAxisSize: MainAxisSize.max,

lib/presentation/appointments/widgets/region_bottomsheet/region_list_widget.dart

@@ -4,6 +4,7 @@ import 'package:flutter/material.dart';
 import 'package:hmg_patient_app_new/core/utils/size_utils.dart';
 import 'package:hmg_patient_app_new/core/utils/utils.dart' show Utils;
 import 'package:hmg_patient_app_new/extensions/widget_extensions.dart';
+import 'package:hmg_patient_app_new/features/book_appointments/book_appointments_view_model.dart';
 import 'package:hmg_patient_app_new/features/my_appointments/appointment_via_region_viewmodel.dart';
 import 'package:hmg_patient_app_new/features/my_appointments/my_appointments_view_model.dart'
     show MyAppointmentsViewModel;
@@ -20,7 +21,7 @@ class RegionBottomSheetBody extends StatefulWidget {
 }
 
 class _RegionBottomSheetBodyState extends State<RegionBottomSheetBody> {
-  late MyAppointmentsViewModel myAppointmentsViewModel;
+  late BookAppointmentsViewModel myAppointmentsViewModel;
   late AppointmentViaRegionViewmodel regionalViewModel;
 
   @override
@@ -33,9 +34,9 @@ class _RegionBottomSheetBodyState extends State<RegionBottomSheetBody> {
 
   @override
   Widget build(BuildContext context) {
-    myAppointmentsViewModel = Provider.of<MyAppointmentsViewModel>(context);
+    myAppointmentsViewModel = Provider.of<BookAppointmentsViewModel>(context);
     regionalViewModel = Provider.of<AppointmentViaRegionViewmodel>(context);
-    return Consumer<MyAppointmentsViewModel>(
+    return Consumer<BookAppointmentsViewModel>(
       builder: (context, myAppointmentsVM, child) {
         if (myAppointmentsVM.isRegionListLoading) {
           return Container(