Merge pull request #427 from Leptopoda/cleanup/requireData

settings: use requireData for AsyncSnapshots
2 years ago · 7fe089f733
2 changed files with 3 additions and 3 deletions
--- a/packages/settings/lib/src/widgets/checkbox_settings_tile.dart
+++ b/packages/settings/lib/src/widgets/checkbox_settings_tile.dart
@ -18,7 +18,7 @@ class CheckBoxSettingsTile extends InputSettingsTile<ToggleOption> {
                    : CheckboxListTile(
                        title: Text(option.label(context)),
                        value: value,
-                        onChanged: enabledSnapshot.data!
+                        onChanged: enabledSnapshot.requireData
                            ? (final value) async {
                                await option.set(value!);
                              }
--- a/packages/settings/lib/src/widgets/dropdown_button_settings_tile.dart
+++ b/packages/settings/lib/src/widgets/dropdown_button_settings_tile.dart
@ -42,12 +42,12 @@ class DropdownButtonSettingsTile<T> extends InputSettingsTile<SelectOption<T>> {
                          child: DropdownButton<T>(
                            isExpanded: true,
                            value: value,
-                            items: valuesSnapshot.data!.keys
+                            items: valuesSnapshot.requireData.keys
                                .map(
                                  (final k) => DropdownMenuItem(
                                    value: k,
                                    child: Text(
-                                      valuesSnapshot.data![k]!(context),
+                                      valuesSnapshot.requireData[k]!(context),
                                      overflow: TextOverflow.ellipsis,
                                    ),
                                  ),