Bills were persisted correctly but read back wrong. The read path rebuilt a cart from its lines alone, dropping bill-level discounts and loyalty, so every figure derived from a stored bill was overstated: the upload payload, the day archive and the shift report. A discounted 529 bill read back as 620. Money and data integrity - order_dao: restore bill_discount and points_redeemed when rebuilding a cart; keep the reconstruction tier-less so the membership discount is not applied twice. Trust the recorded total and points via SaleTransaction.storedTotal. - checkout_sale + order_dao.commitSale: write the bill, its stock movement and the loyalty update in one transaction. Previously a failure part-way through left a persisted bill the cashier believed had failed, inviting a duplicate. - checkout_sale: re-check every line against live stock. A parked bill resumed after its stock was sold passed validation and oversold. - catalogue_dao: allocate the invoice sequence in one transaction; the previous read-modify-write could hand two sales the same number and fail UNIQUE. - local_store: replay unsynced sales after a catalogue import, so a mid-shift re-import cannot restore stock that has already been sold. - payment_controller: stamp the signed-in operator on the bill instead of the hardcoded seed session, and pass the terminal id through. - cart: reconcile per-slab GST against the bill total so the parts sum to the whole on a tax invoice. Sync and reporting - sync_repository: drain unsynced bills in a loop rather than silently capping at one page; stop on rejection so rejected rows cannot loop forever. - sync_log_dao (new): persist the sync history to the sync_log table, which the schema already defined but nothing used. It was in memory, so the only record that bills had been uploaded died at restart. - Scope shift reports by cashier. day_archive is re-keyed to (business_date, cashier_name) so a till stays settleable after its bills are uploaded and deleted. Schema v4 with a migration that carries v3 rows across. Input and UI - barcode_service: consume machine-paced keystrokes so a scan cannot also land in the focused field, and raise the bar to 60ms/char while a text field has focus so typing a mobile number is not read as a scan. Clock and focus check injected so the behaviour is testable. - primary_button: make the label flexible; label plus trailing total overflowed the Charge button by up to 131px. - app_router: redirect instead of null-casting when the receipt route is entered without its transaction. - customer_repository: reduce the search query to digits so a punctuated mobile number matches. Cleanup - Remove TransactionRepository.save, CustomerRepository.recordSale and OrderDao.insertOrder, all superseded by commitSale. - dart fix across the tree; 251 analyzer issues down to 3 info-level. Tests: 23 passing / 15 failing -> 90 passing. Fixed the two defects that broke the existing suite (containsAll type argument, reset() needing a catalogue) and deleted the leftover template test. Added coverage for the order round trip, the day archive after a real sync, stock safety, checkout atomicity, the v3->v4 migration, scanner-versus-human input, and an app-level smoke test that renders every module. Note: bills already uploaded with a discount went up overstated. This stops it happening again but does not correct historical server data. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
210 lines
7.8 KiB
Dart
210 lines
7.8 KiB
Dart
import 'package:flutter/material.dart';
|
|
import 'package:flutter_riverpod/flutter_riverpod.dart';
|
|
|
|
import '../../../app/providers.dart';
|
|
import '../../../core/theme/app_colors.dart';
|
|
import '../../../core/theme/app_dimens.dart';
|
|
import '../../../core/utils/formatters.dart';
|
|
import '../../../core/widgets/status_pill.dart';
|
|
import '../../../domain/entities/customer.dart';
|
|
import '../widgets/module_widgets.dart';
|
|
|
|
/// Full customer book, independent of the six shown during billing.
|
|
final allCustomersProvider = FutureProvider<List<Customer>>(
|
|
(ref) => ref.watch(customerRepositoryProvider).recent(limit: 200),
|
|
);
|
|
|
|
class CustomersView extends ConsumerStatefulWidget {
|
|
const CustomersView({super.key});
|
|
|
|
@override
|
|
ConsumerState<CustomersView> createState() => _CustomersViewState();
|
|
}
|
|
|
|
class _CustomersViewState extends ConsumerState<CustomersView> {
|
|
String _query = '';
|
|
MembershipTier? _tier;
|
|
|
|
Color _tierColor(MembershipTier t) => switch (t) {
|
|
MembershipTier.bronze => AppColors.tierBronze,
|
|
MembershipTier.silver => AppColors.tierSilver,
|
|
MembershipTier.gold => AppColors.tierGold,
|
|
MembershipTier.platinum => AppColors.tierPlatinum,
|
|
};
|
|
|
|
@override
|
|
Widget build(BuildContext context) {
|
|
final all = ref.watch(allCustomersProvider).value ?? const <Customer>[];
|
|
|
|
final filtered = all.where((c) {
|
|
final q = _query.trim().toLowerCase();
|
|
final matchesQuery = q.isEmpty ||
|
|
c.name.toLowerCase().contains(q) ||
|
|
c.mobile.contains(q);
|
|
return matchesQuery && (_tier == null || c.tier == _tier);
|
|
}).toList();
|
|
|
|
final lifetime = all.fold<double>(0, (s, c) => s + c.lifetimeSpend);
|
|
final points = all.fold<int>(0, (s, c) => s + c.loyaltyPoints);
|
|
|
|
return ModulePage(
|
|
children: [
|
|
Wrap(
|
|
spacing: AppSpacing.lg,
|
|
runSpacing: AppSpacing.lg,
|
|
children: [
|
|
StatTile(
|
|
label: 'Total Customers',
|
|
value: '${all.length}',
|
|
icon: Icons.people_alt_rounded,
|
|
caption: 'registered',
|
|
),
|
|
StatTile(
|
|
label: 'Lifetime Value',
|
|
value: Formatters.moneyCompact(lifetime),
|
|
icon: Icons.payments_rounded,
|
|
color: AppColors.success,
|
|
caption: 'all customers',
|
|
),
|
|
StatTile(
|
|
label: 'Points Outstanding',
|
|
value: '$points',
|
|
icon: Icons.stars_rounded,
|
|
color: AppColors.tierGold,
|
|
caption: 'worth ${Formatters.money(points * 0.25)}',
|
|
),
|
|
StatTile(
|
|
label: 'Avg Spend',
|
|
value: Formatters.money(all.isEmpty ? 0 : lifetime / all.length),
|
|
icon: Icons.trending_up_rounded,
|
|
color: AppColors.info,
|
|
caption: 'per customer',
|
|
),
|
|
],
|
|
),
|
|
const SizedBox(height: AppSpacing.lg),
|
|
|
|
PanelCard(
|
|
title: 'Tier distribution',
|
|
child: Column(
|
|
mainAxisSize: MainAxisSize.min,
|
|
children: [
|
|
for (final t in MembershipTier.values)
|
|
ProgressRow(
|
|
label: '${t.label} · '
|
|
'${(t.discountRate * 100).toStringAsFixed(0)}% off',
|
|
value: '${all.where((c) => c.tier == t).length}',
|
|
fraction: all.isEmpty
|
|
? 0
|
|
: all.where((c) => c.tier == t).length / all.length,
|
|
color: _tierColor(t),
|
|
),
|
|
],
|
|
),
|
|
),
|
|
const SizedBox(height: AppSpacing.lg),
|
|
|
|
PanelCard(
|
|
title: 'Customer book',
|
|
subtitle: '${filtered.length} shown',
|
|
action: FilledButton.icon(
|
|
onPressed: () {},
|
|
icon: const Icon(Icons.person_add_alt_1_rounded, size: 17),
|
|
label: const Text('Add customer'),
|
|
style: FilledButton.styleFrom(
|
|
backgroundColor: AppColors.primary,
|
|
minimumSize: const Size(0, 40),
|
|
),
|
|
),
|
|
child: Column(
|
|
crossAxisAlignment: CrossAxisAlignment.stretch,
|
|
children: [
|
|
TextField(
|
|
onChanged: (v) => setState(() => _query = v),
|
|
decoration: const InputDecoration(
|
|
hintText: 'Search by name or mobile number…',
|
|
prefixIcon: Icon(Icons.search_rounded),
|
|
isDense: true,
|
|
),
|
|
),
|
|
const SizedBox(height: AppSpacing.md),
|
|
Wrap(
|
|
spacing: AppSpacing.sm,
|
|
runSpacing: AppSpacing.sm,
|
|
children: [
|
|
ChoiceChip(
|
|
label: const Text('All tiers'),
|
|
selected: _tier == null,
|
|
onSelected: (_) => setState(() => _tier = null),
|
|
labelStyle: TextStyle(
|
|
fontSize: 12.5,
|
|
fontWeight: FontWeight.w600,
|
|
color: _tier == null
|
|
? Colors.white
|
|
: AppColors.textSecondary,
|
|
),
|
|
),
|
|
for (final t in MembershipTier.values)
|
|
ChoiceChip(
|
|
label: Text(t.label),
|
|
selected: _tier == t,
|
|
onSelected: (_) =>
|
|
setState(() => _tier = _tier == t ? null : t),
|
|
labelStyle: TextStyle(
|
|
fontSize: 12.5,
|
|
fontWeight: FontWeight.w600,
|
|
color: _tier == t
|
|
? Colors.white
|
|
: AppColors.textSecondary,
|
|
),
|
|
),
|
|
],
|
|
),
|
|
const SizedBox(height: AppSpacing.lg),
|
|
ResponsiveTable(
|
|
columns: const [
|
|
TableCol('Customer', flex: 4),
|
|
TableCol('Mobile', flex: 3, priority: 1),
|
|
TableCol('Tier', flex: 2),
|
|
TableCol('Points', flex: 2, numeric: true, priority: 1),
|
|
TableCol('Lifetime', flex: 2, numeric: true),
|
|
TableCol('Visits', flex: 2, numeric: true, priority: 1),
|
|
],
|
|
rows: filtered
|
|
.map((c) => [
|
|
Row(
|
|
mainAxisSize: MainAxisSize.min,
|
|
children: [
|
|
CircleAvatar(
|
|
radius: 14,
|
|
backgroundColor: AppColors.primarySurface,
|
|
child: Text(
|
|
Formatters.initials(c.name),
|
|
style: const TextStyle(
|
|
fontSize: 10.5,
|
|
fontWeight: FontWeight.w700,
|
|
color: AppColors.primary,
|
|
),
|
|
),
|
|
),
|
|
const SizedBox(width: AppSpacing.sm),
|
|
Flexible(child: Cell(c.name, bold: true)),
|
|
],
|
|
),
|
|
Cell(Formatters.mobile(c.mobile), mono: true),
|
|
StatusPill.tier(c.tier, dense: true),
|
|
Cell('${c.loyaltyPoints}', mono: true),
|
|
Cell(Formatters.moneyCompact(c.lifetimeSpend),
|
|
mono: true, bold: true,),
|
|
Cell('${c.visitCount}', mono: true),
|
|
],)
|
|
.toList(),
|
|
),
|
|
],
|
|
),
|
|
),
|
|
],
|
|
);
|
|
}
|
|
}
|