From 03233429f4475de558c58707224d1cf72aa28c42 Mon Sep 17 00:00:00 2001 From: Miro Prasil Date: Sun, 16 Feb 2020 20:28:50 +0000 Subject: [PATCH] Remove check from Invitation:take() I've checked the spots when `Invitation::new()` and `Invitation::take()` are used and it seems like all spots are already correctly gated. So to enable invitations via admin API even when invitations are otherwise disabled, this check can be removed. --- src/db/models/user.rs | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/db/models/user.rs b/src/db/models/user.rs index 9646ae58..3a6e95d2 100644 --- a/src/db/models/user.rs +++ b/src/db/models/user.rs @@ -319,10 +319,9 @@ impl Invitation { } pub fn take(mail: &str, conn: &DbConn) -> bool { - CONFIG.invitations_allowed() - && match Self::find_by_mail(mail, &conn) { - Some(invitation) => invitation.delete(&conn).is_ok(), - None => false, - } + match Self::find_by_mail(mail, &conn) { + Some(invitation) => invitation.delete(&conn).is_ok(), + None => false, + } } }