Move getAllCities() from CountryController to CityController

core/openapi.yaml

@@ -799,7 +799,7 @@ paths:
   /api/cities:
     get:
       tags:
-        - Country
+        - City
       summary: Get all cities.
       description: Returns an array of objects representing cities.
       operationId: getAllCities

core/src/main/java/cz/muni/fi/pa165/core/rest/CityController.java

@@ -7,6 +7,8 @@ import cz.muni.fi.pa165.core.model.NewCityDtoRequest;
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.RestController;
 
+import java.util.List;
+
 @RestController
 public class CityController implements CityApiDelegate {
 
@@ -78,6 +80,18 @@ public class CityController implements CityApiDelegate {
         return CityApiDelegate.super.getCityById(id);
     }
 
+    /**
+     * GET /api/cities : Get all cities.
+     * Returns an array of objects representing cities.
+     *
+     * @return OK (status code 200)
+     * @see CityApi#getAllCities
+     */
+    @Override
+    public ResponseEntity<List<CityDto>> getAllCities() {
+        return CityApiDelegate.super.getAllCities();
+    }
+
     /**
      * PUT /api/cities/{id} : Update city by id.
      * Updates a city by id and returns it as a response.

core/src/main/java/cz/muni/fi/pa165/core/rest/CountryController.java

@@ -3,7 +3,6 @@ package cz.muni.fi.pa165.core.rest;
 import cz.muni.fi.pa165.core.api.CountryApi;
 import cz.muni.fi.pa165.core.api.CountryApiDelegate;
 import cz.muni.fi.pa165.core.facade.country.CountryFacade;
-import cz.muni.fi.pa165.core.model.CityDto;
 import cz.muni.fi.pa165.core.model.CountryDto;
 import cz.muni.fi.pa165.core.model.NewCountryDtoRequest;
 import org.springframework.beans.factory.annotation.Autowired;
@@ -89,10 +88,4 @@ public class CountryController implements CountryApiDelegate {
     public ResponseEntity<CountryDto> updateCountry(Long id, NewCountryDtoRequest newCountryDtoRequest) {
         return ResponseEntity.ok(countryFacade.update(id, newCountryDtoRequest));
     }
-
-    @Override
-    public ResponseEntity<List<CityDto>> getAllCities() {
-        // TODO: Implement me
-        return CountryApiDelegate.super.getAllCities();
-    }
 }