From be0533645eee2ec131a32dba7b496c9bbee06b7b Mon Sep 17 00:00:00 2001 From: Faris Armoush Date: Tue, 15 Apr 2025 11:20:15 +0300 Subject: [PATCH] Refactor dialog header text assignment in `CeilingSensorDialog` for clarity and readability. --- .../ceiling_sensor/ceiling_sensor_dialog.dart | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/lib/pages/routines/widgets/routine_dialogs/ceiling_sensor/ceiling_sensor_dialog.dart b/lib/pages/routines/widgets/routine_dialogs/ceiling_sensor/ceiling_sensor_dialog.dart index 7ce6a9b1..c18706f0 100644 --- a/lib/pages/routines/widgets/routine_dialogs/ceiling_sensor/ceiling_sensor_dialog.dart +++ b/lib/pages/routines/widgets/routine_dialogs/ceiling_sensor/ceiling_sensor_dialog.dart @@ -35,6 +35,7 @@ class CeilingSensorDialog extends StatefulWidget { class _CeilingSensorDialogState extends State { late final List _cpsFunctions; + late final String _dialogHeaderText; @override void initState() { @@ -46,6 +47,9 @@ class _CeilingSensorDialogState extends State { } return function.type == 'IF' || function.type == 'BOTH'; }).toList(); + + final isIfDialog = widget.dialogType == 'IF'; + _dialogHeaderText = isIfDialog ? 'Conditions' : 'Functions'; } @override @@ -67,18 +71,18 @@ class _CeilingSensorDialogState extends State { child: Column( mainAxisSize: MainAxisSize.min, children: [ - DialogHeader( - 'Presence Sensor ${widget.dialogType == "IF" ? 'Conditions' : 'Functions'}', - ), + DialogHeader('Presence Sensor $_dialogHeaderText'), Expanded(child: _buildMainContent(context, state)), DialogFooter( onCancel: () => Navigator.pop(context), onConfirm: state.addedFunctions.isNotEmpty ? () { + final functions = _updateValuesForAddedFunctions( + state.addedFunctions, + ); context.read().add( AddFunctionToRoutine( - _updateValuesForAddedFunctions( - state.addedFunctions), + functions, '${widget.uniqueCustomId}', ), );