From aff7ceeac4a36f522c46289587750b7da5fe4680 Mon Sep 17 00:00:00 2001 From: hannathkadher Date: Mon, 9 Dec 2024 10:02:20 +0400 Subject: [PATCH 1/3] added condition to check parent name --- .../widgets/dialogs/create_space_dialog.dart | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/lib/pages/spaces_management/all_spaces/widgets/dialogs/create_space_dialog.dart b/lib/pages/spaces_management/all_spaces/widgets/dialogs/create_space_dialog.dart index e0853e3a..496c1147 100644 --- a/lib/pages/spaces_management/all_spaces/widgets/dialogs/create_space_dialog.dart +++ b/lib/pages/spaces_management/all_spaces/widgets/dialogs/create_space_dialog.dart @@ -133,13 +133,7 @@ class CreateSpaceDialogState extends State { isNameFieldInvalid = value.isEmpty; if (!isNameFieldInvalid) { - if ((widget.parentSpace?.children.any( - (child) => child.name == value) ?? - false) || - (widget.parentSpace?.name == value) || - (widget.editSpace?.children.any( - (child) => child.name == value) ?? - false)) { + if (_isNameConflict(value)) { isNameFieldExist = true; isOkButtonEnabled = false; } else { @@ -387,7 +381,13 @@ class CreateSpaceDialogState extends State { ); } - + bool _isNameConflict(String value) { + return (widget.parentSpace?.children.any((child) => child.name == value) ?? + false) || + (widget.editSpace?.parent?.name == value) || + (widget.editSpace?.children.any((child) => child.name == value) ?? + false); + } String _mapIconToProduct(String uuid, List products) { // Find the product with the matching UUID From 4055265b0aa58affa17c5b2015429abf72abd747 Mon Sep 17 00:00:00 2001 From: hannathkadher Date: Mon, 9 Dec 2024 10:02:57 +0400 Subject: [PATCH 2/3] for create --- .../all_spaces/widgets/dialogs/create_space_dialog.dart | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/pages/spaces_management/all_spaces/widgets/dialogs/create_space_dialog.dart b/lib/pages/spaces_management/all_spaces/widgets/dialogs/create_space_dialog.dart index 496c1147..f6a02833 100644 --- a/lib/pages/spaces_management/all_spaces/widgets/dialogs/create_space_dialog.dart +++ b/lib/pages/spaces_management/all_spaces/widgets/dialogs/create_space_dialog.dart @@ -384,6 +384,7 @@ class CreateSpaceDialogState extends State { bool _isNameConflict(String value) { return (widget.parentSpace?.children.any((child) => child.name == value) ?? false) || + (widget.parentSpace?.name == value) || (widget.editSpace?.parent?.name == value) || (widget.editSpace?.children.any((child) => child.name == value) ?? false); From dd5fe10a217bbb32392be3122840d27655ce0652 Mon Sep 17 00:00:00 2001 From: hannathkadher Date: Mon, 9 Dec 2024 10:03:40 +0400 Subject: [PATCH 3/3] removed print --- .../all_spaces/bloc/space_management_bloc.dart | 3 --- 1 file changed, 3 deletions(-) diff --git a/lib/pages/spaces_management/all_spaces/bloc/space_management_bloc.dart b/lib/pages/spaces_management/all_spaces/bloc/space_management_bloc.dart index a171f2a7..ffa5687e 100644 --- a/lib/pages/spaces_management/all_spaces/bloc/space_management_bloc.dart +++ b/lib/pages/spaces_management/all_spaces/bloc/space_management_bloc.dart @@ -309,8 +309,6 @@ class SpaceManagementBloc await _api.deleteSpace(communityUuid, parent.uuid!); } } catch (e) { - print( - 'Error deleting space ${parent.name} (UUID: ${parent.uuid}, Community UUID: $communityUuid): $e'); rethrow; // Decide whether to stop execution or continue } } @@ -342,7 +340,6 @@ class SpaceManagementBloc space.uuid = response?.uuid; } } catch (e) { - print('Error creating space ${space.name}: $e'); rethrow; // Stop further execution on failure } }