@@ -94,7 +94,7 @@ |
||
94 | 94 | * |
95 | 95 | * @return Response |
96 | 96 | */ |
97 | - public function new(Request $request, EntityManagerInterface $em, EntityImporter $importer, ?Supplier $entity = null): Response |
|
97 | + public function new(Request $request, EntityManagerInterface $em, EntityImporter $importer, ?Supplier $entity = null) : Response |
|
98 | 98 | { |
99 | 99 | return $this->_new($request, $em, $importer, $entity); |
100 | 100 | } |
@@ -86,7 +86,7 @@ |
||
86 | 86 | * |
87 | 87 | * @return Response |
88 | 88 | */ |
89 | - public function new(Request $request, EntityManagerInterface $em, EntityImporter $importer, ?Group $entity = null): Response |
|
89 | + public function new(Request $request, EntityManagerInterface $em, EntityImporter $importer, ?Group $entity = null) : Response |
|
90 | 90 | { |
91 | 91 | return $this->_new($request, $em, $importer, $entity); |
92 | 92 | } |
@@ -50,12 +50,12 @@ |
||
50 | 50 | |
51 | 51 | //Check if the user has set a timezone |
52 | 52 | $user = $this->security->getUser(); |
53 | - if ($user instanceof User && ! empty($user->getTimezone())) { |
|
53 | + if ($user instanceof User && !empty($user->getTimezone())) { |
|
54 | 54 | $timezone = $user->getTimezone(); |
55 | 55 | } |
56 | 56 | |
57 | 57 | //Fill with default value if needed |
58 | - if (null === $timezone && ! empty($this->default_timezone)) { |
|
58 | + if (null === $timezone && !empty($this->default_timezone)) { |
|
59 | 59 | $timezone = $this->default_timezone; |
60 | 60 | } |
61 | 61 |
@@ -74,15 +74,15 @@ discard block |
||
74 | 74 | $user = $this->security->getUser(); |
75 | 75 | $request = $event->getRequest(); |
76 | 76 | |
77 | - if (! $event->isMasterRequest()) { |
|
77 | + if (!$event->isMasterRequest()) { |
|
78 | 78 | return; |
79 | 79 | } |
80 | - if (! $user instanceof User) { |
|
80 | + if (!$user instanceof User) { |
|
81 | 81 | return; |
82 | 82 | } |
83 | 83 | |
84 | 84 | //Abort if we dont need to redirect the user. |
85 | - if (! $user->isNeedPwChange() && ! static::TFARedirectNeeded($user)) { |
|
85 | + if (!$user->isNeedPwChange() && !static::TFARedirectNeeded($user)) { |
|
86 | 86 | return; |
87 | 87 | } |
88 | 88 | |
@@ -125,7 +125,7 @@ discard block |
||
125 | 125 | { |
126 | 126 | $tfa_enabled = $user->isU2FAuthEnabled() || $user->isGoogleAuthenticatorEnabled(); |
127 | 127 | |
128 | - if (null !== $user->getGroup() && $user->getGroup()->isEnforce2FA() && ! $tfa_enabled) { |
|
128 | + if (null !== $user->getGroup() && $user->getGroup()->isEnforce2FA() && !$tfa_enabled) { |
|
129 | 129 | return true; |
130 | 130 | } |
131 | 131 |
@@ -80,9 +80,9 @@ |
||
80 | 80 | public function onRegister(RegisterEvent $event): void |
81 | 81 | { |
82 | 82 | //Skip adding of U2F key on demo mode |
83 | - if (! $this->demo_mode) { |
|
83 | + if (!$this->demo_mode) { |
|
84 | 84 | $user = $event->getUser(); |
85 | - if (! $user instanceof User) { |
|
85 | + if (!$user instanceof User) { |
|
86 | 86 | throw new \InvalidArgumentException('Only User objects can be registered for U2F!'); |
87 | 87 | } |
88 | 88 |
@@ -269,11 +269,11 @@ discard block |
||
269 | 269 | { |
270 | 270 | unset($fields['lastModified']); |
271 | 271 | |
272 | - if (! $this->hasFieldRestrictions($element)) { |
|
272 | + if (!$this->hasFieldRestrictions($element)) { |
|
273 | 273 | return $fields; |
274 | 274 | } |
275 | 275 | |
276 | - return array_filter($fields, function ($value, $key) use ($element) { |
|
276 | + return array_filter($fields, function($value, $key) use ($element) { |
|
277 | 277 | //Associative array (save changed data) case |
278 | 278 | if (is_string($key)) { |
279 | 279 | return $this->shouldFieldBeSaved($element, $key); |
@@ -287,7 +287,7 @@ discard block |
||
287 | 287 | { |
288 | 288 | $uow = $em->getUnitOfWork(); |
289 | 289 | |
290 | - if (! $logEntry instanceof ElementEditedLogEntry && ! $logEntry instanceof ElementDeletedLogEntry) { |
|
290 | + if (!$logEntry instanceof ElementEditedLogEntry && !$logEntry instanceof ElementDeletedLogEntry) { |
|
291 | 291 | throw new \InvalidArgumentException('$logEntry must be ElementEditedLogEntry or ElementDeletedLogEntry!'); |
292 | 292 | } |
293 | 293 | |
@@ -300,7 +300,7 @@ discard block |
||
300 | 300 | $old_data = $this->filterFieldRestrictions($entity, $old_data); |
301 | 301 | |
302 | 302 | //Restrict length of string fields, to save memory... |
303 | - $old_data = array_map(function ($value) { |
|
303 | + $old_data = array_map(function($value) { |
|
304 | 304 | if (is_string($value)) { |
305 | 305 | return mb_strimwidth($value, 0, self::MAX_STRING_LENGTH, '...'); |
306 | 306 | } |
@@ -319,7 +319,7 @@ discard block |
||
319 | 319 | protected function validEntity(object $entity): bool |
320 | 320 | { |
321 | 321 | //Dont log logentries itself! |
322 | - if ($entity instanceof AbstractDBElement && ! $entity instanceof AbstractLogEntry) { |
|
322 | + if ($entity instanceof AbstractDBElement && !$entity instanceof AbstractLogEntry) { |
|
323 | 323 | return true; |
324 | 324 | } |
325 | 325 |
@@ -182,7 +182,7 @@ discard block |
||
182 | 182 | $this->addFlash('info', 'part.edited_flash'); |
183 | 183 | //Reload form, so the SIUnitType entries use the new part unit |
184 | 184 | $form = $this->createForm(PartBaseType::class, $part); |
185 | - } elseif ($form->isSubmitted() && ! $form->isValid()) { |
|
185 | + } elseif ($form->isSubmitted() && !$form->isValid()) { |
|
186 | 186 | $this->addFlash('error', 'part.edited_flash.invalid'); |
187 | 187 | } |
188 | 188 | |
@@ -226,7 +226,7 @@ discard block |
||
226 | 226 | * @return Response |
227 | 227 | */ |
228 | 228 | public function new(Request $request, EntityManagerInterface $em, TranslatorInterface $translator, |
229 | - AttachmentSubmitHandler $attachmentSubmitHandler, ?Part $part = null): Response |
|
229 | + AttachmentSubmitHandler $attachmentSubmitHandler, ?Part $part = null) : Response |
|
230 | 230 | { |
231 | 231 | if (null === $part) { |
232 | 232 | $new_part = new Part(); |
@@ -304,7 +304,7 @@ discard block |
||
304 | 304 | return $this->redirectToRoute('part_edit', ['id' => $new_part->getID()]); |
305 | 305 | } |
306 | 306 | |
307 | - if ($form->isSubmitted() && ! $form->isValid()) { |
|
307 | + if ($form->isSubmitted() && !$form->isValid()) { |
|
308 | 308 | $this->addFlash('error', 'part.created_flash.invalid'); |
309 | 309 | } |
310 | 310 |
@@ -75,7 +75,7 @@ |
||
75 | 75 | |
76 | 76 | public function buildForm(FormBuilderInterface $builder, array $options): void |
77 | 77 | { |
78 | - $builder->addEventListener(FormEvents::PRE_SET_DATA, function (FormEvent $event) use ($options): void { |
|
78 | + $builder->addEventListener(FormEvents::PRE_SET_DATA, function(FormEvent $event) use ($options): void { |
|
79 | 79 | $data = $event->getData(); |
80 | 80 | $config = $event->getForm()->getConfig(); |
81 | 81 | //If enabled do a reindexing of the collection |
@@ -106,7 +106,7 @@ discard block |
||
106 | 106 | 'required' => false, |
107 | 107 | 'label' => 'attachment.edit.secure_file', |
108 | 108 | 'mapped' => false, |
109 | - 'disabled' => ! $this->security->isGranted('@parts_attachments.show_private'), |
|
109 | + 'disabled' => !$this->security->isGranted('@parts_attachments.show_private'), |
|
110 | 110 | 'attr' => [ |
111 | 111 | 'class' => 'form-control-sm', |
112 | 112 | ], |
@@ -134,7 +134,7 @@ discard block |
||
134 | 134 | 'required' => false, |
135 | 135 | 'label' => 'attachment.edit.download_url', |
136 | 136 | 'mapped' => false, |
137 | - 'disabled' => ! $this->allow_attachments_download, |
|
137 | + 'disabled' => !$this->allow_attachments_download, |
|
138 | 138 | 'attr' => [ |
139 | 139 | 'class' => 'form-control-sm', |
140 | 140 | ], |
@@ -163,7 +163,7 @@ discard block |
||
163 | 163 | //Check the secure file checkbox, if file is in securefile location |
164 | 164 | $builder->get('secureFile')->addEventListener( |
165 | 165 | FormEvents::PRE_SET_DATA, |
166 | - function (FormEvent $event): void { |
|
166 | + function(FormEvent $event): void { |
|
167 | 167 | $attachment = $event->getForm()->getParent()->getData(); |
168 | 168 | if ($attachment instanceof Attachment) { |
169 | 169 | $event->setData($attachment->isSecure()); |