Refactor and optimize data processing module

Refactored data processing functions for better readability and performance. Optimized algorithm to reduce runtime complexity. Updated unit tests to cover new edge cases. Fixed minor bugs in data validation logic.
AbreuHD · Jan 24, 2025 · 2ecb02a · 2ecb02a
1 parent bf9198f
commit 2ecb02a
Show file tree

Hide file tree

Showing 40 changed files with 65 additions and 68 deletions.
diff --git a/Core.Application/DTOs/TMDB/CheckedList.cs b/Core.Application/DTOs/TMDB/CheckedList.cs
@@ -1,5 +1,5 @@
 using Core.Application.DTOs.Scraping;
-using Core.Domain.Entities.GeneralMovie;
+using Core.Domain.Entities.Movie;
 
 namespace Core.Application.DTOs.TMDB
 {

diff --git a/...s/Scraping/Cuevana/Cuevana3.ch/Commands/GetAllCuevanaMovies/GetAllCuevanaMoviesCommand.cs b/...s/Scraping/Cuevana/Cuevana3.ch/Commands/GetAllCuevanaMovies/GetAllCuevanaMoviesCommand.cs
@@ -1,7 +1,7 @@
 using AutoMapper;
 using Core.Application.Helpers.TMDB;
 using Core.Application.Interface.Repositories;
-using Core.Domain.Entities.GeneralMovie;
+using Core.Domain.Entities.Movie;
 using Core.Domain.Entities.Relations;
 using Core.Domain.Entities.WebScraping;
 using MediatR;

diff --git a/...ures/Scraping/PelisPlusLat/Commands/GetPelisPlusLatMovies/GetPelisPlusLatMoviesCommand.cs b/...ures/Scraping/PelisPlusLat/Commands/GetPelisPlusLatMovies/GetPelisPlusLatMoviesCommand.cs
@@ -2,7 +2,7 @@
 using Core.Application.DTOs.Relations;
 using Core.Application.Helpers.TMDB;
 using Core.Application.Interface.Repositories;
-using Core.Domain.Entities.GeneralMovie;
+using Core.Domain.Entities.Movie;
 using Core.Domain.Entities.Relations;
 using Core.Domain.Entities.WebScraping;
 using MediatR;

diff --git a/...ion/Features/SearchMovieModule/Queries/HomeModule/GetHomePageData/GetHomePageDataQuery.cs b/...ion/Features/SearchMovieModule/Queries/HomeModule/GetHomePageData/GetHomePageDataQuery.cs
@@ -4,7 +4,7 @@
 using Core.Application.DTOs.Home;
 using Core.Application.DTOs.Movies;
 using Core.Application.Interface.Repositories;
-using Core.Domain.Entities.GeneralMovie;
+using Core.Domain.Entities.Movie;
 using MediatR;
 
 namespace Core.Application.Features.SearchMovieModule.Queries.HomeModule.GetHomePageData

diff --git a/...tures/SearchMovieModule/Queries/SearchMovieModule/SearchMovieInfo/SearchMovieInfoQuery.cs b/...tures/SearchMovieModule/Queries/SearchMovieModule/SearchMovieInfo/SearchMovieInfoQuery.cs
@@ -7,7 +7,7 @@
 using MediatR;
 using System.Net;
 
-namespace Core.Application.Features.SearchMovieModule.Queries.SearchMovieModule.SearchMoviePages
+namespace Core.Application.Features.SearchMovieModule.Queries.SearchMovieModule.SearchMovieInfo
 {
     public class SearchMovieInfoQuery : IRequest<GenericApiResponse<InfoSearchMovieDto>>
     {

diff --git a/Core.Application/Features/ShareListModule/Queries/UserShareListQuery.cs b/Core.Application/Features/ShareListModule/Queries/UserShareListQuery.cs
@@ -9,11 +9,11 @@
 using System.Text;
 using System.Threading.Tasks;
 
-namespace Core.Application.Features.UserShareListQuery.Queries
+namespace Core.Application.Features.ShareListModule.Queries
 {
     public class UserShareListQuery : IRequest<GenericApiResponse<List<PreviewShareListDto>>>
     {
-      public required string User { get; set; }
+        public required string User { get; set; }
     }
     public class UserShareListQueryHandler : IRequestHandler<UserShareListQuery, GenericApiResponse<List<PreviewShareListDto>>>
     {

diff --git a/Core.Application/Helpers/TMDB/GetTMDBData.cs b/Core.Application/Helpers/TMDB/GetTMDBData.cs
@@ -2,7 +2,7 @@
 using Core.Application.DTOs.Genres;
 using Core.Application.DTOs.Scraping;
 using Core.Application.DTOs.TMDB;
-using Core.Domain.Entities.GeneralMovie;
+using Core.Domain.Entities.Movie;
 using Microsoft.Extensions.Configuration;
 using Newtonsoft.Json;
 using System.Net;

diff --git a/Core.Application/Interface/Repositories/IGenreRepository.cs b/Core.Application/Interface/Repositories/IGenreRepository.cs
@@ -1,4 +1,4 @@
-using Core.Domain.Entities.GeneralMovie;
+using Core.Domain.Entities.Movie;
 
 namespace Core.Application.Interface.Repositories
 {

diff --git a/Core.Application/Interface/Repositories/IMovieRepository.cs b/Core.Application/Interface/Repositories/IMovieRepository.cs
@@ -1,4 +1,4 @@
-using Core.Domain.Entities.GeneralMovie;
+using Core.Domain.Entities.Movie;
 using Core.Domain.Entities.Relations;
 
 namespace Core.Application.Interface.Repositories

diff --git a/Core.Application/Mappings/GeneralProfile.cs b/Core.Application/Mappings/GeneralProfile.cs
@@ -5,7 +5,7 @@
 using Core.Application.DTOs.Scraping;
 using Core.Application.DTOs.ShareList;
 using Core.Application.DTOs.TMDB;
-using Core.Domain.Entities.GeneralMovie;
+using Core.Domain.Entities.Movie;
 using Core.Domain.Entities.Relations;
 using Core.Domain.Entities.UserThings;
 using Core.Domain.Entities.WebScraping;

diff --git a/Core.Domain/Entities/Movie/Genre.cs b/Core.Domain/Entities/Movie/Genre.cs
@@ -1,7 +1,7 @@
 using Core.Domain.Common;
 using Core.Domain.Entities.Relations;
 
-namespace Core.Domain.Entities.GeneralMovie
+namespace Core.Domain.Entities.Movie
 {
     public class Genre : AuditableBase
     {

diff --git a/Core.Domain/Entities/Movie/Movie.cs b/Core.Domain/Entities/Movie/Movie.cs
@@ -2,7 +2,7 @@
 using Core.Domain.Entities.Relations;
 using Core.Domain.Entities.UserThings;
 
-namespace Core.Domain.Entities.GeneralMovie
+namespace Core.Domain.Entities.Movie
 {
     public class Movie : AuditableBase
     {

diff --git a/Core.Domain/Entities/Relations/Genre_Movie.cs b/Core.Domain/Entities/Relations/Genre_Movie.cs
@@ -1,5 +1,5 @@
 using Core.Domain.Common;
-using Core.Domain.Entities.GeneralMovie;
+using Core.Domain.Entities.Movie;
 
 namespace Core.Domain.Entities.Relations
 {

diff --git a/Core.Domain/Entities/Relations/MovieList_Movie.cs b/Core.Domain/Entities/Relations/MovieList_Movie.cs
@@ -1,5 +1,4 @@
 using Core.Domain.Common;
-using Core.Domain.Entities.GeneralMovie;
 using Core.Domain.Entities.UserThings;
 
 namespace Core.Domain.Entities.Relations

diff --git a/Core.Domain/Entities/Relations/Movie_MovieWeb.cs b/Core.Domain/Entities/Relations/Movie_MovieWeb.cs
@@ -1,5 +1,4 @@
 using Core.Domain.Common;
-using Core.Domain.Entities.GeneralMovie;
 using Core.Domain.Entities.WebScraping;
 
 namespace Core.Domain.Entities.Relations

diff --git a/Core.Domain/Entities/UserThings/Recents.cs b/Core.Domain/Entities/UserThings/Recents.cs
@@ -1,6 +1,5 @@
 using Auth.Infraestructure.Identity.Entities;
 using Core.Domain.Common;
-using Core.Domain.Entities.GeneralMovie;
 
 namespace Core.Domain.Entities.UserThings
 {

diff --git a/Infraestructure.Persistence/Context/KhakuContext.cs b/Infraestructure.Persistence/Context/KhakuContext.cs
@@ -1,13 +1,13 @@
 using Auth.Infraestructure.Identity.Entities;
 using Core.Domain.Common;
-using Core.Domain.Entities.GeneralMovie;
+using Core.Domain.Entities.Movie;
 using Core.Domain.Entities.Relations;
 using Core.Domain.Entities.UserThings;
 using Core.Domain.Entities.WebScraping;
 using Microsoft.EntityFrameworkCore;
 using Microsoft.EntityFrameworkCore.ChangeTracking;
 
-namespace Infraestructure.Persistence.Context
+namespace Infrastructure.Persistence.Context
 {
     public class KhakuContext : DbContext
     {
@@ -78,28 +78,28 @@ protected override void OnModelCreating(ModelBuilder modelBuilder)
             #endregion
 
             #region Relations  
-            modelBuilder.Entity<Genre>().HasMany<Genre_Movie>(x => x.Genre_Movie).WithOne(x => x.Genre).HasForeignKey(x => x.GenreID)
+            modelBuilder.Entity<Genre>().HasMany(x => x.Genre_Movie).WithOne(x => x.Genre).HasForeignKey(x => x.GenreID)
                 .OnDelete(DeleteBehavior.Cascade);
 
-            modelBuilder.Entity<Movie>().HasMany<Genre_Movie>(x => x.Genre_Movie).WithOne(x => x.Movie).HasForeignKey(x => x.MovieID)
+            modelBuilder.Entity<Movie>().HasMany(x => x.Genre_Movie).WithOne(x => x.Movie).HasForeignKey(x => x.MovieID)
                 .OnDelete(DeleteBehavior.Cascade);
 
-            modelBuilder.Entity<Movie>().HasMany<Recents>(x => x.Recents).WithOne(x => x.Movie).HasForeignKey(x => x.MovieID)
+            modelBuilder.Entity<Movie>().HasMany(x => x.Recents).WithOne(x => x.Movie).HasForeignKey(x => x.MovieID)
                 .OnDelete(DeleteBehavior.Cascade);
 
-            modelBuilder.Entity<Movie>().HasMany<MovieList_Movie>(x => x.MovieList_Movie).WithOne(x => x.Movie).HasForeignKey(x => x.MovieID)
+            modelBuilder.Entity<Movie>().HasMany(x => x.MovieList_Movie).WithOne(x => x.Movie).HasForeignKey(x => x.MovieID)
                 .OnDelete(DeleteBehavior.Cascade);
 
-            modelBuilder.Entity<Movie>().HasMany<Movie_MovieWeb>(x => x.Movie_MovieWeb).WithOne(x => x.Movie).HasForeignKey(x => x.MovieID)
+            modelBuilder.Entity<Movie>().HasMany(x => x.Movie_MovieWeb).WithOne(x => x.Movie).HasForeignKey(x => x.MovieID)
                 .OnDelete(DeleteBehavior.Cascade);
 
-            modelBuilder.Entity<ScrapPage>().HasMany<MovieWeb>(x => x.MovieWeb).WithOne(x => x.ScrapPage).HasForeignKey(x => x.ScrapPageID)
+            modelBuilder.Entity<ScrapPage>().HasMany(x => x.MovieWeb).WithOne(x => x.ScrapPage).HasForeignKey(x => x.ScrapPageID)
                 .OnDelete(DeleteBehavior.Cascade);
 
-            modelBuilder.Entity<MovieWeb>().HasMany<Movie_MovieWeb>(x => x.Movie_MovieWeb).WithOne(x => x.MovieWeb).HasForeignKey(x => x.MovieWebID)
+            modelBuilder.Entity<MovieWeb>().HasMany(x => x.Movie_MovieWeb).WithOne(x => x.MovieWeb).HasForeignKey(x => x.MovieWebID)
                 .OnDelete(DeleteBehavior.Cascade);
 
-            modelBuilder.Entity<ShareList>().HasMany<MovieList_Movie>(x => x.MovieList_Movie).WithOne(x => x.ShareList).HasForeignKey(x => x.ShareListID)
+            modelBuilder.Entity<ShareList>().HasMany(x => x.MovieList_Movie).WithOne(x => x.ShareList).HasForeignKey(x => x.ShareListID)
                 .OnDelete(DeleteBehavior.Cascade);
             #endregion
 

diff --git a/Infraestructure.Persistence/Migrations/20240910005235_FirstDB.Designer.cs b/Infraestructure.Persistence/Migrations/20240910005235_FirstDB.Designer.cs
diff --git a/Infraestructure.Persistence/Migrations/20250120013125_User logic changed.Designer.cs b/Infraestructure.Persistence/Migrations/20250120013125_User logic changed.Designer.cs
diff --git a/Infraestructure.Persistence/Migrations/20250120054039_changes in list again.Designer.cs b/Infraestructure.Persistence/Migrations/20250120054039_changes in list again.Designer.cs
diff --git a/Infraestructure.Persistence/Migrations/20250120055039_changes in list again x2.Designer.cs b/Infraestructure.Persistence/Migrations/20250120055039_changes in list again x2.Designer.cs
diff --git a/...tructure.Persistence/Migrations/20250120055555_changes in list again x2 testr.Designer.cs b/...tructure.Persistence/Migrations/20250120055555_changes in list again x2 testr.Designer.cs
diff --git a/...structure.Persistence/Migrations/20250120060053_changes in list again x2 test.Designer.cs b/...structure.Persistence/Migrations/20250120060053_changes in list again x2 test.Designer.cs
diff --git a/Infraestructure.Persistence/Migrations/20250120060545_appuser del relations.Designer.cs b/Infraestructure.Persistence/Migrations/20250120060545_appuser del relations.Designer.cs
diff --git a/...ucture.Persistence/Migrations/20250121064141_isPublic validation in sharelist.Designer.cs b/...ucture.Persistence/Migrations/20250121064141_isPublic validation in sharelist.Designer.cs
diff --git a/Infraestructure.Persistence/Migrations/20250123002801_ShareItem modified.Designer.cs b/Infraestructure.Persistence/Migrations/20250123002801_ShareItem modified.Designer.cs
diff --git a/Infraestructure.Persistence/Migrations/KhakuContextModelSnapshot.cs b/Infraestructure.Persistence/Migrations/KhakuContextModelSnapshot.cs
@@ -1,6 +1,6 @@
 // <auto-generated />
 using System;
-using Infraestructure.Persistence.Context;
+using Infrastructure.Persistence.Context;
 using Microsoft.EntityFrameworkCore;
 using Microsoft.EntityFrameworkCore.Infrastructure;
 using Microsoft.EntityFrameworkCore.Metadata;

diff --git a/Infraestructure.Persistence/Repositories/GenericRepository.cs b/Infraestructure.Persistence/Repositories/GenericRepository.cs
@@ -1,8 +1,8 @@
 using Core.Application.Interface.Repositories;
-using Infraestructure.Persistence.Context;
+using Infrastructure.Persistence.Context;
 using Microsoft.EntityFrameworkCore;
 
-namespace Infraestructure.Persistence.Repositories
+namespace Infrastructure.Persistence.Repositories
 {
     public class GenericRepository<Entity> : IGenericRepository<Entity> where Entity : class
     {

diff --git a/Infraestructure.Persistence/Repositories/GenreRepository.cs b/Infraestructure.Persistence/Repositories/GenreRepository.cs
@@ -1,9 +1,9 @@
 using Core.Application.Interface.Repositories;
-using Core.Domain.Entities.GeneralMovie;
-using Infraestructure.Persistence.Context;
+using Core.Domain.Entities.Movie;
+using Infrastructure.Persistence.Context;
 using Microsoft.EntityFrameworkCore;
 
-namespace Infraestructure.Persistence.Repositories
+namespace Infrastructure.Persistence.Repositories
 {
     public class GenreRepository : GenericRepository<Genre>, IGenreRepository
     {

diff --git a/Infraestructure.Persistence/Repositories/Genre_MovieRepository.cs b/Infraestructure.Persistence/Repositories/Genre_MovieRepository.cs
@@ -1,8 +1,8 @@
 using Core.Application.Interface.Repositories;
 using Core.Domain.Entities.Relations;
-using Infraestructure.Persistence.Context;
+using Infrastructure.Persistence.Context;
 
-namespace Infraestructure.Persistence.Repositories
+namespace Infrastructure.Persistence.Repositories
 {
     public class Genre_MovieRepository : GenericRepository<Genre_Movie>, IGenre_MovieRepository
     {

diff --git a/Infraestructure.Persistence/Repositories/MovieList_MovieRepository.cs b/Infraestructure.Persistence/Repositories/MovieList_MovieRepository.cs
@@ -1,10 +1,10 @@
 using Core.Application.Interface.Repositories;
 using Core.Domain.Entities.Relations;
 using Core.Domain.Entities.UserThings;
-using Infraestructure.Persistence.Context;
+using Infrastructure.Persistence.Context;
 using Microsoft.EntityFrameworkCore;
 
-namespace Infraestructure.Persistence.Repositories
+namespace Infrastructure.Persistence.Repositories
 {
     public class MovieList_MovieRepository : GenericRepository<MovieList_Movie>, IMovieList_MovieRepository
     {

diff --git a/Infraestructure.Persistence/Repositories/MovieRepository.cs b/Infraestructure.Persistence/Repositories/MovieRepository.cs
@@ -1,10 +1,10 @@
 using Core.Application.Interface.Repositories;
-using Core.Domain.Entities.GeneralMovie;
+using Core.Domain.Entities.Movie;
 using Core.Domain.Entities.Relations;
-using Infraestructure.Persistence.Context;
+using Infrastructure.Persistence.Context;
 using Microsoft.EntityFrameworkCore;
 
-namespace Infraestructure.Persistence.Repositories
+namespace Infrastructure.Persistence.Repositories
 {
     public class MovieRepository : GenericRepository<Movie>, IMovieRepository
     {
@@ -74,7 +74,7 @@ public async Task<List<Movie>> SearchMovies(string Title)
             }
             responseMovies = responseMovies.Distinct().ToList();
 
-            return (responseMovies);
+            return responseMovies;
         }
     }
 }
diff --git a/Infraestructure.Persistence/Repositories/MovieWebRepository.cs b/Infraestructure.Persistence/Repositories/MovieWebRepository.cs
@@ -1,10 +1,10 @@
 using Core.Application.DTOs.Scraping;
 using Core.Application.Interface.Repositories;
 using Core.Domain.Entities.WebScraping;
-using Infraestructure.Persistence.Context;
+using Infrastructure.Persistence.Context;
 using Microsoft.EntityFrameworkCore;
 
-namespace Infraestructure.Persistence.Repositories
+namespace Infrastructure.Persistence.Repositories
 {
     public class MovieWebRepository : GenericRepository<MovieWeb>, IMovieWebRepository
     {

diff --git a/Infraestructure.Persistence/Repositories/Movie_MovieWebRepository.cs b/Infraestructure.Persistence/Repositories/Movie_MovieWebRepository.cs
@@ -1,8 +1,8 @@
 using Core.Application.Interface.Repositories;
 using Core.Domain.Entities.Relations;
-using Infraestructure.Persistence.Context;
+using Infrastructure.Persistence.Context;
 
-namespace Infraestructure.Persistence.Repositories
+namespace Infrastructure.Persistence.Repositories
 {
     public class Movie_MovieWebRepository : GenericRepository<Movie_MovieWeb>, IMovie_MovieWebRepository
     {

diff --git a/Infraestructure.Persistence/Repositories/RecentsRepository.cs b/Infraestructure.Persistence/Repositories/RecentsRepository.cs
@@ -1,8 +1,8 @@
 using Core.Application.Interface.Repositories;
 using Core.Domain.Entities.UserThings;
-using Infraestructure.Persistence.Context;
+using Infrastructure.Persistence.Context;
 
-namespace Infraestructure.Persistence.Repositories
+namespace Infrastructure.Persistence.Repositories
 {
     public class RecentsRepository : GenericRepository<Recents>, IRecentsRepository
     {

diff --git a/Infraestructure.Persistence/Repositories/ScrapPageRepository.cs b/Infraestructure.Persistence/Repositories/ScrapPageRepository.cs
@@ -1,8 +1,8 @@
 using Core.Application.Interface.Repositories;
 using Core.Domain.Entities.WebScraping;
-using Infraestructure.Persistence.Context;
+using Infrastructure.Persistence.Context;
 
-namespace Infraestructure.Persistence.Repositories
+namespace Infrastructure.Persistence.Repositories
 {
     public class ScrapPageRepository : GenericRepository<ScrapPage>, IScrapPageRepository
     {