Completed
Pull Request — develop (#246)
by ANTHONIUS
07:21
created
module/Core/src/Core/Mail/MailService.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -123,7 +123,7 @@
 block discarded – undo
123 123
     /**
124 124
      * Gets the default from address
125 125
      *
126
-     * @return null|String|Address|AddressList|array
126
+     * @return string
127 127
      */
128 128
     public function getFrom()
129 129
     {
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
         $self = $this;
88 88
 
89 89
         $this->addInitializer(
90
-            function ($instance) use ($self) {
90
+            function($instance) use ($self) {
91 91
                 if ($instance instanceof TranslatorAwareInterface) {
92 92
                     $translator = $self->getServiceLocator()->get('translator');
93 93
                     $instance->setTranslator($translator);
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
             false
101 101
         );
102 102
         $this->addInitializer(
103
-            function ($instance) {
103
+            function($instance) {
104 104
                 if (method_exists($instance, 'setServiceLocator')) {
105 105
                     $instance->setServiceLocator($this);
106 106
                 }
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
             false
109 109
         );
110 110
         $this->addInitializer(
111
-            function ($instance) {
111
+            function($instance) {
112 112
                 if (method_exists($instance, 'init')) {
113 113
                     $instance->init();
114 114
                 }
@@ -199,11 +199,11 @@  discard block
 block discarded – undo
199 199
         if ($this->overrideRecipient instanceof AddressList) {
200 200
             $originalRecipient = $headers->get('to')->toString();
201 201
             if ($headers->has('cc')) {
202
-                $originalRecipient .= '; ' . $headers->get('cc')->toString();
202
+                $originalRecipient .= '; '.$headers->get('cc')->toString();
203 203
                 $headers->removeHeader('cc');
204 204
             }
205 205
             if ($headers->has('bcc')) {
206
-                $originalRecipient .= '; ' . $headers->get('bcc')->toString();
206
+                $originalRecipient .= '; '.$headers->get('bcc')->toString();
207 207
                 $headers->removeHeader('bcc');
208 208
             }
209 209
             $headers->addHeaderLine('X-Original-Recipients', $originalRecipient);
Please login to merge, or discard this patch.
module/Core/src/Core/ModuleManager/ModuleConfigLoader.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -34,6 +34,9 @@
 block discarded – undo
34 34
     {
35 35
     }
36 36
     
37
+    /**
38
+     * @param string $directory
39
+     */
37 40
     public static function load($directory)
38 41
     {
39 42
         $directory = rtrim($directory, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR;
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@
 block discarded – undo
36 36
     
37 37
     public static function load($directory)
38 38
     {
39
-        $directory = rtrim($directory, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR;
39
+        $directory = rtrim($directory, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR;
40 40
         $env       = getenv('APPLICATION_ENV') ?: 'production';
41 41
         $pattern   = sprintf('%s{,*.}{config,%s}.php', $directory, $env);
42 42
         $config    = array();
Please login to merge, or discard this patch.
module/Core/src/Core/Paginator/PaginatorFactoryAbstract.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@
 block discarded – undo
24 24
 
25 25
     /**
26 26
      * @param ServiceLocatorInterface $serviceLocator
27
-     * @return mixed|Paginator
27
+     * @return Paginator
28 28
      */
29 29
     public function createService(ServiceLocatorInterface $serviceLocator)
30 30
     {
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@
 block discarded – undo
50 50
         $repository     = $repositories->get($this->getRepository());
51 51
         $queryBuilder   = $repository->createQueryBuilder();
52 52
         $filter         = $serviceLocator->getServiceLocator()->get('filterManager')->get($this->getFilter());
53
-        $adapter       = new \Core\Paginator\Adapter\DoctrineMongoLateCursor($queryBuilder, $filter, $this->options);
53
+        $adapter = new \Core\Paginator\Adapter\DoctrineMongoLateCursor($queryBuilder, $filter, $this->options);
54 54
         $service        = new Paginator($adapter);
55 55
 
56 56
         $this->setCreationOptions([]);
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/Core/Repository/DoctrineMongoODM/PersistenceListener.php 1 patch
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -29,7 +29,6 @@
 block discarded – undo
29 29
      * Attach to an event manager
30 30
      *
31 31
      * @param  EventManagerInterface $events
32
-     * @param  integer $priority
33 32
      */
34 33
     public function attach(EventManagerInterface $events)
35 34
     {
Please login to merge, or discard this patch.
module/Core/src/Core/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/Core/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/Core/View/Helper/AbstractEventsHelper.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
      * Set the View object
26 26
      *
27 27
      * @param  Renderer $view
28
-     * @return AbstractHelper
28
+     * @return AbstractEventsHelper
29 29
      */
30 30
     public function setView(Renderer $view)
31 31
     {
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
     /**
37 37
      * Get the view object
38 38
      *
39
-     * @return null|Renderer
39
+     * @return Renderer
40 40
      */
41 41
     public function getView()
42 42
     {
Please login to merge, or discard this patch.
module/Core/src/Core/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.