[pve-devel] [PATCH pve_flutter_frontend 3/4] fix: dart-analyze: remove deprecated props and use `RadioGroup`

[Shan Shaji <s.shaji@proxmox.com>]

Starting from flutter v3.32 [0] the `groupValue` and `onChanged` has
been deprecated from `RadioListTile` and need to use `RadioGroup`
widget. Fixed the issue by wrapping the group of `RadioListTile`
widgets with `RadioGroup` [1].

- [0] http://api.flutter.dev/flutter/material/RadioListTile/groupValue.html
- [1] https://api.flutter.dev/flutter/widgets/RadioGroup-class.html

Signed-off-by: Shan Shaji <s.shaji@proxmox.com>
---
 lib/widgets/pve_cd_selector_widget.dart | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/lib/widgets/pve_cd_selector_widget.dart b/lib/widgets/pve_cd_selector_widget.dart
index d1f260e..b4591d0 100644
--- a/lib/widgets/pve_cd_selector_widget.dart
+++ b/lib/widgets/pve_cd_selector_widget.dart
@@ -22,12 +22,13 @@ class PveCdSelector extends StatelessWidget {
         builder: (context, snapshot) {
           if (snapshot.hasData) {
             final state = snapshot.data!;
-            return Column(children: [
+            return RadioGroup(
+              groupValue: state.value,
+              onChanged: (value) => cdBloc.events.add(ChangeValue(value)),
+              child: Column(children: [
               RadioListTile<CdType>(
                 title: const Text('Use CD/DVD disc image file (iso)'),
                 value: CdType.iso,
-                groupValue: state.value,
-                onChanged: (value) => cdBloc.events.add(ChangeValue(value)),
               ),
               if (state.value == CdType.iso)
                 OutlinedButton(
@@ -63,16 +64,12 @@ class PveCdSelector extends StatelessWidget {
               RadioListTile<CdType>(
                 title: const Text('Use physical CD/DVD Drive'),
                 value: CdType.cdrom,
-                groupValue: state.value,
-                onChanged: (value) => cdBloc.events.add(ChangeValue(value)),
               ),
               RadioListTile<CdType>(
                 title: const Text('Do not use any media'),
                 value: CdType.none,
-                groupValue: state.value,
-                onChanged: (value) => cdBloc.events.add(ChangeValue(value)),
               ),
-            ]);
+            ]));
           }
 
           return Container();
-- 
2.47.2



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel