Skip to content

Commit 489e168

Browse files
committed
fix: solve unscoped player issue
1 parent de51a9b commit 489e168

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

app/modules/authentication/controllers/auth_controller.rb

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ def player_login
151151

152152
return render_missing_credentials if player_email.blank? || password.blank?
153153

154-
player = Player.find_by(player_email: player_email)
154+
player = Player.unscoped.find_by(player_email: player_email)
155155

156156
unless player&.has_player_access? && player.authenticate_player_password(password)
157157
return render_error(
@@ -209,7 +209,9 @@ def player_register
209209

210210
player = build_free_agent_player(player_email, summoner_name, password, discord)
211211

212-
unless player.save
212+
saved = Player.unscoped { player.save }
213+
214+
unless saved
213215
return render_error(
214216
message: 'Erro ao criar conta',
215217
code: 'VALIDATION_ERROR',
@@ -322,7 +324,7 @@ def forgot_password
322324
end
323325

324326
user = User.unscoped.find_by(email: email)
325-
player = Player.find_by(player_email: email) unless user
327+
player = Player.unscoped.find_by(player_email: email) unless user
326328

327329
if user
328330
handle_user_password_reset(user)
@@ -483,15 +485,15 @@ def validate_player_register_params(player_email, summoner_name, password)
483485
)
484486
end
485487

486-
if Player.exists?(player_email: player_email)
488+
if Player.unscoped.exists?(player_email: player_email)
487489
return render_error(
488490
message: 'Já existe uma conta de jogador com este email',
489491
code: 'DUPLICATE_EMAIL',
490492
status: :unprocessable_entity
491493
)
492494
end
493495

494-
if Player.exists?(['LOWER(summoner_name) = ?', summoner_name.downcase])
496+
if Player.unscoped.exists?(['LOWER(summoner_name) = ?', summoner_name.downcase])
495497
return render_error(
496498
message: 'Summoner name já cadastrado na plataforma',
497499
code: 'DUPLICATE_SUMMONER',

0 commit comments

Comments
 (0)