LeadController - Code Metrics - Inspection of "Gestion de base des leads" - starker-xp/lapero.net - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 778ba1...db4111 )

by Guillaume

created 2017-07-21 23:48 UTC

LeadController A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	209
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	7

Importance

Changes

Metric	Value
wmc	16
lcom	1
cbo	7
dl	209
loc	209
rs	10
c	0
b	0
f	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	cgetAction()	21	21	3
A	getAction()	16	16	3
A	postAction()	21	21	3
B	putAction()	22	22	4
A	deleteAction()	16	16	3

How to fix Duplicated Code

<?php

namespace Starkerxp\LeadBundle\Controller;

use Nelmio\ApiDocBundle\Annotation\ApiDoc;
use Starkerxp\LeadBundle\Entity\LeadAction;
use Starkerxp\LeadBundle\Events;
use Starkerxp\LeadBundle\Form\Type\LeadType;
use Starkerxp\StructureBundle\Controller\StructureController;
use Symfony\Component\EventDispatcher\GenericEvent;
use Symfony\Component\HttpFoundation\JsonResponse;
use Symfony\Component\HttpFoundation\Request;


class LeadController extends StructureController

{
	/**
     * @ApiDoc(
     *      resource=true,
     *      description="Liste les leads.",
     *      section="starkerxp_lead.lead",
     *      parameters={
     *          {
     *              "name"="offset",
     *              "dataType"="integer",
     *              "requirement"="\d+",
     *              "description"="starkerxp_structure.doc.offset.result",
     *              "required"="false"
     *          },
     *          {
     *              "name"="limit",
     *              "dataType"="integer",
     *              "requirement"="\d+",
     *              "description"="starkerxp_structure.doc.limit.result",
     *              "required"="false"
     *          },
     *          {
     *              "name"="fields",
     *              "dataType"="string",
     *              "requirement"="\w+",
     *              "description"="starkerxp_structure.doc.list_field.entity",
     *              "required"="false"
     *          },
     *          {
     *              "name"="sort",
     *              "dataType"="string",
     *              "requirement"="\w+",
     *              "description"="starkerxp_structure.doc.sort.result",
     *              "required"="false"
     *          }
     *      },
     *      views = { "default" }
     * )
     * 
     */
    public function cgetAction(Request $request)
    {
        $manager = $this->get("starkerxp_lead.manager.lead");
        try {
            $options = $this->resolveParams()->resolve($request->query->all());
            $orderBy = $this->getOrderBy($options['sort']);
            $resultSets = $manager->findBy([], $orderBy, $options['limit'], $options['offset']);
        } catch (\Exception $e) {
            return new JsonResponse(["payload" => $e->getMessage()], 400);
        }
        if (empty($resultSets)) {
            return new JsonResponse([]);
        }
        $retour = array_map(
            function ($element) use ($manager, $options) {
                return $manager->toArray($element, $this->getFields($options['fields']));
            },
            $resultSets
        );
        return new JsonResponse($retour);
    }

	
	/**
     * @ApiDoc(
     *      resource=true,
     *      description="Affiche un lead.",
     *      section="starkerxp_lead.lead",
	 *      requirements={
     *          {
     *              "name"="lead_id",
     *              "dataType"="integer",
     *              "requirement"="\d+",
     *              "description"="Permet d'afficher l'élément choisis"
     *          }
     *      },
     *      parameters={
     *          {
     *              "name"="fields",
     *              "dataType"="string",
     *              "requirement"="\w+",
     *              "description"="starkerxp_structure.doc.list_field.entity",
     *              "required"="false"
     *          }
     *      },
     *      views = { "default" }
     * )
     */
    public function getAction(Request $request)
    {
        $manager = $this->get("starkerxp_lead.manager.lead");
        try {
            $options = $this->resolveParams()->resolve($request->query->all());
			if (!$entite = $manager->findOneBy(['id' => $request->get('lead_id')])) {
                return new JsonResponse(["payload" => $this->translate("entity.not_found", "lead")], 404);
            }
        } catch (\Exception $e) {
            return new JsonResponse(["payload" => $e->getMessage()], 400);
        }
        
        $retour = $manager->toArray($entite, $this->getFields($options['fields']));

        return new JsonResponse($retour);
    }

