From 4f22e88e427cc9f08303a2a94cc94f95ac133c54 Mon Sep 17 00:00:00 2001 From: aAbed <39409020+TheAabedKhan@users.noreply.github.com> Date: Fri, 12 Jan 2024 04:00:37 +0545 Subject: [PATCH] fix: Incorrect strings and logics (#1619) --- lib/services/manager_api.dart | 4 +-- lib/services/revanced_api.dart | 4 +-- .../contributors/contributors_viewmodel.dart | 4 +-- lib/ui/views/home/home_viewmodel.dart | 3 +-- .../settings_manage_api_url.dart | 5 +++- .../settings_manage_sources.dart | 25 +++++++++++-------- 6 files changed, 26 insertions(+), 19 deletions(-) diff --git a/lib/services/manager_api.dart b/lib/services/manager_api.dart index c4a7ccd514..72c6fa46e4 100644 --- a/lib/services/manager_api.dart +++ b/lib/services/manager_api.dart @@ -52,12 +52,12 @@ class ManagerAPI { String? integrationsVersion = ''; bool isDefaultPatchesRepo() { - return getPatchesRepo().toLowerCase() == 'revanced/revanced-patches'; + return getPatchesRepo().toLowerCase() == defaultPatchesRepo; } bool isDefaultIntegrationsRepo() { return getIntegrationsRepo().toLowerCase() == - 'revanced/revanced-integrations'; + defaultIntegrationsRepo; } Future initialize() async { diff --git a/lib/services/revanced_api.dart b/lib/services/revanced_api.dart index 70d2064c98..2f5c210b70 100644 --- a/lib/services/revanced_api.dart +++ b/lib/services/revanced_api.dart @@ -32,7 +32,7 @@ class RevancedAPI { final response = await _dio.get('/contributors'); final List repositories = response.data['repositories']; for (final Map repo in repositories) { - final String name = repo['name']; + final String name = repo['name'].toLowerCase(); contributors[name] = repo['contributors']; } } on Exception catch (e) { @@ -54,7 +54,7 @@ class RevancedAPI { final List tools = response.data['tools']; return tools.firstWhereOrNull( (t) => - t['repository'] == repoName && + (t['repository'] as String).toLowerCase() == repoName.toLowerCase() && (t['name'] as String).endsWith(extension), ); } on Exception catch (e) { diff --git a/lib/ui/views/contributors/contributors_viewmodel.dart b/lib/ui/views/contributors/contributors_viewmodel.dart index c8c7975fd8..3085683c54 100644 --- a/lib/ui/views/contributors/contributors_viewmodel.dart +++ b/lib/ui/views/contributors/contributors_viewmodel.dart @@ -14,9 +14,9 @@ class ContributorsViewModel extends BaseViewModel { final Map> contributors = await _managerAPI.getContributors(); patcherContributors = contributors[_managerAPI.defaultPatcherRepo] ?? []; - patchesContributors = contributors[_managerAPI.getPatchesRepo()] ?? []; + patchesContributors = contributors[_managerAPI.getPatchesRepo().toLowerCase()] ?? []; integrationsContributors = - contributors[_managerAPI.getIntegrationsRepo()] ?? []; + contributors[_managerAPI.getIntegrationsRepo().toLowerCase()] ?? []; cliContributors = contributors[_managerAPI.defaultCliRepo] ?? []; managerContributors = contributors[_managerAPI.defaultManagerRepo] ?? []; notifyListeners(); diff --git a/lib/ui/views/home/home_viewmodel.dart b/lib/ui/views/home/home_viewmodel.dart index e5448c5e84..6913e409e1 100644 --- a/lib/ui/views/home/home_viewmodel.dart +++ b/lib/ui/views/home/home_viewmodel.dart @@ -40,12 +40,11 @@ class HomeViewModel extends BaseViewModel { Future initialize(BuildContext context) async { _managerAPI.rePatchedSavedApps().then((_) => _getPatchedApps()); + _latestManagerVersion = await _managerAPI.getLatestManagerVersion(); if (!_managerAPI.getPatchesConsent()) { await showPatchesConsent(context); } - _latestManagerVersion = await _managerAPI.getLatestManagerVersion(); - await _patcherAPI.initialize(); await flutterLocalNotificationsPlugin.initialize( diff --git a/lib/ui/views/settings/settingsFragment/settings_manage_api_url.dart b/lib/ui/views/settings/settingsFragment/settings_manage_api_url.dart index 20d2ef2b64..0784fe879a 100644 --- a/lib/ui/views/settings/settingsFragment/settings_manage_api_url.dart +++ b/lib/ui/views/settings/settingsFragment/settings_manage_api_url.dart @@ -43,7 +43,10 @@ class SManageApiUrl extends BaseViewModel { color: Theme.of(context).colorScheme.onSurfaceVariant, ), border: const OutlineInputBorder(), - labelText: I18nText('settingsView.selectApiURL').toString(), + labelText: FlutterI18n.translate( + context, + 'settingsView.selectApiURL', + ), hintText: apiUrl, ), ), diff --git a/lib/ui/views/settings/settingsFragment/settings_manage_sources.dart b/lib/ui/views/settings/settingsFragment/settings_manage_sources.dart index 9cf33bbc91..363dab8af4 100644 --- a/lib/ui/views/settings/settingsFragment/settings_manage_sources.dart +++ b/lib/ui/views/settings/settingsFragment/settings_manage_sources.dart @@ -58,9 +58,10 @@ class SManageSources extends BaseViewModel { color: Theme.of(context).colorScheme.onSurfaceVariant, ), border: const OutlineInputBorder(), - labelText: I18nText( + labelText: FlutterI18n.translate( + context, 'settingsView.hostRepositoryLabel', - ).toString(), + ), hintText: hostRepository, ), ), @@ -76,9 +77,10 @@ class SManageSources extends BaseViewModel { color: Theme.of(context).colorScheme.onSurfaceVariant, ), border: const OutlineInputBorder(), - labelText: I18nText( + labelText: FlutterI18n.translate( + context, 'settingsView.orgPatchesLabel', - ).toString(), + ), hintText: patchesRepo.split('/')[0], ), ), @@ -94,9 +96,10 @@ class SManageSources extends BaseViewModel { color: Colors.transparent, ), border: const OutlineInputBorder(), - labelText: I18nText( + labelText: FlutterI18n.translate( + context, 'settingsView.sourcesPatchesLabel', - ).toString(), + ), hintText: patchesRepo.split('/')[1], ), ), @@ -112,9 +115,10 @@ class SManageSources extends BaseViewModel { color: Theme.of(context).colorScheme.onSurfaceVariant, ), border: const OutlineInputBorder(), - labelText: I18nText( + labelText: FlutterI18n.translate( + context, 'settingsView.orgIntegrationsLabel', - ).toString(), + ), hintText: integrationsRepo.split('/')[0], ), ), @@ -130,9 +134,10 @@ class SManageSources extends BaseViewModel { color: Colors.transparent, ), border: const OutlineInputBorder(), - labelText: I18nText( + labelText: FlutterI18n.translate( + context, 'settingsView.sourcesIntegrationsLabel', - ).toString(), + ), hintText: integrationsRepo.split('/')[1], ), ),