Refactor dialog header text assignment in CeilingSensorDialog for clarity and readability.

This commit is contained in:
Faris Armoush
2025-04-15 11:20:15 +03:00
parent 254e03e3c7
commit be0533645e

View File

@ -35,6 +35,7 @@ class CeilingSensorDialog extends StatefulWidget {
class _CeilingSensorDialogState extends State<CeilingSensorDialog> {
late final List<CpsFunctions> _cpsFunctions;
late final String _dialogHeaderText;
@override
void initState() {
@ -46,6 +47,9 @@ class _CeilingSensorDialogState extends State<CeilingSensorDialog> {
}
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<CeilingSensorDialog> {
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<RoutineBloc>().add(
AddFunctionToRoutine(
_updateValuesForAddedFunctions(
state.addedFunctions),
functions,
'${widget.uniqueCustomId}',
),
);