From fe068ee38573b8f70c9fd778e876967d39cbee72 Mon Sep 17 00:00:00 2001 From: jld3103 Date: Wed, 19 Apr 2023 07:12:38 +0200 Subject: [PATCH] neon,neon_notifications: Cleanup finding app implementation --- packages/neon/neon/lib/src/app.dart | 34 +++++++++---------- packages/neon/neon/lib/src/blocs/apps.dart | 7 ++-- packages/neon/neon/lib/src/pages/home.dart | 4 +-- .../lib/src/utils/app_implementation.dart | 12 +++++++ .../lib/src/utils/settings_export_helper.dart | 6 ++-- .../neon_notifications/lib/pages/main.dart | 14 +++----- 6 files changed, 40 insertions(+), 37 deletions(-) diff --git a/packages/neon/neon/lib/src/app.dart b/packages/neon/neon/lib/src/app.dart index a39647e8..cc6a5b33 100644 --- a/packages/neon/neon/lib/src/app.dart +++ b/packages/neon/neon/lib/src/app.dart @@ -171,22 +171,18 @@ class _NeonAppState extends State with WidgetsBindingObserver, tray.Tra if (account == null) { return; } - final appImplementation = Provider.of>(context, listen: false) - .singleWhere((final a) => a.id == 'notifications'); - await _accountsBloc.getAppsBloc(account).getAppBloc(appImplementation).refresh(); + final app = Provider.of>(context, listen: false).find('notifications'); + if (app != null) { + await _accountsBloc.getAppsBloc(account).getAppBloc(app).refresh(); + } }; Global.onPushNotificationClicked = (final pushNotificationWithAccountID) async { final allAppImplementations = Provider.of>(context, listen: false); - final matchingAppImplementations = - allAppImplementations.where((final a) => a.id == pushNotificationWithAccountID.subject.app); - - late AppImplementation appImplementation; - if (matchingAppImplementations.isNotEmpty) { - appImplementation = matchingAppImplementations.single; - } else { - appImplementation = allAppImplementations.singleWhere((final a) => a.id == 'notifications'); - } + final app = (pushNotificationWithAccountID.subject.app != null + ? allAppImplementations.find(pushNotificationWithAccountID.subject.app!) + : null) ?? + allAppImplementations.find('notifications'); final account = _accountsBloc.accounts.value.find(pushNotificationWithAccountID.accountID); if (account == null) { @@ -194,13 +190,15 @@ class _NeonAppState extends State with WidgetsBindingObserver, tray.Tra } _accountsBloc.setActiveAccount(account); - if (appImplementation.id != 'notifications') { - _accountsBloc - .getAppsBloc(account) - .getAppBloc(appImplementation) - .deleteNotification(pushNotificationWithAccountID.subject.nid!); + if (app != null) { + if (app.id != 'notifications') { + _accountsBloc + .getAppsBloc(account) + .getAppBloc(app) + .deleteNotification(pushNotificationWithAccountID.subject.nid!); + } + await _openAppFromExternal(account, app.id); } - await _openAppFromExternal(account, appImplementation.id); }; final details = await localNotificationsPlugin.getNotificationAppLaunchDetails(); diff --git a/packages/neon/neon/lib/src/blocs/apps.dart b/packages/neon/neon/lib/src/blocs/apps.dart index 319eca16..129539f5 100644 --- a/packages/neon/neon/lib/src/blocs/apps.dart +++ b/packages/neon/neon/lib/src/blocs/apps.dart @@ -37,7 +37,7 @@ class AppsBloc extends InteractiveBloc implements AppsBlocEvents, AppsBlocStates unawaited( options.initialApp.stream.first.then((var initialApp) async { if (initialApp == null) { - if (result.data!.where((final a) => a.id == 'files').isNotEmpty) { + if (result.data!.find('files') != null) { initialApp = 'files'; } else if (result.data!.isNotEmpty) { // This should never happen, because the files app is always installed and can not be removed, but just in @@ -126,10 +126,7 @@ class AppsBloc extends InteractiveBloc implements AppsBlocEvents, AppsBlocStates @override Future setActiveApp(final String? appID) async { - if ((await appImplementations.firstWhere((final a) => a.data != null)) - .data! - .where((final app) => app.id == appID) - .isNotEmpty) { + if (appID != null && (await appImplementations.firstWhere((final a) => a.data != null)).data!.find(appID) != null) { if (activeAppID.valueOrNull != appID) { activeAppID.add(appID); } diff --git a/packages/neon/neon/lib/src/pages/home.dart b/packages/neon/neon/lib/src/pages/home.dart index 796f3f67..c2fd12af 100644 --- a/packages/neon/neon/lib/src/pages/home.dart +++ b/packages/neon/neon/lib/src/pages/home.dart @@ -474,7 +474,7 @@ class _HomePageState extends State { Flexible( child: Text( appImplementations.data! - .singleWhere((final a) => a.id == activeAppIDSnapshot.data!) + .find(activeAppIDSnapshot.data!)! .name(context), ), ), @@ -583,7 +583,7 @@ class _HomePageState extends State { if (activeAppIDSnapshot.hasData) ...[ Expanded( child: appImplementations.data! - .singleWhere((final a) => a.id == activeAppIDSnapshot.data!) + .find(activeAppIDSnapshot.data!)! .buildPage(context, _appsBloc), ), ], diff --git a/packages/neon/neon/lib/src/utils/app_implementation.dart b/packages/neon/neon/lib/src/utils/app_implementation.dart index ac0245c4..a3159d40 100644 --- a/packages/neon/neon/lib/src/utils/app_implementation.dart +++ b/packages/neon/neon/lib/src/utils/app_implementation.dart @@ -46,3 +46,15 @@ abstract class AppImplementation { + AppImplementation? find(final String appID) { + for (final app in this) { + if (app.id == appID) { + return app; + } + } + + return null; + } +} diff --git a/packages/neon/neon/lib/src/utils/settings_export_helper.dart b/packages/neon/neon/lib/src/utils/settings_export_helper.dart index 5057fdba..e62b1a7c 100644 --- a/packages/neon/neon/lib/src/utils/settings_export_helper.dart +++ b/packages/neon/neon/lib/src/utils/settings_export_helper.dart @@ -20,13 +20,13 @@ class SettingsExportHelper { final appImplementationsData = data['apps'] as Map; for (final appId in appImplementationsData.keys) { - final matchingAppImplementations = appImplementations.where((final app) => app.id == appId).toList(); - if (matchingAppImplementations.length != 1) { + final app = appImplementations.find(appId); + if (app == null) { return; } final appImplementationData = appImplementationsData[appId]! as Map; await _applyOptionsMapToOptions( - matchingAppImplementations[0].options.options, + app.options.options, appImplementationData, ); } diff --git a/packages/neon/neon_notifications/lib/pages/main.dart b/packages/neon/neon_notifications/lib/pages/main.dart index 21ffc71e..7e0c0c18 100644 --- a/packages/neon/neon_notifications/lib/pages/main.dart +++ b/packages/neon/neon_notifications/lib/pages/main.dart @@ -50,9 +50,7 @@ class _NotificationsMainPageState extends State { final BuildContext context, final NextcloudNotificationsNotification notification, ) { - final matchingAppImplementations = Provider.of>(context, listen: false) - .where((final a) => a.id == notification.app) - .toList(); + final app = Provider.of>(context, listen: false).find(notification.app); return ListTile( title: Text(notification.subject), @@ -73,8 +71,8 @@ class _NotificationsMainPageState extends State { ), ], ), - leading: matchingAppImplementations.isNotEmpty - ? matchingAppImplementations.single.buildIcon( + leading: app != null + ? app.buildIcon( context, width: 40, height: 40, @@ -93,11 +91,9 @@ class _NotificationsMainPageState extends State { if (notification.app == 'notifications') { return; } - final allAppImplementations = Provider.of>(context, listen: false); - final matchingAppImplementations = allAppImplementations.where((final a) => a.id == notification.app); - if (matchingAppImplementations.isNotEmpty) { + if (app != null) { final accountsBloc = Provider.of(context, listen: false); - await accountsBloc.getAppsBloc(accountsBloc.activeAccount.value!).setActiveApp(notification.app); + await accountsBloc.getAppsBloc(accountsBloc.activeAccount.value!).setActiveApp(app.id); } else { await showDialog( context: context,