Merge pull request #20 from headlines-toolkit/fix_duplicate_table_entry

Fix duplicate table entry

Author: fulleni

lib/content_management/bloc/content_management_bloc.dart

@@ -30,15 +30,12 @@ class ContentManagementBloc
        super(const ContentManagementState()) {
     on<ContentManagementTabChanged>(_onContentManagementTabChanged);
     on<LoadHeadlinesRequested>(_onLoadHeadlinesRequested);
-    on<HeadlineAdded>(_onHeadlineAdded);
     on<HeadlineUpdated>(_onHeadlineUpdated);
     on<DeleteHeadlineRequested>(_onDeleteHeadlineRequested);
     on<LoadCategoriesRequested>(_onLoadCategoriesRequested);
-    on<CategoryAdded>(_onCategoryAdded);
     on<CategoryUpdated>(_onCategoryUpdated);
     on<DeleteCategoryRequested>(_onDeleteCategoryRequested);
     on<LoadSourcesRequested>(_onLoadSourcesRequested);
-    on<SourceAdded>(_onSourceAdded);
     on<SourceUpdated>(_onSourceUpdated);
     on<DeleteSourceRequested>(_onOnDeleteSourceRequested);
   }
@@ -92,31 +89,6 @@ class ContentManagementBloc
     }
   }
 
-  void _onHeadlineAdded(
-    HeadlineAdded event,
-    Emitter<ContentManagementState> emit,
-  ) {
-    final updatedHeadlines = [event.headline, ...state.headlines];
-    emit(
-      state.copyWith(
-        headlines: updatedHeadlines,
-        headlinesStatus: ContentManagementStatus.success,
-      ),
-    );
-  }
-
-  void _onHeadlineUpdated(
-    HeadlineUpdated event,
-    Emitter<ContentManagementState> emit,
-  ) {
-    final updatedHeadlines = List<Headline>.from(state.headlines);
-    final index = updatedHeadlines.indexWhere((h) => h.id == event.headline.id);
-    if (index != -1) {
-      updatedHeadlines[index] = event.headline;
-      emit(state.copyWith(headlines: updatedHeadlines));
-    }
-  }
-
   Future<void> _onDeleteHeadlineRequested(
     DeleteHeadlineRequested event,
     Emitter<ContentManagementState> emit,
@@ -144,6 +116,18 @@ class ContentManagementBloc
     }
   }
 
+  void _onHeadlineUpdated(
+    HeadlineUpdated event,
+    Emitter<ContentManagementState> emit,
+  ) {
+    final updatedHeadlines = List<Headline>.from(state.headlines);
+    final index = updatedHeadlines.indexWhere((h) => h.id == event.headline.id);
+    if (index != -1) {
+      updatedHeadlines[index] = event.headline;
+      emit(state.copyWith(headlines: updatedHeadlines));
+    }
+  }
+
   Future<void> _onLoadCategoriesRequested(
     LoadCategoriesRequested event,
     Emitter<ContentManagementState> emit,
@@ -182,33 +166,6 @@ class ContentManagementBloc
     }
   }
 
-  void _onCategoryAdded(
-    CategoryAdded event,
-    Emitter<ContentManagementState> emit,
-  ) {
-    final updatedCategories = [event.category, ...state.categories];
-    emit(
-      state.copyWith(
-        categories: updatedCategories,
-        categoriesStatus: ContentManagementStatus.success,
-      ),
-    );
-  }
-
-  void _onCategoryUpdated(
-    CategoryUpdated event,
-    Emitter<ContentManagementState> emit,
-  ) {
-    final updatedCategories = List<Category>.from(state.categories);
-    final index = updatedCategories.indexWhere(
-      (c) => c.id == event.category.id,
-    );
-    if (index != -1) {
-      updatedCategories[index] = event.category;
-      emit(state.copyWith(categories: updatedCategories));
-    }
-  }
-
   Future<void> _onDeleteCategoryRequested(
     DeleteCategoryRequested event,
     Emitter<ContentManagementState> emit,
@@ -236,6 +193,20 @@ class ContentManagementBloc
     }
   }
 
+  void _onCategoryUpdated(
+    CategoryUpdated event,
+    Emitter<ContentManagementState> emit,
+  ) {
+    final updatedCategories = List<Category>.from(state.categories);
+    final index = updatedCategories.indexWhere(
+      (c) => c.id == event.category.id,
+    );
+    if (index != -1) {
+      updatedCategories[index] = event.category;
+      emit(state.copyWith(categories: updatedCategories));
+    }
+  }
+
   Future<void> _onLoadSourcesRequested(
     LoadSourcesRequested event,
     Emitter<ContentManagementState> emit,
@@ -274,28 +245,6 @@ class ContentManagementBloc
     }
   }
 
