Completed
Pull Request — develop (#501)
by ANTHONIUS
10:57
created
module/Core/src/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
@@ -120,7 +120,7 @@
 block discarded – undo
120 120
         return $this;
121 121
     }
122 122
 
123
-    public function remove($entity, $flush=false)
123
+    public function remove($entity, $flush = false)
124 124
     {
125 125
         $this->checkEntityType($entity);
126 126
         $this->dm->remove($entity);
Please login to merge, or discard this patch.
Repository/DoctrineMongoODM/Event/AbstractUpdatePermissionsSubscriber.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -108,6 +108,9 @@
 block discarded – undo
108 108
         return $entities;
109 109
     }
110 110
     
111
+    /**
112
+     * @return string
113
+     */
111 114
     protected function getRepositoryName()
112 115
     {
113 116
         if (0 === strpos($this->repositoryName, '\\')) {
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -98,7 +98,7 @@
 block discarded – undo
98 98
         $repository     = $dm->getRepository($repositoryName);
99 99
         
100 100
         $criteria = array(
101
-            'permissions.assigned.' . $resourceId => array(
101
+            'permissions.assigned.'.$resourceId => array(
102 102
                 '$exists' => true
103 103
             )
104 104
         );
Please login to merge, or discard this patch.
module/Core/src/Repository/Filter/AbstractPaginationQuery.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@
 block discarded – undo
31 31
 
32 32
     /**
33 33
      * @param mixed $value
34
-     * @param null $queryBuilder
34
+     * @param \Doctrine\ODM\MongoDB\Query\Builder $queryBuilder
35 35
      * @return mixed
36 36
      * @throws \DomainException
37 37
      */
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@
 block discarded – undo
43 43
         
44 44
         if ($this->repositoryName) {
45 45
             $nameParts = explode('/', $this->repositoryName);
46
-            $entityClass = $nameParts[0] . '\\Entity\\' . $nameParts[1];
46
+            $entityClass = $nameParts[0].'\\Entity\\'.$nameParts[1];
47 47
             $queryBuilder->find($entityClass);
48 48
         }
49 49
         return $this->createQuery($value, $queryBuilder);
Please login to merge, or discard this patch.
module/Core/src/Service/Config.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -33,6 +33,9 @@
 block discarded – undo
33 33
         return $this;
34 34
     }
35 35
     
36
+    /**
37
+     * @param string $name
38
+     */
36 39
     public function setNamespace($name)
37 40
     {
38 41
         $this->namespace = $name;
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -89,11 +89,11 @@  discard block
 block discarded – undo
89 89
             $config    = $this->serviceManager->get('Config');
90 90
             $appConfig = $this->serviceManager->get('ApplicationConfig');
91 91
             foreach ($appConfig['modules'] as $module) {
92
-                $this->config[$module] = array_key_exists($module, $config)?$config[$module]:array();
92
+                $this->config[$module] = array_key_exists($module, $config) ? $config[$module] : array();
93 93
             }
94 94
         }
95 95
         if (isset($namespace)) {
96
-            return array_key_exists($namespace, $this->config)?$this->config[$namespace]: array();
96
+            return array_key_exists($namespace, $this->config) ? $this->config[$namespace] : array();
97 97
         }
98 98
         return $this->config;
99 99
     }
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
         }
107 107
         if (!empty($this->map[$key])) {
108 108
             $module          = $this->getNamespace();
109
-            return isset($this->map[$key][$module])?$this->map[$key][$module]:array();
109
+            return isset($this->map[$key][$module]) ? $this->map[$key][$module] : array();
110 110
         }
111 111
         return array();
112 112
     }
Please login to merge, or discard this patch.
module/Core/src/Service/EntityEraser/AbstractDependenciesListener.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -79,6 +79,9 @@
 block discarded – undo
79 79
         return $this->dependencyCheck($event);
80 80
     }
81 81
 
82
+    /**
83
+     * @param \Core\Entity\EntityInterface $entity
84
+     */
82 85
     private function checkEntityClasses($entity)
