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

@@ -76,23 +76,13 @@ class _InheritedListeners extends StatelessWidget {
       onDidChangeDependencies: (value) {
         context.read<MeteringBloc>().add(EquipmentProfileChangedEvent(value));
       },
-      child: InheritedModelAspectListener<MeteringScreenLayoutFeature, bool>(
-        aspect: MeteringScreenLayoutFeature.equipmentProfiles,
-        onDidChangeDependencies: (value) {
-          if (!value) {
-            EquipmentProfileProvider.of(context).setProfile(context.get<EquipmentProfiles>().first);
-          }
-        },
       child: InheritedModelAspectListener<MeteringScreenLayoutFeature, bool>(
         aspect: MeteringScreenLayoutFeature.filmPicker,
         onDidChangeDependencies: (value) {
-            if (!value) {
-              context.read<MeteringBloc>().add(const FilmChangedEvent(Film.other()));
-            }
+          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,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', () {
+  test('fromJson', () {
     expect(
-          MeteringScreenLayoutConfigJson.fromJson(
-            {
-              '0': true,
-              '1': true,
-              '2': true,
-            },
-          ),
+      MeteringScreenLayoutConfigJson.fromJson({'0': true, '1': true}),
       {
         MeteringScreenLayoutFeature.extremeExposurePairs: true,
         MeteringScreenLayoutFeature.filmPicker: true,
-            MeteringScreenLayoutFeature.equipmentProfiles: true,
+      },
+    );
+    expect(
+      MeteringScreenLayoutConfigJson.fromJson({'0': false, '1': false}),
+      {
+        MeteringScreenLayoutFeature.extremeExposurePairs: false,
+        MeteringScreenLayoutFeature.filmPicker: false,
       },
     );
   });
 
-      test('Legacy (no equipment profiles)', () {
-        expect(
-          MeteringScreenLayoutConfigJson.fromJson(
-            {
-              '0': true,
-              '1': true,
-            },
-          ),
-          {
-            MeteringScreenLayoutFeature.extremeExposurePairs: true,
-            MeteringScreenLayoutFeature.filmPicker: true,
-            MeteringScreenLayoutFeature.equipmentProfiles: true,
-          },
-        );
-      });
-    },
-  );
-
-  test('toJson()', () {
+  test('toJson', () {
     expect(
       {
-        MeteringScreenLayoutFeature.equipmentProfiles: true,
         MeteringScreenLayoutFeature.extremeExposurePairs: true,
         MeteringScreenLayoutFeature.filmPicker: true,
       }.toJson(),
-      {
-        '2': true,
-        '0': true,
-        '1': true,
-      },
+      {'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,
         },
       );
     });