-  void _onSourceAdded(SourceAdded event, Emitter<ContentManagementState> emit) {
-    final updatedSources = [event.source, ...state.sources];
-    emit(
-      state.copyWith(
-        sources: updatedSources,
-        sourcesStatus: ContentManagementStatus.success,
-      ),
-    );
-  }
-
-  void _onSourceUpdated(
-    SourceUpdated event,
-    Emitter<ContentManagementState> emit,
-  ) {
-    final updatedSources = List<Source>.from(state.sources);
-    final index = updatedSources.indexWhere((s) => s.id == event.source.id);
-    if (index != -1) {
-      updatedSources[index] = event.source;
-      emit(state.copyWith(sources: updatedSources));
-    }
-  }
-
   Future<void> _onOnDeleteSourceRequested(
     DeleteSourceRequested event,
     Emitter<ContentManagementState> emit,
@@ -322,4 +271,16 @@ class ContentManagementBloc
       );
     }
   }
+
+  void _onSourceUpdated(
+    SourceUpdated event,
+    Emitter<ContentManagementState> emit,
+  ) {
+    final updatedSources = List<Source>.from(state.sources);
+    final index = updatedSources.indexWhere((s) => s.id == event.source.id);
+    if (index != -1) {
+      updatedSources[index] = event.source;
+      emit(state.copyWith(sources: updatedSources));
+    }
+  }
 }

lib/content_management/bloc/content_management_event.dart

@@ -38,18 +38,18 @@ final class LoadHeadlinesRequested extends ContentManagementEvent {
   List<Object?> get props => [startAfterId, limit];
 }
 
-/// {@template headline_added}
-/// Event to add a new headline to the local state.
+/// {@template delete_headline_requested}
+/// Event to request deletion of a headline.
 /// {@endtemplate}