	/**
     * @ApiDoc(
     *      resource=true,
     *      description="Ajoute un lead.",
     *      section="starkerxp_lead.lead",
     *      views = { "default" }
     * )
     */
    public function postAction(Request $request)
    {
        $manager = $this->get("starkerxp_lead.manager.lead");
        try {
            $entite = new Lead();
            $form = $this->createForm(LeadType::class, $entite, ['method' => 'POST']);
            $form->submit($this->getRequestData($request));
            if ($form->isValid()) {
                $lead = $form->getData();
                $lead->setUuid($this->getUuid());
                $manager->insert($lead);
				$this->dispatch(Events::LEAD_CREATED, new GenericEvent($entite));
                return new JsonResponse(["payload" => $this->translate("leadaction.entity.created", "lead")], 201);
            }
        } catch (\Exception $e) {
            $manager->rollback();
            return new JsonResponse(["payload" => $e->getMessage()], 400);
        }

        return new JsonResponse(["payload" => $this->getFormErrors($form)], 400);
    }

	/**
     * @ApiDoc(
     *      resource=true,
     *      description="Modifie un lead.",
     *      section="starkerxp_lead.lead",
	 *      requirements={
     *          {
     *              "name"="lead_id",
     *              "dataType"="integer",
     *              "requirement"="\d+",
     *              "description"="Permet de modifier l'élément choisi."
     *          }
     *      },
     *      views = { "default" }
     * )
     */
    public function putAction(Request $request)
    {
        $manager = $this->get("starkerxp_lead.manager.lead");
		if (!$entite = $manager->findOneBy(['id' => $request->get('lead_id')])) {
			return new JsonResponse(["payload" => $this->translate("entity.not_found", "lead")], 404);
		}
        $manager->beginTransaction();
        try {
            $form = $this->createForm(LeadType::class, $entite, ['method' => 'PUT']);
            $form->submit($this->getRequestData($request));
            if ($form->isValid()) {
                $entite = $form->getData();
                $manager->update($lead);

				$this->dispatch(Events::LEAD_UPDATED, new GenericEvent($entite));
                return new JsonResponse(["payload" => $this->translate("leadaction.entity.updated", "lead")], 204);
            }
        } catch (\Exception $e) {
            $manager->rollback();
            return new JsonResponse(["payload" => $e->getMessage()], 400);
        }
        return new JsonResponse(["payload" => $this->getFormErrors($form)], 400);
    }

	/**
     * @ApiDoc(
     *      resource=true,
     *      description="Supprime un lead.",
     *      section="starkerxp_lead.lead",
	 *      requirements={
     *          {
     *              "name"="lead_id",
     *              "dataType"="integer",
     *              "requirement"="\d+",
     *              "description"="Permet de supprimer l'élément choisi."
     *          }
     *      },
     *      views = { "default" }
     * )
     */
    public function deleteAction(Request $request)
    {
        $manager = $this->get("starkerxp_lead.manager.lead");
        if (!$entite = $manager->findOneBy(['id' => $request->get('lead_id')])) {
			return new JsonResponse(["payload" => $this->translate("entity.not_found", "lead")], 404);
		}
        try {
            $manager->delete($entite);
        } catch (\Exception $e) {
            $manager->rollback();
            return new JsonResponse(["payload" => $e->getMessage()], 400);
        }
		$this->dispatch(Events::LEAD_DELETED, new GenericEvent($request->get('lead_id')));
		
        return new JsonResponse(["payload" => $this->translate("leadaction.entity.deleted", "lead")], 204);
    }

} 



