refactor(content): use country_picker for create source

Author: fulleni

lib/content_management/bloc/create_source/create_source_bloc.dart

@@ -1,8 +1,10 @@
 import 'package:bloc/bloc.dart';
 import 'package:core/core.dart';
+import 'package:country_picker/country_picker.dart' as picker;
 import 'package:data_repository/data_repository.dart';
 import 'package:equatable/equatable.dart';
 import 'package:flutter/foundation.dart';
+import 'package:flutter_news_app_web_dashboard_full_source_code/shared/shared.dart';
 import 'package:uuid/uuid.dart';
 
 part 'create_source_event.dart';
@@ -13,9 +15,7 @@ class CreateSourceBloc extends Bloc<CreateSourceEvent, CreateSourceState> {
   /// {@macro create_source_bloc}
   CreateSourceBloc({
     required DataRepository<Source> sourcesRepository,
-    required DataRepository<Country> countriesRepository,
   }) : _sourcesRepository = sourcesRepository,
-       _countriesRepository = countriesRepository,
        super(const CreateSourceState()) {
     on<CreateSourceDataLoaded>(_onDataLoaded);
     on<CreateSourceNameChanged>(_onNameChanged);
@@ -29,36 +29,15 @@ class CreateSourceBloc extends Bloc<CreateSourceEvent, CreateSourceState> {
   }
 
   final DataRepository<Source> _sourcesRepository;
-  final DataRepository<Country> _countriesRepository;
   final _uuid = const Uuid();
 
   Future<void> _onDataLoaded(
     CreateSourceDataLoaded event,
     Emitter<CreateSourceState> emit,
   ) async {
-    emit(state.copyWith(status: CreateSourceStatus.loading));
-    try {
-      final countriesResponse = await _countriesRepository.readAll(
-        sort: [const SortOption('updatedAt', SortOrder.desc)],
-      );
-      final countries = countriesResponse.items;
-
-      emit(
-        state.copyWith(
-          status: CreateSourceStatus.initial,
-          countries: countries,
-        ),
-      );
-    } 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'),
-        ),
-      );
-    }
+    // This event is now a no-op since we don't need to load countries.
+    // We just ensure the BLoC is in the initial state.
+    emit(state.copyWith(status: CreateSourceStatus.initial));
   }
 
   void _onNameChanged(
@@ -100,7 +79,13 @@ class CreateSourceBloc extends Bloc<CreateSourceEvent, CreateSourceState> {
     CreateSourceHeadquartersChanged event,
     Emitter<CreateSourceState> emit,
   ) {
-    emit(state.copyWith(headquarters: () => event.headquarters));
+    final packageCountry = event.headquarters;
+    if (packageCountry == null) {
+      emit(state.copyWith(headquarters: () => null));
+    } else {
+      final coreCountry = adaptPackageCountryToCoreCountry(packageCountry);
+      emit(state.copyWith(headquarters: () => coreCountry));
+    }
   }
 
   void _onStatusChanged(

lib/content_management/bloc/create_source/create_source_event.dart

@@ -56,7 +56,7 @@ final class CreateSourceLanguageChanged extends CreateSourceEvent {
 /// Event for when the source's headquarters is changed.
 final class CreateSourceHeadquartersChanged extends CreateSourceEvent {
   const CreateSourceHeadquartersChanged(this.headquarters);
-  final Country? headquarters;
+  final picker.Country? headquarters;
   @override
   List<Object?> get props => [headquarters];
 }

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.countries = const [],
     this.contentStatus = ContentStatus.active,
     this.exception,
     this.createdSource,
@@ -42,7 +41,6 @@ final class CreateSourceState extends Equatable {
   final SourceType? sourceType;
   final String language;
   final Country? headquarters;
-  final List<Country> countries;
   final ContentStatus contentStatus;
   final HttpException? exception;
   final Source? createdSource;
@@ -64,7 +62,6 @@ final class CreateSourceState extends Equatable {
     ValueGetter<SourceType?>? sourceType,
     String? language,
     ValueGetter<Country?>? headquarters,
-    List<Country>? countries,
     ContentStatus? contentStatus,
     HttpException? exception,
     Source? createdSource,
@@ -77,7 +74,6 @@ final class CreateSourceState extends Equatable {
       sourceType: sourceType != null ? sourceType() : this.sourceType,
       language: language ?? this.language,
       headquarters: headquarters != null ? headquarters() : this.headquarters,
-      countries: countries ?? this.countries,
       contentStatus: contentStatus ?? this.contentStatus,
       exception: exception,
       createdSource: createdSource ?? this.createdSource,
@@ -93,7 +89,6 @@ final class CreateSourceState extends Equatable {
     sourceType,
     language,
     headquarters,
-    countries,
     contentStatus,
     exception,
     createdSource,

lib/content_management/view/create_source_page.dart

@@ -1,4 +1,5 @@
 import 'package:core/core.dart';
+import 'package:country_picker/country_picker.dart' as picker;
 import 'package:data_repository/data_repository.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
@@ -23,7 +24,6 @@ class CreateSourcePage extends StatelessWidget {
     return BlocProvider(
       create: (context) => CreateSourceBloc(
         sourcesRepository: context.read<DataRepository<Source>>(),
-        countriesRepository: context.read<DataRepository<Country>>(),
       )..add(const CreateSourceDataLoaded()),
       child: const _CreateSourceView(),
     );
@@ -109,8 +109,7 @@ class _CreateSourceViewState extends State<_CreateSourceView> {
             );
           }
 
-          if (state.status == CreateSourceStatus.failure &&
-              state.countries.isEmpty) {
+          if (state.status == CreateSourceStatus.failure) {
             return FailureStateWidget(
               exception: state.exception!,
               onRetry: () => context.read<CreateSourceBloc>().add(
@@ -192,24 +191,18 @@ class _CreateSourceViewState extends State<_CreateSourceView> {
                           .add(CreateSourceTypeChanged(value)),
                     ),
                     const SizedBox(height: AppSpacing.lg),
-                    DropdownButtonFormField<Country?>(
-                      value: state.headquarters,
-                      decoration: InputDecoration(
-                        labelText: l10n.headquarters,
-                        border: const OutlineInputBorder(),
-                      ),
-                      items: [
-                        DropdownMenuItem(value: null, child: Text(l10n.none)),
-                        ...state.countries.map(
-                          (country) => DropdownMenuItem(
-                            value: country,
-                            child: Text(country.name),
-                          ),
-                        ),
-                      ],
-                      onChanged: (value) => context
-                          .read<CreateSourceBloc>()
-                          .add(CreateSourceHeadquartersChanged(value)),
+                    CountryPickerFormField(
+                      labelText: l10n.headquarters,
+                      initialValue: state.headquarters != null
+                          ? adaptCoreCountryToPackageCountry(
+                              state.headquarters!,
+                            )
+                          : null,
+                      onChanged: (picker.Country country) {
+                        context.read<CreateSourceBloc>().add(
+                              CreateSourceHeadquartersChanged(country),
+                            );
+                      },
                     ),
                     const SizedBox(height: AppSpacing.lg),
                     DropdownButtonFormField<ContentStatus>(