Completed
Pull Request — develop (#359)
by Mathias
23:51
created
module/Core/src/Core/Entity/Snapshot.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@
 block discarded – undo
23 23
 {
24 24
     /**
25 25
      * @param $data
26
-     * @return mixed
26
+     * @return Snapshot
27 27
      * @ODM\PreUpdate
28 28
      */
29 29
     public function __invoke($data)
Please login to merge, or discard this patch.
module/Core/src/Core/Repository/AbstractRepository.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
     }
37 37
 
38 38
     /**
39
-     * @param $name
39
+     * @param string $name
40 40
      *
41 41
      * @return mixed
42 42
      */
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
     /**
60 60
      * @param array $data
61 61
      *
62
-     * @return mixed
62
+     * @return EntityInterface
63 63
      */
64 64
     public function create(array $data = null, $persist = false)
65 65
     {
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -130,7 +130,7 @@
 block discarded – undo
130 130
 
131 131
     protected function checkEntityType($entity)
132 132
     {
133
-        if ( !($entity instanceOf $this->entityPrototype) ) {
133
+        if (!($entity instanceOf $this->entityPrototype)) {
134 134
             throw new \InvalidArgumentException(sprintf(
135 135
                                                     'Entity must be of type %s but recieved %s instead',
136 136
                                                     get_class($this->entityPrototype),
Please login to merge, or discard this patch.
module/Core/src/Core/Repository/DoctrineMongoODM/PersistenceListener.php 1 patch
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -26,7 +26,6 @@
 block discarded – undo
26 26
      * Attach to an event manager
27 27
      *
28 28
      * @param  EventManagerInterface $events
29
-     * @param  integer $priority
30 29
      */
31 30
     public function attach(EventManagerInterface $events)
32 31
     {
Please login to merge, or discard this patch.
module/Organizations/src/Organizations/Controller/IndexController.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -250,6 +250,9 @@
 block discarded – undo
250 250
         return $container;
251 251
     }
252 252
 
253
+    /**
254
+     * @param string $script
255
+     */
253 256
     protected function getErrorViewModel($script)
254 257
     {
255 258
         $this->getResponse()->setStatusCode(Response::STATUS_CODE_500);
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
         try {
125 125
             /* @var $handler \Organizations\Controller\Plugin\GetOrganizationHandler */
126 126
             $handler = $this->plugin('Organizations/GetOrganizationHandler');
127
-            $org  = $handler->process($this->params(), true);
127
+            $org = $handler->process($this->params(), true);
128 128
         } catch (MissingParentOrganizationException $e) {
129 129
             return $this->getErrorViewModel('no-parent');
130 130
         } catch (NotFoundException $e) {
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
             ];
136 136
         }
137 137
 
138
-        $container       = $this->getFormular($org);
138
+        $container = $this->getFormular($org);
139 139
 
140 140
         if (isset($formIdentifier) && $request->isPost()) {
141 141
             /* @var $form \Zend\Form\FormInterface */
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
             $form = $container->get($formIdentifier);
161 161
             $form->setData(array_merge($postData, $filesData));
162 162
             if (!isset($form)) {
163
-                throw new \RuntimeException('No form found for "' . $formIdentifier . '"');
163
+                throw new \RuntimeException('No form found for "'.$formIdentifier.'"');
164 164
             }
165 165
             $isValid = $form->isValid();
166 166
 
Please login to merge, or discard this patch.
src/Organizations/Controller/Plugin/GetOrganizationHandler.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -59,7 +59,7 @@
 block discarded – undo
59 59
      * @param Params $params
60 60
      * @param bool   $allowDraft
61 61
      *
62
-     * @return object|\Organizations\Entity\Organization
62
+     * @return \Organizations\Entity\OrganizationInterface
63 63
      * @throws UnauthorizedAccessException
64 64
      * @throws \Doctrine\ODM\MongoDB\LockException
65 65
      * @throws NotFoundException
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -44,10 +44,10 @@  discard block
 block discarded – undo
44 44
      */
45 45
     protected $acl;
46 46
 
47
-    public function __construct(RepositoryService $repositoryService,AuthenticationService $auth, Acl $acl) {
48
-        $this->repositoryService=$repositoryService;
49
-        $this->auth=$auth;
50
-        $this->acl=$acl;
47
+    public function __construct(RepositoryService $repositoryService, AuthenticationService $auth, Acl $acl) {
48
+        $this->repositoryService = $repositoryService;
49
+        $this->auth = $auth;
50
+        $this->acl = $acl;
51 51
     }
52 52
 
53 53
     public function __invoke()
@@ -64,9 +64,9 @@  discard block
 block discarded – undo
64 64
      * @throws \Doctrine\ODM\MongoDB\LockException
65 65
      * @throws NotFoundException
66 66
      */
67
-    public function process(Params $params,$allowDraft = true)
67
+    public function process(Params $params, $allowDraft = true)
68 68
     {
69
-        $repositories   = $this->repositoryService;
69
+        $repositories = $this->repositoryService;
70 70
         /* @var \Organizations\Repository\Organization $organizationRepository */
71 71
         $organizationRepository = $this->repositoryService->get('Organizations/Organization');
72 72
 
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
         $user = $this->auth->getUser(); /* @var $user \Auth\Entity\UserInterface */
76 76
 
77 77
         /* @var $organizationId string */
78
-        $organizationId = empty($idFromRoute)?$idFromSubForm:$idFromRoute;
78
+        $organizationId = empty($idFromRoute) ? $idFromSubForm : $idFromRoute;
79 79
 
80 80
         $editOwnOrganization = '__my__' === $organizationId;
81 81
 
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
             return $organization;
111 111
         }
112 112
 
113
-        $organization      = $organizationRepository->find($organizationId);
113
+        $organization = $organizationRepository->find($organizationId);
114 114
         if (!$organization) {
115 115
             throw new NotFoundException($organizationId);
116 116
         }
Please login to merge, or discard this patch.
module/Settings/src/Settings/Form/FormAbstract.php 1 patch
Doc Comments   +8 added lines patch added patch discarded remove patch
@@ -31,6 +31,11 @@  discard block
 block discarded – undo
31 31
         return $this->hydrator;
32 32
     }
33 33
     
34
+    /**
35
+     * @param string $name
36
+     *
37
+     * @return callable
38
+     */
34 39
     protected function getPlugin($name)
35 40
     {
36 41
         $plugin = null;
@@ -69,6 +74,9 @@  discard block
 block discarded – undo
69 74
         $this->add($this->formManager->get('DefaultButtonsFieldset'));
70 75
     }
71 76
     
77
+    /**
78
+     * @return string
79
+     */
72 80
     abstract public function getCoreFieldset();
73 81
     
74 82
     /**
Please login to merge, or discard this patch.
module/Core/src/Core/Controller/AdminControllerEvent.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@
 block discarded – undo
61 61
      *
62 62
      * @return self
63 63
      */
64
-    public function addViewModel($name, $model, $priority=0)
64
+    public function addViewModel($name, $model, $priority = 0)
65 65
     {
66 66
         $this->models->insert($name, $model, $priority);
67 67
 
Please login to merge, or discard this patch.
module/Core/src/Core/Listener/NotificationListener.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -40,12 +40,12 @@
 block discarded – undo
40 40
      */
41 41
     public function attachShared(SharedEventManagerInterface $events)
42 42
     {
43
-        $events->attach('*', NotificationEvent::EVENT_NOTIFICATION_ADD, array($this,'add'), 1);
44
-        $events->attach('Zend\Mvc\Application', MvcEvent::EVENT_DISPATCH, array($this,'renderJSON'), -240);
45
-        $events->attach('Zend\Mvc\Application', MvcEvent::EVENT_DISPATCH, array($this,'renderHTML'), -250);
43
+        $events->attach('*', NotificationEvent::EVENT_NOTIFICATION_ADD, array($this, 'add'), 1);
44
+        $events->attach('Zend\Mvc\Application', MvcEvent::EVENT_DISPATCH, array($this, 'renderJSON'), -240);
45
+        $events->attach('Zend\Mvc\Application', MvcEvent::EVENT_DISPATCH, array($this, 'renderHTML'), -250);
46 46
         // Sometimes the Dispatch-Event is not reached, for instance with a route-direct
47 47
         // but also for Events, that are happening after the Dispatch
48
-        $events->attach('Zend\Mvc\Application', MvcEvent::EVENT_FINISH, array($this,'renderHTML'), -250);
48
+        $events->attach('Zend\Mvc\Application', MvcEvent::EVENT_FINISH, array($this, 'renderHTML'), -250);
49 49
         return $this;
50 50
     }
51 51
 
Please login to merge, or discard this patch.
module/Core/src/Core/Form/View/Helper/FormElement.php 1 patch
Indentation   -1 removed lines patch added patch discarded remove patch
@@ -1,6 +1,5 @@
 block discarded – undo
1 1
 <?php
2 2
 /**
3
-
4 3
  */
5 4
 
6 5
 namespace Core\Form\View\Helper;
Please login to merge, or discard this patch.