Merge pull request #80 from flutter-news-app-full-source-code/Implementing-a-Robust-Drafting-Feature-for-Headline-Content-Management

Implementing a robust drafting feature for headline content management

Author: fulleni

README.md

@@ -27,10 +27,10 @@ Click on any category to explore.
 
 ### 📰 Comprehensive Content Management
 Manage the entire lifecycle of your news content with full CRUD (Create, Read, Update, Delete) capabilities, complemented by advanced archiving and restoration features:
-- **Headlines:** Create, edit, publish, archive, restore, and permanently delete news articles.
-- **Topics:** Organize, define, archive, and restore news topics.
-- **Sources:** Maintain, update, archive, and restore news sources.
-> **💡 Your Advantage:** Gain detailed control over your content. This centralized system ensures accuracy and consistency, allowing you to manage active content and easily retrieve or remove archived items.
+- **Headlines:** Draft, Create, edit, publish, archive, restore, and permanently delete news articles.
+- **Topics:** Draft, Create, edit, archive, and restore news topics.
+- **Sources:** Draft, Create, edit, archive, and restore news sources.
+> **💡 Your Advantage:** Gain detailed control over your content. This centralized system ensures accuracy and consistency, allowing you to manage draft/active content and easily retrieve or remove archived items.
 
 ---
 

lib/bloc_observer.dart

@@ -15,39 +15,16 @@ class AppBlocObserver extends BlocObserver {
 
     // Initialize state information strings.
     // By default, truncate the full string representation of the state
-    // to the first 250 characters to prevent excessively long logs.
-    var oldStateInfo = oldState.toString().substring(
+    // to the first 150 characters to prevent excessively long logs.
+    final oldStateInfo = oldState.toString().substring(
       0,
-      oldState.toString().length > 250 ? 250 : oldState.toString().length,
+      oldState.toString().length > 150 ? 150 : oldState.toString().length,
     );
-    var newStateInfo = newState.toString().substring(
+    final newStateInfo = newState.toString().substring(
       0,
-      newState.toString().length > 250 ? 250 : newState.toString().length,
+      newState.toString().length > 150 ? 150 : newState.toString().length,
     );
 
-    try {
-      // Attempt to access a 'status' property on the state objects.
-      // Many BLoC states use a 'status' property (e.g., Loading, Success, Failure)
-      // to represent their current lifecycle phase. If this property exists
-      // and is not null, prioritize logging its value for conciseness.
-      if (oldState.status != null) {
-        oldStateInfo = 'status: ${oldState.status}';
-      }
-      if (newState.status != null) {
-        newStateInfo = 'status: ${newState.status}';
-      }
-    } catch (e) {
-      // This catch block handles cases where:
-      // 1. The 'status' property does not exist on the state object (NoSuchMethodError).
-      // 2. Accessing 'status' throws any other runtime error.
-      // In such scenarios, the `oldStateInfo` and `newStateInfo` variables
-      // will retain their initially truncated string representations,
-      // providing a fallback for states without a 'status' property.
-      // Log the error for debugging purposes, but do not rethrow to avoid
-      // crashing the observer.
-      log('Error accessing status property for ${bloc.runtimeType}: $e');
-    }
-
     // Log the state change, including the BLoC type and the old and new state information.
     log('onChange(${bloc.runtimeType}, $oldStateInfo -> $newStateInfo)');
   }

lib/content_management/bloc/content_management_state.dart

@@ -108,7 +108,6 @@ class ContentManagementState extends Equatable {
       sources: sources ?? this.sources,
       sourcesCursor: sourcesCursor ?? this.sourcesCursor,
       sourcesHasMore: sourcesHasMore ?? this.sourcesHasMore,
-
       exception: exception ?? this.exception,
     );
   }

lib/content_management/bloc/create_headline/create_headline_bloc.dart

@@ -15,7 +15,6 @@ class CreateHeadlineBloc
   CreateHeadlineBloc({
     required DataRepository<Headline> headlinesRepository,
   }) : _headlinesRepository = headlinesRepository,
-
        super(const CreateHeadlineState()) {
     on<CreateHeadlineTitleChanged>(_onTitleChanged);
     on<CreateHeadlineExcerptChanged>(_onExcerptChanged);
@@ -24,8 +23,8 @@ class CreateHeadlineBloc
     on<CreateHeadlineSourceChanged>(_onSourceChanged);
     on<CreateHeadlineTopicChanged>(_onTopicChanged);
     on<CreateHeadlineCountryChanged>(_onCountryChanged);
-    on<CreateHeadlineStatusChanged>(_onStatusChanged);
-    on<CreateHeadlineSubmitted>(_onSubmitted);
+    on<CreateHeadlineSavedAsDraft>(_onSavedAsDraft);
+    on<CreateHeadlinePublished>(_onPublished);
   }
 
   final DataRepository<Headline> _headlinesRepository;
@@ -81,24 +80,52 @@ class CreateHeadlineBloc
     emit(state.copyWith(eventCountry: () => event.country));
   }
 
