Changeset 22439d3 for iknow-api/Services/Implementations

Timestamp:

12/09/25 13:21:49 (10 months ago)

Author:

Stefan-Saveski <stefan@…>

Branches:

master

Children:

2d64b2e

Parents:

bb656bc

Message:

Changed appdb context injection location.

File:

• iknow-api/Services/Implementations/AuthService.cs (modified) (5 diffs)

iknow-api/Services/Implementations/AuthService.cs

@@ -8 +8 @@
 using iknow_api.Repositories;
 using Microsoft.IdentityModel.Tokens;
+using iknow_api.Data;
+using Microsoft.EntityFrameworkCore;
 
 namespace iknow_api.Services
@@ -17 +19 @@
         private readonly IRefreshTokenService _refreshTokenService;
         private readonly IConfiguration _configuration;
+        private readonly AppDbContext _context;
 
-        public AuthService(IUserRepository userRepository, IConfiguration configuration
-                            , IRefreshTokenService refreshTokenService, IRefreshTokenRepository refreshTokenRepository)
+        public AuthService(IUserRepository userRepository, IConfiguration configuration,
+                            IRefreshTokenService refreshTokenService, IRefreshTokenRepository refreshTokenRepository,
+                            AppDbContext context)
         {
             _userRepository = userRepository;
@@ -25 +29 @@
             _refreshTokenService = refreshTokenService;
             _refreshTokenRepository = refreshTokenRepository;
+            _context = context;
         }
 
@@ -44 +49 @@
                 throw new InvalidOperationException("User with this email already exists");
 
-            var user = new User
+            // Use a transaction to ensure all entities are saved together
+            using var transaction = await _context.Database.BeginTransactionAsync();
+            try
             {
-                Name = registerDto.Name,
-                Surname = registerDto.Surname,
-                Index = registerDto.Index,
-                Email = registerDto.Email,
-                PasswordHash = BCrypt.Net.BCrypt.HashPassword(registerDto.Password),
-                Bday = DateTime.SpecifyKind(registerDto.Bday, DateTimeKind.Utc),
-                CreatedAt = DateTime.UtcNow,
-                Role = (Models.UserRole)registerDto.Role,
-                EMBG = registerDto.EMBG
-            };
+                var user = new User
+                {
+                    Name = registerDto.Name,
+                    Surname = registerDto.Surname,
+                    Index = registerDto.Index,
+                    Email = registerDto.Email,
+                    PasswordHash = BCrypt.Net.BCrypt.HashPassword(registerDto.Password),
+                    Bday = DateTime.SpecifyKind(registerDto.Bday, DateTimeKind.Utc),
+                    CreatedAt = DateTime.UtcNow,
+                    Role = (Models.UserRole)registerDto.Role,
+                    EMBG = registerDto.EMBG
+                };
 
-            await _userRepository.AddUserAsync(user);
-            int userId = user.Id;
-            var contactInfo = new ContactInfo
+                _context.User.Add(user);
+                await _context.SaveChangesAsync();
+                
+                int userId = user.Id;
+                
+                var contactInfo = new ContactInfo
+                {
+                    UserId = userId,
+                    city = registerDto.city,
+                    address = registerDto.address,
+                    municipality = registerDto.municipality,
+                    phoneNumber = registerDto.phoneNumber,
+                    microsoftEmail = registerDto.microsoftEmail
+                };
+                
+                var enrollmentInfo = new EnrollmentInfo
+                {
+                    UserId = userId,
+                    enrollmentYear = registerDto.enrollmentYear,
+                    quota = (Models.Quota)registerDto.quotaType,
+                    MajorId = registerDto.majorType
+                };
+                
+                var highSchool = new HighSchool
+                {
+                    UserId = userId,
+                    GPA = registerDto.gpa,
+                    tip = (Models.type)registerDto.tip
+                };
+
+                _context.ContactInfo.Add(contactInfo);
+                _context.EnrollmentInfo.Add(enrollmentInfo);
+                _context.HighSchool.Add(highSchool);
+                
+                // Save all related entities in a single transaction
+                await _context.SaveChangesAsync();
+                await transaction.CommitAsync();
+                
+                return true;
+            }
+            catch
             {
-                UserId = userId,
-                city = registerDto.city,
-                address = registerDto.address,
-                municipality = registerDto.municipality,
-                phoneNumber = registerDto.phoneNumber,
-                microsoftEmail = registerDto.microsoftEmail
-            };
-            await _userRepository.AddContactAsync(contactInfo);
-            var enrollmentInfo = new EnrollmentInfo
-            {
-                UserId = userId,
-                enrollmentYear = registerDto.enrollmentYear,
-                quota = (Models.Quota)registerDto.quotaType,
-                MajorId = registerDto.majorType
-            };
-            await _userRepository.AddEnrollmentAsync(enrollmentInfo);
-            await _userRepository.AddHighSchoolAsync(new HighSchool
-            {
-                UserId = userId,
-                GPA = registerDto.gpa,
-                tip = (Models.type)registerDto.tip
-            });
-            return true;
+                await transaction.RollbackAsync();
+                throw;
+            }
         }
 
@@ -99 +126 @@
             if (loginDto.GenerateRefreshToken)
             {
-                int userid = _userRepository.GetUserByUsernameAsync(loginDto.Email).Id;
-                result.RefreshToken = await _refreshTokenService.GenerateRefreshToken(userid);
+                result.RefreshToken = await _refreshTokenService.GenerateRefreshToken(user.Id);
             }