package de.techdev.trackr.domain.employee.vacation;

import de.techdev.trackr.domain.employee.vacation.support.VacationRequestEmployeeToDaysTotalService;
import lombok.extern.slf4j.Slf4j;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.http.HttpStatus;
import org.springframework.security.access.prepost.PreAuthorize;
import org.springframework.stereotype.Controller;
import org.springframework.web.bind.annotation.*;

import java.security.Principal;
import java.util.Date;
import java.util.Map;

/**
 * @author Moritz Schulze
 */
@Controller
@RequestMapping("/vacationRequests")
@Slf4j
public class VacationRequestController {

    @Autowired
    private VacationRequestApproveService vacationRequestApproveService;

    @Autowired
    private VacationRequestEmployeeToDaysTotalService vacationRequestEmployeeToDaysTotalService;

    /**
     * Approve a vacation request. Can only be done by supervisors, but they are not allowed to approve their own requests.
     * Only works on pending requests.
     *
     * @param vacationRequest The vacation request to approve
     * @param principal       The Spring Security principal to extract the user from.
     * @return The vacation request object.
     */
    @PreAuthorize("hasRole('ROLE_SUPERVISOR')")
    @RequestMapping(value = "/{id}/approve", method = RequestMethod.PUT)
    @ResponseStatus(HttpStatus.OK)
    @ResponseBody
    public VacationRequest approve(@PathVariable("id") VacationRequest vacationRequest, Principal principal) {
        return vacationRequestApproveService.approve(vacationRequest, principal.getName());
    }

    /**
     * Reject a vacation request. Can only be done by supervisors, but they are not allowed to reject their own requests.
     * Only works on pending requests.
     *
     * @param vacationRequest The vacation request to reject.
     * @param principal       The Spring Security principal to extract the user from.
     * @return The vacation request object.
     */
    @PreAuthorize("hasRole('ROLE_SUPERVISOR')")
    @RequestMapping(value = "/{id}/reject", method = RequestMethod.PUT)
    @ResponseStatus(HttpStatus.OK)
    @ResponseBody
    public VacationRequest reject(@PathVariable("id") VacationRequest vacationRequest, Principal principal) {
        return vacationRequestApproveService.reject(vacationRequest, principal.getName());
    }

    /**
     * Create a mapping of employee names to the effective number of vacation days between a start and end date.
     *
     * All approved vacation requests that coincide with the period are collected. Only the cut of every vacation request with the period is used for the calculation.
     * Public holidays and weekends are exluded.
     */
    @PreAuthorize("hasRole('ROLE_ADMIN')")
    @ResponseBody
    @RequestMapping(value = "/daysPerEmployeeBetween", method = RequestMethod.GET, produces = "application/hal+json")
    @ResponseStatus(HttpStatus.OK)
    public Map<String, Integer> daysPerEmployeeBetween(
            @RequestParam("start") Date start,
            @RequestParam("end") Date end
    ) {
        return vacationRequestEmployeeToDaysTotalService.mapVacationRequestsToTotalDays(start, end);
    }
}

@@ -2722,16 +2722,17 @@
 s are ex
+c
 luded.%0A 

fix typo in JavaDoc