1		<?php
2
3		namespace Starkerxp\LeadBundle\Controller;
4
5		use Nelmio\ApiDocBundle\Annotation\ApiDoc;
6		use Starkerxp\LeadBundle\Entity\LeadAction;
7		use Starkerxp\LeadBundle\Events;
8		use Starkerxp\LeadBundle\Form\Type\LeadType;
9		use Starkerxp\StructureBundle\Controller\StructureController;
10		use Symfony\Component\EventDispatcher\GenericEvent;
11		use Symfony\Component\HttpFoundation\JsonResponse;
12		use Symfony\Component\HttpFoundation\Request;
13
14
15	View Code Duplication	class LeadController extends StructureController
		0 ignored issues – show Duplication introduced 2017-05-10 00:39 UTC by Report Bug Copy Issue Report This class seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
16		{
17		/**
18		* @ApiDoc(
19		* resource=true,
20		* description="Liste les leads.",
21		* section="starkerxp_lead.lead",
22		* parameters={
23		* {
24		* "name"="offset",
25		* "dataType"="integer",
26		* "requirement"="\d+",
27		* "description"="starkerxp_structure.doc.offset.result",
28		* "required"="false"
29		* },
30		* {
31		* "name"="limit",
32		* "dataType"="integer",
33		* "requirement"="\d+",
34		* "description"="starkerxp_structure.doc.limit.result",
35		* "required"="false"
36		* },
37		* {
38		* "name"="fields",
39		* "dataType"="string",
40		* "requirement"="\w+",
41		* "description"="starkerxp_structure.doc.list_field.entity",
42		* "required"="false"
43		* },
44		* {
45		* "name"="sort",
46		* "dataType"="string",
47		* "requirement"="\w+",
48		* "description"="starkerxp_structure.doc.sort.result",
49		* "required"="false"
50		* }
51		* },
52		* views = { "default" }
53		* )
54		*
55		*/
56		public function cgetAction(Request $request)
57		{
58		$manager = $this->get("starkerxp_lead.manager.lead");
59		try {
60		$options = $this->resolveParams()->resolve($request->query->all());
61		$orderBy = $this->getOrderBy($options['sort']);
62		$resultSets = $manager->findBy([], $orderBy, $options['limit'], $options['offset']);
63		} catch (\Exception $e) {
64		return new JsonResponse(["payload" => $e->getMessage()], 400);
65		}
66		if (empty($resultSets)) {
67		return new JsonResponse([]);
68		}
69		$retour = array_map(
70		function ($element) use ($manager, $options) {
71		return $manager->toArray($element, $this->getFields($options['fields']));
72		},
73		$resultSets
74		);
75		return new JsonResponse($retour);
76		}
77
78
79		/**
80		* @ApiDoc(
81		* resource=true,
82		* description="Affiche un lead.",
83		* section="starkerxp_lead.lead",
84		* requirements={
85		* {
86		* "name"="lead_id",
87		* "dataType"="integer",
88		* "requirement"="\d+",
89		* "description"="Permet d'afficher l'élément choisis"
90		* }
91		* },
92		* parameters={
93		* {
94		* "name"="fields",
95		* "dataType"="string",
96		* "requirement"="\w+",
97		* "description"="starkerxp_structure.doc.list_field.entity",
98		* "required"="false"
99		* }
100		* },
101		* views = { "default" }
102		* )
103		*/
104		public function getAction(Request $request)
105		{
106		$manager = $this->get("starkerxp_lead.manager.lead");
107		try {
108		$options = $this->resolveParams()->resolve($request->query->all());
109		if (!$entite = $manager->findOneBy(['id' => $request->get('lead_id')])) {
110		return new JsonResponse(["payload" => $this->translate("entity.not_found", "lead")], 404);
111		}
112		} catch (\Exception $e) {
113		return new JsonResponse(["payload" => $e->getMessage()], 400);
114		}
115
116		$retour = $manager->toArray($entite, $this->getFields($options['fields']));
117
118		return new JsonResponse($retour);
119		}
120
121		/**
122		* @ApiDoc(
123		* resource=true,
124		* description="Ajoute un lead.",
125		* section="starkerxp_lead.lead",
126		* views = { "default" }
127		* )
128		*/
129		public function postAction(Request $request)
130		{
131		$manager = $this->get("starkerxp_lead.manager.lead");
132		try {
133		$entite = new Lead();
134		$form = $this->createForm(LeadType::class, $entite, ['method' => 'POST']);
135		$form->submit($this->getRequestData($request));
136		if ($form->isValid()) {
137		$lead = $form->getData();
138		$lead->setUuid($this->getUuid());
139		$manager->insert($lead);
140		$this->dispatch(Events::LEAD_CREATED, new GenericEvent($entite));
141		return new JsonResponse(["payload" => $this->translate("leadaction.entity.created", "lead")], 201);
142		}
143		} catch (\Exception $e) {
144		$manager->rollback();
145		return new JsonResponse(["payload" => $e->getMessage()], 400);
146		}
147
148		return new JsonResponse(["payload" => $this->getFormErrors($form)], 400);
149		}
150
151		/**
152		* @ApiDoc(
153		* resource=true,
154		* description="Modifie un lead.",
155		* section="starkerxp_lead.lead",
156		* requirements={
157		* {
158		* "name"="lead_id",
159		* "dataType"="integer",
160		* "requirement"="\d+",
161		* "description"="Permet de modifier l'élément choisi."
162		* }
163		* },
164		* views = { "default" }
165		* )
166		*/
167		public function putAction(Request $request)
168		{
169		$manager = $this->get("starkerxp_lead.manager.lead");
170		if (!$entite = $manager->findOneBy(['id' => $request->get('lead_id')])) {
171		return new JsonResponse(["payload" => $this->translate("entity.not_found", "lead")], 404);
172		}
173		$manager->beginTransaction();
174		try {
175		$form = $this->createForm(LeadType::class, $entite, ['method' => 'PUT']);
176		$form->submit($this->getRequestData($request));
177		if ($form->isValid()) {
178		$entite = $form->getData();
179		$manager->update($lead);
		0 ignored issues – show Bug introduced 2017-07-21 23:51 UTC by Report Bug Copy Issue Report The variable `$lead` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history...
180		$this->dispatch(Events::LEAD_UPDATED, new GenericEvent($entite));
181		return new JsonResponse(["payload" => $this->translate("leadaction.entity.updated", "lead")], 204);
182		}
183		} catch (\Exception $e) {
184		$manager->rollback();
185		return new JsonResponse(["payload" => $e->getMessage()], 400);
186		}
187		return new JsonResponse(["payload" => $this->getFormErrors($form)], 400);
188		}
189
190		/**
191		* @ApiDoc(
192		* resource=true,
193		* description="Supprime un lead.",
194		* section="starkerxp_lead.lead",
195		* requirements={
196		* {
197		* "name"="lead_id",
198		* "dataType"="integer",
199		* "requirement"="\d+",
200		* "description"="Permet de supprimer l'élément choisi."
201		* }
202		* },
203		* views = { "default" }
204		* )
205		*/
206		public function deleteAction(Request $request)
207		{
208		$manager = $this->get("starkerxp_lead.manager.lead");
209		if (!$entite = $manager->findOneBy(['id' => $request->get('lead_id')])) {
210		return new JsonResponse(["payload" => $this->translate("entity.not_found", "lead")], 404);
211		}
212		try {
213		$manager->delete($entite);
214		} catch (\Exception $e) {
215		$manager->rollback();
216		return new JsonResponse(["payload" => $e->getMessage()], 400);
217		}
218		$this->dispatch(Events::LEAD_DELETED, new GenericEvent($request->get('lead_id')));
219
220		return new JsonResponse(["payload" => $this->translate("leadaction.entity.deleted", "lead")], 204);
221		}
222
223		}
		0 ignored issues – show Coding Style introduced 2017-05-10 00:39 UTC by Report Bug Copy Issue Report As per coding style, files should not end with a newline character. This check marks files that end in a newline character, i.e. an empy line. Loading history...
224

starker-xp / lapero.net

Push — master ( 778ba1...db4111 )

LeadController A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

5 Methods

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like