refactor: improve Equatable stringify handling and clean up props usage (#722)

* refactor: add factory constructor and update docs for EquatablePropToString

* fix: use raw fields in props instead of EquatablePropToString wrapper

* feat: enable Equatable.stringify based on local or remote log level

* chore: remove unused utils import from bloc files

* chore: update Equatable props to use nullable object list

Author: SERDUN

lib/app/notifications/bloc/notifications_bloc.dart

@@ -3,9 +3,7 @@ import 'package:equatable/equatable.dart';
 import 'package:bloc_concurrency/bloc_concurrency.dart';
 import 'package:freezed_annotation/freezed_annotation.dart';
 
-import 'package:webtrit_phone/utils/utils.dart';
-
-import '../models/models.dart';
+import 'package:webtrit_phone/app/notifications/notifications.dart';
 
 part 'notifications_bloc.freezed.dart';
 

lib/app/notifications/bloc/notifications_event.dart

@@ -4,7 +4,7 @@ sealed class NotificationsEvent extends Equatable {
   const NotificationsEvent();
 
   @override
-  List<Object> get props => [];
+  List<Object?> get props => [];
 }
 
 class NotificationsSubmitted extends NotificationsEvent {
@@ -13,9 +13,7 @@ class NotificationsSubmitted extends NotificationsEvent {
   final Notification notification;
 
   @override
-  List<Object> get props => [
-        EquatablePropToString([notification], listPropToString),
-      ];
+  List<Object?> get props => [notification];
 }
 
 class NotificationsCleared extends NotificationsEvent {

lib/blocs/app/app_event.dart

@@ -4,7 +4,7 @@ sealed class AppEvent extends Equatable {
   const AppEvent();
 
   @override
-  List<Object> get props => [];
+  List<Object?> get props => [];
 }
 
 class _SessionUpdated extends AppEvent {
@@ -13,9 +13,7 @@ class _SessionUpdated extends AppEvent {
   final Session? session;
 
   @override
-  List<Object> get props => [
-        EquatablePropToString([session], listPropToString),
-      ];
+  List<Object?> get props => [session];
 }
 
 class AppThemeSettingsChanged extends AppEvent {
@@ -24,9 +22,7 @@ class AppThemeSettingsChanged extends AppEvent {
   final ThemeSettings value;
 
   @override
-  List<Object> get props => [
-        EquatablePropToString([value], listPropToString),
-      ];
+  List<Object?> get props => [value];
 }
 
 class AppThemeModeChanged extends AppEvent {
@@ -35,9 +31,7 @@ class AppThemeModeChanged extends AppEvent {
   final ThemeMode value;
 
   @override
-  List<Object> get props => [
-        EquatablePropToString([value], listPropToString),
-      ];
+  List<Object?> get props => [value];
 }
 
 class AppLocaleChanged extends AppEvent {
@@ -46,9 +40,7 @@ class AppLocaleChanged extends AppEvent {
   final Locale value;
 
   @override
-  List<Object> get props => [
-        EquatablePropToString([value], listPropToString),
-      ];
+  List<Object?> get props => [value];
 }
 
 sealed class AppAgreementAccepted extends AppEvent {
@@ -65,9 +57,7 @@ class _UserAppAgreementUpdate extends AppAgreementAccepted {
   final AgreementStatus status;
 
   @override
-  List<Object> get props => [
-        EquatablePropToString([status], listPropToString),
-      ];
+  List<Object?> get props => [status];
 }
 
 class _ContactsAppAgreementUpdate extends AppAgreementAccepted {
@@ -76,7 +66,5 @@ class _ContactsAppAgreementUpdate extends AppAgreementAccepted {
   final AgreementStatus status;
 
   @override
-  List<Object> get props => [
-        EquatablePropToString([status], listPropToString),
-      ];
+  List<Object?> get props => [status];
 }

lib/blocs/external_contacts_sync/external_contacts_sync_event.dart

@@ -4,7 +4,7 @@ sealed class ExternalContactsSyncEvent extends Equatable {
   const ExternalContactsSyncEvent();
 
   @override
-  List<Object> get props => [];
+  List<Object?> get props => [];
 }
 
 class ExternalContactsSyncStarted extends ExternalContactsSyncEvent {
@@ -18,12 +18,8 @@ class ExternalContactsSyncRefreshed extends ExternalContactsSyncEvent {
 class _ExternalContactsSyncUpdated extends ExternalContactsSyncEvent {
   final List<ExternalContact> contacts;
 
-  const _ExternalContactsSyncUpdated({
-    required this.contacts,
-  });
+  const _ExternalContactsSyncUpdated({required this.contacts});
 
   @override
-  List<Object> get props => [
-        EquatablePropToString(contacts, listPropToString),
-      ];
+  List<Object?> get props => [EquatablePropToString.list(contacts)];
 }

lib/blocs/external_contacts_sync/external_contacts_sync_state.dart

@@ -5,7 +5,7 @@ abstract class ExternalContactsSyncState extends Equatable {
   const ExternalContactsSyncState();
 
   @override
-  List<Object> get props => [];
+  List<Object?> get props => [];
 }
 
 class ExternalContactsSyncInitial extends ExternalContactsSyncState {

lib/blocs/local_contacts_sync/local_contacts_sync_event.dart

@@ -4,7 +4,7 @@ sealed class LocalContactsSyncEvent extends Equatable {
   const LocalContactsSyncEvent();
 
   @override
-  List<Object> get props => [];
+  List<Object?> get props => [];
 }
 
 class LocalContactsSyncStarted extends LocalContactsSyncEvent {
@@ -18,12 +18,8 @@ class LocalContactsSyncRefreshed extends LocalContactsSyncEvent {
 class _LocalContactsSyncUpdated extends LocalContactsSyncEvent {
   final List<LocalContact> contacts;
 
-  const _LocalContactsSyncUpdated({
-    required this.contacts,
-  });
+  const _LocalContactsSyncUpdated({required this.contacts});
 
   @override
-  List<Object> get props => [
-        EquatablePropToString(contacts, listPropToString),
-      ];
+  List<Object?> get props => [EquatablePropToString.list(contacts)];
 }

lib/blocs/local_contacts_sync/local_contacts_sync_state.dart

@@ -5,7 +5,7 @@ abstract class LocalContactsSyncState extends Equatable {
   const LocalContactsSyncState();
 
   @override
-  List<Object> get props => [];
+  List<Object?> get props => [];
 }
 
 class LocalContactsSyncInitial extends LocalContactsSyncState {

lib/data/app_logger.dart

@@ -1,3 +1,4 @@
+import 'package:equatable/equatable.dart';
 import 'package:logging/logging.dart';
 import 'package:logging_appenders/logging_appenders.dart';
 
@@ -21,6 +22,8 @@ class AppLogger {
     Logger.root.clearListeners();
     Logger.root.level = localLogLevel;
 
+    EquatableConfig.stringify = localLogLevel <= Level.FINE || logzioLogLevel <= Level.FINE;
+
     // Set up local logs printing with a color formatter
     PrintAppender(formatter: const ColorFormatter()).attachToLogger(Logger.root);
 

lib/data/app_themes.dart

@@ -111,7 +111,5 @@ class AppTheme extends Equatable {
   final ThemeSettings settings;
 
   @override
-  List<Object> get props => [
-        settings,
-      ];
+  List<Object?> get props => [settings];
 }