Skip to content

Commit 16794d5

Browse files
committed
refactor(app): restructure app version check logic
- Invert condition to check if current version is up to date or newer - Move update required logic to else block - Improve code readability and maintain the same functionality
1 parent 960ca24 commit 16794d5

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

lib/app/bloc/app_bloc.dart

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -633,18 +633,7 @@ class AppBloc extends Bloc<AppEvent, AppState> {
633633
remoteConfig.appStatus.latestAppVersion,
634634
);
635635

636-
if (currentVersion < latestRequiredVersion) {
637-
_logger.info(
638-
'[AppBloc] App version ($currentVersion) is older than '
639-
'required ($latestRequiredVersion). Transitioning to updateRequired state.',
640-
);
641-
emit(
642-
state.copyWith(
643-
status: AppLifeCycleStatus.updateRequired,
644-
currentAppVersion: currentAppVersionString,
645-
),
646-
);
647-
} else {
636+
if (currentVersion >= latestRequiredVersion) {
648637
_logger.info(
649638
'[AppBloc] App version ($currentVersion) is up to date '
650639
'or newer than required ($latestRequiredVersion).',
@@ -664,6 +653,17 @@ class AppBloc extends Bloc<AppEvent, AppState> {
664653
} else {
665654
emit(state.copyWith(currentAppVersion: currentAppVersionString));
666655
}
656+
} else {
657+
_logger.info(
658+
'[AppBloc] App version ($currentVersion) is older than '
659+
'required ($latestRequiredVersion). Transitioning to updateRequired state.',
660+
);
661+
emit(
662+
state.copyWith(
663+
status: AppLifeCycleStatus.updateRequired,
664+
currentAppVersion: currentAppVersionString,
665+
),
666+
);
667667
}
668668
} on FormatException catch (e, s) {
669669
_logger.severe(

0 commit comments

Comments
 (0)