TF-1115 Handle onError for all controller

This commit is contained in:
dab246
2022-10-28 13:12:14 +07:00
committed by Dat H. Pham
parent ef08949e98
commit 490787fc11
28 changed files with 11 additions and 103 deletions
+1 -3
View File
@@ -60,9 +60,7 @@ abstract class BaseController extends GetxController
messageError = error.message ?? error.type.value; messageError = error.message ?? error.type.value;
} else { } else {
if (currentContext != null) { if (currentContext != null) {
messageError = AppLocalizations messageError = AppLocalizations.of(currentContext!).unknownError;
.of(currentContext!)
.unknown_error_login_message;
} }
} }
@@ -281,18 +281,6 @@ class ComposerController extends BaseController {
}); });
} }
@override
void onError(error) {
if (currentOverlayContext != null && currentContext != null) {
_appToast.showToastWithIcon(
currentOverlayContext!,
textColor: AppColor.toastErrorBackgroundColor,
message: AppLocalizations.of(currentContext!).message_has_been_sent_failure,
icon: _imagePaths.icSendToast);
}
popBack();
}
void _listenWorker() { void _listenWorker() {
uploadInlineImageWorker = ever(uploadController.uploadInlineViewState, (state) { uploadInlineImageWorker = ever(uploadController.uploadInlineViewState, (state) {
log('ComposerController::_listenWorker(): $state'); log('ComposerController::_listenWorker(): $state');
@@ -196,7 +196,4 @@ class ContactController extends BaseController {
@override @override
void onDone() {} void onDone() {}
@override
void onError(error) {}
} }
@@ -148,9 +148,6 @@ class DestinationPickerController extends BaseMailboxController {
); );
} }
@override
void onError(error) {}
@override @override
void onClose() { void onClose() {
_disposeWidget(); _disposeWidget();
@@ -262,11 +262,6 @@ class EmailController extends BaseController with AppLoaderMixin {
}); });
} }
@override
void onError(error) {
log('EmailController::onError(): $error');
}
void _getEmailContentSuccess(GetEmailContentSuccess success) { void _getEmailContentSuccess(GetEmailContentSuccess success) {
emailContents.value = success.emailContentsDisplayed; emailContents.value = success.emailContentsDisplayed;
initialEmailContents = success.emailContents; initialEmailContents = success.emailContents;
@@ -59,9 +59,6 @@ class EmailsForwardCreatorController extends BaseController {
@override @override
void onDone() {} void onDone() {}
@override
void onError(error) {}
@override @override
void injectAutoCompleteBindings(Session? session, AccountId? accountId) { void injectAutoCompleteBindings(Session? session, AccountId? accountId) {
try { try {
@@ -132,9 +132,6 @@ class IdentityCreatorController extends BaseController {
); );
} }
@override
void onError(error) {}
void _setUpValueFromIdentity() { void _setUpValueFromIdentity() {
_nameIdentity = identity?.name ?? ''; _nameIdentity = identity?.name ?? '';
inputNameIdentityController?.text = identity?.name ?? ''; inputNameIdentityController?.text = identity?.name ?? '';
@@ -48,7 +48,7 @@ abstract class BaseLoginView extends GetWidget<LoginController> {
} else if (failure is GetTokenOIDCFailure || failure is LoginCanNotGetTokenAction) { } else if (failure is GetTokenOIDCFailure || failure is LoginCanNotGetTokenAction) {
return AppLocalizations.of(context).canNotGetToken; return AppLocalizations.of(context).canNotGetToken;
} else { } else {
return AppLocalizations.of(context).unknown_error_login_message; return AppLocalizations.of(context).unknownError;
} }
}, },
(success) { (success) {
@@ -392,7 +392,4 @@ class LoginController extends ReloadableController {
@override @override
void onDone() {} void onDone() {}
@override
void onError(error) {}
} }
@@ -190,9 +190,6 @@ class MailboxController extends BaseMailboxController {
); );
} }
@override
void onError(error) {}
void _initWorker() { void _initWorker() {
accountIdWorker = ever(mailboxDashBoardController.accountId, (accountId) { accountIdWorker = ever(mailboxDashBoardController.accountId, (accountId) {
if (accountId is AccountId) { if (accountId is AccountId) {
@@ -64,11 +64,7 @@ class MailboxCreatorController extends BaseController {
} }
@override @override
void onDone() { void onDone() {}
}
@override
void onError(error) {}
@override @override
void onClose() { void onClose() {
@@ -370,7 +370,4 @@ class AdvancedFilterController extends BaseController {
@override @override
void onDone() {} void onDone() {}
@override
void onError(error) {}
} }
@@ -278,9 +278,6 @@ class MailboxDashBoardController extends ReloadableController {
); );
} }
@override
void onError(error) {}
void _registerPendingEmailAddress() { void _registerPendingEmailAddress() {
_emailReceiveManagerStreamSubscription = _emailReceiveManagerStreamSubscription =
_emailReceiveManager.pendingEmailAddressInfo.stream.listen((emailAddress) { _emailReceiveManager.pendingEmailAddressInfo.stream.listen((emailAddress) {
@@ -241,9 +241,6 @@ class SearchController extends BaseController {
@override @override
void onDone() {} void onDone() {}
@override
void onError(error) {}
@override @override
void onClose() { void onClose() {
searchInputController.dispose(); searchInputController.dispose();
@@ -69,9 +69,6 @@ class EmailRulesController extends BaseController {
}); });
} }
@override
void onError(error) {}
@override @override
void onInit() { void onInit() {
_getAllRules(); _getAllRules();
@@ -83,9 +83,6 @@ class ForwardController extends BaseController {
}); });
} }
@override
void onError(error) {}
@override @override
void onInit() { void onInit() {
_getForward(); _getForward();
@@ -33,9 +33,6 @@ class LanguageAndRegionController extends BaseController {
}); });
} }
@override
void onError(error) {}
void _setUpSupportedLanguages() { void _setUpSupportedLanguages() {
listSupportedLanguages.value = LocalizationService.supportedLocales; listSupportedLanguages.value = LocalizationService.supportedLocales;
@@ -84,10 +84,6 @@ class ManageAccountDashBoardController extends ReloadableController {
); );
} }
@override
void onError(error) {
}
@override @override
void handleReloaded(Session session) { void handleReloaded(Session session) {
accountId.value = session.accounts.keys.first; accountId.value = session.accounts.keys.first;
@@ -56,12 +56,7 @@ class ManageAccountMenuController extends BaseController {
} }
@override @override
void onDone() { void onDone() {}
}
@override
void onError(error) {
}
void selectAccountMenuItem(AccountMenuItem newAccountMenuItem) { void selectAccountMenuItem(AccountMenuItem newAccountMenuItem) {
dashBoardController.selectAccountMenuItem(newAccountMenuItem); dashBoardController.selectAccountMenuItem(newAccountMenuItem);
@@ -100,10 +100,6 @@ class IdentitiesController extends BaseController {
); );
} }
@override
void onError(error) {
}
void _initWorker() { void _initWorker() {
accountIdWorker = ever(_accountDashBoardController.accountId, (accountId) { accountIdWorker = ever(_accountDashBoardController.accountId, (accountId) {
if (accountId is AccountId) { if (accountId is AccountId) {
@@ -4,10 +4,5 @@ import 'package:tmail_ui_user/features/base/base_controller.dart';
class ProfilesController extends BaseController { class ProfilesController extends BaseController {
@override @override
void onDone() { void onDone() {}
}
@override
void onError(error) {
}
} }
@@ -91,9 +91,6 @@ class VacationController extends BaseController {
); );
} }
@override
void onError(error) {}
void _initWorker() { void _initWorker() {
vacationWorker = ever(_accountDashBoardController.vacationResponse, (vacation) { vacationWorker = ever(_accountDashBoardController.vacationResponse, (vacation) {
if (vacation is VacationResponse) { if (vacation is VacationResponse) {
@@ -42,9 +42,6 @@ class NetworkConnectionController extends BaseController {
@override @override
void onDone() {} void onDone() {}
@override
void onError(error) {}
void setNetworkConnectivityState(ConnectivityResult newConnectivityResult) { void setNetworkConnectivityState(ConnectivityResult newConnectivityResult) {
connectivityResult.value = newConnectivityResult; connectivityResult.value = newConnectivityResult;
} }
@@ -114,9 +114,6 @@ class RulesFilterCreatorController extends BaseMailboxController {
}); });
} }
@override
void onError(error) {}
void _setUpDefaultValueRuleFilter() { void _setUpDefaultValueRuleFilter() {
switch(actionType.value) { switch(actionType.value) {
case CreatorActionType.create: case CreatorActionType.create:
@@ -146,9 +146,6 @@ class SearchEmailController extends BaseController
@override @override
void onDone() {} void onDone() {}
@override
void onError(error) {}
void _initializeDebounceTimeTextSearchChange() { void _initializeDebounceTimeTextSearchChange() {
_deBouncerTime = Debouncer<String>( _deBouncerTime = Debouncer<String>(
const Duration(milliseconds: 500), const Duration(milliseconds: 500),
@@ -79,7 +79,7 @@ class SessionController extends ReloadableController {
if (sessionException.message != null) { if (sessionException.message != null) {
_appToast.showErrorToast('[${sessionException.code}] ${sessionException.message}'); _appToast.showErrorToast('[${sessionException.code}] ${sessionException.message}');
} else { } else {
_appToast.showErrorToast(AppLocalizations.of(currentContext!).unknown_error_login_message); _appToast.showErrorToast(AppLocalizations.of(currentContext!).unknownError);
} }
} }
} }
@@ -111,7 +111,4 @@ class SessionController extends ReloadableController {
@override @override
void onDone() {} void onDone() {}
@override
void onError(error) {}
} }
+3 -3
View File
@@ -1,5 +1,5 @@
{ {
"@@last_modified": "2022-10-27T15:02:38.515955", "@@last_modified": "2022-10-28T13:12:30.889722",
"initializing_data": "Initializing data...", "initializing_data": "Initializing data...",
"@initializing_data": { "@initializing_data": {
"type": "text", "type": "text",
@@ -108,8 +108,8 @@
"placeholders_order": [], "placeholders_order": [],
"placeholders": {} "placeholders": {}
}, },
"unknown_error_login_message": "Unknown error occurred, please try again", "unknownError": "Unknown error occurred, please try again",
"@unknown_error_login_message": { "@unknownError": {
"type": "text", "type": "text",
"placeholders_order": [], "placeholders_order": [],
"placeholders": {} "placeholders": {}
@@ -105,9 +105,9 @@ class AppLocalizations {
return Intl.message('Multiple integrations', name: 'multipleIntegrations'); return Intl.message('Multiple integrations', name: 'multipleIntegrations');
} }
String get unknown_error_login_message { String get unknownError {
return Intl.message('Unknown error occurred, please try again', return Intl.message('Unknown error occurred, please try again',
name: 'unknown_error_login_message'); name: 'unknownError');
} }
String get search_folder { String get search_folder {