lib/data/models/metering_screen_layout_config.dart

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

lib/screens/metering/screen_metering.dart

@@ -77,21 +77,11 @@ class _InheritedListeners extends StatelessWidget {
         context.read<MeteringBloc>().add(EquipmentProfileChangedEvent(value));
       },
       child: InheritedModelAspectListener<MeteringScreenLayoutFeature, bool>(
-        aspect: MeteringScreenLayoutFeature.equipmentProfiles,
+        aspect: MeteringScreenLayoutFeature.filmPicker,
         onDidChangeDependencies: (value) {
-          if (!value) {
-            EquipmentProfileProvider.of(context).setProfile(context.get<EquipmentProfiles>().first);
-          }
+          if (!value) context.read<MeteringBloc>().add(const FilmChangedEvent(Film.other()));
         },
-        child: InheritedModelAspectListener<MeteringScreenLayoutFeature, bool>(
-          aspect: MeteringScreenLayoutFeature.filmPicker,
-          onDidChangeDependencies: (value) {
-            if (!value) {
-              context.read<MeteringBloc>().add(const FilmChangedEvent(Film.other()));
-            }
-          },
-          child: child,
-        ),
+        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,9 +33,18 @@ class _MeteringScreenLayoutFeaturesDialogState extends State<MeteringScreenLayou
               child: Text(S.of(context).meteringScreenLayoutHint),
             ),
             const SizedBox(height: Dimens.grid16),
-            _featureListTile(MeteringScreenLayoutFeature.equipmentProfiles),
-            _featureListTile(MeteringScreenLayoutFeature.extremeExposurePairs),
-            _featureListTile(MeteringScreenLayoutFeature.filmPicker),
+            ...MeteringScreenLayoutFeature.values.map(
+              (f) => SwitchListTile(
+                contentPadding: EdgeInsets.symmetric(horizontal: Dimens.dialogTitlePadding.left),
+                title: Text(_toStringLocalized(context, f)),
+                value: _features[f]!,
+                onChanged: (value) {
+                  setState(() {
+                    _features.update(f, (_) => value);
+                  });
+                },
+              ),
+            ),
           ],
         ),
       ),
@@ -56,19 +65,6 @@ 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,56 +2,30 @@ import 'package:lightmeter/data/models/metering_screen_layout_config.dart';
 import 'package:test/test.dart';
 
 void main() {
-  group(
-    'fromJson()',
-    () {
-      test('All keys', () {
-        expect(
-          MeteringScreenLayoutConfigJson.fromJson(
-            {
-              '0': true,
-              '1': true,
-              '2': true,
-            },
-          ),
-          {
-            MeteringScreenLayoutFeature.extremeExposurePairs: true,
-            MeteringScreenLayoutFeature.filmPicker: true,
-            MeteringScreenLayoutFeature.equipmentProfiles: true,
-          },
-        );
-      });
-
-      test('Legacy (no equipment profiles)', () {
-        expect(
-          MeteringScreenLayoutConfigJson.fromJson(
-            {
-              '0': true,
-              '1': true,
-            },
-          ),
-          {
-            MeteringScreenLayoutFeature.extremeExposurePairs: true,
-            MeteringScreenLayoutFeature.filmPicker: true,
-            MeteringScreenLayoutFeature.equipmentProfiles: true,
-          },
-        );
-      });
-    },
-  );
-
-  test('toJson()', () {
+  test('fromJson', () {
     expect(
+      MeteringScreenLayoutConfigJson.fromJson({'0': true, '1': true}),
       {
-        MeteringScreenLayoutFeature.equipmentProfiles: true,
         MeteringScreenLayoutFeature.extremeExposurePairs: true,
         MeteringScreenLayoutFeature.filmPicker: true,
-      }.toJson(),
+      },
+    );
+    expect(
+      MeteringScreenLayoutConfigJson.fromJson({'0': false, '1': false}),
       {
-        '2': true,
-        '0': true,
-        '1': true,
+        MeteringScreenLayoutFeature.extremeExposurePairs: false,
+        MeteringScreenLayoutFeature.filmPicker: false,
       },
     );
   });
+
+  test('toJson', () {
+    expect(
+      {
+        MeteringScreenLayoutFeature.extremeExposurePairs: true,
+        MeteringScreenLayoutFeature.filmPicker: true,
+      }.toJson(),
+      {'0': true, '1': true},
+    );
+  });
 }

test/data/shared_prefs_service_test.dart

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