Invis1ble/assistant

<?php
 
namespace AppBundle\Controller;
 
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\Form\Form;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Security;
use FOS\RestBundle\Controller\FOSRestController;
use FOS\RestBundle\Controller\Annotations;
use FOS\RestBundle\View\View;
use Nelmio\ApiDocBundle\Annotation\ApiDoc;
 
use AppBundle\Entity\Task;
use AppBundle\EntityCollection\TaskPeriodCollection;
use AppBundle\Form\Type\TaskPeriodFormType;
 
/**
 * TaskPeriodController
 *
 * @author     Max Invis1ble
 * @copyright  (c) 2016, Max Invis1ble
 * @license    http://www.opensource.org/licenses/mit-license.php MIT
 */
class TaskPeriodController extends FOSRestController
{
    /**
     * List task's periods
     *
     * @ApiDoc(
     *     resource = true,
     *     requirements = {
     *         {
     *             "name" = "id",
     *             "dataType" = "UUID string",
     *             "description" = "ID of the task for which periods are requested"
     *         }
     *     },
     *     headers = {
     *         {
     *             "name" = "Authorization",
     *             "default" = "Bearer ",
     *             "description" = "api_doc.jwt",
     *         }
     *     },
     *     statusCodes = {
     *         200 = "Returned when successful",
     *         401 = "Returned when unauthorized",
     *         403 = "Returned when not permitted",
     *         404 = "Returned when the task is not found"
     *     }
     * )
     *
     * @Annotations\Route(path="tasks/{id}/periods")
     *
     * @Security("is_granted('period_list', task)")
     *
     * @Annotations\View()
     *
     * @param Task $task
     *
     * @return TaskPeriodCollection
     */
    public function getPeriodsAction(Task $task): TaskPeriodCollection
    {
        return new TaskPeriodCollection(
            $task->getPeriods()->toArray(),
            $task
        );
    }
 
    /**
     * Creates a new period from the submitted data.
     *
     * @ApiDoc(
     *     input = {
     *         "class" = "AppBundle\Form\Type\TaskPeriodFormType",
     *         "name" = ""
     *     },
     *     requirements = {
     *         {
     *             "name" = "id",
     *             "dataType" = "UUID string",
     *             "description" = "ID of the task for which period is created"
     *         }
     *     },
     *     headers = {
     *         {
     *             "name" = "Authorization",
     *             "default" = "Bearer ",
     *             "description" = "api_doc.jwt",
     *         }
     *     },
     *     statusCodes = {
     *         201 = "Returned when a new period is created",
     *         400 = "Returned when the form has errors",
     *         401 = "Returned when unauthorized",
     *         403 = "Returned when not permitted",
     *         404 = "Returned when the task is not found"
     *     }
     * )
     *
     * @Annotations\Route(path="tasks/{id}/periods")
     *
     * @Security("is_granted('period_create', task)")
     *
     * @Annotations\View()
     *
     * @param Request $request
     * @param Task    $task
     *
     * @return View|Form
     */
Similar blocks of code found in 3 locations. Consider refactoring.
    public function postPeriodAction(Request $request, Task $task)
    {
        $periodManager = $this->get('app.manager.period_manager');
        $period = $periodManager->createPeriod();
        $period->setTask($task);
 
        $form = $this->createForm(TaskPeriodFormType::class, $period);
        $form->submit(json_decode($request->getContent(), true));
 
        if ($form->isValid()) {
            $periodManager->saveAndFlush($period);
 
            return $this->routeRedirectView('api_get_period', [
                'id' => $period->getId(),
            ]);
        }
 
        return $form;
    }
}