Merge pull request #1225 from LemmyNet/fix_remove_icons

Fix invalid url error when removing icons. Fixes #1219
This commit is contained in:
Dessalines 2020-10-22 14:56:55 -04:00 committed by GitHub
commit cfa60dcb76
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 9 additions and 9 deletions

View File

@ -140,8 +140,8 @@ impl Perform for CreateCommunity {
let icon = diesel_option_overwrite(&data.icon);
let banner = diesel_option_overwrite(&data.banner);
check_optional_url(&data.icon)?;
check_optional_url(&data.banner)?;
check_optional_url(&icon)?;
check_optional_url(&banner)?;
// When you create a community, make sure the user becomes a moderator and a follower
let keypair = generate_actor_keypair()?;
@ -240,8 +240,8 @@ impl Perform for EditCommunity {
let icon = diesel_option_overwrite(&data.icon);
let banner = diesel_option_overwrite(&data.banner);
check_optional_url(&data.icon)?;
check_optional_url(&data.banner)?;
check_optional_url(&icon)?;
check_optional_url(&banner)?;
let community_form = CommunityForm {
name: read_community.name,

View File

@ -100,8 +100,8 @@ pub(in crate) async fn check_community_ban(
}
}
pub(in crate) fn check_optional_url(item: &Option<String>) -> Result<(), LemmyError> {
if let Some(item) = &item {
pub(in crate) fn check_optional_url(item: &Option<Option<String>>) -> Result<(), LemmyError> {
if let Some(Some(item)) = &item {
if Url::parse(item).is_err() {
return Err(APIError::err("invalid_url").into());
}

View File

@ -59,7 +59,7 @@ impl Perform for CreatePost {
check_community_ban(user.id, data.community_id, context.pool()).await?;
check_optional_url(&data.url)?;
check_optional_url(&Some(data.url.to_owned()))?;
// Fetch Iframely and pictrs cached image
let (iframely_title, iframely_description, iframely_html, pictrs_thumbnail) =

View File

@ -349,8 +349,8 @@ impl Perform for SaveUserSettings {
let matrix_user_id = diesel_option_overwrite(&data.matrix_user_id);
// Check to make sure the avatar and banners are urls
check_optional_url(&data.avatar)?;
check_optional_url(&data.banner)?;
check_optional_url(&avatar)?;
check_optional_url(&banner)?;
if let Some(Some(bio)) = &bio {
if bio.chars().count() > 300 {