Completed
Push — master ( 4bb011...4b99be )
by Guillaume
13:23
created
src/Starkerxp/UserBundle/Controller/UserController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
             return new JsonResponse([]);
65 65
         }
66 66
         $retour = array_map(
67
-            function ($element) use ($manager, $options) {
67
+            function($element) use ($manager, $options) {
68 68
                 return $manager->toArray($element, $this->getFields($options['fields']));
69 69
             },
70 70
             $resultSets
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
184 184
             return new JsonResponse(["payload" => $this->translate("user.entity.not_found", "user")], 404);
185 185
         }
186 186
         // Un user ne peut modifier un autre user sauf si ce dernier est un super admin.
187
-        if($this->getUser()->getId() !=  $user->getId() && !$this->isGranted("ROLE_SUPER_ADMIN")){
187
+        if ($this->getUser()->getId() != $user->getId() && !$this->isGranted("ROLE_SUPER_ADMIN")) {
188 188
             return new JsonResponse(["payload" => $this->translate("user.entity.not_updated_is_not_admin", "user")], 400);
189 189
         }
190 190
         $manager->beginTransaction();
Please login to merge, or discard this patch.
src/Starkerxp/UserBundle/Entity/User.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -88,7 +88,7 @@
 block discarded – undo
88 88
      */
89 89
     public function setRoles($roles)
90 90
     {
91
-        if($roles instanceof RoleUser || empty($this->roles) ){
91
+        if ($roles instanceof RoleUser || empty($this->roles)) {
92 92
             $this->roles = new RoleUser($roles);
93 93
             return true;
94 94
         }
Please login to merge, or discard this patch.
src/Starkerxp/UserBundle/Tests/Manager/UserManagerTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
      */
25 25
     public function testFindAll()
26 26
     {
27
-        $this->loadFixtureFiles(['@StarkerxpUserBundle/Tests/DataFixtures/UserManager/UserManager.yml',]);
27
+        $this->loadFixtureFiles(['@StarkerxpUserBundle/Tests/DataFixtures/UserManager/UserManager.yml', ]);
28 28
         $this->assertCount(10, $this->manager->findAll());
29 29
     }
30 30
 
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
      */
35 35
     public function testInsertNewUser()
36 36
     {
37
-        $this->loadFixtureFiles(['@StarkerxpUserBundle/Tests/DataFixtures/UserManager/DefaultUser.yml',]);
37
+        $this->loadFixtureFiles(['@StarkerxpUserBundle/Tests/DataFixtures/UserManager/DefaultUser.yml', ]);
38 38
         $user = new User('[email protected]', ["ROLE_SUPER_ADMIN"]);
39 39
         $user->setPlainPassword("azerty");
40 40
         $this->manager->insert($user);
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
      */
48 48
     public function testUpdateUser()
49 49
     {
50
-        $this->loadFixtureFiles(['@StarkerxpUserBundle/Tests/DataFixtures/UserManager/DefaultUser.yml',]);
50
+        $this->loadFixtureFiles(['@StarkerxpUserBundle/Tests/DataFixtures/UserManager/DefaultUser.yml', ]);
51 51
         $criteria = ['createdAt' => new \DateTime("2017-05-20 00:31:47")];
52 52
         /** @var User $user */
53 53
         $user = $this->manager->findOneBy($criteria);
Please login to merge, or discard this patch.
src/Starkerxp/CampaignBundle/Form/Type/EventType.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
                 'class'         => Campaign::class,
35 35
                 'multiple'      => false,
36 36
                 'required'      => false,
37
-                'query_builder' => function (CampaignRepository $repository) {
37
+                'query_builder' => function(CampaignRepository $repository) {
38 38
                     return $repository->getQueryListe();
39 39
                 },
40 40
                 'constraints'   => [
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
                 'class'         => Template::class,
52 52
                 'multiple'      => false,
53 53
                 'required'      => false,
54
-                'query_builder' => function (TemplateRepository $repository) {
54
+                'query_builder' => function(TemplateRepository $repository) {
55 55
                     return $repository->getQueryListe();
56 56
                 },
57 57
                 'constraints'   => [
Please login to merge, or discard this patch.
src/Starkerxp/CampaignBundle/Tests/Controller/CampaignControllerTest.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@  discard block
 block discarded – undo
14 14
      */
15 15
     public function testPostValide()
16 16
     {
17
-        $this->loadFixtureFiles(['@StarkerxpUserBundle/Tests/DataFixtures/UserManager/DefaultUser.yml',]);
17
+        $this->loadFixtureFiles(['@StarkerxpUserBundle/Tests/DataFixtures/UserManager/DefaultUser.yml', ]);
18 18
         $data = [
19 19
             'name' => "Ma première campaign",
20 20
         ];
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
      */
38 38
     public function testPostInvalide()
39 39
     {
40
-        $this->loadFixtureFiles(['@StarkerxpUserBundle/Tests/DataFixtures/UserManager/DefaultUser.yml',]);
40
+        $this->loadFixtureFiles(['@StarkerxpUserBundle/Tests/DataFixtures/UserManager/DefaultUser.yml', ]);
41 41
         $client = $this->getAuthClient();
42 42
         $client->request('POST', '/api/campaigns', []);
43 43
         $response = $client->getResponse();
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
         $manager = $this->getContainer()->get('starkerxp_campaign.manager.campaign');
62 62
         $listeCampaigns = $manager->getRepository()->findAll();
63 63
         $this->assertCount(1, $listeCampaigns);
64
-        $campaignDepart = $manager->toArray($listeCampaigns[0], ['name']);// Exemple
64
+        $campaignDepart = $manager->toArray($listeCampaigns[0], ['name']); // Exemple
65 65
         $data = [
66 66
             'name' => "Mon nom", //exemple
67 67
         ];
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
         $manager->clear();
73 73
         $campaigns = $manager->findAll();
74 74
         $this->assertCount(1, $campaigns);
75
-        $campaignFinal = $manager->toArray($campaigns[0], ['name']);// Exemple
75
+        $campaignFinal = $manager->toArray($campaigns[0], ['name']); // Exemple
76 76
         $this->assertNotEquals($campaignDepart, $campaignFinal);
77 77
         $this->assertEquals($data, $campaignFinal);
78 78
     }
Please login to merge, or discard this patch.