Equipment profiles issues (#112)

* Fixed equipment profiles sections collapsing

* Fixed range picker dialog

* Refined equipment profiles sections handling

lib/screens/settings/components/metering/components/equipment_profiles/components/equipment_profile_screen/components/equipment_profile_container/components/equipment_list_tiles/components/dialog_range_picker/widget_dialog_picker_range.dart

@@ -68,8 +68,8 @@ class _DialogRangePickerState<T extends PhotographyValue> extends State<DialogRa
                   divisions: widget.values.length - 1,
                   onChanged: (value) {
                     setState(() {
-                      _start = value.start.toInt();
+                      _start = value.start.round();
-                      _end = value.end.toInt();
+                      _end = value.end.round();
                     });
                   },
                 ),

lib/screens/settings/components/metering/components/equipment_profiles/components/equipment_profile_screen/components/equipment_profile_container/widget_container_equipment_profile.dart

@@ -143,10 +143,13 @@ class EquipmentProfileContainerState extends State<EquipmentProfileContainer>
     widget.onExpand();
     _controller.forward();
     SchedulerBinding.instance.addPostFrameCallback((_) {
-      Scrollable.ensureVisible(
+      Future.delayed(_controller.duration!).then((_) {
-        context,
+        Scrollable.ensureVisible(
-        alignmentPolicy: ScrollPositionAlignmentPolicy.keepVisibleAtEnd,
+          context,
-      );
+          alignmentPolicy: ScrollPositionAlignmentPolicy.keepVisibleAtEnd,
+          duration: _controller.duration!,
+        );
+      });
     });
   }
 

lib/screens/settings/components/metering/components/equipment_profiles/components/equipment_profile_screen/screen_equipment_profile.dart

@@ -17,17 +17,13 @@ class EquipmentProfilesScreen extends StatefulWidget {
 }
 
 class _EquipmentProfilesScreenState extends State<EquipmentProfilesScreen> {
-  static const maxProfiles = 5 + 1; // replace with a constant from iap
+  final Map<String, GlobalKey<EquipmentProfileContainerState>> keysMap = {};
-
+  int get profilesCount => keysMap.length;
-  late List<GlobalKey<EquipmentProfileContainerState>> profileContainersKeys = [];
-  int get profilesCount => EquipmentProfiles.of(context).length;
 
   @override
   void didChangeDependencies() {
     super.didChangeDependencies();
-    profileContainersKeys = EquipmentProfiles.of(context)
+    _updateProfilesKeys();
-        .map((e) => GlobalKey<EquipmentProfileContainerState>(debugLabel: e.id))
-        .toList();
   }
 
   @override
@@ -35,11 +31,10 @@ class _EquipmentProfilesScreenState extends State<EquipmentProfilesScreen> {
     return SliverScreen(
       title: S.of(context).equipmentProfiles,
       appBarActions: [
-        if (profilesCount < maxProfiles)
+        IconButton(
-          IconButton(
+          onPressed: _addProfile,
-            onPressed: _addProfile,
+          icon: const Icon(Icons.add),
-            icon: const Icon(Icons.add),
+        ),
-          ),
         IconButton(
           onPressed: Navigator.of(context).pop,
           icon: const Icon(Icons.close),
@@ -55,24 +50,30 @@ class _EquipmentProfilesScreenState extends State<EquipmentProfilesScreen> {
           : [
               SliverList(
                 delegate: SliverChildBuilderDelegate(
-                  (context, index) => index > 0 // skip default
+                  (context, index) {
-                      ? Padding(
+                    if (index == 0) {
-                          padding: EdgeInsets.fromLTRB(
+                      // skip default profile
-                            Dimens.paddingM,
+                      return const SizedBox.shrink();
-                            index == 0 ? Dimens.paddingM : 0,
+                    }
-                            Dimens.paddingM,
+
-                            Dimens.paddingM,
+                    final profile = EquipmentProfiles.of(context)[index];
-                          ),
+                    return Padding(
-                          child: EquipmentProfileContainer(
+                      padding: EdgeInsets.fromLTRB(
-                            key: profileContainersKeys[index],
+                        Dimens.paddingM,
-                            data: EquipmentProfiles.of(context)[index],
+                        index == 0 ? Dimens.paddingM : 0,
-                            onExpand: () => _keepExpandedAt(index),
+                        Dimens.paddingM,
-                            onUpdate: (profileData) => _updateProfileAt(profileData, index),
+                        Dimens.paddingM,
-                            onDelete: () => _removeProfileAt(index),
+                      ),
-                          ),
+                      child: EquipmentProfileContainer(
-                        )
+                        key: keysMap[profile.id],
-                      : const SizedBox.shrink(),
+                        data: profile,
-                  childCount: profilesCount,
+                        onExpand: () => _keepExpandedAt(index),
+                        onUpdate: _updateProfileAt,
+                        onDelete: () => _removeProfileAt(profile),
+                      ),
+                    );
+                  },
+                  childCount: EquipmentProfiles.of(context).length,
                 ),
               ),
               SliverToBoxAdapter(child: SizedBox(height: MediaQuery.paddingOf(context).bottom)),
@@ -91,22 +92,47 @@ class _EquipmentProfilesScreenState extends State<EquipmentProfilesScreen> {
     });
   }
 
-  void _updateProfileAt(EquipmentProfile data, int index) {
+  void _updateProfileAt(EquipmentProfile data) {
     EquipmentProfileProvider.of(context).updateProdile(data);
   }
 
-  void _removeProfileAt(int index) {
+  void _removeProfileAt(EquipmentProfile data) {
-    EquipmentProfileProvider.of(context).deleteProfile(EquipmentProfiles.of(context)[index]);
+    EquipmentProfileProvider.of(context).deleteProfile(data);
   }
 
   void _keepExpandedAt(int index) {
-    profileContainersKeys.getRange(0, index).forEach((element) {
+    keysMap.values.toList().getRange(0, index).forEach((element) {
       element.currentState?.collapse();
     });
-    profileContainersKeys.getRange(index + 1, profilesCount).forEach((element) {
+    keysMap.values.toList().getRange(index + 1, profilesCount).forEach((element) {
       element.currentState?.collapse();
     });
   }
+
+  void _updateProfilesKeys() {
+    final profiles = EquipmentProfiles.of(context);
+    if (profiles.length > keysMap.length) {
+      // profile added
+      final List<String> idsToAdd = [];
+      for (final profile in profiles) {
+        if (!keysMap.keys.contains(profile.id)) idsToAdd.add(profile.id);
+      }
+      for (final id in idsToAdd) {
+        keysMap[id] = GlobalKey<EquipmentProfileContainerState>(debugLabel: id);
+      }
+      idsToAdd.clear();
+    } else if (profiles.length < keysMap.length) {
+      // profile deleted
+      final List<String> idsToDelete = [];
+      for (final id in keysMap.keys) {
+        if (!profiles.any((p) => p.id == id)) idsToDelete.add(id);
+      }
+      idsToDelete.forEach(keysMap.remove);
+      idsToDelete.clear();
+    } else {
+      // profile updated, no need to updated keys
+    }
+  }
 }
 
 class _EquipmentProfilesListPlaceholder extends StatelessWidget {