feat(flags): Feature-Flags aus Config-Datei; Gutschriften-Abschnitt abschaltbar

- Schalter stehen in assets/feature_flags.json (Key -> enabled +
  Beschreibung) statt als Konstanten im Code
- Enum Feature buendelt die stabilen Keys; FeatureFlags liest die Datei,
  ignoriert unbekannte Keys und nutzt fuer fehlende den Default aus dem Enum
- Laden beim App-Start im AppBloc, bereitgestellt per RepositoryProvider;
  FeatureGate blendet Bereiche bei inaktivem Feature komplett aus
- articles.credit_section = false: Abschnitt "Gutschriften" im
  Artikel-Step ist ausgeblendet
- Bisheriges Stepper-Flag migriert (articles.credit_amount_stepper); die
  fuenf ungenutzten statischen Flags mit veralteten Kommentaren entfernt
- Tests: Asset enthaelt alle Keys, Parsing, Defaults, FeatureGate

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
Dennis Nemec
2026-09-25 15:07:40 +02:00
parent 6b934e4534
commit 200992262a
15 changed files with 307 additions and 87 deletions

View File

@ -5,6 +5,7 @@ import 'package:hl_lieferservice/bloc/app_states.dart';
import 'package:hl_lieferservice/data/network/backend_config.dart';
import 'package:hl_lieferservice/data/network/backend_environment.dart';
import 'package:hl_lieferservice/data/network/environment_repository.dart';
import 'package:hl_lieferservice/feature/feature_flags/feature_flags_repository.dart';
/// Aktiviert das Networking für eine Umgebung (Registrierung im Locator).
typedef EnvironmentActivator = void Function(BackendConfig config);
@ -13,14 +14,18 @@ typedef EnvironmentActivator = void Function(BackendConfig config);
///
/// Liest die Server-Umgebungen aus `assets/hl_server_config.json`,
/// bestimmt die aktive Umgebung (gespeicherte Wahl, sonst Default) und
/// registriert dafür das Networking. Ein Umgebungswechsel läuft über
/// denselben Weg; die UI baut ihren Bloc-Baum pro Umgebung neu auf.
/// registriert dafür das Networking. Außerdem lädt er die Feature-Schalter
/// aus `assets/feature_flags.json`. Ein Umgebungswechsel läuft über denselben
/// Weg; die UI baut ihren Bloc-Baum pro Umgebung neu auf.
class AppBloc extends Bloc<AppEvents, AppState> {
AppBloc({
required EnvironmentRepository repository,
required EnvironmentActivator activate,
FeatureFlagsRepository featureFlagsRepository =
const FeatureFlagsRepository(),
}) : _repository = repository,
_activate = activate,
_featureFlagsRepository = featureFlagsRepository,
super(const AppInitial()) {
on<AppLoadConfig>(_onLoad);
on<AppSwitchEnvironment>(_onSwitch);
@ -28,20 +33,26 @@ class AppBloc extends Bloc<AppEvents, AppState> {
final EnvironmentRepository _repository;
final EnvironmentActivator _activate;
final FeatureFlagsRepository _featureFlagsRepository;
Future<void> _onLoad(AppLoadConfig event, Emitter<AppState> emit) async {
emit(const AppConfigLoading());
try {
final catalog = await _repository.loadCatalog('assets/${event.path}');
final featureFlags = await _featureFlagsRepository.load();
final storedId = await _repository.loadSelectedId();
// Unbekannte ID (Umgebung aus der Config entfernt) → Default.
final active = catalog.byId(storedId) ?? catalog.defaultEnvironment;
_activate(catalog.configFor(active));
emit(AppConfigLoaded(catalog: catalog, active: active));
emit(AppConfigLoaded(
catalog: catalog,
active: active,
featureFlags: featureFlags,
));
} catch (e, st) {
debugPrint('AppBloc: Server-Config nicht ladbar: $e\n$st');
debugPrint('AppBloc: Konfiguration nicht ladbar: $e\n$st');
emit(AppConfigLoadingFailed(
message: 'Server-Konfiguration konnte nicht geladen werden.\n$e',
message: 'Konfiguration konnte nicht geladen werden.\n$e',
));
}
}
@ -59,6 +70,10 @@ class AppBloc extends Bloc<AppEvents, AppState> {
await _repository.saveSelectedId(next.id);
_activate(current.catalog.configFor(next));
emit(AppConfigLoaded(catalog: current.catalog, active: next));
emit(AppConfigLoaded(
catalog: current.catalog,
active: next,
featureFlags: current.featureFlags,
));
}
}

View File

@ -1,9 +1,11 @@
import 'package:hl_lieferservice/data/network/backend_environment.dart';
import 'package:hl_lieferservice/feature/feature_flags/feature_flags.dart';
/// Lifecycle-States des App-Bootstraps.
///
/// `AppConfigLoaded` heißt: Server-Config ist gelesen, die aktive Umgebung
/// steht fest und das Networking dafür ist im Locator registriert.
/// steht fest, das Networking dafür ist im Locator registriert und die
/// Feature-Schalter sind geladen.
abstract class AppState {
const AppState();
}
@ -17,13 +19,20 @@ class AppConfigLoading extends AppState {
}
class AppConfigLoaded extends AppState {
const AppConfigLoaded({required this.catalog, required this.active});
const AppConfigLoaded({
required this.catalog,
required this.active,
required this.featureFlags,
});
/// Alle auswählbaren Umgebungen aus der Config-Datei.
final ServerCatalog catalog;
/// Die Umgebung, gegen die die App gerade spricht.
final BackendEnvironment active;
/// Feature-Schalter aus `assets/feature_flags.json`.
final FeatureFlags featureFlags;
}
class AppConfigLoadingFailed extends AppState {

View File

@ -11,10 +11,9 @@ enum DeliveryState { active, held, canceled, completed }
/// Eine einzelne Auslieferung an einen Kunden innerhalb einer Tour.
///
/// Anders als im alten Modell trägt `Delivery` hier ausschließlich
/// Logistik-Daten — keine Preise, keine Rabatte, keine Zahlungsoptionen.
/// Diese ERP-Themen sind in Phase C+D-2 absichtlich nicht migriert und
/// hängen hinter `FeatureFlags`.
/// Trägt die Logistik-Daten der Lieferung plus die Beträge, die der
/// Zahlungs-Step braucht (Anzahlung, Zahlungsmethode vom Beleg; Stückpreise
/// liegen an den Positionen).
class Delivery {
const Delivery({
required this.id,

View File

@ -12,6 +12,8 @@ import 'package:hl_lieferservice/feature/delivery/bloc/tour_event.dart';
import 'package:hl_lieferservice/feature/delivery/detail/presentation/widget/discount_editor.dart';
import 'package:hl_lieferservice/feature/loading/widget/reason_catalog.dart';
import 'package:hl_lieferservice/feature/loading/widget/reason_picker_sheet.dart';
import 'package:hl_lieferservice/feature/feature_flags/feature.dart';
import 'package:hl_lieferservice/feature/feature_flags/feature_gate.dart';
/// Step 3 — Artikel & Gutschriften.
///
@ -99,17 +101,26 @@ class StepArticles extends StatelessWidget {
],
),
),
const SizedBox(height: 24),
_SectionHeader(text: 'Gutschriften'),
const SizedBox(height: 8),
Card(
margin: EdgeInsets.zero,
child: Padding(
padding: const EdgeInsets.all(16),
child: DiscountEditor(
deliveryId: delivery.id,
active: delivery.state == DeliveryState.active,
),
// Betrags-Gutschrift, per Feature-Flag komplett ausblendbar.
FeatureGate(
feature: Feature.articlesCreditSection,
child: Column(
crossAxisAlignment: CrossAxisAlignment.stretch,
children: [
const SizedBox(height: 24),
_SectionHeader(text: 'Gutschriften'),
const SizedBox(height: 8),
Card(
margin: EdgeInsets.zero,
child: Padding(
padding: const EdgeInsets.all(16),
child: DiscountEditor(
deliveryId: delivery.id,
active: delivery.state == DeliveryState.active,
),
),
),
],
),
),
],

View File

@ -3,7 +3,8 @@ import 'package:flutter_bloc/flutter_bloc.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/feature_flags.dart';
import 'package:hl_lieferservice/feature/feature_flags/feature.dart';
import 'package:hl_lieferservice/feature/feature_flags/feature_flags.dart';
import 'package:hl_lieferservice/feature/delivery/bloc/tour_bloc.dart';
import 'package:hl_lieferservice/feature/delivery/bloc/tour_event.dart';
import 'package:hl_lieferservice/feature/delivery/bloc/tour_state.dart';
@ -165,8 +166,10 @@ class _DiscountEditorState extends State<DiscountEditor> {
),
const SizedBox(height: 8),
// Default: freies Betrags-Textfeld. Hinter dem Feature-Flag
// `discountAmountStepper` liegt die ursprüngliche +/−-Variante.
if (FeatureFlags.discountAmountStepper)
// `articles.credit_amount_stepper` liegt die +/−-Variante.
if (context
.read<FeatureFlags>()
.isEnabled(Feature.articlesCreditAmountStepper))
Row(
mainAxisAlignment: MainAxisAlignment.center,
children: [

View File

@ -1,46 +0,0 @@
/// Globale, statische Feature-Schalter.
///
/// Dient als Übergangs-Geländer während der Migration vom alten
/// ERPframe-Backend auf das neue Rust-Backend: Funktionen, die im neuen
/// Backend (noch) nicht modelliert sind — Rabatte, Zahlungsoptionen,
/// flexible Lieferoptionen, Preisanzeigen, Unterschriften-Upload —
/// werden hier gebündelt ausgeschaltet, statt sie in jedem UI-Widget
/// einzeln auszukommentieren.
///
/// **Konvention**: jeder Flag bekommt einen kurzen Kommentar, *warum*
/// er gerade auf `false` steht und in welcher Phase der Migration
/// das gegebenenfalls wieder geöffnet wird. So bleibt nachvollziehbar,
/// was hier nur „pausiert" und nicht „weg" ist.
class FeatureFlags {
const FeatureFlags._();
/// Rabatt/Gutschrift-Funktion in der Detail-Ansicht.
/// Backend-Modell fehlt — nicht Teil der Logistik-Migration. Wird
/// frühestens nach C+D-2 wiedereröffnet, wenn überhaupt jemals.
static const bool discountsEnabled = false;
/// Auswahl der Zahlungsart (Bar/EC/Vorkasse) am Ende der Lieferung.
/// Backend modelliert das nicht; die Logistik-App soll bewusst keinen
/// Zahlungs-Workflow tragen.
static const bool paymentsEnabled = false;
/// Anzeige von Brutto-/Netto-Preisen und Vorauszahlung in der UI.
/// Backend liefert keine Preise — Logistik ≠ Buchhaltung.
static const bool pricesEnabled = false;
/// Konfigurierbare Lieferoptionen (Treppe, Anschluss, Altgerät, …).
/// Backend-Schema noch nicht vorhanden; geplant für Phase E.
static const bool deliveryOptionsEnabled = false;
/// Fahrer- und Kunden-Signatur beim Abschluss einer Lieferung. Verkabelt:
/// `SignatureView` → `CompleteDelivery` → multipart `/complete` (Signaturen
/// liegen lokal im Backend-Server).
static const bool signaturesEnabled = true;
/// Eingabeart der Betrags-Gutschrift im Artikel-Step.
/// `false` → freies Betrags-Textfeld (Default); `true` → der ursprüngliche
/// +/−-Stepper (10-€-Schritte). Hinter dem Flag versteckt, falls der
/// Stepper wieder gewünscht wird. In beiden Fällen gilt die Backend-Regel:
/// >0, ≤150 €, Vielfaches von 10 €.
static const bool discountAmountStepper = false;
}

View File

@ -0,0 +1,20 @@
/// Alle schaltbaren Features der App.
///
/// [key] ist der stabile Bezeichner in `assets/feature_flags.json`, über den
/// ein Feature aktiviert oder deaktiviert wird. Der Wert selbst steht nie im
/// Code — nur, was gilt, falls ein Key in der Datei fehlt ([defaultEnabled]).
enum Feature {
/// Abschnitt „Gutschriften" (Betrags-Gutschrift) im Artikel-Step.
articlesCreditSection('articles.credit_section', defaultEnabled: true),
/// Betrags-Gutschrift per +/−-Stepper statt freiem Textfeld.
articlesCreditAmountStepper(
'articles.credit_amount_stepper',
defaultEnabled: false,
);
const Feature(this.key, {required this.defaultEnabled});
final String key;
final bool defaultEnabled;
}

View File

@ -0,0 +1,45 @@
import 'package:flutter/foundation.dart';
import 'feature.dart';
/// Unveränderlicher Satz an Feature-Schaltern, geladen aus
/// `assets/feature_flags.json` (siehe [FeatureFlagsRepository]).
///
/// Erwartetes Format:
/// ```json
/// { "features": { "<key>": { "enabled": true, "description": "…" } } }
/// ```
/// `description` ist nur Doku für Menschen. Unbekannte Keys werden
/// ignoriert, fehlende fallen auf [Feature.defaultEnabled] zurück.
@immutable
class FeatureFlags {
const FeatureFlags(this._enabledByKey);
factory FeatureFlags.fromJson(Map<String, dynamic> json) {
final features = json['features'];
if (features is! Map<String, dynamic>) {
throw const FormatException('feature_flags.json: "features" fehlt.');
}
final knownKeys = {for (final f in Feature.values) f.key};
final enabledByKey = <String, bool>{};
features.forEach((key, value) {
final enabled = value is Map<String, dynamic> ? value['enabled'] : null;
if (enabled is! bool) {
throw FormatException(
'feature_flags.json: "$key.enabled" muss true oder false sein.',
);
}
if (!knownKeys.contains(key)) {
debugPrint('FeatureFlags: unbekannter Key "$key" wird ignoriert.');
return;
}
enabledByKey[key] = enabled;
});
return FeatureFlags(enabledByKey);
}
final Map<String, bool> _enabledByKey;
bool isEnabled(Feature feature) =>
_enabledByKey[feature.key] ?? feature.defaultEnabled;
}

View File

@ -0,0 +1,20 @@
import 'dart:convert';
import 'package:flutter/services.dart' show rootBundle;
import 'feature_flags.dart';
/// Lädt die Feature-Schalter aus der mitgelieferten Asset-Datei.
class FeatureFlagsRepository {
const FeatureFlagsRepository({this.assetPath = 'assets/feature_flags.json'});
final String assetPath;
Future<FeatureFlags> load() async {
final json = jsonDecode(await rootBundle.loadString(assetPath));
if (json is! Map<String, dynamic>) {
throw const FormatException('feature_flags.json ist kein JSON-Objekt.');
}
return FeatureFlags.fromJson(json);
}
}

View File

@ -0,0 +1,22 @@
import 'package:flutter/widgets.dart';
import 'package:flutter_bloc/flutter_bloc.dart';
import 'feature.dart';
import 'feature_flags.dart';
/// Zeigt [child] nur, wenn [feature] aktiviert ist — sonst nichts (kein
/// Platzhalter, kein Abstand). Die Schalter kommen aus dem [FeatureFlags],
/// das oberhalb per `RepositoryProvider` bereitsteht.
class FeatureGate extends StatelessWidget {
const FeatureGate({super.key, required this.feature, required this.child});
final Feature feature;
final Widget child;
@override
Widget build(BuildContext context) {
return context.read<FeatureFlags>().isEnabled(feature)
? child
: const SizedBox.shrink();
}
}

View File

@ -23,6 +23,7 @@ import 'package:hl_lieferservice/feature/delivery/bloc/phase_bloc.dart';
import 'package:hl_lieferservice/feature/delivery/bloc/tour_bloc.dart';
import 'package:hl_lieferservice/feature/delivery/bloc/tour_date_cubit.dart';
import 'package:hl_lieferservice/feature/delivery/bloc/tour_state.dart';
import 'package:hl_lieferservice/feature/feature_flags/feature_flags.dart';
import 'package:hl_lieferservice/widget/home/bloc/navigation_bloc.dart';
import 'package:hl_lieferservice/widget/operations/bloc/operation_bloc.dart';
import 'package:hl_lieferservice/widget/operations/presentation/operation_view_enforcer.dart';
@ -49,6 +50,9 @@ class _DeliveryAppState extends State<DeliveryApp> {
// API-Client. Kein Bloc hält Daten der alten Umgebung.
key: ValueKey('environment-${state.active.id}'),
providers: [
// Feature-Schalter aus `assets/feature_flags.json`; Widgets lesen
// sie über `FeatureGate` bzw. `context.read<FeatureFlags>()`.
RepositoryProvider<FeatureFlags>.value(value: state.featureFlags),
BlocProvider(create: (context) => NavigationBloc()),
BlocProvider(create: (context) => OperationBloc()),
BlocProvider(