83 86
     {
84 87
         if (empty($this->entityClasses)) {
Please login to merge, or discard this patch.
module/Core/src/Service/EntityEraser/DependencyResult.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -131,6 +131,9 @@
 block discarded – undo
131 131
         return $this->description;
132 132
     }
133 133
 
134
+    /**
135
+     * @param string $mode
136
+     */
134 137
     public function isMode($mode)
135 138
     {
136 139
         return $this->mode == $mode;
Please login to merge, or discard this patch.
module/Core/src/Service/RestClient.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -88,7 +88,7 @@
 block discarded – undo
88 88
 
89 89
     /**
90 90
      * @throws \RuntimeException
91
-     * @return mixed
91
+     * @return ZendClient
92 92
      */
93 93
     protected function authetificate()
94 94
     {
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -92,8 +92,8 @@
 block discarded – undo
92 92
      */
93 93
     protected function authetificate()
94 94
     {
95
-        $auth = array_key_exists('user', $this->config)?$this->config['user']:'';
96
-        $pass = array_key_exists('pass', $this->config)?$this->config['pass']:'';
95
+        $auth = array_key_exists('user', $this->config) ? $this->config['user'] : '';
96
+        $pass = array_key_exists('pass', $this->config) ? $this->config['pass'] : '';
97 97
         return $this->setAuth($auth, $pass);
98 98
     }
99 99
 
Please login to merge, or discard this patch.
module/Core/src/View/Helper/AbstractEventsHelper.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@
 block discarded – undo
20 20
      * Set the View object
21 21
      *
22 22
      * @param  Renderer $view
23
-     * @return AbstractHelper
23
+     * @return AbstractEventsHelper
24 24
      */
25 25
     public function setView(Renderer $view)
26 26
     {
Please login to merge, or discard this patch.
module/Core/src/View/Helper/Alert.php 3 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -160,7 +160,7 @@
 block discarded – undo
160 160
     /**
161 161
      * Starts content capturing.
162 162
      *
163
-     * @param string|array $type
163
+     * @param string|null $type
164 164
      * @param array        $options see {@łink render()} for information.
165 165
      *
166 166
      * @return self
Please login to merge, or discard this patch.
Indentation   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -52,7 +52,6 @@  discard block
 block discarded – undo
52 52
  * @method warning()
53 53
  * @method success()
54 54
  * @method danger()
55
-
56 55
  *
57 56
 *@author Mathias Gelhausen <[email protected]>
58 57
  */
@@ -149,7 +148,7 @@  discard block
 block discarded – undo
149 148
         ) {
150 149
             $class .= ' alert-dismissable';
151 150
             $content = '<button type="button" class="close" data-dismiss="alert">&times;</button>'
152
-                     . '<span class="notification-content">' . $content . '</span>';
151
+                        . '<span class="notification-content">' . $content . '</span>';
153 152
         }
154 153
 
155 154
         $target = array_key_exists('target', $options)?' target="' . $options['target'] . '"':'';
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -142,18 +142,18 @@  discard block
 block discarded – undo
142 142
             return $this->start($type, $options);
143 143
         }
144 144
 
145
-        $id    = isset($options['id']) ? ' id="' . $options['id'] . '"' : '';
146
-        $class = isset($options['class']) ? ' ' . $options['class'] : '';
145
+        $id    = isset($options['id']) ? ' id="'.$options['id'].'"' : '';
146
+        $class = isset($options['class']) ? ' '.$options['class'] : '';
147 147
         if ((isset($options['dismissable']) && $options['dismissable'])
148 148
             || !isset($options['dismissable'])
149 149
         ) {
150 150
             $class .= ' alert-dismissable';
151 151
             $content = '<button type="button" class="close" data-dismiss="alert">&times;</button>'
152
-                     . '<span class="notification-content">' . $content . '</span>';
152
+                     . '<span class="notification-content">'.$content.'</span>';
153 153
         }
154 154
 
155
-        $target = array_key_exists('target', $options)?' target="' . $options['target'] . '"':'';
156
-        $markup = '<div ' . $id . ' class="alert alert-' . $type . $class . '" ' . $target . '>' . $content . '</div>' . PHP_EOL;
155
+        $target = array_key_exists('target', $options) ? ' target="'.$options['target'].'"' : '';
156
+        $markup = '<div '.$id.' class="alert alert-'.$type.$class.'" '.$target.'>'.$content.'</div>'.PHP_EOL;
157 157
         return $markup;
158 158
     }
159 159
 
@@ -191,7 +191,7 @@  discard block
 block discarded – undo
191 191
     public function __call($method, $args)
192 192
     {
193 193
         if (!in_array($method, array(self::TYPE_INFO, self::TYPE_DANGER, self::TYPE_SUCCESS, self::TYPE_WARNING))) {
194
-            throw new \BadMethodCallException('Unknown method: ' . $method);
194
+            throw new \BadMethodCallException('Unknown method: '.$method);
195 195
         }
196 196
 
197 197
         // We know that $method is one of the valid types, so we can safely use it as "type"-parameter for the
Please login to merge, or discard this patch.