-final class HeadlineAdded extends ContentManagementEvent {
-  /// {@macro headline_added}
-  const HeadlineAdded(this.headline);
+final class DeleteHeadlineRequested extends ContentManagementEvent {
+  /// {@macro delete_headline_requested}
+  const DeleteHeadlineRequested(this.id);
 
-  /// The headline that was added.
-  final Headline headline;
+  /// The ID of the headline to delete.
+  final String id;
 
   @override
-  List<Object?> get props => [headline];
+  List<Object?> get props => [id];
 }
 
 /// {@template headline_updated}
@@ -66,20 +66,6 @@ final class HeadlineUpdated extends ContentManagementEvent {
   List<Object?> get props => [headline];
 }
 
-/// {@template delete_headline_requested}
-/// Event to request deletion of a headline.
-/// {@endtemplate}
-final class DeleteHeadlineRequested extends ContentManagementEvent {
-  /// {@macro delete_headline_requested}
-  const DeleteHeadlineRequested(this.id);
-
-  /// The ID of the headline to delete.
-  final String id;
-
-  @override
-  List<Object?> get props => [id];
-}
-
 /// {@template load_categories_requested}
 /// Event to request loading of categories.
 /// {@endtemplate}
@@ -97,18 +83,18 @@ final class LoadCategoriesRequested extends ContentManagementEvent {
   List<Object?> get props => [startAfterId, limit];
 }
 
-/// {@template category_added}
-/// Event to add a new category to the local state.
+/// {@template delete_category_requested}
+/// Event to request deletion of a category.
 /// {@endtemplate}
-final class CategoryAdded extends ContentManagementEvent {
-  /// {@macro category_added}
-  const CategoryAdded(this.category);
+final class DeleteCategoryRequested extends ContentManagementEvent {
+  /// {@macro delete_category_requested}
+  const DeleteCategoryRequested(this.id);
 
-  /// The category that was added.
-  final Category category;
+  /// The ID of the category to delete.
+  final String id;
 
   @override
-  List<Object?> get props => [category];
+  List<Object?> get props => [id];
 }
 
 /// {@template category_updated}
@@ -125,20 +111,6 @@ final class CategoryUpdated extends ContentManagementEvent {
   List<Object?> get props => [category];
 }
 
-/// {@template delete_category_requested}
-/// Event to request deletion of a category.
-/// {@endtemplate}
-final class DeleteCategoryRequested extends ContentManagementEvent {
-  /// {@macro delete_category_requested}
-  const DeleteCategoryRequested(this.id);
-
-  /// The ID of the category to delete.
-  final String id;
-
-  @override
-  List<Object?> get props => [id];
-}
-
 /// {@template load_sources_requested}
 /// Event to request loading of sources.
 /// {@endtemplate}
@@ -156,18 +128,18 @@ final class LoadSourcesRequested extends ContentManagementEvent {
   List<Object?> get props => [startAfterId, limit];
 }
 
-/// {@template source_added}
-/// Event to add a new source to the local state.
+/// {@template delete_source_requested}
+/// Event to request deletion of a source.
 /// {@endtemplate}
-final class SourceAdded extends ContentManagementEvent {
-  /// {@macro source_added}
-  const SourceAdded(this.source);
+final class DeleteSourceRequested extends ContentManagementEvent {
+  /// {@macro delete_source_requested}
+  const DeleteSourceRequested(this.id);
 
-  /// The source that was added.
-  final Source source;
+  /// The ID of the source to delete.
+  final String id;
 
   @override
-  List<Object?> get props => [source];
+  List<Object?> get props => [id];
 }
 
 /// {@template source_updated}
@@ -183,17 +155,3 @@ final class SourceUpdated extends ContentManagementEvent {
   @override
   List<Object?> get props => [source];
 }
-
-/// {@template delete_source_requested}
-/// Event to request deletion of a source.
-/// {@endtemplate}
-final class DeleteSourceRequested extends ContentManagementEvent {
-  /// {@macro delete_source_requested}
-  const DeleteSourceRequested(this.id);
-
-  /// The ID of the source to delete.
-  final String id;
-
-  @override
-  List<Object?> get props => [id];
-}

lib/content_management/view/create_category_page.dart

@@ -4,6 +4,7 @@ import 'package:go_router/go_router.dart';
 import 'package:ht_dashboard/content_management/bloc/content_management_bloc.dart';
 import 'package:ht_dashboard/content_management/bloc/create_category/create_category_bloc.dart';
 import 'package:ht_dashboard/l10n/l10n.dart';
+import 'package:ht_dashboard/shared/constants/pagination_constants.dart';
 import 'package:ht_dashboard/shared/shared.dart';
 import 'package:ht_data_repository/ht_data_repository.dart';
 import 'package:ht_shared/ht_shared.dart';
@@ -81,7 +82,8 @@ class _CreateCategoryViewState extends State<_CreateCategoryView> {
                 SnackBar(content: Text(l10n.categoryCreatedSuccessfully)),
               );
             context.read<ContentManagementBloc>().add(
-              CategoryAdded(state.createdCategory!),
+                  // Refresh the list to show the new category
+                  const LoadCategoriesRequested(limit: kDefaultRowsPerPage),
             );
             context.pop();
           }

lib/content_management/view/create_headline_page.dart

@@ -4,6 +4,7 @@ import 'package:go_router/go_router.dart';
 import 'package:ht_dashboard/content_management/bloc/content_management_bloc.dart';
 import 'package:ht_dashboard/content_management/bloc/create_headline/create_headline_bloc.dart';
 import 'package:ht_dashboard/l10n/l10n.dart';
+import 'package:ht_dashboard/shared/constants/pagination_constants.dart';
 import 'package:ht_dashboard/shared/shared.dart';
 import 'package:ht_data_repository/ht_data_repository.dart';
 import 'package:ht_shared/ht_shared.dart';
@@ -83,7 +84,8 @@ class _CreateHeadlineViewState extends State<_CreateHeadlineView> {
                 SnackBar(content: Text(l10n.headlineCreatedSuccessfully)),
               );
             context.read<ContentManagementBloc>().add(
-              HeadlineAdded(state.createdHeadline!),
+                  // Refresh the list to show the new headline
+                  const LoadHeadlinesRequested(limit: kDefaultRowsPerPage),
             );
             context.pop();
           }

lib/content_management/view/create_source_page.dart

@@ -5,6 +5,7 @@ import 'package:ht_dashboard/content_management/bloc/content_management_bloc.dar
 import 'package:ht_dashboard/content_management/bloc/create_source/create_source_bloc.dart';
 import 'package:ht_dashboard/content_management/bloc/edit_source/edit_source_bloc.dart';
 import 'package:ht_dashboard/l10n/l10n.dart';
+import 'package:ht_dashboard/shared/constants/pagination_constants.dart';
 import 'package:ht_dashboard/shared/shared.dart';
 import 'package:ht_data_repository/ht_data_repository.dart';
 import 'package:ht_shared/ht_shared.dart';
@@ -83,7 +84,8 @@ class _CreateSourceViewState extends State<_CreateSourceView> {
                 SnackBar(content: Text(l10n.sourceCreatedSuccessfully)),
               );
             context.read<ContentManagementBloc>().add(
-              SourceAdded(state.createdSource!),
+                  // Refresh the list to show the new source
+                  const LoadSourcesRequested(limit: kDefaultRowsPerPage),
             );
             context.pop();
           }