Merge pull request 'lab prescription details are added andgraph visibility added to the graph page for result that are IRR' (#73) from feature/lab_result into master

Reviewed-on: #73

lib/features/lab/lab_view_model.dart

@@ -22,6 +22,8 @@ class LabViewModel extends ChangeNotifier {
   bool isLabResultsLoading = false;
   bool isLabResultByHospitalLoading = false;
   bool isSpecialResultsLoading = false;
+  bool isGraphVisible = true;
+  bool shouldShowGraph = true;
 
   LabRepo labRepo;
   ErrorHandlerService errorHandlerService;
@@ -225,6 +227,7 @@ class LabViewModel extends ChangeNotifier {
             } catch (e) {}
           });
           LabResult recentResult = recentThree.first;
+          checkIfGraphShouldBeDisplayed(recentResult);
           recentResult.verifiedOn = resultDate(DateUtil.convertStringToDate(recentResult.verifiedOnDateTime!));
           // filteredGraphValues = [filteredGraphValues.first];
           navigationService.push(MaterialPageRoute(
@@ -237,6 +240,12 @@ class LabViewModel extends ChangeNotifier {
     );
   }
 
+  void checkIfGraphShouldBeDisplayed(LabResult recentResult){
+    shouldShowGraph = recentResult.checkIfGraphShouldBeDisplayed();
+    isGraphVisible = shouldShowGraph;
+    notifyListeners();
+  }
+
   Future<void> getPatientSpecialResult(
       PatientLabOrdersResponseModel laborder) async {
     isSpecialResultsLoading = true;
@@ -551,4 +560,9 @@ class LabViewModel extends ChangeNotifier {
         return "normal";
     }
   }
+
+  alterGraphVisibility(){
+    isGraphVisible = !isGraphVisible;
+    notifyListeners();
+  }
 }

lib/features/lab/models/resp_models/lab_result.dart

