Merge pull request #22 from SyncrowIOT/fix_hour_picker_dialog

new HourPicker

lib/pages/common/hour_picker_dialog.dart

@@ -1,7 +1,10 @@
+
+
 import 'package:flutter/material.dart';
 
 class HourPickerDialog extends StatefulWidget {
   final TimeOfDay initialTime;
+
   const HourPickerDialog({super.key, required this.initialTime});
 
   @override
@@ -9,70 +12,50 @@ class HourPickerDialog extends StatefulWidget {
 }
 
 class _HourPickerDialogState extends State<HourPickerDialog> {
-  late int _selectedHour;
-  bool _isPm = false;
+  late String selectedHour;
 
   @override
   void initState() {
     super.initState();
-    _selectedHour = widget.initialTime.hour > 12
-        ? widget.initialTime.hour - 12
-        : widget.initialTime.hour;
-    _isPm = widget.initialTime.period == DayPeriod.pm;
+    // Initialize the selectedHour with the initial time passed to the dialog
+    selectedHour = widget.initialTime.hour.toString().padLeft(2, '0') + ':00';
   }
 
   @override
   Widget build(BuildContext context) {
     return AlertDialog(
       title: const Text('Select Hour'),
-      content: Row(
-        mainAxisAlignment: MainAxisAlignment.center,
-        children: [
-          DropdownButton<int>(
-            value: _selectedHour,
-            items: List.generate(12, (index) {
-              int displayHour = index + 1;
-              return DropdownMenuItem(
-                value: displayHour,
-                child: Text(displayHour.toString()),
+      content: DropdownButton<String>(
+        value: selectedHour, // Show the currently selected hour
+        items: List.generate(24, (index) {
+          String hour = index.toString().padLeft(2, '0');
+          return DropdownMenuItem<String>(
+            value: '$hour:00',
+            child: Text('$hour:00'),
           );
         }),
-            onChanged: (value) {
+        onChanged: (String? newValue) {
+          if (newValue != null) {
             setState(() {
-                _selectedHour = value!;
+              selectedHour = newValue; // Update the selected hour without closing the dialog
             });
+          }
         },
       ),
-          SizedBox(width: 16.0),
-          DropdownButton<bool>(
-            value: _isPm,
-            items: const [
-              DropdownMenuItem(
-                value: false,
-                child: Text('AM'),
-              ),
-              DropdownMenuItem(
-                value: true,
-                child: Text('PM'),
-              ),
-            ],
-            onChanged: (value) {
-              setState(() {
-                _isPm = value!;
-              });
-            },
-          ),
-        ],
-      ),
       actions: [
         TextButton(
-          onPressed: () => Navigator.of(context).pop(null),
+          onPressed: () => Navigator.of(context).pop(null), // Close the dialog without selection
           child: const Text('Cancel'),
         ),
         TextButton(
           onPressed: () {
-            int hour = _isPm ? _selectedHour + 12 : _selectedHour;
-            Navigator.of(context).pop(TimeOfDay(hour: hour, minute: 0));
+            // Close the dialog and return the selected time
+            Navigator.of(context).pop(
+              TimeOfDay(
+                hour: int.parse(selectedHour.split(':')[0]),
+                minute: 0,
+              ),
+            );
           },
           child: const Text('OK'),
         ),
@@ -86,6 +69,7 @@ Future<TimeOfDay?> showHourPicker({
   required TimeOfDay initialTime,
 }) {
   return showDialog<TimeOfDay>(
+    barrierDismissible: false,
     context: context,
     builder: (context) => HourPickerDialog(initialTime: initialTime),
   );

lib/pages/visitor_password/bloc/visitor_password_bloc.dart

@@ -14,6 +14,7 @@ import 'package:syncrow_web/services/access_mang_api.dart';
 import 'package:syncrow_web/utils/color_manager.dart';
 import 'package:syncrow_web/utils/constants/assets.dart';
 import 'package:syncrow_web/utils/snack_bar.dart';
+import 'package:syncrow_web/utils/style.dart';
 
 class VisitorPasswordBloc extends Bloc<VisitorPasswordEvent, VisitorPasswordState> {
   VisitorPasswordBloc() : super(VisitorPasswordInitial()) {
@@ -32,9 +33,9 @@ class VisitorPasswordBloc extends Bloc<VisitorPasswordEvent, VisitorPasswordStat
     on<SelectTimeEvent>(selectTimeOfLinePassword);
     on<ChangeTimeEvent>(changeTime);
   }
+
   final TextEditingController userNameController = TextEditingController();
   final TextEditingController emailController = TextEditingController();
-
   final TextEditingController deviceNameController = TextEditingController();
   final TextEditingController deviceIdController = TextEditingController();
   final TextEditingController unitNameController = TextEditingController();
@@ -51,6 +52,7 @@ class VisitorPasswordBloc extends Bloc<VisitorPasswordEvent, VisitorPasswordStat
   String accessTypeSelected = 'Online Password';
   String usageFrequencySelected = 'One-Time';
   String passwordController = '';
+  String accessPeriodValidate = '';
 
   bool repeat = false;
 
@@ -377,19 +379,18 @@ class VisitorPasswordBloc extends Bloc<VisitorPasswordEvent, VisitorPasswordStat
             1000; // Divide by 1000 to remove milliseconds
         if (event.isEffective) {
           if (expirationTimeTimeStamp != null && selectedTimestamp > expirationTimeTimeStamp!) {
-            CustomSnackBar.displaySnackBar('Effective Time cannot be later than Expiration Time.');
+            accessPeriodValidate="Effective Time cannot be later than Expiration Time.";
           } else {
-            effectiveTime =
-                selectedDateTime.toString().split('.').first; // Remove seconds and milliseconds
+            accessPeriodValidate='';
+            effectiveTime = selectedDateTime.toString().split('.').first;
             effectiveTimeTimeStamp = selectedTimestamp;
           }
         } else {
           if (effectiveTimeTimeStamp != null && selectedTimestamp < effectiveTimeTimeStamp!) {
-            CustomSnackBar.displaySnackBar(
-                'Expiration Time cannot be earlier than Effective Time.');
+            accessPeriodValidate=  'Expiration Time cannot be earlier than Effective Time.';
           } else {
-            expirationTime =
-                selectedDateTime.toString().split('.').first; // Remove seconds and milliseconds
+            accessPeriodValidate='';
+            expirationTime = selectedDateTime.toString().split('.').first;
             expirationTimeTimeStamp = selectedTimestamp;
           }
         }

lib/pages/visitor_password/view/visitor_password_dialog.dart

@@ -404,6 +404,9 @@ class VisitorPasswordDialog extends StatelessWidget {
                                         ? visitorBloc.expirationTime
                                         : visitorBloc.endTimeAccess.toString(),
                                     icon: Assets.calendarIcon),
+                              const SizedBox(height: 10,),
+                              Text(visitorBloc.accessPeriodValidate,
+                                style: Theme.of(context).textTheme.bodyMedium!.copyWith(color: ColorsManager.red),),
                               const SizedBox(
                                 height: 20,
                               ),

lib/utils/style.dart

@@ -41,3 +41,4 @@ BoxDecoration containerDecoration = BoxDecoration(
     ],
     color: ColorsManager.boxColor,
     borderRadius: const BorderRadius.all(Radius.circular(10)));
+

pubspec.yaml

@@ -50,7 +50,6 @@ dependencies:
   dropdown_search: ^5.0.6
   flutter_dotenv: ^5.1.0
 
-
 dev_dependencies:
   flutter_test:
     sdk: flutter