From 17f64e24e0bde45840f70d9aac00674d9fadfed4 Mon Sep 17 00:00:00 2001 From: krawieck Date: Tue, 6 Oct 2020 16:19:03 +0200 Subject: [PATCH] implement cleanupurl in addInstancePage --- lib/pages/add_instance.dart | 22 +++------------------- lib/util/cleanup_url.dart | 3 +++ 2 files changed, 6 insertions(+), 19 deletions(-) diff --git a/lib/pages/add_instance.dart b/lib/pages/add_instance.dart index de8bb8a..7e988e4 100644 --- a/lib/pages/add_instance.dart +++ b/lib/pages/add_instance.dart @@ -1,6 +1,7 @@ import 'package:cached_network_image/cached_network_image.dart'; import 'package:flutter/material.dart'; import 'package:flutter_hooks/flutter_hooks.dart'; +import 'package:lemmur/util/cleanup_url.dart'; import 'package:lemmy_api_client/lemmy_api_client.dart'; import '../hooks/debounce.dart'; @@ -26,7 +27,7 @@ class AddInstancePage extends HookWidget { final debounce = useDebounce(() async { if (prevInput == instanceController.text) return; - final inst = _fixInstanceUrl(instanceController.text); + final inst = cleanUpUrl(instanceController.text); if (inst.isEmpty) { isSite.value = null; return; @@ -47,7 +48,7 @@ class AddInstancePage extends HookWidget { instanceController.removeListener(debounce); }; }, []); - final inst = _fixInstanceUrl(instanceController.text); + final inst = cleanUpUrl(instanceController.text); handleOnAdd() async { try { await accountsStore.addInstance(inst, assumeValid: true); @@ -145,20 +146,3 @@ class AddInstancePage extends HookWidget { ); } } - -/// removes protocol and trailing slash -String _fixInstanceUrl(String inst) { - if (inst.startsWith('https://')) { - inst = inst.substring(8); - } - - if (inst.startsWith('http://')) { - inst = inst.substring(7); - } - - if (inst.endsWith('/')) { - inst = inst.substring(0, inst.length - 1); - } - - return inst; -} diff --git a/lib/util/cleanup_url.dart b/lib/util/cleanup_url.dart index af542be..b380eb6 100644 --- a/lib/util/cleanup_url.dart +++ b/lib/util/cleanup_url.dart @@ -5,6 +5,9 @@ String cleanUpUrl(String url) { if (url.startsWith('www.')) { url = url.substring(4); } + if (url.endsWith('/')) { + url = url.substring(0, url.length - 1); + } return url; }