Merge pull request #8 from FlutterWiz/fix-bloc-comm

remove unnecessary cubit

Author: FlutterWiz

lib/core/di/dependency_injector.dart

@@ -1,9 +1,10 @@
 import 'package:get_it/get_it.dart';
+import 'package:mapmotion_flutter/core/interfaces/i_app_life_cycle_service.dart';
 import 'package:mapmotion_flutter/core/interfaces/i_location_permission_service.dart';
 import 'package:mapmotion_flutter/core/interfaces/i_location_service.dart';
+import 'package:mapmotion_flutter/data/services/app_life_cycle_service.dart';
 import 'package:mapmotion_flutter/data/services/location_permission_service.dart';
 import 'package:mapmotion_flutter/data/services/location_service.dart';
-import 'package:mapmotion_flutter/presentation/blocs/application_life_cycle/application_life_cycle_cubit.dart';
 import 'package:mapmotion_flutter/presentation/blocs/location/location_cubit.dart';
 import 'package:mapmotion_flutter/presentation/blocs/permission/permission_cubit.dart';
 
@@ -13,9 +14,11 @@ void injectionSetup() {
   // Services
   getIt.registerLazySingleton<ILocationPermissionService>(() => LocationPermissionService());
   getIt.registerLazySingleton<ILocationService>(() => LocationService());
+  getIt.registerLazySingleton<IAppLifeCycleService>(() => AppLifeCycleService());
 
   // Cubits
-  getIt.registerLazySingleton<PermissionCubit>(() => PermissionCubit(getIt<ILocationPermissionService>()));
-  getIt.registerLazySingleton<ApplicationLifeCycleCubit>(() => ApplicationLifeCycleCubit());
+  getIt.registerLazySingleton<PermissionCubit>(
+    () => PermissionCubit(getIt<ILocationPermissionService>(), getIt<IAppLifeCycleService>()),
+  );
   getIt.registerLazySingleton<LocationCubit>(() => LocationCubit());
 }

lib/core/init/app_widget.dart

@@ -1,7 +1,6 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:mapmotion_flutter/core/di/dependency_injector.dart';
-import 'package:mapmotion_flutter/presentation/blocs/application_life_cycle/application_life_cycle_cubit.dart';
 import 'package:mapmotion_flutter/presentation/blocs/location/location_cubit.dart';
 import 'package:mapmotion_flutter/presentation/blocs/permission/permission_cubit.dart';
 import 'package:mapmotion_flutter/presentation/views/map/map_view.dart';
@@ -16,9 +15,6 @@ class AppWidget extends StatelessWidget {
         BlocProvider(
           create: (context) => getIt<PermissionCubit>()..initialize(),
         ),
-        BlocProvider(
-          create: (context) => getIt<ApplicationLifeCycleCubit>(),
-        ),
         BlocProvider(
           create: (context) => getIt<LocationCubit>()..initialize(),
         ),

lib/core/interfaces/i_app_life_cycle_service.dart

@@ -0,0 +1,4 @@
+abstract class IAppLifeCycleService {
+  Stream<bool> get isResumedStream;
+  Future<void> close();
+}

lib/data/services/app_life_cycle_service.dart

@@ -0,0 +1,27 @@
+import 'dart:async';
+
+import 'package:flutter/material.dart';
+import 'package:mapmotion_flutter/core/interfaces/i_app_life_cycle_service.dart';
+
+class AppLifeCycleService with WidgetsBindingObserver implements IAppLifeCycleService {
+  final StreamController<bool> _isResumedController = StreamController<bool>();
+
+  AppLifeCycleService() {
+    WidgetsBinding.instance.addObserver(this);
+  }
+
+  @override
+  void didChangeAppLifecycleState(AppLifecycleState state) {
+    _isResumedController.add(state == AppLifecycleState.resumed);
+    super.didChangeAppLifecycleState(state);
+  }
+
+  @override
+  Stream<bool> get isResumedStream => _isResumedController.stream;
+
+  @override
+  Future<void> close() async {
+    WidgetsBinding.instance.removeObserver(this);
+    await _isResumedController.close();
+  }
+}

lib/presentation/blocs/application_life_cycle/application_life_cycle_cubit.dart

@@ -2,21 +2,19 @@ import 'dart:async';
 
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:mapmotion_flutter/core/constants/enums/location_permission_status_enum.dart';
-import 'package:mapmotion_flutter/core/di/dependency_injector.dart';
+import 'package:mapmotion_flutter/core/interfaces/i_app_life_cycle_service.dart';
 import 'package:mapmotion_flutter/core/interfaces/i_location_permission_service.dart';
-import 'package:mapmotion_flutter/presentation/blocs/application_life_cycle/application_life_cycle_cubit.dart';
 import 'package:mapmotion_flutter/presentation/blocs/permission/permission_state.dart';
 
 class PermissionCubit extends Cubit<PermissionState> {
-  PermissionCubit(this._permissionService) : super(PermissionState.initial());
+  PermissionCubit(this._permissionService, this._appLifeCycleService) : super(PermissionState.initial());
 
   final ILocationPermissionService _permissionService;
+  final IAppLifeCycleService _appLifeCycleService;
 
   StreamSubscription<bool>? _locationPermissionStatusSubscription;
   StreamSubscription<bool>? _appLifeCycleSubscription;
 
-  ApplicationLifeCycleCubit get _appLifeCycleCubit => getIt<ApplicationLifeCycleCubit>();
-
   @override
   Future<void> close() {
     _appLifeCycleSubscription?.cancel();
@@ -34,7 +32,7 @@ class PermissionCubit extends Cubit<PermissionState> {
     });
 
     _appLifeCycleSubscription?.cancel();
-    _appLifeCycleSubscription = _appLifeCycleCubit.stream.map((s) => s.isResumed).listen((isResumed) async {
+    _appLifeCycleSubscription = _appLifeCycleService.isResumedStream.listen((isResumed) async {
       if (isResumed) {
         final isGranted = await _permissionService.isLocationPermissionGranted();