Remove unused parameters from CreateSubSpaceDialog constructor

Refactor FactoryResetModel and MainDoorSensorBatchView

lib/pages/device_managment/all_devices/models/factory_reset_model.dart

@@ -19,6 +19,7 @@ class FactoryResetModel {
   Map<String, dynamic> toJson() {
     return {
       'devicesUuid': devicesUuid,
+      'operationType': operationType,
     };
   }
 
@@ -33,6 +34,7 @@ class FactoryResetModel {
   Map<String, dynamic> toMap() {
     return {
       'devicesUuid': devicesUuid,
+      'operationType': operationType,
     };
   }
 
@@ -56,3 +58,4 @@ class FactoryResetModel {
   @override
   int get hashCode => devicesUuid.hashCode;
 }
+

lib/pages/device_managment/main_door_sensor/view/main_door_sensor_batch_view.dart

@@ -4,7 +4,6 @@ import 'package:syncrow_web/pages/device_managment/all_devices/models/factory_re
 import 'package:syncrow_web/pages/device_managment/main_door_sensor/bloc/main_door_sensor_bloc.dart';
 import 'package:syncrow_web/pages/device_managment/main_door_sensor/bloc/main_door_sensor_event.dart';
 import 'package:syncrow_web/pages/device_managment/shared/batch_control/factory_reset.dart';
-// import 'package:syncrow_web/pages/device_managment/shared/batch_control/firmware_update.dart';
 
 class MainDoorSensorBatchView extends StatelessWidget {
   const MainDoorSensorBatchView({super.key, required this.devicesIds});
@@ -13,26 +12,19 @@ class MainDoorSensorBatchView extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
+    return BlocProvider(
+      create: (context) => MainDoorSensorBloc(),
+      child: Builder(
+        builder: (innerContext) {
           return Row(
             mainAxisAlignment: MainAxisAlignment.center,
             children: [
-        // SizedBox(
-        //   width: 170,
-        //   height: 140,
-        //   child: FirmwareUpdateWidget(
-        //     deviceId: devicesIds.first,
-        //     version: 12,
-        //   ),
-        // ),
-        // const SizedBox(
-        //   width: 12,
-        // ),
               SizedBox(
                 width: 170,
                 height: 140,
                 child: FactoryResetWidget(
                   callFactoryReset: () {
-              BlocProvider.of<MainDoorSensorBloc>(context).add(
+                    BlocProvider.of<MainDoorSensorBloc>(innerContext).add(
                       MainDoorSensorFactoryReset(
                         deviceId: devicesIds.first,
                         factoryReset: FactoryResetModel(devicesUuid: devicesIds),
@@ -43,5 +35,8 @@ class MainDoorSensorBatchView extends StatelessWidget {
               ),
             ],
           );
+        },
+      ),
+    );
   }
 }

lib/pages/spaces_management/all_spaces/widgets/dialogs/create_space_dialog.dart

@@ -592,8 +592,6 @@ class CreateSpaceDialogState extends State<CreateSpaceDialog> {
         return CreateSubSpaceDialog(
             spaceName: name,
             dialogTitle: isEdit ? 'Edit Sub-space' : 'Create Sub-space',
-            spaceTags: spaceTags,
-            isEdit: isEdit,
             products: products,
             existingSubSpaces: existingSubSpaces,
             onSave: (slectedSubspaces) {

lib/pages/spaces_management/create_subspace/views/create_subspace_model_dialog.dart

@@ -4,169 +4,155 @@ import 'package:syncrow_web/pages/common/buttons/cancel_button.dart';
 import 'package:syncrow_web/pages/common/buttons/default_button.dart';
 import 'package:syncrow_web/pages/spaces_management/all_spaces/model/product_model.dart';
 import 'package:syncrow_web/pages/spaces_management/all_spaces/model/subspace_model.dart';
-import 'package:syncrow_web/pages/spaces_management/all_spaces/model/tag.dart';
 import 'package:syncrow_web/pages/spaces_management/create_subspace/bloc/subspace_bloc.dart';
 import 'package:syncrow_web/pages/spaces_management/create_subspace/bloc/subspace_event.dart';
 import 'package:syncrow_web/pages/spaces_management/create_subspace/bloc/subspace_state.dart';
+import 'package:syncrow_web/pages/spaces_management/create_subspace_model/widgets/subspace_chip.dart';
+import 'package:syncrow_web/pages/spaces_management/space_model/models/subspace_template_model.dart';
 import 'package:syncrow_web/utils/color_manager.dart';
+import 'package:syncrow_web/utils/extension/build_context_x.dart';
 
-class CreateSubSpaceDialog extends StatelessWidget {
+class CreateSubSpaceDialog extends StatefulWidget {
-  final bool isEdit;
   final String dialogTitle;
   final List<SubspaceModel>? existingSubSpaces;
   final String? spaceName;
-  final List<Tag>? spaceTags;
   final List<ProductModel>? products;
-  final Function(List<SubspaceModel>?)? onSave;
+  final void Function(List<SubspaceModel>?)? onSave;
 
-  const CreateSubSpaceDialog(
+  const CreateSubSpaceDialog({
-      {Key? key,
-      required this.isEdit,
     required this.dialogTitle,
-      this.existingSubSpaces,
     required this.spaceName,
-      required this.spaceTags,
     required this.products,
-      required this.onSave})
+    required this.onSave,
-      : super(key: key);
+    this.existingSubSpaces,
+    super.key,
+  });
+
+  @override
+  State<CreateSubSpaceDialog> createState() => _CreateSubSpaceDialogState();
+}
+
+class _CreateSubSpaceDialogState extends State<CreateSubSpaceDialog> {
+  late final TextEditingController _subspaceNameController;
+
+  @override
+  void initState() {
+    _subspaceNameController = TextEditingController();
+    super.initState();
+  }
+
+  @override
+  void dispose() {
+    _subspaceNameController.dispose();
+    super.dispose();
+  }
 
   @override
   Widget build(BuildContext context) {
-    final screenWidth = MediaQuery.of(context).size.width;
+    return BlocProvider(
-    final textController = TextEditingController();
-
-    return Dialog(
-      shape: RoundedRectangleBorder(
-        borderRadius: BorderRadius.circular(20),
-      ),
-      child: BlocProvider(
       create: (_) {
         final bloc = SubSpaceBloc();
-          if (existingSubSpaces != null) {
+        if (widget.existingSubSpaces != null) {
-            for (var subSpace in existingSubSpaces!) {
+          for (final subSpace in widget.existingSubSpaces ?? []) {
             bloc.add(AddSubSpace(subSpace));
           }
         }
         return bloc;
       },
+      child: Dialog(
+        shape: RoundedRectangleBorder(
+          borderRadius: BorderRadius.circular(20),
+        ),
         child: BlocBuilder<SubSpaceBloc, SubSpaceState>(
           builder: (context, state) {
             return Container(
+              width: context.screenWidth * 0.35,
               color: ColorsManager.whiteColors,
-                child: SizedBox(
+              padding: const EdgeInsets.all(16),
-                  width: screenWidth * 0.35,
-                  child: Padding(
-                    padding: const EdgeInsets.all(16.0),
               child: Column(
                 crossAxisAlignment: CrossAxisAlignment.start,
                 mainAxisSize: MainAxisSize.min,
                 children: [
                   Text(
-                          dialogTitle,
+                    widget.dialogTitle,
-                          style: Theme.of(context)
+                    style: context.textTheme.headlineLarge?.copyWith(
-                              .textTheme
+                      color: ColorsManager.blackColor,
-                              .headlineLarge
+                    ),
-                              ?.copyWith(color: ColorsManager.blackColor),
                   ),
                   const SizedBox(height: 16),
                   Container(
-                          width: screenWidth * 0.35,
+                    width: context.screenWidth * 0.35,
                     padding: const EdgeInsets.symmetric(
-                              vertical: 10.0, horizontal: 16.0),
+                      vertical: 10,
+                      horizontal: 16,
+                    ),
                     decoration: BoxDecoration(
                       color: ColorsManager.boxColor,
                       borderRadius: BorderRadius.circular(10),
                     ),
                     child: Wrap(
-                            spacing: 8.0,
+                      spacing: 8,
-                            runSpacing: 8.0,
+                      runSpacing: 8,
+                      alignment: WrapAlignment.start,
+                      crossAxisAlignment: WrapCrossAlignment.center,
                       children: [
                         ...state.subSpaces.asMap().entries.map(
                           (entry) {
                             final index = entry.key;
                             final subSpace = entry.value;
 
-                                  final lowerName =
+                            final lowerName = subSpace.subspaceName.toLowerCase();
-                                      subSpace.subspaceName.toLowerCase();
 
                             final duplicateIndices = state.subSpaces
                                 .asMap()
                                 .entries
                                 .where((e) =>
-                                          e.value.subspaceName.toLowerCase() ==
+                                    e.value.subspaceName.toLowerCase() == lowerName)
-                                          lowerName)
                                 .map((e) => e.key)
                                 .toList();
-                                  final isDuplicate =
+                            final isDuplicate = duplicateIndices.length > 1 &&
-                                      duplicateIndices.length > 1 &&
                                 duplicateIndices.indexOf(index) != 0;
-
+                            return SubspaceChip(
-                                  return Chip(
+                              subSpace: SubspaceTemplateModel(
-                                    label: Text(subSpace.subspaceName,
+                                subspaceName: entry.value.subspaceName,
-                                        style: Theme.of(context)
+                                disabled: entry.value.disabled,
-                                            .textTheme
-                                            .bodyMedium
-                                            ?.copyWith(
-                                                color:
-                                                    ColorsManager.spaceColor)),
-                                    backgroundColor: ColorsManager.whiteColors,
-                                    shape: RoundedRectangleBorder(
-                                      borderRadius: BorderRadius.circular(10),
-                                      side: BorderSide(
-                                        color: isDuplicate
-                                            ? ColorsManager.red
-                                            : ColorsManager.transparentColor,
-                                        width: 0,
                               ),
+                              isDuplicate: isDuplicate,
+                              onDeleted: () => context.read<SubSpaceBloc>().add(
+                                    RemoveSubSpace(subSpace),
                                   ),
-                                    deleteIcon: Container(
-                                      width: 24,
-                                      height: 24,
-                                      decoration: BoxDecoration(
-                                        shape: BoxShape.circle,
-                                        border: Border.all(
-                                          color: ColorsManager.lightGrayColor,
-                                          width: 1.5,
-                                        ),
-                                      ),
-                                      child: const Icon(
-                                        Icons.close,
-                                        size: 16,
-                                        color: ColorsManager.lightGrayColor,
-                                      ),
-                                    ),
-                                    onDeleted: () => context
-                                        .read<SubSpaceBloc>()
-                                        .add(RemoveSubSpace(subSpace)),
                             );
                           },
                         ),
                         SizedBox(
                           width: 200,
                           child: TextField(
-                                    controller: textController,
+                            controller: _subspaceNameController,
                             decoration: InputDecoration(
                               border: InputBorder.none,
                               hintText: state.subSpaces.isEmpty
                                   ? 'Please enter the name'
                                   : null,
-                                        hintStyle: Theme.of(context)
+                              hintStyle: context.textTheme.bodySmall?.copyWith(
-                                            .textTheme
+                                color: ColorsManager.lightGrayColor,
-                                            .bodySmall
+                              ),
-                                            ?.copyWith(
+                            ),
-                                                color: ColorsManager
-                                                    .lightGrayColor)),
                             onSubmitted: (value) {
-                                      if (value.trim().isNotEmpty) {
+                              final trimmedValue = value.trim();
+                              if (trimmedValue.isNotEmpty) {
                                 context.read<SubSpaceBloc>().add(
-                                            AddSubSpace(SubspaceModel(
+                                      AddSubSpace(
-                                                subspaceName: value.trim(),
+                                        SubspaceModel(
-                                                disabled: false)));
+                                          subspaceName: trimmedValue,
-                                        textController.clear();
+                                          disabled: false,
+                                        ),
+                                      ),
+                                    );
+                                _subspaceNameController.clear();
                               }
                             },
-                                    style:
+                            style: context.textTheme.bodyMedium,
-                                        Theme.of(context).textTheme.bodyMedium),
+                          ),
                         ),
                       ],
                     ),
@@ -174,13 +160,12 @@ class CreateSubSpaceDialog extends StatelessWidget {
                   if (state.errorMessage.isNotEmpty)
                     Padding(
                       padding: const EdgeInsets.only(top: 8.0),
-                            child: Text(state.errorMessage,
+                      child: Text(
-                                style: Theme.of(context)
+                        state.errorMessage,
-                                    .textTheme
+                        style: context.textTheme.bodySmall?.copyWith(
-                                    .bodySmall
-                                    ?.copyWith(
                           color: ColorsManager.warningRed,
-                                    )),
+                        ),
+                      ),
                     ),
                   const SizedBox(height: 16),
                   Row(
@@ -196,16 +181,14 @@ class CreateSubSpaceDialog extends StatelessWidget {
                       const SizedBox(width: 10),
                       Expanded(
                         child: DefaultButton(
-                                onPressed: (state.errorMessage.isNotEmpty)
+                          onPressed: state.errorMessage.isEmpty
-                                    ? null
+                              ? () {
-                                    : () async {
+                                  final subSpacesBloc = context.read<SubSpaceBloc>();
-                                        final subSpaces = context
+                                  final subSpaces = subSpacesBloc.state.subSpaces;
-                                            .read<SubSpaceBloc>()
+                                  widget.onSave?.call(subSpaces);
-                                            .state
-                                            .subSpaces;
-                                        onSave!(subSpaces);
                                   Navigator.of(context).pop();
-                                      },
+                                }
+                              : null,
                           backgroundColor: ColorsManager.secondaryColor,
                           borderRadius: 10,
                           foregroundColor: state.errorMessage.isNotEmpty
@@ -218,8 +201,7 @@ class CreateSubSpaceDialog extends StatelessWidget {
                   ),
                 ],
               ),
-                  ),
+            );
-                ));
           },
         ),
       ),

lib/pages/spaces_management/create_subspace_model/widgets/create_subspace_model_chips_box.dart

@@ -1,4 +1,7 @@
 import 'package:flutter/material.dart';
+import 'package:flutter_bloc/flutter_bloc.dart';
+import 'package:syncrow_web/pages/spaces_management/create_subspace_model/bloc/subspace_model_bloc.dart';
+import 'package:syncrow_web/pages/spaces_management/create_subspace_model/bloc/subspace_model_event.dart';
 import 'package:syncrow_web/pages/spaces_management/create_subspace_model/widgets/subspace_chip.dart';
 import 'package:syncrow_web/pages/spaces_management/create_subspace_model/widgets/subspaces_textfield.dart';
 import 'package:syncrow_web/pages/spaces_management/space_model/models/subspace_template_model.dart';
@@ -51,6 +54,9 @@ class CreateSubspaceModelChipsBox extends StatelessWidget {
               return SubspaceChip(
                 subSpace: subSpace,
                 isDuplicate: isDuplicate,
+                onDeleted: () => context.read<SubSpaceModelBloc>().add(
+                      RemoveSubSpaceModel(subSpace),
+                    ),
               );
             },
           ),

lib/pages/spaces_management/create_subspace_model/widgets/subspace_chip.dart

@@ -1,7 +1,4 @@
 import 'package:flutter/material.dart';
-import 'package:flutter_bloc/flutter_bloc.dart';
-import 'package:syncrow_web/pages/spaces_management/create_subspace_model/bloc/subspace_model_bloc.dart';
-import 'package:syncrow_web/pages/spaces_management/create_subspace_model/bloc/subspace_model_event.dart';
 import 'package:syncrow_web/pages/spaces_management/space_model/models/subspace_template_model.dart';
 import 'package:syncrow_web/utils/color_manager.dart';
 import 'package:syncrow_web/utils/extension/build_context_x.dart';
@@ -11,10 +8,12 @@ class SubspaceChip extends StatelessWidget {
     required this.subSpace,
     required this.isDuplicate,
     super.key,
+    required this.onDeleted,
   });
 
   final SubspaceTemplateModel subSpace;
   final bool isDuplicate;
+  final void Function() onDeleted;
 
   @override
   Widget build(BuildContext context) {
@@ -50,9 +49,7 @@ class SubspaceChip extends StatelessWidget {
           ),
         ),
       ),
-      onDeleted: () => context.read<SubSpaceModelBloc>().add(
+      onDeleted: onDeleted,
-            RemoveSubSpaceModel(subSpace),
-          ),
     );
   }
 }

lib/services/devices_mang_api.dart

@@ -321,13 +321,14 @@ class DevicesManagementApi {
   Future<bool> factoryReset(FactoryResetModel factoryReset, String uuid) async {
     try {
       final response = await HTTPService().post(
-        path: ApiEndpoints.factoryReset.replaceAll('{deviceUuid}', uuid),
+        path: ApiEndpoints.factoryReset,
         body: factoryReset.toMap(),
         showServerMessage: true,
         expectedResponseModel: (json) {
           return json['success'] ?? false;
         },
       );
+
       return response;
     } catch (e) {
       debugPrint('Error fetching  $e');