Passed
Pull Request — master (#48)
by
unknown
12:49
created
src/DataTables/LogDataTable.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -102,8 +102,8 @@  discard block
 block discarded – undo
102 102
         $optionsResolver->setAllowedTypes('filter_elements', ['array', 'object']);
103 103
         $optionsResolver->setAllowedTypes('mode', 'string');
104 104
 
105
-        $optionsResolver->setNormalizer('filter_elements', function (Options $options, $value) {
106
-            if (! is_array($value)) {
105
+        $optionsResolver->setNormalizer('filter_elements', function(Options $options, $value) {
106
+            if (!is_array($value)) {
107 107
                 return [$value];
108 108
             }
109 109
 
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
 
122 122
         $dataTable->add('symbol', TextColumn::class, [
123 123
             'label' => '',
124
-            'render' => function ($value, AbstractLogEntry $context) {
124
+            'render' => function($value, AbstractLogEntry $context) {
125 125
                 switch ($context->getLevelString()) {
126 126
                     case LogLevel::DEBUG:
127 127
                         $symbol = 'fa-bug';
@@ -182,7 +182,7 @@  discard block
 block discarded – undo
182 182
         $dataTable->add('type', TextColumn::class, [
183 183
             'label' => $this->translator->trans('log.type'),
184 184
             'propertyPath' => 'type',
185
-            'render' => function (string $value, AbstractLogEntry $context) {
185
+            'render' => function(string $value, AbstractLogEntry $context) {
186 186
                 return $this->translator->trans('log.type.'.$value);
187 187
             },
188 188
         ]);
@@ -191,14 +191,14 @@  discard block
 block discarded – undo
191 191
             'label' => $this->translator->trans('log.level'),
192 192
             'visible' => 'system_log' === $options['mode'],
193 193
             'propertyPath' => 'levelString',
194
-            'render' => function (string $value, AbstractLogEntry $context) {
194
+            'render' => function(string $value, AbstractLogEntry $context) {
195 195
                 return $value;
196 196
             },
197 197
         ]);
198 198
 
199 199
         $dataTable->add('user', TextColumn::class, [
200 200
             'label' => $this->translator->trans('log.user'),
201
-            'render' => function ($value, AbstractLogEntry $context) {
201
+            'render' => function($value, AbstractLogEntry $context) {
202 202
                 $user = $context->getUser();
203 203
 
204 204
                 return sprintf(
@@ -212,7 +212,7 @@  discard block
 block discarded – undo
212 212
         $dataTable->add('target_type', TextColumn::class, [
213 213
             'label' => $this->translator->trans('log.target_type'),
214 214
             'visible' => false,
215
-            'render' => function ($value, AbstractLogEntry $context) {
215
+            'render' => function($value, AbstractLogEntry $context) {
216 216
                 $class = $context->getTargetClass();
217 217
                 if (null !== $class) {
218 218
                     return $this->elementTypeNameGenerator->getLocalizedTypeLabel($class);
@@ -234,7 +234,7 @@  discard block
 block discarded – undo
234 234
         $dataTable->add('timeTravel', IconLinkColumn::class, [
235 235
             'label' => '',
236 236
             'icon' => 'fas fa-fw fa-eye',
237
-            'href' => function ($value, AbstractLogEntry $context) {
237
+            'href' => function($value, AbstractLogEntry $context) {
238 238
                 if (
239 239
                     ($context instanceof TimeTravelInterface
240 240
                         && $context->hasOldDataInformations())
@@ -252,10 +252,10 @@  discard block
 block discarded – undo
252 252
 
253 253
                 return null;
254 254
             },
255
-            'disabled' => function ($value, AbstractLogEntry $context) {
255
+            'disabled' => function($value, AbstractLogEntry $context) {
256 256
                 return
257
-                    ! $this->security->isGranted('@tools.timetravel')
258
-                    || ! $this->security->isGranted('show_history', $context->getTargetClass());
257
+                    !$this->security->isGranted('@tools.timetravel')
258
+                    || !$this->security->isGranted('show_history', $context->getTargetClass());
259 259
             },
260 260
         ]);
261 261
 
@@ -267,7 +267,7 @@  discard block
 block discarded – undo
267 267
 
268 268
         $dataTable->createAdapter(ORMAdapter::class, [
269 269
             'entity' => AbstractLogEntry::class,
270
-            'query' => function (QueryBuilder $builder) use ($options): void {
270
+            'query' => function(QueryBuilder $builder) use ($options): void {
271 271
                 $this->getQuery($builder, $options);
272 272
             },
273 273
         ]);
@@ -293,7 +293,7 @@  discard block
 block discarded – undo
293 293
             ]);
294 294
         }
295 295
 
296
-        if (! empty($options['filter_elements'])) {
296
+        if (!empty($options['filter_elements'])) {
297 297
             foreach ($options['filter_elements'] as $element) {
298 298
                 /** @var AbstractDBElement $element */
299 299
 
Please login to merge, or discard this patch.
src/Services/Attachments/AttachmentURLGenerator.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -126,11 +126,11 @@
 block discarded – undo
126 126
      */
127 127
     public function getThumbnailURL(Attachment $attachment, string $filter_name = 'thumbnail_sm'): string
128 128
     {
129
-        if (! $attachment->isPicture()) {
129
+        if (!$attachment->isPicture()) {
130 130
             throw new InvalidArgumentException('Thumbnail creation only works for picture attachments!');
131 131
         }
132 132
 
133
-        if ($attachment->isExternal() && ! empty($attachment->getURL())) {
133
+        if ($attachment->isExternal() && !empty($attachment->getURL())) {
134 134
             return $attachment->getURL();
135 135
         }
136 136
 
Please login to merge, or discard this patch.
src/Entity/LogSystem/SecurityEventLogEntry.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -59,7 +59,7 @@
 block discarded – undo
59 59
 
60 60
     public function setTargetElement(?AbstractDBElement $element): AbstractLogEntry
61 61
     {
62
-        if (! $element instanceof User) {
62
+        if (!$element instanceof User) {
63 63
             throw new \InvalidArgumentException('Target element must be a User object!');
64 64
         }
65 65
 
Please login to merge, or discard this patch.
src/Entity/UserSystem/User.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -508,7 +508,7 @@  discard block
 block discarded – undo
508 508
     {
509 509
         $tmp = $this->getFirstName();
510 510
         //Dont add a space, if the name has only one part (it would look strange)
511
-        if (! empty($this->getFirstName()) && ! empty($this->getLastName())) {
511
+        if (!empty($this->getFirstName()) && !empty($this->getLastName())) {
512 512
             $tmp .= ' ';
513 513
         }
514 514
         $tmp .= $this->getLastName();
@@ -530,7 +530,7 @@  discard block
 block discarded – undo
530 530
     public function setName(string $new_name): AbstractNamedDBElement
531 531
     {
532 532
         // Anonymous user is not allowed to change its username
533
-        if (! $this->isAnonymousUser()) {
533
+        if (!$this->isAnonymousUser()) {
534 534
             $this->name = $new_name;
535 535
         }
536 536
 
Please login to merge, or discard this patch.