diff --git a/lib/data/models/account_model.dart b/lib/data/models/account_model.dart index 64c527ea2..5977db30b 100644 --- a/lib/data/models/account_model.dart +++ b/lib/data/models/account_model.dart @@ -138,6 +138,7 @@ abstract class AccountEntity // ignore: unused_element static void _initializeBuilder(AccountEntityBuilder builder) => builder ..key = '' + ..currentVersion = '' ..debugEnabled = false ..isDocker = false ..isSchedulerRunning = true diff --git a/lib/main.dart b/lib/main.dart index eb900f32e..0585dcfd3 100644 --- a/lib/main.dart +++ b/lib/main.dart @@ -185,7 +185,7 @@ void main({bool isTesting = false}) async { options.beforeSend = (SentryEvent event, {dynamic hint}) { final state = store.state; final account = state.account; - final reportErrors = account.reportErrors ?? false; + final reportErrors = account.reportErrors; if (!reportErrors) { return null; @@ -194,7 +194,7 @@ void main({bool isTesting = false}) async { event = event.copyWith( environment: '${store.state.environment}'.split('.').last, extra: { - 'server_version': account.currentVersion ?? 'Unknown', + 'server_version': account.currentVersion, 'route': state.uiState.currentRoute, }, ); diff --git a/lib/redux/app/app_state.dart b/lib/redux/app/app_state.dart index e0b6ecd93..0ecc8cd7d 100644 --- a/lib/redux/app/app_state.dart +++ b/lib/redux/app/app_state.dart @@ -190,8 +190,7 @@ abstract class AppState implements Built { return color.isNotEmpty; } - bool get showReviewApp => - !prefState.hideReviewApp && company.daysActive > 60; + bool get showReviewApp => !prefState.hideReviewApp && company.daysActive > 60; bool get showOneYearReviewApp => !prefState.hideOneYearReviewApp && company.daysActive > 365; @@ -222,7 +221,7 @@ abstract class AppState implements Built { String get appVersion { String version = 'v'; - version += account.currentVersion ?? ''; + version += account.currentVersion; if (version.isNotEmpty) { version += '-'; @@ -826,7 +825,7 @@ abstract class AppState implements Built { } } - bool get reportErrors => account.reportErrors ?? false; + bool get reportErrors => account.reportErrors; bool get isHosted => account == null ? authState.isHosted : account.isHosted; @@ -1012,7 +1011,7 @@ abstract class AppState implements Built { '\nIs Loading: ${isLoading ? 'Yes' : 'No'}' '\nIs Saving: ${isSaving ? 'Yes' : 'No'}' '\nIs Loaded: ${isLoaded ? 'Yes' : 'No'}' - '\nis Large: ${(company.isLarge ?? false) ? 'Yes' : 'No'}' + '\nis Large: ${(company.isLarge) ? 'Yes' : 'No'}' '\nCompany: $companyUpdated${userCompanyState.isStale ? ' [S]' : ''}' '\nStatic: $staticUpdated${staticState.isStale ? ' [S]' : ''}' '\nPassword: $passwordUpdated${hasRecentlyEnteredPassword ? '' : ' [S]'}' diff --git a/lib/ui/client/client_screen.dart b/lib/ui/client/client_screen.dart index a778dc0b6..446c6031a 100644 --- a/lib/ui/client/client_screen.dart +++ b/lib/ui/client/client_screen.dart @@ -67,7 +67,7 @@ class ClientScreen extends StatelessWidget { }, sortFields: [ ClientFields.name, - ClientFields.idNumber, + ClientFields.number, ClientFields.balance, ClientFields.updatedAt, ], diff --git a/lib/ui/settings/client_portal.dart b/lib/ui/settings/client_portal.dart index 2304ce311..a974056dd 100644 --- a/lib/ui/settings/client_portal.dart +++ b/lib/ui/settings/client_portal.dart @@ -533,7 +533,7 @@ class _ClientPortalState extends State onChanged: (value) => viewModel.onCompanyChanged( company.rebuild((b) => b..clientCanRegister = value)), ), - if (state.company.clientCanRegister ?? false) ...[ + if (state.company.clientCanRegister) ...[ SizedBox(height: 16), ListTile( title: Text(localization.registrationUrl), diff --git a/lib/ui/settings/expense_settings.dart b/lib/ui/settings/expense_settings.dart index fafaa1bd8..01934f2df 100644 --- a/lib/ui/settings/expense_settings.dart +++ b/lib/ui/settings/expense_settings.dart @@ -62,7 +62,7 @@ class _ExpenseSettingsState extends State { SwitchListTile( activeColor: Theme.of(context).colorScheme.secondary, title: Text(localization.shouldBeInvoiced), - value: company.markExpensesInvoiceable ?? false, + value: company.markExpensesInvoiceable, subtitle: Text(localization.shouldBeInvoicedHelp), onChanged: (value) => viewModel.onCompanyChanged( company.rebuild((b) => b..markExpensesInvoiceable = value)), @@ -70,7 +70,7 @@ class _ExpenseSettingsState extends State { SwitchListTile( activeColor: Theme.of(context).colorScheme.secondary, title: Text(localization.markPaid), - value: company.markExpensesPaid ?? false, + value: company.markExpensesPaid, subtitle: Text(localization.markPaidHelp), onChanged: (value) => viewModel.onCompanyChanged( company.rebuild((b) => b..markExpensesPaid = value)), @@ -92,7 +92,7 @@ class _ExpenseSettingsState extends State { SwitchListTile( activeColor: Theme.of(context).colorScheme.secondary, title: Text(localization.convertCurrency), - value: company.convertExpenseCurrency ?? false, + value: company.convertExpenseCurrency, subtitle: Text(localization.convertExpenseCurrencyHelp!), onChanged: (value) => viewModel.onCompanyChanged( company.rebuild((b) => b..convertExpenseCurrency = value)), @@ -100,7 +100,7 @@ class _ExpenseSettingsState extends State { SwitchListTile( activeColor: Theme.of(context).colorScheme.secondary, title: Text(localization.addDocumentsToInvoice), - value: company.invoiceExpenseDocuments ?? false, + value: company.invoiceExpenseDocuments, subtitle: Text(localization.addDocumentsToInvoiceHelp), onChanged: (value) => viewModel.onCompanyChanged( company.rebuild((b) => b..invoiceExpenseDocuments = value)), @@ -111,7 +111,7 @@ class _ExpenseSettingsState extends State { SwitchListTile( activeColor: Theme.of(context).colorScheme.secondary, title: Text(localization.notifyVendorWhenPaid!), - value: company.notifyVendorWhenPaid ?? false, + value: company.notifyVendorWhenPaid, subtitle: Text(localization.notifyVendorWhenPaidHelp!), onChanged: (value) => viewModel.onCompanyChanged( company.rebuild((b) => b..notifyVendorWhenPaid = value)), @@ -124,7 +124,7 @@ class _ExpenseSettingsState extends State { label: localization.enterTaxes, enabledLabel: localization.byAmount, disabledLabel: localization.byRate, - value: company.calculateExpenseTaxByAmount ?? false, + value: company.calculateExpenseTaxByAmount, onChanged: (value) => viewModel.onCompanyChanged(company .rebuild((b) => b..calculateExpenseTaxByAmount = value)), ), @@ -132,7 +132,7 @@ class _ExpenseSettingsState extends State { SwitchListTile( activeColor: Theme.of(context).colorScheme.secondary, title: Text(localization.inclusiveTaxes), - value: company.expenseInclusiveTaxes ?? false, + value: company.expenseInclusiveTaxes, subtitle: Text( '\n${localization.exclusive}: 100 + 10% = 100 + 10\n${localization.inclusive}: 100 + 10% = 90.91 + 9.09'), onChanged: (value) => viewModel.onCompanyChanged( diff --git a/lib/ui/settings/invoice_design.dart b/lib/ui/settings/invoice_design.dart index 94b74777a..2e9924d70 100644 --- a/lib/ui/settings/invoice_design.dart +++ b/lib/ui/settings/invoice_design.dart @@ -1289,8 +1289,8 @@ class _PdfPreviewState extends State<_PdfPreview> { entityType: widget.entityType.apiValue, settingsType: settingsUIState.entityType.apiValue, settings: widget.settings!, - groupId: settingsUIState.group.id ?? '', - clientId: settingsUIState.client.id ?? '', + groupId: settingsUIState.group.id, + clientId: settingsUIState.client.id, ); setState(() => isLoading = true);