LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Packages Projects Releases 238 Wiki Activity

fix: valid URL handing in wunderbar

Browse source 
Once parseURI can handle the spaces / return if the URI is valid, we can remove that part and the try/catch.
This commit is contained in:
Thomas Zarebczan 2020-03-10 10:08:27 -04:00 committed by Sean Yesmunt
parent 3eae6f8fdf
commit 46d98ff0c0
1 changed files with 16 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
ui/page/search/view.jsx
View file

@ -2,7 +2,7 @@
import * as ICONS from 'constants/icons';
import * as PAGES from 'constants/pages';
import React, { useEffect, Fragment } from 'react';
import { regexInvalidURI } from 'lbry-redux';
import { regexInvalidURI, parseURI } from 'lbry-redux';
import ClaimPreview from 'component/claimPreview';
import ClaimList from 'component/claimList';
import Page from 'component/page';
@ -35,10 +35,21 @@ export default function SearchPage(props: Props) {
}
const INVALID_URI_CHARS = new RegExp(regexInvalidURI, 'gu');
const modifiedUrlQuery = urlQuery
.trim()
.replace(/\s+/g, '-')
.replace(INVALID_URI_CHARS, '');
let isValid = false;
let path;
try {
({ path } = parseURI(urlQuery.replace(/ /g, '-').replace(/:/g, '#')));
isValid = true;
} catch (e) {
isValid = false;
}
const modifiedUrlQuery = isValid
? path
: urlQuery
.trim()
.replace(/\s+/g, '-')
.replace(INVALID_URI_CHARS, '');
const uriFromQuery = `lbry://${modifiedUrlQuery}`;
useEffect(() => {