Skip to content

Commit fd5f54c

Browse files
committed
refactor: code formatting and minor adjustments
- Improved code readability - Updated constructor formatting - Adjusted conditional logic
1 parent c26fa14 commit fd5f54c

File tree

5 files changed

+28
-33
lines changed

5 files changed

+28
-33
lines changed

lib/account/bloc/account_bloc.dart

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ import 'package:ht_shared/ht_shared.dart';
1010
part 'account_event.dart';
1111
part 'account_state.dart';
1212

13-
1413
class AccountBloc extends Bloc<AccountEvent, AccountState> {
1514
AccountBloc({
1615
required HtAuthRepository authenticationRepository,

lib/app/view/app.dart

Lines changed: 14 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -29,22 +29,22 @@ class App extends StatelessWidget {
2929
required HtDataRepository<Source> htSourcesRepository,
3030
required HtDataRepository<UserAppSettings> htUserAppSettingsRepository,
3131
required HtDataRepository<UserContentPreferences>
32-
htUserContentPreferencesRepository,
32+
htUserContentPreferencesRepository,
3333
required HtDataRepository<AppConfig> htAppConfigRepository,
3434
required HtKVStorageService kvStorageService,
3535
required AppEnvironment environment,
3636
this.demoDataMigrationService,
3737
super.key,
38-
}) : _htAuthenticationRepository = htAuthenticationRepository,
39-
_htHeadlinesRepository = htHeadlinesRepository,
40-
_htCategoriesRepository = htCategoriesRepository,
41-
_htCountriesRepository = htCountriesRepository,
42-
_htSourcesRepository = htSourcesRepository,
43-
_htUserAppSettingsRepository = htUserAppSettingsRepository,
44-
_htUserContentPreferencesRepository = htUserContentPreferencesRepository,
45-
_htAppConfigRepository = htAppConfigRepository,
46-
_kvStorageService = kvStorageService,
47-
_environment = environment;
38+
}) : _htAuthenticationRepository = htAuthenticationRepository,
39+
_htHeadlinesRepository = htHeadlinesRepository,
40+
_htCategoriesRepository = htCategoriesRepository,
41+
_htCountriesRepository = htCountriesRepository,
42+
_htSourcesRepository = htSourcesRepository,
43+
_htUserAppSettingsRepository = htUserAppSettingsRepository,
44+
_htUserContentPreferencesRepository = htUserContentPreferencesRepository,
45+
_htAppConfigRepository = htAppConfigRepository,
46+
_kvStorageService = kvStorageService,
47+
_environment = environment;
4848

4949
final HtAuthRepository _htAuthenticationRepository;
5050
final HtDataRepository<Headline> _htHeadlinesRepository;
@@ -53,7 +53,7 @@ class App extends StatelessWidget {
5353
final HtDataRepository<Source> _htSourcesRepository;
5454
final HtDataRepository<UserAppSettings> _htUserAppSettingsRepository;
5555
final HtDataRepository<UserContentPreferences>
56-
_htUserContentPreferencesRepository;
56+
_htUserContentPreferencesRepository;
5757
final HtDataRepository<AppConfig> _htAppConfigRepository;
5858
final HtKVStorageService _kvStorageService;
5959
final AppEnvironment _environment;
@@ -78,8 +78,8 @@ class App extends StatelessWidget {
7878
BlocProvider(
7979
create: (context) => AppBloc(
8080
authenticationRepository: context.read<HtAuthRepository>(),
81-
userAppSettingsRepository:
82-
context.read<HtDataRepository<UserAppSettings>>(),
81+
userAppSettingsRepository: context
82+
.read<HtDataRepository<UserAppSettings>>(),
8383
appConfigRepository: context.read<HtDataRepository<AppConfig>>(),
8484
environment: _environment,
8585
demoDataMigrationService: demoDataMigrationService,
@@ -108,7 +108,6 @@ class App extends StatelessWidget {
108108
}
109109
}
110110

111-
112111
class _AppView extends StatefulWidget {
113112
const _AppView({
114113
required this.htAuthenticationRepository,

lib/bootstrap.dart

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -212,11 +212,11 @@ Future<Widget> bootstrap(
212212
// Conditionally instantiate DemoDataMigrationService
213213
final DemoDataMigrationService? demoDataMigrationService =
214214
appConfig.environment == app_config.AppEnvironment.demo
215-
? DemoDataMigrationService(
216-
userAppSettingsRepository: userAppSettingsRepository,
217-
userContentPreferencesRepository: userContentPreferencesRepository,
218-
)
219-
: null;
215+
? DemoDataMigrationService(
216+
userAppSettingsRepository: userAppSettingsRepository,
217+
userContentPreferencesRepository: userContentPreferencesRepository,
218+
)
219+
: null;
220220

221221
return App(
222222
htAuthenticationRepository: authenticationRepository,

lib/shared/services/demo_data_migration_service.dart

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -14,13 +14,13 @@ class DemoDataMigrationService {
1414
const DemoDataMigrationService({
1515
required HtDataRepository<UserAppSettings> userAppSettingsRepository,
1616
required HtDataRepository<UserContentPreferences>
17-
userContentPreferencesRepository,
18-
}) : _userAppSettingsRepository = userAppSettingsRepository,
19-
_userContentPreferencesRepository = userContentPreferencesRepository;
17+
userContentPreferencesRepository,
18+
}) : _userAppSettingsRepository = userAppSettingsRepository,
19+
_userContentPreferencesRepository = userContentPreferencesRepository;
2020

2121
final HtDataRepository<UserAppSettings> _userAppSettingsRepository;
2222
final HtDataRepository<UserContentPreferences>
23-
_userContentPreferencesRepository;
23+
_userContentPreferencesRepository;
2424

2525
/// Migrates user settings and content preferences from an old anonymous
2626
/// user ID to a new authenticated user ID.
@@ -69,10 +69,7 @@ class DemoDataMigrationService {
6969
}
7070
}
7171

72-
await _userAppSettingsRepository.delete(
73-
id: oldUserId,
74-
userId: oldUserId,
75-
);
72+
await _userAppSettingsRepository.delete(id: oldUserId, userId: oldUserId);
7673
print(
7774
'[DemoDataMigrationService] UserAppSettings migrated successfully '
7875
'from $oldUserId to $newUserId.',

lib/shared/services/feed_injector_service.dart

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -108,14 +108,14 @@ class FeedInjectorService {
108108
if (userRole == UserRole.guestUser) {
109109
daysThreshold = daysBetweenActionsConfig.guestDaysBetweenAccountActions;
110110
actionType = AccountActionType.linkAccount;
111-
} else if (userRole == UserRole.standardUser) {
111+
} else if (userRole == UserRole.standardUser) {
112112
daysThreshold =
113113
daysBetweenActionsConfig.standardUserDaysBetweenAccountActions;
114-
115-
// todo(fulleni): once account upgrade feature is implemented,
114+
115+
// todo(fulleni): once account upgrade feature is implemented,
116116
// uncomment the action type line
117117
// and remove teh null return line.
118-
118+
119119
// actionType = AccountActionType.upgrade;
120120
return null;
121121
} else {

0 commit comments

Comments
 (0)