Final commit.
This commit is contained in:
Dennis Nemec
@ -1,11 +1,10 @@
|
||||
import 'package:flutter/material.dart';
|
||||
import 'package:flutter_bloc/flutter_bloc.dart';
|
||||
import 'package:hl_lieferservice/feature/authentication/bloc/auth_bloc.dart';
|
||||
import 'package:hl_lieferservice/feature/authentication/bloc/auth_state.dart';
|
||||
import 'package:hl_lieferservice/feature/car_selection/bloc/bloc.dart';
|
||||
import 'package:hl_lieferservice/feature/car_selection/bloc/state.dart';
|
||||
import 'package:hl_lieferservice/feature/car_selection/presentation/selected_car_bar.dart';
|
||||
import 'package:hl_lieferservice/feature/cars/presentation/car_management_page.dart';
|
||||
import 'package:hl_lieferservice/feature/cars/bloc/cars_bloc.dart';
|
||||
import 'package:hl_lieferservice/feature/cars/bloc/cars_event.dart';
|
||||
import 'package:hl_lieferservice/feature/cars/bloc/cars_state.dart';
|
||||
import 'package:hl_lieferservice/feature/delivery/bloc/phase_bloc.dart';
|
||||
import 'package:hl_lieferservice/feature/delivery/bloc/phase_event.dart';
|
||||
import 'package:hl_lieferservice/feature/delivery/bloc/phase_state.dart';
|
||||
@ -17,10 +16,6 @@ import 'package:hl_lieferservice/feature/delivery/overview/presentation/delivery
|
||||
import 'package:hl_lieferservice/feature/delivery/overview/presentation/delivery_selection_page.dart';
|
||||
import 'package:hl_lieferservice/feature/delivery/overview/presentation/delivery_sort_page.dart';
|
||||
import 'package:hl_lieferservice/feature/loading/presentation/loading_overview_page.dart';
|
||||
import 'package:hl_lieferservice/feature/settings/presentation/settings_page.dart';
|
||||
import 'package:hl_lieferservice/widget/home/bloc/navigation_bloc.dart';
|
||||
import 'package:hl_lieferservice/widget/home/bloc/navigation_state.dart';
|
||||
import 'package:hl_lieferservice/widget/navigation_bar/presentation/navigation_bar.dart';
|
||||
|
||||
/// Wurzel-Widget des authentifizierten Bereichs. Routet anhand der aktuellen
|
||||
/// Phase des ausgewählten Fahrzeugs:
|
||||
@ -40,28 +35,75 @@ class Home extends StatefulWidget {
|
||||
class _HomeState extends State<Home> {
|
||||
String? _initializedCarId;
|
||||
|
||||
/// Merkt, für welches Auto die automatische Lieferungs-Zuordnung bereits
|
||||
/// erledigt ist (Ein-Auto-Teams, s. [_ensureSingleCarAssignment]).
|
||||
String? _autoAssignedForCar;
|
||||
|
||||
@override
|
||||
void initState() {
|
||||
super.initState();
|
||||
|
||||
// Tour beim ersten Aufbau laden.
|
||||
final authState = context.read<AuthBloc>().state as Authenticated;
|
||||
context.read<TourBloc>().add(LoadTour(teamId: authState.user.number));
|
||||
// Tour beim ersten Aufbau laden. Account-Filter sitzt jetzt
|
||||
// serverseitig im JWT — kein Personalnummer-Argument mehr nötig.
|
||||
context.read<TourBloc>().add(const LoadTour());
|
||||
|
||||
// CarsBloc auch hier triggern: wenn der CarSelectBloc beim App-Start
|
||||
// eine valide Tages-Auswahl aus den SharedPreferences fand, wurde die
|
||||
// CarSelectionPage übersprungen — und damit auch ihr `CarLoad`-Trigger.
|
||||
// Ohne diesen Aufruf hängen drei Stellen am leeren CarsBloc-State:
|
||||
// `PhaseStepper.carCount`, `app.carCountResolver` (Eintrittsphase) und
|
||||
// `delivery_selection_page._plateFor` (Vergeben-Tab zeigt sonst "?").
|
||||
// Der Bloc-interne `if (state is CarsLoaded && !event.force) return;`
|
||||
// macht den Aufruf idempotent.
|
||||
context.read<CarsBloc>().add(const CarLoad());
|
||||
}
|
||||
|
||||
/// Stellt sicher, dass für das aktuell gewählte Auto eine Phase im
|
||||
/// [PhaseBloc] existiert. Wird im build() reaktiv aufgerufen, daher mit
|
||||
/// `_initializedCarId` gegen mehrfache Loads gesichert.
|
||||
///
|
||||
/// Wichtig: Wir feuern den Load erst, sobald die Tour geladen ist —
|
||||
/// sonst kennt der PhaseBloc die Anzahl der Team-Fahrzeuge nicht und
|
||||
/// würde fälschlich mit `sortieren` einsteigen, statt mit `auswaehlen`.
|
||||
/// Wichtig: Wir feuern den Load erst, sobald sowohl Tour als auch Cars
|
||||
/// einen geladen-Zustand haben — sonst fragt der `PhaseBloc` den
|
||||
/// `carCountResolver` ab, bekommt `null` und entscheidet für Mehr-Auto-
|
||||
/// Teams fälschlich auf `sortieren` statt `auswaehlen`.
|
||||
void _ensurePhaseLoaded(String carId) {
|
||||
if (_initializedCarId == carId) return;
|
||||
final carsState = context.read<CarsBloc>().state;
|
||||
if (carsState is! CarsLoaded) return;
|
||||
_initializedCarId = carId;
|
||||
context.read<PhaseBloc>().add(PhaseLoadForCar(carId: carId));
|
||||
}
|
||||
|
||||
/// Ein-Auto-Teams überspringen die „Auswählen"-Phase (s. [PhaseBloc]),
|
||||
/// in der Lieferungen normalerweise einem Fahrzeug zugeordnet werden.
|
||||
/// Ohne Zuordnung (`assignedCarId`) blieben sie in „Ausliefern" unsichtbar.
|
||||
/// Daher hier einmalig: ist genau EIN Fahrzeug im Team, werden alle noch
|
||||
/// nicht zugeordneten Lieferungen automatisch diesem Fahrzeug zugewiesen.
|
||||
///
|
||||
/// Reaktiv (im build) aufgerufen, aber per [_autoAssignedForCar] gegen
|
||||
/// Mehrfachausführung gesichert. Greift erst, wenn Tour UND Cars geladen
|
||||
/// sind (sonst Abbruch ohne Flag → erneuter Versuch beim nächsten Build).
|
||||
void _ensureSingleCarAssignment(String carId) {
|
||||
if (_autoAssignedForCar == carId) return;
|
||||
final carsState = context.read<CarsBloc>().state;
|
||||
final tourState = context.read<TourBloc>().state;
|
||||
if (carsState is! CarsLoaded || tourState is! TourLoaded) return;
|
||||
|
||||
_autoAssignedForCar = carId;
|
||||
// Nur Ein-Auto-Teams: bei ≥2 Fahrzeugen entscheidet der Fahrer aktiv in
|
||||
// der „Auswählen"-Phase, hier wird bewusst nichts automatisch zugewiesen.
|
||||
if (carsState.cars.length != 1) return;
|
||||
|
||||
final tourBloc = context.read<TourBloc>();
|
||||
for (final delivery in tourState.details.deliveries) {
|
||||
if (delivery.assignedCarId != carId) {
|
||||
tourBloc.add(
|
||||
AssignCarToDelivery(deliveryId: delivery.id, carId: carId),
|
||||
);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@override
|
||||
Widget build(BuildContext context) {
|
||||
return BlocBuilder<CarSelectBloc, CarSelectState>(
|
||||
@ -74,32 +116,49 @@ class _HomeState extends State<Home> {
|
||||
|
||||
final carId = carState.selectedCar.id.toString();
|
||||
|
||||
return BlocBuilder<TourBloc, TourState>(
|
||||
// Tour-Status mitnehmen, weil die Eintrittsphase davon abhängt.
|
||||
// Nur bei TourLoaded triggern wir den Phasen-Load.
|
||||
builder: (context, tourState) {
|
||||
if (tourState is TourLoaded) {
|
||||
_ensurePhaseLoaded(carId);
|
||||
}
|
||||
|
||||
return BlocBuilder<PhaseBloc, PhaseState>(
|
||||
builder: (context, phaseState) {
|
||||
final phase = phaseState is PhaseReady
|
||||
? phaseState.phaseFor(carId)
|
||||
: null;
|
||||
|
||||
// Solange weder Tour noch Phase geladen sind, kurzen Spinner
|
||||
// zeigen — das dauert in der Praxis maximal einen Frame.
|
||||
if (phase == null) {
|
||||
return const Scaffold(
|
||||
body: Center(child: CircularProgressIndicator()),
|
||||
);
|
||||
}
|
||||
|
||||
return _buildForPhase(context, phase, carState.selectedCar.id);
|
||||
},
|
||||
);
|
||||
// Nachzieh-Trigger: wenn Cars erst NACH dem Tour-Build fertig
|
||||
// werden, würde `_ensurePhaseLoaded` beim Tour-Build noch
|
||||
// skippen. Dieser Listener feuert sobald `CarsLoaded` da ist.
|
||||
return BlocListener<CarsBloc, CarsState>(
|
||||
listenWhen: (prev, curr) =>
|
||||
prev is! CarsLoaded && curr is CarsLoaded,
|
||||
listener: (context, _) {
|
||||
_ensurePhaseLoaded(carId);
|
||||
_ensureSingleCarAssignment(carId);
|
||||
},
|
||||
child: BlocBuilder<TourBloc, TourState>(
|
||||
builder: (context, tourState) {
|
||||
if (tourState is TourLoaded || tourState is TourEmpty) {
|
||||
_ensurePhaseLoaded(carId);
|
||||
}
|
||||
if (tourState is TourLoaded) {
|
||||
_ensureSingleCarAssignment(carId);
|
||||
}
|
||||
|
||||
return BlocBuilder<PhaseBloc, PhaseState>(
|
||||
builder: (context, phaseState) {
|
||||
final phase = phaseState is PhaseReady
|
||||
? phaseState.phaseFor(carId)
|
||||
: null;
|
||||
|
||||
// Solange Tour, Cars oder Phase noch laden, kurzen
|
||||
// Spinner zeigen — das dauert in der Praxis maximal
|
||||
// ein paar Frames.
|
||||
if (phase == null) {
|
||||
return const Scaffold(
|
||||
body: Center(child: CircularProgressIndicator()),
|
||||
);
|
||||
}
|
||||
|
||||
return _buildForPhase(
|
||||
context,
|
||||
phase,
|
||||
carState.selectedCar.id,
|
||||
);
|
||||
},
|
||||
);
|
||||
},
|
||||
),
|
||||
);
|
||||
},
|
||||
);
|
||||
@ -129,46 +188,15 @@ class _HomeState extends State<Home> {
|
||||
}
|
||||
}
|
||||
|
||||
/// Klassisches Home für die Auslieferungs-Phase: BottomNavigationBar mit
|
||||
/// drei Tabs (Auslieferung / Fahrzeuge / Einstellungen). Die Beladung als
|
||||
/// Tab entfällt bewusst — wer in dieser Phase zurück zur Beladung möchte,
|
||||
/// nutzt den Phasen-Stepper auf den jeweiligen Pages oder den Drawer.
|
||||
/// Home für die Auslieferungs-Phase. Reine Hülle um die Übersicht — es gibt
|
||||
/// keine BottomNavigationBar mehr: Fahrzeug-Verwaltung und Einstellungen
|
||||
/// sind über den Drawer erreichbar, der Fahrzeug-Wechsel direkt aus dem
|
||||
/// PhaseStepper (Icon neben dem Plate).
|
||||
class _DeliveryHome extends StatelessWidget {
|
||||
const _DeliveryHome();
|
||||
|
||||
Widget _buildPage(int index) {
|
||||
switch (index) {
|
||||
case 0:
|
||||
return const DeliveryOverviewPage();
|
||||
case 1:
|
||||
return const CarManagementPage();
|
||||
case 2:
|
||||
return const SettingsPage();
|
||||
default:
|
||||
return const SizedBox.shrink();
|
||||
}
|
||||
}
|
||||
|
||||
@override
|
||||
Widget build(BuildContext context) {
|
||||
return BlocBuilder<NavigationBloc, NavigationState>(
|
||||
builder: (context, state) {
|
||||
final navIndex = state is NavigationInfo ? state.navigationIndex : 0;
|
||||
// Bei einem Tab-Index, der außerhalb des neuen Bereichs liegt
|
||||
// (z. B. vom alten 4-Tab-Layout: 0..3), normieren wir defensiv auf 0.
|
||||
final safeIndex = (navIndex >= 0 && navIndex <= 2) ? navIndex : 0;
|
||||
|
||||
return Scaffold(
|
||||
body: _buildPage(safeIndex),
|
||||
bottomNavigationBar: Column(
|
||||
mainAxisSize: MainAxisSize.min,
|
||||
children: const [
|
||||
SelectedCarBar(),
|
||||
AppNavigationBar(),
|
||||
],
|
||||
),
|
||||
);
|
||||
},
|
||||
);
|
||||
return const DeliveryOverviewPage();
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user