Skip to content

Commit 9f96218

Browse files
committed
refactor(logger): adjust log levels in demo services
- Change most logger calls to fine() or finner() from info() - This reduces log verbosity for normal operation - Keeps important migration steps at info level
1 parent 5cc4cdb commit 9f96218

File tree

2 files changed

+27
-27
lines changed

2 files changed

+27
-27
lines changed

lib/app/services/demo_data_initializer_service.dart

Lines changed: 19 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ class DemoDataInitializerService {
8383
/// This prevents "READ FAILED" errors when the application attempts to
8484
/// access these user-specific data points for a newly signed-in anonymous
8585
/// user in the demo environment.
86-
Future<void> initializeUserSpecificData(User user) async {
86+
Future<void> initializeUserSpecificData(User user) async {
8787
_logger.info('Initializing user-specific data for user ID: ${user.id}');
8888

8989
await Future.wait([
@@ -105,9 +105,9 @@ class DemoDataInitializerService {
105105
Future<void> _ensureAppSettingsExist(String userId) async {
106106
try {
107107
await _appSettingsRepository.read(id: userId, userId: userId);
108-
_logger.info('AppSettings found for user ID: $userId.');
108+
_logger.finer('AppSettings found for user ID: $userId.');
109109
} on NotFoundException {
110-
_logger.info(
110+
_logger.fine(
111111
'AppSettings not found for user ID: '
112112
'$userId. Creating settings from fixture.',
113113
);
@@ -126,7 +126,7 @@ class DemoDataInitializerService {
126126
item: fixtureSettings,
127127
userId: userId,
128128
);
129-
_logger.info(
129+
_logger.fine(
130130
'AppSettings from fixture created for '
131131
'user ID: $userId.',
132132
);
@@ -146,9 +146,9 @@ class DemoDataInitializerService {
146146
Future<void> _ensureUserContentPreferencesExist(String userId) async {
147147
try {
148148
await _userContentPreferencesRepository.read(id: userId, userId: userId);
149-
_logger.info('UserContentPreferences found for user ID: $userId.');
149+
_logger.finer('UserContentPreferences found for user ID: $userId.');
150150
} on NotFoundException {
151-
_logger.info(
151+
_logger.fine(
152152
'UserContentPreferences not found for '
153153
'user ID: $userId. Creating preferences from fixture.',
154154
);
@@ -166,7 +166,7 @@ class DemoDataInitializerService {
166166
item: fixturePreferences,
167167
userId: userId,
168168
);
169-
_logger.info(
169+
_logger.fine(
170170
'UserContentPreferences from fixture created '
171171
'for user ID: $userId.',
172172
);
@@ -193,11 +193,11 @@ class DemoDataInitializerService {
193193
userId: userId,
194194
);
195195
if (existingNotifications.items.isNotEmpty) {
196-
_logger.info('InAppNotifications already exist for user ID: $userId.');
196+
_logger.finer('InAppNotifications already exist for user ID: $userId.');
197197
return;
198198
}
199199

200-
_logger.info(
200+
_logger.fine(
201201
'No InAppNotifications found for user ID: $userId. Creating from fixture.',
202202
);
203203

@@ -222,7 +222,7 @@ class DemoDataInitializerService {
222222
(n) => _inAppNotificationRepository.create(item: n, userId: userId),
223223
),
224224
);
225-
_logger.info(
225+
_logger.fine(
226226
'${userNotifications.length} InAppNotifications from fixture created for user ID: $userId.',
227227
);
228228
} catch (e, s) {
@@ -241,11 +241,11 @@ class DemoDataInitializerService {
241241
try {
242242
final existing = await _engagementRepository.readAll(userId: userId);
243243
if (existing.items.isNotEmpty) {
244-
_logger.info('Engagements already exist for user ID: $userId.');
244+
_logger.finer('Engagements already exist for user ID: $userId.');
245245
return;
246246
}
247247

248-
_logger.info(
248+
_logger.fine(
249249
'No Engagements found for user ID: $userId. Creating from fixture.',
250250
);
251251

@@ -268,7 +268,7 @@ class DemoDataInitializerService {
268268
(item) => _engagementRepository.create(item: item, userId: userId),
269269
),
270270
);
271-
_logger.info(
271+
_logger.fine(
272272
'${userItems.length} Engagements from fixture created for user ID: $userId.',
273273
);
274274
} catch (e, s) {
@@ -285,11 +285,11 @@ class DemoDataInitializerService {
285285
try {
286286
final existing = await _reportRepository.readAll(userId: userId);
287287
if (existing.items.isNotEmpty) {
288-
_logger.info('Reports already exist for user ID: $userId.');
288+
_logger.finer('Reports already exist for user ID: $userId.');
289289
return;
290290
}
291291

292-
_logger.info(
292+
_logger.fine(
293293
'No Reports found for user ID: $userId. Creating from fixture.',
294294
);
295295

@@ -307,7 +307,7 @@ class DemoDataInitializerService {
307307
(item) => _reportRepository.create(item: item, userId: userId),
308308
),
309309
);
310-
_logger.info(
310+
_logger.fine(
311311
'${userItems.length} Reports from fixture created for user ID: $userId.',
312312
);
313313
} catch (e, s) {
@@ -324,11 +324,11 @@ class DemoDataInitializerService {
324324
try {
325325
final existing = await _appReviewRepository.readAll(userId: userId);
326326
if (existing.items.isNotEmpty) {
327-
_logger.info('AppReviews already exist for user ID: $userId.');
327+
_logger.finer('AppReviews already exist for user ID: $userId.');
328328
return;
329329
}
330330

331-
_logger.info(
331+
_logger.fine(
332332
'No AppReviews found for user ID: $userId. Creating from fixture.',
333333
);
334334

@@ -346,7 +346,7 @@ class DemoDataInitializerService {
346346
(item) => _appReviewRepository.create(item: item, userId: userId),
347347
),
348348
);
349-
_logger.info(
349+
_logger.fine(
350350
'${userItems.length} AppReviews from fixture created for user ID: $userId.',
351351
);
352352
} catch (e, s) {

lib/app/services/demo_data_migration_service.dart

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ class DemoDataMigrationService {
4343
required String oldUserId,
4444
required String newUserId,
4545
}) async {
46-
_logger.info(
46+
_logger.fine(
4747
'[DemoDataMigrationService] Attempting to migrate data from '
4848
'anonymous user ID: $oldUserId to authenticated user ID: $newUserId',
4949
);
@@ -82,12 +82,12 @@ class DemoDataMigrationService {
8282
}
8383

8484
await _appSettingsRepository.delete(id: oldUserId, userId: oldUserId);
85-
_logger.info(
85+
_logger.fine(
8686
'[DemoDataMigrationService] AppSettings migrated successfully '
8787
'from $oldUserId to $newUserId.',
8888
);
8989
} on NotFoundException {
90-
_logger.info(
90+
_logger.fine(
9191
'[DemoDataMigrationService] No AppSettings found for old user ID: '
9292
'$oldUserId. Skipping migration for settings.',
9393
);
@@ -137,12 +137,12 @@ class DemoDataMigrationService {
137137
id: oldUserId,
138138
userId: oldUserId,
139139
);
140-
_logger.info(
140+
_logger.fine(
141141
'[DemoDataMigrationService] UserContentPreferences migrated '
142142
'successfully from $oldUserId to $newUserId.',
143143
);
144144
} on NotFoundException {
145-
_logger.info(
145+
_logger.fine(
146146
'[DemoDataMigrationService] No UserContentPreferences found for old '
147147
'user ID: $oldUserId. Skipping migration for preferences.',
148148
);
@@ -175,7 +175,7 @@ class DemoDataMigrationService {
175175
/* ignore, already exists */
176176
}
177177
}
178-
_logger.info(
178+
_logger.fine(
179179
'[DemoDataMigrationService] ${oldEngagements.items.length} '
180180
'engagements migrated successfully from $oldUserId to $newUserId.',
181181
);
@@ -196,7 +196,7 @@ class DemoDataMigrationService {
196196
await _reportRepository.create(item: newReport, userId: newUserId);
197197
await _reportRepository.delete(id: oldReport.id, userId: oldUserId);
198198
}
199-
_logger.info(
199+
_logger.fine(
200200
'[DemoDataMigrationService] ${oldReports.items.length} '
201201
'reports migrated successfully from $oldUserId to $newUserId.',
202202
);
@@ -225,7 +225,7 @@ class DemoDataMigrationService {
225225
userId: oldUserId,
226226
);
227227
}
228-
_logger.info(
228+
_logger.fine(
229229
'[DemoDataMigrationService] ${oldAppReviews.items.length} '
230230
'app reviews migrated successfully from $oldUserId to $newUserId.',
231231
);

0 commit comments

Comments
 (0)