-  void _onStatusChanged(
-    CreateHeadlineStatusChanged event,
+  /// Handles saving the headline as a draft.
+  Future<void> _onSavedAsDraft(
+    CreateHeadlineSavedAsDraft event,
     Emitter<CreateHeadlineState> emit,
-  ) {
-    emit(
-      state.copyWith(
-        contentStatus: event.status,
-        status: CreateHeadlineStatus.initial,
-      ),
-    );
+  ) async {
+    emit(state.copyWith(status: CreateHeadlineStatus.submitting));
+    try {
+      final now = DateTime.now();
+      final newHeadline = Headline(
+        id: _uuid.v4(),
+        title: state.title,
+        excerpt: state.excerpt,
+        url: state.url,
+        imageUrl: state.imageUrl,
+        source: state.source!,
+        eventCountry: state.eventCountry!,
+        topic: state.topic!,
+        createdAt: now,
+        updatedAt: now,
+        status: ContentStatus.draft,
+      );
+
+      await _headlinesRepository.create(item: newHeadline);
+      emit(
+        state.copyWith(
+          status: CreateHeadlineStatus.success,
+          createdHeadline: newHeadline,
+        ),
+      );
+    } on HttpException catch (e) {
+      emit(state.copyWith(status: CreateHeadlineStatus.failure, exception: e));
+    } catch (e) {
+      emit(
+        state.copyWith(
+          status: CreateHeadlineStatus.failure,
+          exception: UnknownException('An unexpected error occurred: $e'),
+        ),
+      );
+    }
   }
 
-  Future<void> _onSubmitted(
-    CreateHeadlineSubmitted event,
+  /// Handles publishing the headline.
+  Future<void> _onPublished(
+    CreateHeadlinePublished event,
     Emitter<CreateHeadlineState> emit,
   ) async {
-    if (!state.isFormValid) return;
-
     emit(state.copyWith(status: CreateHeadlineStatus.submitting));
     try {
       final now = DateTime.now();
@@ -113,7 +140,7 @@ class CreateHeadlineBloc
         topic: state.topic!,
         createdAt: now,
         updatedAt: now,
-        status: state.contentStatus,
+        status: ContentStatus.active,
       );
 
       await _headlinesRepository.create(item: newHeadline);

lib/content_management/bloc/create_headline/create_headline_event.dart

@@ -64,17 +64,12 @@ final class CreateHeadlineCountryChanged extends CreateHeadlineEvent {
   List<Object?> get props => [country];
 }
 
-/// Event for when the headline's status is changed.
-final class CreateHeadlineStatusChanged extends CreateHeadlineEvent {
-  const CreateHeadlineStatusChanged(this.status);
-
-  final ContentStatus status;
-
-  @override
-  List<Object?> get props => [status];
+/// Event to save the headline as a draft.
+final class CreateHeadlineSavedAsDraft extends CreateHeadlineEvent {
+  const CreateHeadlineSavedAsDraft();
 }
 
-/// Event to signal that the form should be submitted.
-final class CreateHeadlineSubmitted extends CreateHeadlineEvent {
-  const CreateHeadlineSubmitted();
+/// Event to publish the headline.
+final class CreateHeadlinePublished extends CreateHeadlineEvent {
+  const CreateHeadlinePublished();
 }

lib/content_management/bloc/create_headline/create_headline_state.dart

@@ -29,7 +29,6 @@ final class CreateHeadlineState extends Equatable {
     this.source,
     this.topic,
     this.eventCountry,
-    this.contentStatus = ContentStatus.active,
     this.exception,
     this.createdHeadline,
   });
@@ -42,7 +41,6 @@ final class CreateHeadlineState extends Equatable {
   final Source? source;
   final Topic? topic;
   final Country? eventCountry;
-  final ContentStatus contentStatus;
   final HttpException? exception;
   final Headline? createdHeadline;
 
@@ -65,7 +63,6 @@ final class CreateHeadlineState extends Equatable {
     ValueGetter<Source?>? source,
     ValueGetter<Topic?>? topic,
     ValueGetter<Country?>? eventCountry,
-    ContentStatus? contentStatus,
     HttpException? exception,
     Headline? createdHeadline,
   }) {
@@ -78,7 +75,6 @@ final class CreateHeadlineState extends Equatable {
       source: source != null ? source() : this.source,
       topic: topic != null ? topic() : this.topic,
       eventCountry: eventCountry != null ? eventCountry() : this.eventCountry,
-      contentStatus: contentStatus ?? this.contentStatus,
       exception: exception,
       createdHeadline: createdHeadline ?? this.createdHeadline,
     );
@@ -94,7 +90,6 @@ final class CreateHeadlineState extends Equatable {
     source,
     topic,
     eventCountry,
-    contentStatus,
     exception,
     createdHeadline,
   ];

lib/content_management/bloc/create_source/create_source_bloc.dart

@@ -21,8 +21,8 @@ class CreateSourceBloc extends Bloc<CreateSourceEvent, CreateSourceState> {
     on<CreateSourceTypeChanged>(_onSourceTypeChanged);
     on<CreateSourceLanguageChanged>(_onLanguageChanged);
     on<CreateSourceHeadquartersChanged>(_onHeadquartersChanged);
-    on<CreateSourceStatusChanged>(_onStatusChanged);
-    on<CreateSourceSubmitted>(_onSubmitted);
+    on<CreateSourceSavedAsDraft>(_onSavedAsDraft);
+    on<CreateSourcePublished>(_onPublished);
   }
 
   final DataRepository<Source> _sourcesRepository;
@@ -70,24 +70,51 @@ class CreateSourceBloc extends Bloc<CreateSourceEvent, CreateSourceState> {
     emit(state.copyWith(headquarters: () => event.headquarters));
   }
 
-  void _onStatusChanged(
-    CreateSourceStatusChanged event,
+  /// Handles saving the source as a draft.
+  Future<void> _onSavedAsDraft(
+    CreateSourceSavedAsDraft event,
     Emitter<CreateSourceState> emit,
-  ) {
-    emit(
-      state.copyWith(
-        contentStatus: event.status,
-        status: CreateSourceStatus.initial,
-      ),
-    );
+  ) async {
+    emit(state.copyWith(status: CreateSourceStatus.submitting));
+    try {
+      final now = DateTime.now();
+      final newSource = Source(
+        id: _uuid.v4(),
+        name: state.name,
+        description: state.description,
+        url: state.url,
+        sourceType: state.sourceType!,
+        language: state.language!,
+        createdAt: now,
+        updatedAt: now,
+        headquarters: state.headquarters!,
+        status: ContentStatus.draft,
+      );
+
+      await _sourcesRepository.create(item: newSource);
+      emit(
+        state.copyWith(
+          status: CreateSourceStatus.success,
+          createdSource: newSource,
+        ),
+      );
+    } on HttpException catch (e) {
+      emit(state.copyWith(status: CreateSourceStatus.failure, exception: e));
+    } catch (e) {
+      emit(
+        state.copyWith(
+          status: CreateSourceStatus.failure,
+          exception: UnknownException('An unexpected error occurred: $e'),
+        ),
+      );
+    }
   }
 
-  Future<void> _onSubmitted(
-    CreateSourceSubmitted event,
+  /// Handles publishing the source.
+  Future<void> _onPublished(
+    CreateSourcePublished event,
     Emitter<CreateSourceState> emit,
   ) async {
-    if (!state.isFormValid) return;
-
     emit(state.copyWith(status: CreateSourceStatus.submitting));
     try {
       final now = DateTime.now();
@@ -101,7 +128,7 @@ class CreateSourceBloc extends Bloc<CreateSourceEvent, CreateSourceState> {
         createdAt: now,
         updatedAt: now,
         headquarters: state.headquarters!,
-        status: state.contentStatus,
+        status: ContentStatus.active,
       );
 
       await _sourcesRepository.create(item: newSource);

lib/content_management/bloc/create_source/create_source_event.dart

@@ -56,17 +56,12 @@ final class CreateSourceHeadquartersChanged extends CreateSourceEvent {
   List<Object?> get props => [headquarters];
 }
 
-/// Event for when the source's status is changed.
-final class CreateSourceStatusChanged extends CreateSourceEvent {
-  const CreateSourceStatusChanged(this.status);
-
-  final ContentStatus status;
-
-  @override
-  List<Object?> get props => [status];
+/// Event to save the source as a draft.
+final class CreateSourceSavedAsDraft extends CreateSourceEvent {
+  const CreateSourceSavedAsDraft();
 }
 
-/// Event to signal that the form should be submitted.
-final class CreateSourceSubmitted extends CreateSourceEvent {
-  const CreateSourceSubmitted();
+/// Event to publish the source.
+final class CreateSourcePublished extends CreateSourceEvent {
+  const CreateSourcePublished();
 }

lib/content_management/bloc/create_source/create_source_state.dart

@@ -29,7 +29,6 @@ final class CreateSourceState extends Equatable {
     this.sourceType,
     this.language,
     this.headquarters,
-    this.contentStatus = ContentStatus.active,
     this.exception,
     this.createdSource,
   });
@@ -41,7 +40,6 @@ final class CreateSourceState extends Equatable {
   final SourceType? sourceType;
   final Language? language;
   final Country? headquarters;
-  final ContentStatus contentStatus;
   final HttpException? exception;
   final Source? createdSource;
 
@@ -62,7 +60,6 @@ final class CreateSourceState extends Equatable {
     ValueGetter<SourceType?>? sourceType,
     ValueGetter<Language?>? language,
     ValueGetter<Country?>? headquarters,
-    ContentStatus? contentStatus,
     HttpException? exception,
     Source? createdSource,
   }) {
@@ -74,7 +71,6 @@ final class CreateSourceState extends Equatable {
       sourceType: sourceType != null ? sourceType() : this.sourceType,
       language: language != null ? language() : this.language,
       headquarters: headquarters != null ? headquarters() : this.headquarters,
-      contentStatus: contentStatus ?? this.contentStatus,
       exception: exception,
       createdSource: createdSource ?? this.createdSource,
     );
@@ -89,7 +85,6 @@ final class CreateSourceState extends Equatable {
     sourceType,
     language,
     headquarters,
-    contentStatus,
     exception,
     createdSource,
   ];