Trying to make csharp like the usage of uow in tourservice

EspadaKomanda · Nov 17, 2024 · f10e09a · f10e09a
1 parent dd22010
commit f10e09a
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 11 deletions.
diff --git a/EntertaimentService/Services/EntertaimentService/EntertaimentService.cs b/EntertaimentService/Services/EntertaimentService/EntertaimentService.cs
@@ -64,7 +64,7 @@ public IQueryable<EntertaimentDto> GetEntertaiments(GetEntertaimentsRequest getE
                 IQueryable<Entertaiment> entertaiments;
                 if(getEntertaiments.Categories!=null)
                 {
-                    var entertainments = _unitOfWork.Entertaiments.GetAll()
+                    entertainments = _unitOfWork.Entertaiments.GetAll()
                         .Where(e => 
                             _unitOfWork.EntertaimentCategories.GetAll()
                                 .Where(ec => getEntertaiments.Categories.Contains(ec.CategoryId))
@@ -81,7 +81,7 @@ public IQueryable<EntertaimentDto> GetEntertaiments(GetEntertaimentsRequest getE
                 }
                 else
                 {
-                    var entertainments = _unitOfWork.Entertaiments.GetAll()
+                    entertainments = _unitOfWork.Entertaiments.GetAll()
                         .Where(e => 
                             e.Rating >= getEntertaiments.MinimalRating && e.Rating <= getEntertaiments.MaximalRating &&
                             e.Price >= getEntertaiments.MinimalPrice && e.Price <= getEntertaiments.MaximalPrice &&

diff --git a/TourService/Services/TourService/TourService.cs b/TourService/Services/TourService/TourService.cs
@@ -125,23 +125,22 @@ public IQueryable<TourDto> GetTours(GetToursRequest getTours)
                 {
                     tours = _unitOfWork.Tours.GetAll()
                         .Where(t => 
-                            _unitOfWork.TourCategories.GetAll()
-                                .Where(tc => getTours.Categories.Contains(tc.CategoryId))
-                                .Select(tc => tc.TourId)
-                                .Contains(t.Id)
-                        )
-                        .Where(t => t.Rating >= getTours.MinimalRating && t.Rating <= getTours.MaximalRating)
-                        .Where(t => t.Price >= getTours.MinimalPrice && t.Price <= getTours.MaximalPrice)
-                        .Distinct()
+                            t.Rating >= getTours.MinimalRating && 
+                            t.Rating <= getTours.MaximalRating && 
+                            t.Price >= getTours.MinimalPrice && 
+                            t.Price <= getTours.MaximalPrice &&
+                            _unitOfWork.TourCategories.GetAll(new FindOptions())
+                                .Any(tc => tc.TourId == t.Id && getTours.Categories.Contains(tc.CategoryId)))
                         .Skip((getTours.Page - 1) * 10)
                         .Take(10)
                         .ToList();
+
                 }
                 else if(getTours.TourTags!=null)
                 {
                     tours = _unitOfWork.Tours.GetAll()
                         .Where(t => 
-                            _unitOfWork.TourTags.GetAll()
+                            _unitOfWork.TourTags.GetAll(new FindOptions())
                                 .Where(tt => getTours.TourTags.Contains(tt.TagId))
                                 .Select(tt => tt.TourId)
                                 .Contains(t.Id)