refactor(api): use more semantic whereILike instruction instead of ra…

…w SQL commands
1024pix · Mar 9, 2025 · 12b5095 · 12b5095
1 parent 5df4cf9
commit 12b5095
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/api/src/identity-access-management/infrastructure/repositories/user.repository.js b/api/src/identity-access-management/infrastructure/repositories/user.repository.js
@@ -22,7 +22,7 @@ import { UserDetailsForAdmin } from '../../domain/models/UserDetailsForAdmin.js'
 import { UserLogin } from '../../domain/models/UserLogin.js';
 
 const getByEmail = async function (email) {
-  const foundUser = await knex.from('users').whereRaw('LOWER("email") = ?', email.toLowerCase()).first();
+  const foundUser = await knex.from('users').whereILike('email', email).first();
   if (!foundUser) {
     throw new UserNotFoundError(`User not found for email ${email}`);
   }
@@ -55,7 +55,7 @@ const getFullById = async function (userId) {
 
 const getByUsernameOrEmailWithRolesAndPassword = async function (username) {
   const userDTO = await knex('users')
-    .whereRaw('LOWER("email") = ?', username.toLowerCase())
+    .whereILike('email', username)
     .orWhere({ username: username.toLowerCase() })
     .first();
 
@@ -257,15 +257,15 @@ const updateWithEmailConfirmed = function ({ id, userAttributes }) {
 };
 
 const checkIfEmailIsAvailable = async function (email) {
-  const existingUserEmail = await knex('users').whereRaw('LOWER("email") = ?', email.toLowerCase()).first();
+  const existingUserEmail = await knex('users').whereILike('email', email).first();
 
   if (existingUserEmail) throw new InvalidOrAlreadyUsedEmailError();
 
   return email;
 };
 
 const isUserExistingByEmail = async function (email) {
-  const existingUser = await knex('users').whereRaw('LOWER("email") = ?', email.toLowerCase()).first();
+  const existingUser = await knex('users').whereILike('email', email).first();
   if (!existingUser) throw new UserNotFoundError();
   return true;
 };
@@ -390,7 +390,7 @@ const findByExternalIdentifier = async function ({ externalIdentityId, identityP
 };
 
 const findAnotherUserByEmail = async function (userId, email) {
-  const anotherUsers = await knex('users').whereNot('id', userId).whereRaw('LOWER("email") = ?', email.toLowerCase());
+  const anotherUsers = await knex('users').whereNot('id', userId).whereILike('email', email);
 
   return anotherUsers.map((anotherUser) => new User(anotherUser));
 };