Move require Logger to module level

Move require Logger statements from function/case level to module level
for better code organization and consistency with Elixir best practices
This commit is contained in:
Moritz 2026-01-21 08:35:34 +01:00 committed by Simon
parent d893c0680a
commit d9f5579350
Signed by: simon
GPG key ID: 40E7A58C4AA1EDB2
3 changed files with 4 additions and 12 deletions

View file

@ -9,6 +9,8 @@ defmodule Mv.Accounts.User.Validations.EmailNotUsedByOtherMember do
"""
use Ash.Resource.Validation
require Logger
@doc """
Validates email uniqueness across linked User-Member pairs.
@ -92,8 +94,6 @@ defmodule Mv.Accounts.User.Validations.EmailNotUsedByOtherMember do
{:error, field: :email, message: "is already used by another member", value: email}
{:error, reason} ->
require Logger
Logger.warning(
"Email uniqueness validation query failed for user email '#{email}': #{inspect(reason)}. Allowing operation to proceed (fail-open)."
)

View file

@ -10,6 +10,8 @@ defmodule Mv.Membership.Member.Validations.EmailNotUsedByOtherUser do
use Ash.Resource.Validation
alias Mv.Helpers
require Logger
@doc """
Validates email uniqueness across linked Member-User pairs.
@ -64,8 +66,6 @@ defmodule Mv.Membership.Member.Validations.EmailNotUsedByOtherUser do
{:error, field: :email, message: "is already used by another user", value: email}
{:error, reason} ->
require Logger
Logger.warning(
"Email uniqueness validation query failed for member email '#{email}': #{inspect(reason)}. Allowing operation to proceed (fail-open)."
)