Changeset ceaed42 for trip-planner/src/main/java/finki/diplomska/tripplanner/service

Timestamp:

11/11/21 12:59:26 (3 years ago)

Author:

Ema <ema_spirova@…>

Branches:

master

Children:

59329aa

Parents:

6c1585f

Message:

add location to planner

Location:

trip-planner/src/main/java/finki/diplomska/tripplanner/service

Files:

• LocationService.java (modified) (2 diffs)
• PlannerService.java (modified) (1 diff)
• impl/LocationServiceImpl.java (modified) (3 diffs)
• impl/PlannerServiceImpl.java (modified) (4 diffs)

trip-planner/src/main/java/finki/diplomska/tripplanner/service/LocationService.java

@@ -3 +3 @@
 
 import finki.diplomska.tripplanner.models.Location;
+import finki.diplomska.tripplanner.models.Planner;
+import finki.diplomska.tripplanner.models.dto.PlannerLocationDto;
 
 import java.util.List;
@@ -17 +19 @@
     List<Location> findLocationsFromCityForm(Long cityId, Long companionId, Long lengthOfStay, String categoryIds);
     List<Location> findLocationsFromRegionForm(Long regionId, Long companionId, Long lengthOfStay, String categoryIds);
+    Location addLocationToPlanner(PlannerLocationDto plannerLocationDto);
+    List<Location> getAllLocationsForPlanner(Long plannerId);
+
 }

trip-planner/src/main/java/finki/diplomska/tripplanner/service/PlannerService.java

@@ -14 +14 @@
         List<Planner> getAllPlaners();
         Optional<Planner> findById(Long id);
-        Planner addLocationToPlanner(Long plannerId, Long locationId);
         Planner editPlannerWithRequestParams(Long id, String description, String name, List<Location> locationList);
         Optional<Planner> newPlanner(PlannerDto plannerDto);

trip-planner/src/main/java/finki/diplomska/tripplanner/service/impl/LocationServiceImpl.java

@@ -2 +2 @@
 
 import finki.diplomska.tripplanner.models.Location;
+import finki.diplomska.tripplanner.models.Planner;
+import finki.diplomska.tripplanner.models.dto.PlannerLocationDto;
 import finki.diplomska.tripplanner.models.exceptions.LocationNotFoundException;
 import finki.diplomska.tripplanner.repository.jpa.JpaLocationRepository;
+import finki.diplomska.tripplanner.repository.jpa.JpaPlannerRepository;
 import finki.diplomska.tripplanner.service.LocationService;
 import org.springframework.stereotype.Service;
@@ -15 +18 @@
 
     private final JpaLocationRepository locationRepository;
+    private final JpaPlannerRepository plannerRepository;
 
-    public LocationServiceImpl(JpaLocationRepository locationRepository) {
+    public LocationServiceImpl(JpaLocationRepository locationRepository, JpaPlannerRepository plannerRepository) {
         this.locationRepository = locationRepository;
+        this.plannerRepository = plannerRepository;
     }
 
@@ -150 +155 @@
     }
 
+    @Override
+    public Location addLocationToPlanner(PlannerLocationDto plannerLocationDto) {
+        Location location = this.locationRepository.findById(plannerLocationDto.getLocationId())
+                .orElseThrow(() -> new LocationNotFoundException(plannerLocationDto.getLocationId()));
+        Planner planner = this.plannerRepository.getById(plannerLocationDto.getPlannerId());
+        planner.getLocationList().add(location);
+        return this.locationRepository.save(location);
+    }
+
+    @Override
+    public List<Location> getAllLocationsForPlanner(Long plannerId) {
+        return this.locationRepository.getAllLocationsForPlanner(plannerId);
+    }
+
+
 }

trip-planner/src/main/java/finki/diplomska/tripplanner/service/impl/PlannerServiceImpl.java

@@ -22 +22 @@
     private final JpaPlannerRepository plannerRepository;
     private final JpaLocationRepository locationRepository;
-    private final LocationService locationService;
 
-    public PlannerServiceImpl(JpaPlannerRepository plannerRepository, JpaLocationRepository locationRepository, LocationService locationService) {
+    public PlannerServiceImpl(JpaPlannerRepository plannerRepository, JpaLocationRepository locationRepository) {
         this.plannerRepository = plannerRepository;
         this.locationRepository = locationRepository;
-        this.locationService = locationService;
     }
 
@@ -40 +38 @@
     }
 
-    @Override
-    public Planner addLocationToPlanner(Long plannerId, Long locationId) {
-        Location location = this.locationService.findById(locationId)
-                .orElseThrow(() -> new LocationNotFoundException(locationId));
-        Planner planner = this.plannerRepository.getById(plannerId);
-        planner.getLocationList().add(location);
-        return this.plannerRepository.save(planner);
-    }
 
     @Override
@@ -84 +74 @@
         planner.setName(plannerDto.getName());
         planner.setDescription(plannerDto.getDescription());
-
+/*
         List<Location> locationList  = new ArrayList<>();
         for(Long location : plannerDto.getLocationList()){
@@ -92 +82 @@
         }
         planner.setLocationList(locationList);
+
+ */
         return Optional.of(this.plannerRepository.save(planner));
     }