refactor(account): remove duplicate limitation bottom sheet code

fulleni · fulleni · commit bb90381f6cce · 2025-12-12T09:41:33.000+01:00
- Remove duplicate _getBottomSheetContent function from multiple files
- Replace custom bottom sheet logic with showContentLimitationBottomSheet function
- Remove unnecessary imports and simplify code
diff --git a/lib/account/view/followed_contents/countries/add_country_to_follow_page.dart b/lib/account/view/followed_contents/countries/add_country_to_follow_page.dart
@@ -6,10 +6,8 @@ import 'package:flutter_news_app_mobile_client_full_source_code/account/bloc/ava
 import 'package:flutter_news_app_mobile_client_full_source_code/app/bloc/app_bloc.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/l10n/app_localizations.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/l10n/l10n.dart';
-import 'package:flutter_news_app_mobile_client_full_source_code/router/routes.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/shared/services/content_limitation_service.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/shared/widgets/content_limitation_bottom_sheet.dart';
-import 'package:go_router/go_router.dart';
 import 'package:ui_kit/ui_kit.dart';
 
 class _FollowButton extends StatefulWidget {
@@ -52,23 +50,10 @@ class _FollowButtonState extends State<_FollowButton> {
 
         if (status != LimitationStatus.allowed) {
           if (mounted) {
-            final userRole = context.read<AppBloc>().state.user?.appRole;
-            final content = _getBottomSheetContent(
+            showContentLimitationBottomSheet(
               context: context,
-              l10n: l10n,
               status: status,
-              userRole: userRole,
-              defaultBody: l10n.limitReachedBodyFollow,
-            );
-
-            await showModalBottomSheet<void>(
-              context: context,
-              builder: (_) => ContentLimitationBottomSheet(
-                title: content.title,
-                body: content.body,
-                buttonText: content.buttonText,
-                onButtonPressed: content.onPressed,
-              ),
+              action: ContentAction.followCountry,
             );
           }
           return;
@@ -126,50 +111,6 @@ class _FollowButtonState extends State<_FollowButton> {
   }
 }
 
-/// Determines the content for the [ContentLimitationBottomSheet] based on
-/// the user's role and the limitation status.
-({String title, String body, String buttonText, VoidCallback? onPressed})
-_getBottomSheetContent({
-  required BuildContext context,
-  required AppLocalizations l10n,
-  required LimitationStatus status,
-  required AppUserRole? userRole,
-  required String defaultBody,
-}) {
-  switch (status) {
-    case LimitationStatus.anonymousLimitReached:
-      return (
-        title: l10n.anonymousLimitTitle,
-        body: l10n.anonymousLimitBody,
-        buttonText: l10n.anonymousLimitButton,
-        onPressed: () {
-          Navigator.of(context).pop();
-          context.pushNamed(Routes.accountLinkingName);
-        },
-      );
-    case LimitationStatus.standardUserLimitReached:
-      // TODO(fulleni): Implement upgrade flow.
-      return (
-        title: l10n.standardLimitTitle,
-        body: l10n.standardLimitBody,
-        buttonText: l10n.standardLimitButton,
-        onPressed: () => Navigator.of(context).pop(),
-      );
-    case LimitationStatus.premiumUserLimitReached:
-      return (
-        title: l10n.premiumLimitTitle,
-        body: defaultBody,
-        buttonText: l10n.premiumLimitButton,
-        onPressed: () {
-          Navigator.of(context).pop();
-          context.goNamed(Routes.manageFollowedItemsName);
-        },
-      );
-    case LimitationStatus.allowed:
-      return (title: '', body: '', buttonText: '', onPressed: null);
-  }
-}
-
 /// {@template add_country_to_follow_page}
 /// A page that allows users to browse and select countries to follow.
 /// {@endtemplate}
diff --git a/lib/account/view/followed_contents/sources/add_source_to_follow_page.dart b/lib/account/view/followed_contents/sources/add_source_to_follow_page.dart
@@ -6,10 +6,8 @@ import 'package:flutter_news_app_mobile_client_full_source_code/account/bloc/ava
 import 'package:flutter_news_app_mobile_client_full_source_code/app/bloc/app_bloc.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/l10n/app_localizations.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/l10n/l10n.dart';
-import 'package:flutter_news_app_mobile_client_full_source_code/router/routes.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/shared/services/content_limitation_service.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/shared/widgets/content_limitation_bottom_sheet.dart';
-import 'package:go_router/go_router.dart';
 import 'package:ui_kit/ui_kit.dart';
 
 class _FollowButton extends StatefulWidget {
@@ -52,23 +50,10 @@ class _FollowButtonState extends State<_FollowButton> {
 
         if (status != LimitationStatus.allowed) {
           if (mounted) {
-            final userRole = context.read<AppBloc>().state.user?.appRole;
-            final content = _getBottomSheetContent(
+            showContentLimitationBottomSheet(
               context: context,
-              l10n: l10n,
               status: status,
-              userRole: userRole,
-              defaultBody: l10n.limitReachedBodyFollow,
-            );
-
-            await showModalBottomSheet<void>(
-              context: context,
-              builder: (_) => ContentLimitationBottomSheet(
-                title: content.title,
-                body: content.body,
-                buttonText: content.buttonText,
-                onButtonPressed: content.onPressed,
-              ),
+              action: ContentAction.followSource,
             );
           }
           return;
@@ -126,50 +111,6 @@ class _FollowButtonState extends State<_FollowButton> {
   }
 }
 
-/// Determines the content for the [ContentLimitationBottomSheet] based on
-/// the user's role and the limitation status.
-({String title, String body, String buttonText, VoidCallback? onPressed})
-_getBottomSheetContent({
-  required BuildContext context,
-  required AppLocalizations l10n,
-  required LimitationStatus status,
-  required AppUserRole? userRole,
-  required String defaultBody,
-}) {
-  switch (status) {
-    case LimitationStatus.anonymousLimitReached:
-      return (
-        title: l10n.anonymousLimitTitle,
-        body: l10n.anonymousLimitBody,
-        buttonText: l10n.anonymousLimitButton,
-        onPressed: () {
-          Navigator.of(context).pop();
-          context.pushNamed(Routes.accountLinkingName);
-        },
-      );
-    case LimitationStatus.standardUserLimitReached:
-      // TODO(fulleni): Implement upgrade flow.
-      return (
-        title: l10n.standardLimitTitle,
-        body: l10n.standardLimitBody,
-        buttonText: l10n.standardLimitButton,
-        onPressed: () => Navigator.of(context).pop(),
-      );
-    case LimitationStatus.premiumUserLimitReached:
-      return (
-        title: l10n.premiumLimitTitle,
-        body: defaultBody,
-        buttonText: l10n.premiumLimitButton,
-        onPressed: () {
-          Navigator.of(context).pop();
-          context.goNamed(Routes.manageFollowedItemsName);
-        },
-      );
-    case LimitationStatus.allowed:
-      return (title: '', body: '', buttonText: '', onPressed: null);
-  }
-}
-
 /// {@template add_source_to_follow_page}
 /// A page that allows users to browse and select sources to follow.
 /// {@endtemplate}
diff --git a/lib/account/view/followed_contents/topics/add_topic_to_follow_page.dart b/lib/account/view/followed_contents/topics/add_topic_to_follow_page.dart
@@ -6,10 +6,8 @@ import 'package:flutter_news_app_mobile_client_full_source_code/account/bloc/ava
 import 'package:flutter_news_app_mobile_client_full_source_code/app/bloc/app_bloc.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/l10n/app_localizations.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/l10n/l10n.dart';
-import 'package:flutter_news_app_mobile_client_full_source_code/router/routes.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/shared/services/content_limitation_service.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/shared/widgets/content_limitation_bottom_sheet.dart';
-import 'package:go_router/go_router.dart';
 import 'package:ui_kit/ui_kit.dart';
 
 class _FollowButton extends StatefulWidget {
@@ -52,23 +50,10 @@ class _FollowButtonState extends State<_FollowButton> {
 
         if (status != LimitationStatus.allowed) {
           if (mounted) {
-            final userRole = context.read<AppBloc>().state.user?.appRole;
-            final content = _getBottomSheetContent(
+            showContentLimitationBottomSheet(
               context: context,
-              l10n: l10n,
               status: status,
-              userRole: userRole,
-              defaultBody: l10n.limitReachedBodyFollow,
-            );
-
-            await showModalBottomSheet<void>(
-              context: context,
-              builder: (_) => ContentLimitationBottomSheet(
-                title: content.title,
-                body: content.body,
-                buttonText: content.buttonText,
-                onButtonPressed: content.onPressed,
-              ),
+              action: ContentAction.followTopic,
             );
           }
           return;
@@ -126,50 +111,6 @@ class _FollowButtonState extends State<_FollowButton> {
   }
 }
 
-/// Determines the content for the [ContentLimitationBottomSheet] based on
-/// the user's role and the limitation status.
-({String title, String body, String buttonText, VoidCallback? onPressed})
-_getBottomSheetContent({
-  required BuildContext context,
-  required AppLocalizations l10n,
-  required LimitationStatus status,
-  required AppUserRole? userRole,
-  required String defaultBody,
-}) {
-  switch (status) {
-    case LimitationStatus.anonymousLimitReached:
-      return (
-        title: l10n.anonymousLimitTitle,
-        body: l10n.anonymousLimitBody,
-        buttonText: l10n.anonymousLimitButton,
-        onPressed: () {
-          Navigator.of(context).pop();
-          context.pushNamed(Routes.accountLinkingName);
-        },
-      );
-    case LimitationStatus.standardUserLimitReached:
-      // TODO(fulleni): Implement upgrade flow.
-      return (
-        title: l10n.standardLimitTitle,
-        body: l10n.standardLimitBody,
-        buttonText: l10n.standardLimitButton,
-        onPressed: () => Navigator.of(context).pop(),
-      );
-    case LimitationStatus.premiumUserLimitReached:
-      return (
-        title: l10n.premiumLimitTitle,
-        body: defaultBody,
-        buttonText: l10n.premiumLimitButton,
-        onPressed: () {
-          Navigator.of(context).pop();
-          context.goNamed(Routes.manageFollowedItemsName);
-        },
-      );
-    case LimitationStatus.allowed:
-      return (title: '', body: '', buttonText: '', onPressed: null);
-  }
-}
-
 /// {@template add_topic_to_follow_page}
 /// A page that allows users to browse and select topics to follow.
 /// {@endtemplate}