@@ -24,6 +24,7 @@ class LabResult {
   String? referenceHigh;
   String? criticalLow;
   String? referenceLow;
+  num? resultTypeID;
   String? packageShortDescription;
 
   LabResult(
@@ -80,6 +81,7 @@ class LabResult {
     criticalLow = json['CriticalLow'];
     referenceLow = json['ReferenceLow'];
     packageShortDescription = json['PackageShortDescription'];
+    resultTypeID = json['ResultTypeID'];
   }
 
   Map<String, dynamic> toJson() {
@@ -111,6 +113,21 @@ class LabResult {
     return data;
   }
 
+  bool checkIfGraphShouldBeDisplayed(){
+    if (resultTypeID == null) return false;
+    if (resultTypeID == 6) return false;
+    if (referanceRange == null || referanceRange == "" || referanceRange == "\n") return false;
+    bool isDigit =  RegExp(r"\\d+").hasMatch("$resultValue");
+    if(isDigit) return true;
+    try {
+      num.parse(resultValue ?? "");
+    } catch (e) {
+      return false;
+    }
+
+    return true;
+  }
+
   @override
   String toString() {
     return 'LabOrderResult(flag: $calculatedResultFlag, value: $resultValue, verifiedOn: $verifiedOnDateTime)';

lib/presentation/lab/lab_result_via_hospital/LabResultList.dart

@@ -4,7 +4,7 @@ import 'package:hmg_patient_app_new/extensions/string_extensions.dart';
 import 'package:hmg_patient_app_new/features/lab/lab_view_model.dart';
 import 'package:hmg_patient_app_new/features/lab/models/resp_models/lab_result.dart';
 import 'package:hmg_patient_app_new/presentation/lab/lab_result_via_hospital/lab_order_result_item.dart';
-import 'package:provider/provider.dart' show Selector, Provider;
+import 'package:provider/provider.dart' show Selector, Provider, ReadContext;
 
 class LabResultList extends StatelessWidget {
   late LabViewModel model;
@@ -15,7 +15,7 @@ class LabResultList extends StatelessWidget {
     return Selector<LabViewModel, List<LabResult>>(
       selector: (_, model) => model.mainLabResultsByHospitals,
       builder: (__, list, ___) {
-        if (list.isEmpty) {
+        if (list.isEmpty && context.read<LabViewModel>().labSpecialResult.isEmpty) {
           return Utils.getNoDataWidget(context,
               noDataText: "You don't have any lab results yet."
                   .needTranslation);

lib/presentation/lab/lab_result_via_hospital/lab_order_result_item.dart

@@ -7,6 +7,7 @@ import 'package:hmg_patient_app_new/core/utils/date_util.dart';
 import 'package:hmg_patient_app_new/core/utils/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/lab/lab_view_model.dart' show LabViewModel;
 import 'package:hmg_patient_app_new/features/lab/models/resp_models/lab_result.dart';
 import 'package:hmg_patient_app_new/features/lab/models/resp_models/patient_lab_orders_response_model.dart';
 import 'package:hmg_patient_app_new/generated/locale_keys.g.dart';
@@ -14,6 +15,7 @@ import 'package:hmg_patient_app_new/theme/colors.dart';
 import 'package:hmg_patient_app_new/widgets/buttons/custom_button.dart';
 import 'package:hmg_patient_app_new/widgets/chip/app_custom_chip_widget.dart';
 import 'package:hmg_patient_app_new/widgets/chip/custom_chip_widget.dart';
+import 'package:provider/provider.dart';
 
 class LabOrderResultItem extends StatelessWidget {
   final VoidCallback onTap;
@@ -45,8 +47,49 @@ class LabOrderResultItem extends StatelessWidget {
                 child: '${tests!.description}'.toText14(weight: FontWeight.w500),
               ),
               '${tests!.packageShortDescription}'.toText12(fontWeight: FontWeight.w500, color: AppColors.textColorLight),
-              //
+              SizedBox(height: 12.h),
-              SizedBox(height: 24.h),
+              Row(
+                mainAxisSize: MainAxisSize.max,
+                children: [
+                  Flexible(
+                    child: Text(
+                      tests?.resultValue ?? "",
+                      style: TextStyle(
+                        fontSize: 24.fSize,
+                        fontWeight: FontWeight.w600,
+                        fontFamily: 'Poppins',
+                        color: context.read<LabViewModel>().getColor(
+                          tests?.calculatedResultFlag ?? "",
+                        ),
+                        letterSpacing: -2,
+                      ),
+                      overflow: TextOverflow.ellipsis, // prevent overflow
+                      maxLines: 1,
+                      softWrap: false,
+                    ),
+                  ),
+                  SizedBox(width: 4.h,),
+                  Expanded(
+                    flex: 2,
+                    child: Visibility(
+                      visible: tests?.referanceRange != null,
+                      child: Text(
+                        "(Reference range ${tests?.referanceRange})".needTranslation,
+                        style: TextStyle(
+                          fontSize: 12.fSize,
+                          fontWeight: FontWeight.w500,
+                          fontFamily: 'Poppins',
+                          color: AppColors.greyTextColor,
+                        ),
+                        // overflow: TextOverflow.ellipsis,
+                        // maxLines: 2,
+                        softWrap: true,
+                      ),
+                    ),
+                  ),
+                ],
+              ),
+              SizedBox(height: 12.h),
               Row(
                 mainAxisAlignment: MainAxisAlignment.spaceBetween,
                 children: [

lib/presentation/lab/lab_results/lab_result_details.dart

@@ -15,35 +15,38 @@ import 'package:hmg_patient_app_new/presentation/lab/lab_results/lab_result_list
 import 'package:hmg_patient_app_new/theme/colors.dart' show AppColors;
 import 'package:hmg_patient_app_new/widgets/appbar/collapsing_list_view.dart';
 import 'package:hmg_patient_app_new/widgets/graph/custom_graph.dart';
-import 'package:provider/provider.dart' show Consumer, Provider;
+import 'package:provider/provider.dart' show Consumer, Provider, ReadContext;
 
 import '../../../widgets/common_bottom_sheet.dart'
     show showCommonBottomSheetWithoutHeight;
 
 class LabResultDetails extends StatelessWidget {
   final LabResult recentLabResult;
+  final String? testDescription;
 
-  // final List<DataPoint> graphPoint;
+  const LabResultDetails(
-  late LabViewModel model;
+      {super.key,
-  String? testDescription;
+      required this.recentLabResult,
-
+      required this.testDescription});
-  LabResultDetails({super.key, required this.recentLabResult, required this.testDescription});
 
   @override
   Widget build(BuildContext context) {
-    model = Provider.of<LabViewModel>(context, listen: false);
     return CollapsingListView(
       title: 'Lab Result Details'.needTranslation,
       child: SingleChildScrollView(
         child: Column(
           spacing: 16.h,
-          children: [LabNameAndStatus, LabGraph(context)],
+          children: [
+            LabNameAndStatus(context),
+            getLabDescription(context),
+            LabGraph(context)
+          ],
         ).paddingAll(24.h),
       ),
     );
   }
 
-  Widget get LabNameAndStatus => Container(
+  Widget LabNameAndStatus(BuildContext context) => Container(
       decoration: RoundedRectangleBorder().toSmoothCornerDecoration(
         color: AppColors.whiteColor,
         borderRadius: 24.h,
@@ -80,11 +83,12 @@ class LabResultDetails extends StatelessWidget {
           //todo change the text color according to the provided test values
           Row(
             crossAxisAlignment: CrossAxisAlignment.center,
+
             children: [
               Expanded(
                 child: Row(
-                  spacing: 4.h,
 
+                  mainAxisSize: MainAxisSize.max,
                   children: [
                     Flexible(
                       child: Text(
@@ -93,7 +97,7 @@ class LabResultDetails extends StatelessWidget {
                           fontSize: 24.fSize,
                           fontWeight: FontWeight.w600,
                           fontFamily: 'Poppins',
-                          color: model.getColor(
+                          color: context.read<LabViewModel>().getColor(
                                 recentLabResult.calculatedResultFlag ?? "",
                           ),
                           letterSpacing: -2,
@@ -103,7 +107,10 @@ class LabResultDetails extends StatelessWidget {
                         softWrap: false,
                       ),
                     ),
-                    Visibility(
+                    SizedBox(width: 4.h,),
+                    Expanded(
+                      flex: 2,
+                      child: Visibility(
                         visible: recentLabResult.referanceRange != null,
                         child: Text(
                           "(Reference range ${recentLabResult.referanceRange})".needTranslation,
@@ -113,37 +120,44 @@ class LabResultDetails extends StatelessWidget {
                             fontFamily: 'Poppins',
                             color: AppColors.greyTextColor,
                           ),
-                        overflow: TextOverflow.ellipsis,
+                          // overflow: TextOverflow.ellipsis,
-                        maxLines: 1,
+                          // maxLines: 2,
-                        softWrap: false,
+                          softWrap: true,
+                        ),
                       ),
                     ),
                   ],
                 ),
               ),
-              Utils.buildSvgWithAssets(
+              SizedBox(
+                width: 21,
+                child: Utils.buildSvgWithAssets(
                   icon: AppAssets.lab_result_indicator,
                   width: 21,
                   height: 23,
-                iconColor: model.getColor(
+                  iconColor: context.read<LabViewModel>().getColor(
                         recentLabResult.calculatedResultFlag ?? "",
                   ),
                 ),
+              ),
             ],
           )
 
         ],
       ));
 
-  Widget LabGraph(BuildContext context) => Consumer<DateRangeSelectorRangeViewModel>(
+  Widget LabGraph(BuildContext context) => Consumer<LabViewModel>(
-      builder: (_, model, ___) => Consumer<LabViewModel>(
         builder: (_, labmodel, ___) => Container(
             decoration: RoundedRectangleBorder().toSmoothCornerDecoration(
               color: AppColors.whiteColor,
               borderRadius: 24.h,
               hasShadow: true,
             ),
-                height: model.isGraphVisible?260.h:(labmodel.filteredGraphValues.length<3)?(labmodel.filteredGraphValues.length*64)+80.h:260.h,
+            height: labmodel.isGraphVisible
+                ? 260.h
+                : (labmodel.filteredGraphValues.length < 3)
+                    ? (labmodel.filteredGraphValues.length * 64) + 80.h
+                    : 260.h,
             padding: EdgeInsets.all(16.h),
             child: Column(
               mainAxisAlignment: MainAxisAlignment.spaceAround,
@@ -153,11 +167,12 @@ class LabResultDetails extends StatelessWidget {
                   mainAxisAlignment: MainAxisAlignment.spaceBetween,
                   children: [
                     Text(
-                          model.isGraphVisible?LocaleKeys.historyFlowchart.tr(): LocaleKeys.history.tr(),
+                      labmodel.isGraphVisible
+                          ? LocaleKeys.historyFlowchart.tr()
+                          : LocaleKeys.history.tr(),
                       style: TextStyle(
                         fontSize: 16,
                         fontFamily: 'Poppins',
-
                         fontWeight: FontWeight.w600,
                         color: AppColors.textColor,
                       ),
@@ -167,11 +182,15 @@ class LabResultDetails extends StatelessWidget {
                       children: [
                         //todo handle when the graph icon is being displayed
                         Utils.buildSvgWithAssets(
-                                    icon: model.isGraphVisible?AppAssets.ic_list:AppAssets.ic_graph,
+                                icon: labmodel.isGraphVisible
+                                    ? AppAssets.ic_list
+                                    : AppAssets.ic_graph,
                                 width: 24.h,
                                 height: 24.h)
                             .onPress(() {
-                                  model.alterGraphVisibility();
+                          if (labmodel.shouldShowGraph) {
+                            labmodel.alterGraphVisibility();
+                          }
                         }),
                         Utils.buildSvgWithAssets(
                                 icon: AppAssets.ic_date_filter,
@@ -183,7 +202,6 @@ class LabResultDetails extends StatelessWidget {
                             context,
                             child: DateRangeSelector(
                               onRangeSelected: (start, end) {
-
                                 // if (start != null) {
                                 labmodel.getSelectedDateRange(start, end);
                                 // }
@@ -197,11 +215,11 @@ class LabResultDetails extends StatelessWidget {
                       ],
                     )
                   ],
-                    ).paddingOnly(bottom: model.isGraphVisible? 16.h :24.h),
+                ).paddingOnly(bottom: labmodel.isGraphVisible ? 16.h : 24.h),
-                    historyBody(model, labmodel)
+                historyBody(labmodel)
               ],
             )),
-          ));
+      );
 
   Widget leftLabels(String value) {
     return Text(
@@ -229,17 +247,15 @@ class LabResultDetails extends StatelessWidget {
     );
   }
 
-  Widget historyBody(DateRangeSelectorRangeViewModel model, LabViewModel labmodel) {
+  Widget historyBody(LabViewModel labmodel) {
-    if(model.isGraphVisible){
+    if (labmodel.isGraphVisible && labmodel.shouldShowGraph) {
       var graphColor = labmodel.getColor(recentLabResult.calculatedResultFlag??"N");
       return CustomGraph(
           dataPoints: labmodel.filteredGraphValues,
-          // maxY: 100,
-
           makeGraphBasedOnActualValue: true,
           leftLabelReservedSize: 40,
           leftLabelInterval: getInterval(labmodel),
-          maxY: (labmodel.maxY)+(getInterval(labmodel)??0)/5,
+          maxY: (labmodel.maxY)+(getInterval(labmodel)??0)/2,
           maxX: labmodel.filteredGraphValues.length.toDouble()-.75,
           leftLabelFormatter: (value) {
             return leftLabels(value.toStringAsFixed(2).tr());
@@ -279,11 +295,11 @@ class LabResultDetails extends StatelessWidget {
           scrollDirection: Axis.horizontal,
           height: 180.h);
     }else {
-      return labHistoryList(model, labmodel);
+      return labHistoryList(labmodel);
     }
   }
 
-  Widget labHistoryList(DateRangeSelectorRangeViewModel model, LabViewModel labmodel) {
+  Widget labHistoryList(LabViewModel labmodel) {
     return SizedBox(
       height: labmodel.filteredGraphValues.length<3?labmodel.filteredGraphValues.length*64:180.h,
       child: ListView.separated(
@@ -315,4 +331,27 @@ class LabResultDetails extends StatelessWidget {
     if(maxX >100 && maxX < 200) return 30;
     return 50;
   }
+
+  Widget getLabDescription(BuildContext context) {
+    return Container(
+        decoration: RoundedRectangleBorder().toSmoothCornerDecoration(
+          color: AppColors.whiteColor,
+          borderRadius: 24.h,
+          hasShadow: true,
+        ),
+        height: 98.h,
+        padding: EdgeInsets.all(16.h),
+        child: Column(
+          crossAxisAlignment: CrossAxisAlignment.start,
+          spacing: 8.h,
+          children: [
+            "What is this result?"
+                .needTranslation
+                .toText16(weight: FontWeight.w600, color: AppColors.textColor),
+            testDescription?.toText12(
+                    fontWeight: FontWeight.w500, color: AppColors.textColorLight) ??
+                SizedBox.shrink()
+          ],
+        ));
+  }
 }

lib/widgets/date_range_selector/viewmodel/date_range_view_model.dart

@@ -17,7 +17,7 @@ class DateRangeSelectorRangeViewModel extends ChangeNotifier {
     'Nov',
     'Dec'
   ];
-  bool isGraphVisible = true;
+
   Range? _currentlySelectedRange;
 
   Range? get currentlySelectedRange => _currentlySelectedRange;
@@ -77,7 +77,6 @@ class DateRangeSelectorRangeViewModel extends ChangeNotifier {
     toDate = null;
     fromDate = null;
     currentlySelectedRange = null;
-    isGraphVisible = true;
     notifyListeners();
   }
 
@@ -86,8 +85,4 @@ class DateRangeSelectorRangeViewModel extends ChangeNotifier {
     notifyListeners();
   }
 
-  alterGraphVisibility(){
-    isGraphVisible = !isGraphVisible;
-    notifyListeners();
-  }
 }