removing not needed invite update
authorAlex S <alex.strizhakov@gmail.com>
Mon, 8 Apr 2019 13:08:16 +0000 (20:08 +0700)
committerAlex S <alex.strizhakov@gmail.com>
Mon, 8 Apr 2019 13:08:16 +0000 (20:08 +0700)
lib/pleroma/user_invite_token.ex
test/web/twitter_api/twitter_api_test.exs

index 6247831d51a3c407c70af0288ce86f5200d56ee6..86f0a548690c00ff7e482efa536688757f45a1aa 100644 (file)
@@ -96,9 +96,8 @@ defmodule Pleroma.UserInviteToken do
     not_overdue_date?(invite) and invite.uses < invite.max_use and not invite.used
   end
 
-  defp not_overdue_date?(%{expires_at: expires_at} = invite) do
-    Date.compare(Date.utc_today(), expires_at) in [:lt, :eq] ||
-      (update_invite!(invite, %{used: true}) && false)
+  defp not_overdue_date?(%{expires_at: expires_at}) do
+    Date.compare(Date.utc_today(), expires_at) in [:lt, :eq]
   end
 
   @spec update_usage!(UserInviteToken.t()) :: nil | UserInviteToken.t() | no_return()
index 798a009feb174236dc2529c764b4b6780e2af013..a4540e6511a01a817240382422ca867c82b49df0 100644 (file)
@@ -492,7 +492,7 @@ defmodule Pleroma.Web.TwitterAPI.TwitterAPITest do
       refute User.get_by_nickname("vinny")
       invite = Repo.get_by(UserInviteToken, token: invite.token)
 
-      assert invite.used == true
+      refute invite.used
     end
   end