reset selected equipment profile when layout feature is disabled

lib/data/models/metering_screen_layout_config.dart

@@ -1,11 +1,17 @@
-enum MeteringScreenLayoutFeature { equipmentProfiles, extremeExposurePairs, filmPicker }
+enum MeteringScreenLayoutFeature {
+  extremeExposurePairs,
+  filmPicker,
+  equipmentProfiles,
+}
 
 typedef MeteringScreenLayoutConfig = Map<MeteringScreenLayoutFeature, bool>;
 
 extension MeteringScreenLayoutConfigJson on MeteringScreenLayoutConfig {
-  static MeteringScreenLayoutConfig fromJson(Map<String, dynamic> data) => data.map(
+  static MeteringScreenLayoutConfig fromJson(Map<String, dynamic> data) =>
-        (key, value) => MapEntry(MeteringScreenLayoutFeature.values[int.parse(key)], value as bool),
+      <MeteringScreenLayoutFeature, bool>{
-      );
+        for (final f in MeteringScreenLayoutFeature.values)
+          f: data[f.index.toString()] as bool? ?? true
+      };
 
   Map<String, dynamic> toJson() => map((key, value) => MapEntry(key.index.toString(), value));
 }

lib/screens/metering/screen_metering.dart

@@ -77,11 +77,21 @@ class _InheritedListeners extends StatelessWidget {
         context.read<MeteringBloc>().add(EquipmentProfileChangedEvent(value));
       },
       child: InheritedModelAspectListener<MeteringScreenLayoutFeature, bool>(
-        aspect: MeteringScreenLayoutFeature.filmPicker,
+        aspect: MeteringScreenLayoutFeature.equipmentProfiles,
         onDidChangeDependencies: (value) {
-          if (!value) context.read<MeteringBloc>().add(const FilmChangedEvent(Film.other()));
+          if (!value) {
+            EquipmentProfileProvider.of(context).setProfile(context.get<EquipmentProfiles>().first);
+          }
         },
-        child: child,
+        child: InheritedModelAspectListener<MeteringScreenLayoutFeature, bool>(
+          aspect: MeteringScreenLayoutFeature.filmPicker,
+          onDidChangeDependencies: (value) {
+            if (!value) {
+              context.read<MeteringBloc>().add(const FilmChangedEvent(Film.other()));
+            }
+          },
+          child: child,
+        ),
       ),
     );
   }

lib/screens/settings/components/metering/components/metering_screen_layout/components/meterins_screen_layout_features_dialog/widget_dialog_metering_screen_layout_features.dart

@@ -33,18 +33,9 @@ class _MeteringScreenLayoutFeaturesDialogState extends State<MeteringScreenLayou
               child: Text(S.of(context).meteringScreenLayoutHint),
             ),
             const SizedBox(height: Dimens.grid16),
-            ...MeteringScreenLayoutFeature.values.map(
+            _featureListTile(MeteringScreenLayoutFeature.equipmentProfiles),
-              (f) => SwitchListTile(
+            _featureListTile(MeteringScreenLayoutFeature.extremeExposurePairs),
-                contentPadding: EdgeInsets.symmetric(horizontal: Dimens.dialogTitlePadding.left),
+            _featureListTile(MeteringScreenLayoutFeature.filmPicker),
-                title: Text(_toStringLocalized(context, f)),
-                value: _features[f]!,
-                onChanged: (value) {
-                  setState(() {
-                    _features.update(f, (_) => value);
-                  });
-                },
-              ),
-            ),
           ],
         ),
       ),
@@ -65,6 +56,19 @@ class _MeteringScreenLayoutFeaturesDialogState extends State<MeteringScreenLayou
     );
   }
 
+  Widget _featureListTile(MeteringScreenLayoutFeature f) {
+    return SwitchListTile(
+      contentPadding: EdgeInsets.symmetric(horizontal: Dimens.dialogTitlePadding.left),
+      title: Text(_toStringLocalized(context, f)),
+      value: _features[f]!,
+      onChanged: (value) {
+        setState(() {
+          _features.update(f, (_) => value);
+        });
+      },
+    );
+  }
+
   String _toStringLocalized(BuildContext context, MeteringScreenLayoutFeature feature) {
     switch (feature) {
       case MeteringScreenLayoutFeature.equipmentProfiles:

test/data/models/metering_screen_layout_config_test.dart

@@ -2,30 +2,56 @@ import 'package:lightmeter/data/models/metering_screen_layout_config.dart';
 import 'package:test/test.dart';
 
 void main() {
-  test('fromJson', () {
+  group(
-    expect(
+    'fromJson()',
-      MeteringScreenLayoutConfigJson.fromJson({'0': true, '1': true}),
+    () {
-      {
+      test('All keys', () {
-        MeteringScreenLayoutFeature.extremeExposurePairs: true,
+        expect(
-        MeteringScreenLayoutFeature.filmPicker: true,
+          MeteringScreenLayoutConfigJson.fromJson(
-      },
+            {
-    );
+              '0': true,
-    expect(
+              '1': true,
-      MeteringScreenLayoutConfigJson.fromJson({'0': false, '1': false}),
+              '2': true,
-      {
+            },
-        MeteringScreenLayoutFeature.extremeExposurePairs: false,
+          ),
-        MeteringScreenLayoutFeature.filmPicker: false,
+          {
-      },
+            MeteringScreenLayoutFeature.extremeExposurePairs: true,
-    );
+            MeteringScreenLayoutFeature.filmPicker: true,
-  });
+            MeteringScreenLayoutFeature.equipmentProfiles: true,
+          },
+        );
+      });
 
-  test('toJson', () {
+      test('Legacy (no equipment profiles)', () {
+        expect(
+          MeteringScreenLayoutConfigJson.fromJson(
+            {
+              '0': true,
+              '1': true,
+            },
+          ),
+          {
+            MeteringScreenLayoutFeature.extremeExposurePairs: true,
+            MeteringScreenLayoutFeature.filmPicker: true,
+            MeteringScreenLayoutFeature.equipmentProfiles: true,
+          },
+        );
+      });
+    },
+  );
+
+  test('toJson()', () {
     expect(
       {
+        MeteringScreenLayoutFeature.equipmentProfiles: true,
         MeteringScreenLayoutFeature.extremeExposurePairs: true,
         MeteringScreenLayoutFeature.filmPicker: true,
       }.toJson(),
-      {'0': true, '1': true},
+      {
+        '2': true,
+        '0': true,
+        '1': true,
+      },
     );
   });
 }

test/data/shared_prefs_service_test.dart

@@ -193,6 +193,7 @@ void main() {
         {
           MeteringScreenLayoutFeature.extremeExposurePairs: true,
           MeteringScreenLayoutFeature.filmPicker: true,
+          MeteringScreenLayoutFeature.equipmentProfiles: true,
         },
       );
     });
@@ -206,6 +207,7 @@ void main() {
         {
           MeteringScreenLayoutFeature.extremeExposurePairs: false,
           MeteringScreenLayoutFeature.filmPicker: true,
+          MeteringScreenLayoutFeature.equipmentProfiles: true,
         },
       );
     });