🐛 Fix dismissing open url dialog saving checkbox states

OkunaOrg · Aug 25, 2019 · 7d8b59c · 7d8b59c
1 parent 3c728d3
commit 7d8b59c
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/lib/pages/home/bottom_sheets/confirm_open_url.dart b/lib/pages/home/bottom_sheets/confirm_open_url.dart
@@ -122,7 +122,7 @@ class OBConfirmOpenUrlBottomSheetState extends State<OBConfirmOpenUrlBottomSheet
  size: OBButtonSize.medium,
  type: OBButtonType.primary,
  child: Text(_localizationService.post__open_url_continue),
- onPressed: _onConfirmOpen,
+ onPressed: _onConfirmTapped,
  ),
  ),
  ],
@@ -136,17 +136,27 @@ class OBConfirmOpenUrlBottomSheetState extends State<OBConfirmOpenUrlBottomSheet
  void _toggleDontAskForHost() {
  setState(() {
  _askForHost = !_askForHost;
- _preferencesService.setAskToConfirmOpenUrl(_askForHost, host: widget._uri.host);
  });
  }
 
  void _toggleDontAsk() {
  setState(() {
  _ask = !_ask;
- _preferencesService.setAskToConfirmOpenUrl(_ask);
  });
  }
 
+ void _onConfirmTapped() async {
+ if (!_askForHost) {
+ await _preferencesService.setAskToConfirmOpenUrl(
+ _askForHost, host: widget._uri.host);
+ }
+ if (!_ask) {
+ await _preferencesService.setAskToConfirmOpenUrl(_ask);
+ }
+
+ _onConfirmOpen();
+ }
+
  void _onConfirmOpen() {
  Navigator.pop(context, true);
  }