Part-DB / Part-DB-symfony

src/Form/UserAdminForm.php

Spacing +16 added lines, -16 removed lines

@@ -94,7 +94,7 @@ 
                 'attr' => [
                     'placeholder' => 'user.username.placeholder',
                 ],
-                'disabled' => ! $this->security->isGranted('edit_username', $entity),
+                'disabled' => !$this->security->isGranted('edit_username', $entity),
             ])
 
             ->add('group', StructuralEntityType::class, [
@@ -102,7 +102,7 @@ 
                 'required' => false,
                 'label' => 'group.label',
                 'disable_not_selectable' => true,
-                'disabled' => ! $this->security->isGranted('change_group', $entity),
+                'disabled' => !$this->security->isGranted('change_group', $entity),
             ])
 
             ->add('first_name', TextType::class, [
@@ -112,7 +112,7 @@ 
                     'placeholder' => 'user.firstName.placeholder',
                 ],
                 'required' => false,
-                'disabled' => ! $this->security->isGranted('edit_infos', $entity),
+                'disabled' => !$this->security->isGranted('edit_infos', $entity),
             ])
 
             ->add('last_name', TextType::class, [
@@ -122,7 +122,7 @@ 
                     'placeholder' => 'user.lastName.placeholder',
                 ],
                 'required' => false,
-                'disabled' => ! $this->security->isGranted('edit_infos', $entity),
+                'disabled' => !$this->security->isGranted('edit_infos', $entity),
             ])
 
             ->add('email', TextType::class, [
@@ -132,7 +132,7 @@ 
                     'placeholder' => 'user.email.placeholder',
                 ],
                 'required' => false,
-                'disabled' => ! $this->security->isGranted('edit_infos', $entity),
+                'disabled' => !$this->security->isGranted('edit_infos', $entity),
             ])
 
             ->add('department', TextType::class, [
@@ -142,7 +142,7 @@ 
                     'placeholder' => 'user.department.placeholder',
                 ],
                 'required' => false,
-                'disabled' => ! $this->security->isGranted('edit_infos', $entity),
+                'disabled' => !$this->security->isGranted('edit_infos', $entity),
             ])
 
             //Config section
@@ -155,7 +155,7 @@ 
                 'placeholder' => 'user_settings.language.placeholder',
                 'label' => 'user.language_select',
                 'preferred_choices' => ['en', 'de'],
-                'disabled' => ! $this->security->isGranted('change_user_settings', $entity),
+                'disabled' => !$this->security->isGranted('change_user_settings', $entity),
             ])
             ->add('timezone', TimezoneType::class, [
                 'required' => false,
@@ -166,12 +166,12 @@ 
                 'placeholder' => 'user_settings.timezone.placeholder',
                 'label' => 'user.timezone.label',
                 'preferred_choices' => ['Europe/Berlin'],
-                'disabled' => ! $this->security->isGranted('change_user_settings', $entity),
+                'disabled' => !$this->security->isGranted('change_user_settings', $entity),
             ])
             ->add('theme', ChoiceType::class, [
                 'required' => false,
                 'choices' => User::AVAILABLE_THEMES,
-                'choice_label' => function ($entity, $key, $value) {
+                'choice_label' => function($entity, $key, $value) {
                     return $value;
                 },
                 'attr' => [
@@ -179,12 +179,12 @@ 
                 ],
                 'placeholder' => 'user_settings.theme.placeholder',
                 'label' => 'user.theme.label',
-                'disabled' => ! $this->security->isGranted('change_user_settings', $entity),
+                'disabled' => !$this->security->isGranted('change_user_settings', $entity),
             ])
             ->add('currency', CurrencyEntityType::class, [
                 'required' => false,
                 'label' => 'user.currency.label',
-                'disabled' => ! $this->security->isGranted('change_user_settings', $entity),
+                'disabled' => !$this->security->isGranted('change_user_settings', $entity),
             ])
 
             ->add('new_password', RepeatedType::class, [
@@ -198,7 +198,7 @@ 
                 'invalid_message' => 'password_must_match',
                 'required' => false,
                 'mapped' => false,
-                'disabled' => ! $this->security->isGranted('set_password', $entity),
+                'disabled' => !$this->security->isGranted('set_password', $entity),
                 'constraints' => [new Length([
                     'min' => 6,
                     'max' => 128,
@@ -211,7 +211,7 @@ 
                     'class' => 'checkbox-custom',
                 ],
                 'label' => 'user.edit.needs_pw_change',
-                'disabled' => ! $this->security->isGranted('set_password', $entity),
+                'disabled' => !$this->security->isGranted('set_password', $entity),
             ])
 
             ->add('disabled', CheckboxType::class, [
@@ -220,7 +220,7 @@ 
                     'class' => 'checkbox-custom',
                 ],
                 'label' => 'user.edit.user_disabled',
-                'disabled' => ! $this->security->isGranted('set_password', $entity)
+                'disabled' => !$this->security->isGranted('set_password', $entity)
                     || $entity === $this->security->getUser(),
             ])
 
@@ -228,7 +228,7 @@ 
             ->add('permissions', PermissionsType::class, [
                 'mapped' => false,
                 'data' => $builder->getData(),
-                'disabled' => ! $this->security->isGranted('edit_permissions', $entity),
+                'disabled' => !$this->security->isGranted('edit_permissions', $entity),
             ])
         ;
         /*->add('comment', CKEditorType::class, ['required' => false,
@@ -247,7 +247,7 @@ 
                 'data_class' => $options['attachment_class'],
             ],
             'by_reference' => false,
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
         ]);
 
         $builder->add('log_comment', TextType::class, [

src/DataTables/LogDataTable.php

Spacing +15 added lines, -15 removed lines

@@ -111,7 +111,7 @@ 
 
         $dataTable->add('symbol', TextColumn::class, [
             'label' => '',
-            'render' => function ($value, AbstractLogEntry $context) {
+            'render' => function($value, AbstractLogEntry $context) {
                 switch ($context->getLevelString()) {
                     case LogLevel::DEBUG:
                         $symbol = 'fa-bug';
@@ -172,7 +172,7 @@ 
         $dataTable->add('type', TextColumn::class, [
             'label' => $this->translator->trans('log.type'),
             'propertyPath' => 'type',
-            'render' => function (string $value, AbstractLogEntry $context) {
+            'render' => function(string $value, AbstractLogEntry $context) {
                 return $this->translator->trans('log.type.'.$value);
             },
         ]);
@@ -181,14 +181,14 @@ 
             'label' => $this->translator->trans('log.level'),
             'visible' => $options['mode'] === 'system_log',
             'propertyPath' => 'levelString',
-            'render' => function (string $value, AbstractLogEntry $context) {
+            'render' => function(string $value, AbstractLogEntry $context) {
                 return $value;
             },
         ]);
 
         $dataTable->add('user', TextColumn::class, [
             'label' => $this->translator->trans('log.user'),
-            'render' => function ($value, AbstractLogEntry $context) {
+            'render' => function($value, AbstractLogEntry $context) {
                 $user = $context->getUser();
 
                 return sprintf(
@@ -202,7 +202,7 @@ 
         $dataTable->add('target_type', TextColumn::class, [
             'label' => $this->translator->trans('log.target_type'),
             'visible' => false,
-            'render' => function ($value, AbstractLogEntry $context) {
+            'render' => function($value, AbstractLogEntry $context) {
                 $class = $context->getTargetClass();
                 if (null !== $class) {
                     return $this->elementTypeNameGenerator->getLocalizedTypeLabel($class);
@@ -220,10 +220,10 @@ 
             'label' => $this->translator->trans('log.extra'),
         ]);
 
-        $dataTable->add('timeTravel', IconLinkColumn::class,[
+        $dataTable->add('timeTravel', IconLinkColumn::class, [
             'label' => '',
             'icon' => 'fas fa-fw fa-eye',
-            'href' => function ($value, AbstractLogEntry $context) {
+            'href' => function($value, AbstractLogEntry $context) {
                 if (
                     ($context instanceof TimeTravelInterface
                         && $context->hasOldDataInformations())
@@ -231,7 +231,7 @@ 
                 ) {
                     try {
                         $target = $this->logRepo->getTargetElement($context);
-                        if($target !== null) {
+                        if ($target !== null) {
                             $str = $this->entityURLGenerator->timeTravelURL($target, $context->getTimestamp());
                             return $str;
                         }
@@ -241,7 +241,7 @@ 
                 }
                 return null;
             },
-            'disabled' => function ($value, AbstractLogEntry $context) {
+            'disabled' => function($value, AbstractLogEntry $context) {
                 return
                     !$this->security->isGranted('@tools.timetravel')
                     || !$this->security->isGranted('show_history', $context->getTargetClass());
@@ -257,7 +257,7 @@ 
 
         $dataTable->createAdapter(ORMAdapter::class, [
             'entity' => AbstractLogEntry::class,
-            'query' => function (QueryBuilder $builder) use ($options): void {
+            'query' => function(QueryBuilder $builder) use ($options): void {
                 $this->getQuery($builder, $options);
             },
         ]);
@@ -271,11 +271,11 @@ 
             ->leftJoin('log.user', 'user');
 
         if ($options['mode'] === 'last_activity') {
-            $builder->where('log INSTANCE OF ' . ElementCreatedLogEntry::class)
-                ->orWhere('log INSTANCE OF ' . ElementDeletedLogEntry::class)
-                ->orWhere('log INSTANCE OF ' . ElementEditedLogEntry::class)
-                ->orWhere('log INSTANCE OF ' . CollectionElementDeleted::class)
-                ->andWhere('log.target_type NOT IN (:disallowed)');;
+            $builder->where('log INSTANCE OF '.ElementCreatedLogEntry::class)
+                ->orWhere('log INSTANCE OF '.ElementDeletedLogEntry::class)
+                ->orWhere('log INSTANCE OF '.ElementEditedLogEntry::class)
+                ->orWhere('log INSTANCE OF '.CollectionElementDeleted::class)
+                ->andWhere('log.target_type NOT IN (:disallowed)'); ;
 
             $builder->setParameter('disallowed', [
                 AbstractLogEntry::targetTypeClassToID(User::class),

src/Repository/LogEntryRepository.php

Spacing +6 added lines, -6 removed lines

@@ -94,7 +94,7 @@ 
         $qb = $this->createQueryBuilder('log');
         $qb->select('log')
             //->where('log INSTANCE OF App\Entity\LogSystem\ElementEditedLogEntry')
-            ->where('log INSTANCE OF ' . ElementDeletedLogEntry::class)
+            ->where('log INSTANCE OF '.ElementDeletedLogEntry::class)
             ->andWhere('log.target_type = :target_type')
             ->andWhere('log.target_id = :target_id')
             ->orderBy('log.timestamp', 'DESC')
@@ -105,7 +105,7 @@ 
                                'target_id' => $id,
                            ]);
 
-        $query =  $qb->getQuery();
+        $query = $qb->getQuery();
 
         $results = $query->execute();
 
@@ -126,8 +126,8 @@ 
         $qb = $this->createQueryBuilder('log');
         $qb->select('log')
             //->where('log INSTANCE OF App\Entity\LogSystem\ElementEditedLogEntry')
-            ->where('log INSTANCE OF ' . ElementEditedLogEntry::class)
-            ->orWhere('log INSTANCE OF ' . CollectionElementDeleted::class)
+            ->where('log INSTANCE OF '.ElementEditedLogEntry::class)
+            ->orWhere('log INSTANCE OF '.CollectionElementDeleted::class)
             ->andWhere('log.target_type = :target_type')
             ->andWhere('log.target_id = :target_id')
             ->andWhere('log.timestamp >= :until')
@@ -153,7 +153,7 @@ 
     {
         $qb = $this->createQueryBuilder('log');
         $qb->select('count(log)')
-            ->where('log INSTANCE OF ' . ElementCreatedLogEntry::class)
+            ->where('log INSTANCE OF '.ElementCreatedLogEntry::class)
             ->andWhere('log.target_type = :target_type')
             ->andWhere('log.target_id = :target_id')
             ->andWhere('log.timestamp >= :until')
@@ -211,7 +211,7 @@ 
         $qb = $this->createQueryBuilder('log');
         $qb->select('log')
             //->where('log INSTANCE OF App\Entity\LogSystem\ElementEditedLogEntry')
-            ->where('log INSTANCE OF ' . $class)
+            ->where('log INSTANCE OF '.$class)
             ->andWhere('log.target_type = :target_type')
             ->andWhere('log.target_id = :target_id')
             ->orderBy('log.timestamp', 'DESC');

src/EventSubscriber/EventLoggerSubscriber.php

Spacing +4 added lines, -4 removed lines

@@ -129,10 +129,10 @@ 
 
                 $log->setUndoneEvent($undoEvent, $this->eventUndoHelper->getMode());
 
-                if($undoEvent instanceof ElementDeletedLogEntry && $undoEvent->getTargetClass() === $log->getTargetClass()) {
+                if ($undoEvent instanceof ElementDeletedLogEntry && $undoEvent->getTargetClass() === $log->getTargetClass()) {
                     $log->setTargetElementID($undoEvent->getTargetID());
                 }
-                if($undoEvent instanceof CollectionElementDeleted && $undoEvent->getDeletedElementClass() === $log->getTargetClass()) {
+                if ($undoEvent instanceof CollectionElementDeleted && $undoEvent->getDeletedElementClass() === $log->getTargetClass()) {
                     $log->setTargetElementID($undoEvent->getDeletedElementID());
                 }
             }
@@ -246,7 +246,7 @@ 
             return $fields;
         }
 
-        return array_filter($fields, function ($value, $key) use ($element) {
+        return array_filter($fields, function($value, $key) use ($element) {
             //Associative array (save changed data) case
             if (is_string($key)) {
                 return $this->shouldFieldBeSaved($element, $key);
@@ -291,7 +291,7 @@ 
         $old_data = $this->filterFieldRestrictions($entity, $old_data);
 
         //Restrict length of string fields, to save memory...
-        $old_data = array_map(function ($value) {
+        $old_data = array_map(function($value) {
             if (is_string($value)) {
                 return mb_strimwidth($value, 0, self::MAX_STRING_LENGTH, '...');
             }

src/Controller/AdminPages/BaseAdminController.php

Spacing +6 added lines, -6 removed lines

@@ -131,7 +131,7 @@ 
             $this->timeTravel->revertEntityToTimestamp($entity, $timeTravel_timestamp);
         }
 
-        if ($this->isGranted('show_history', $entity) ) {
+        if ($this->isGranted('show_history', $entity)) {
             $table = $this->dataTableFactory->createFromType(
                 LogDataTable::class,
                 [
@@ -157,7 +157,7 @@ 
         $form->handleRequest($request);
         if ($form->isSubmitted() && $form->isValid()) {
             //Check if we editing a user and if we need to change the password of it
-            if ($entity instanceof User && ! empty($form['new_password']->getData())) {
+            if ($entity instanceof User && !empty($form['new_password']->getData())) {
                 $password = $this->passwordEncoder->encodePassword($entity, $form['new_password']->getData());
                 $entity->setPassword($password);
                 //By default the user must change the password afterwards
@@ -192,7 +192,7 @@ 
             //Rebuild form, so it is based on the updated data. Important for the parent field!
             //We can not use dynamic form events here, because the parent entity list is build from database!
             $form = $this->createForm($this->form_class, $entity, ['attachment_class' => $this->attachment_class]);
-        } elseif ($form->isSubmitted() && ! $form->isValid()) {
+        } elseif ($form->isSubmitted() && !$form->isValid()) {
             $this->addFlash('error', 'entity.edit_flash.invalid');
         }
 
@@ -219,7 +219,7 @@ 
         $form->handleRequest($request);
 
         if ($form->isSubmitted() && $form->isValid()) {
-            if ($new_entity instanceof User && ! empty($form['new_password']->getData())) {
+            if ($new_entity instanceof User && !empty($form['new_password']->getData())) {
                 $password = $this->passwordEncoder->encodePassword($new_entity, $form['new_password']->getData());
                 $new_entity->setPassword($password);
                 //By default the user must change the password afterwards
@@ -254,7 +254,7 @@ 
             return $this->redirectToRoute($this->route_base.'_edit', ['id' => $new_entity->getID()]);
         }
 
-        if ($form->isSubmitted() && ! $form->isValid()) {
+        if ($form->isSubmitted() && !$form->isValid()) {
             $this->addFlash('error', 'entity.created_flash.invalid');
         }
 
@@ -274,7 +274,7 @@ 
                 'csv_separator' => $data['csv_separator'],
             ];
 
-            $this->commentHelper->setMessage('Import ' . $file->getClientOriginalName());
+            $this->commentHelper->setMessage('Import '.$file->getClientOriginalName());
 
 
 

src/Services/LogSystem/LogEntryExtraFormatter.php

Spacing +3 added lines, -3 removed lines

@@ -90,7 +90,7 @@ 
         foreach ($arr as $key => $value) {
             $str = '';
             if (is_string($key)) {
-                $str .= '<error>' . $this->translator->trans($key) . '</error>: ';
+                $str .= '<error>'.$this->translator->trans($key).'</error>: ';
             }
             $str .= $value;
             if (!empty($str)) {
@@ -163,7 +163,7 @@ 
                 '%s <i class="fas fa-long-arrow-alt-right"></i> %s (%s)',
                 $context->getOldInstock(),
                 $context->getNewInstock(),
-                (! $context->isWithdrawal() ? '+' : '-').$context->getDifference(true)
+                (!$context->isWithdrawal() ? '+' : '-').$context->getDifference(true)
             );
             $array['log.instock_changed.comment'] = htmlspecialchars($context->getComment());
         }
@@ -198,7 +198,7 @@ 
         foreach ($arr as $key => $value) {
             $str = '';
             if (is_string($key)) {
-                $str .= '<b>' . $this->translator->trans($key) . '</b>: ';
+                $str .= '<b>'.$this->translator->trans($key).'</b>: ';
             }
             $str .= $value;
             if (!empty($str)) {

src/Services/LogSystem/TimeTravel.php

Spacing +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
         $this->setField($element, 'id', $id);
 
         //Let database determine when it will be created
-        $this->setField($element,'addedDate', null);
+        $this->setField($element, 'addedDate', null);
 
         return $element;
     }

src/DataTables/PartsDataTable.php

Spacing +8 added lines, -8 removed lines

@@ -113,7 +113,7 @@ 
         $optionsResolver->setAllowedTypes('search', ['null', 'string']);
 
         //Configure search options
-        $optionsResolver->setDefault('search_options', function (OptionsResolver $resolver) {
+        $optionsResolver->setDefault('search_options', function(OptionsResolver $resolver) {
             $resolver->setDefaults([
                                        'name' => true,
                                        'category' => true,
@@ -149,7 +149,7 @@ 
         $dataTable
             ->add('picture', TextColumn::class, [
                 'label' => '',
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     $preview_attachment = $this->previewGenerator->getTablePreviewAttachment($context);
                     if (null === $preview_attachment) {
                         return '';
@@ -166,7 +166,7 @@ 
             ])
             ->add('name', TextColumn::class, [
                 'label' => $this->translator->trans('part.table.name'),
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     return sprintf(
                         '<a href="%s">%s</a>',
                         $this->urlGenerator->infoURL($context),
@@ -195,7 +195,7 @@ 
             ])
             ->add('storelocation', TextColumn::class, [
                 'label' => $this->translator->trans('part.table.storeLocations'),
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     $tmp = [];
                     foreach ($context->getPartLots() as $lot) {
                         //Ignore lots without storelocation
@@ -214,7 +214,7 @@ 
             ])
             ->add('amount', TextColumn::class, [
                 'label' => $this->translator->trans('part.table.amount'),
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     $amount = $context->getAmountSum();
 
                     return $this->amountFormatter->format($amount, $context->getPartUnit());
@@ -223,7 +223,7 @@ 
             ->add('minamount', TextColumn::class, [
                 'label' => $this->translator->trans('part.table.minamount'),
                 'visible' => false,
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     return $this->amountFormatter->format($value, $context->getPartUnit());
                 },
             ])
@@ -287,12 +287,12 @@ 
             ->addOrderBy('name')
             ->createAdapter(FetchJoinORMAdapter::class, [
                 'simple_total_query' => true,
-                'query' => function (QueryBuilder $builder): void {
+                'query' => function(QueryBuilder $builder): void {
                     $this->getQuery($builder);
                 },
                 'entity' => Part::class,
                 'criteria' => [
-                    function (QueryBuilder $builder) use ($options): void {
+                    function(QueryBuilder $builder) use ($options): void {
                         $this->buildCriteria($builder, $options);
                     },
                     new SearchCriteriaProvider(),

src/Entity/Parts/PartLot.php

Spacing +2 added lines, -2 removed lines

@@ -293,7 +293,7 @@ 
      */
     public function getAmount(): float
     {
-        if ($this->part instanceof Part && ! $this->part->useFloatAmount()) {
+        if ($this->part instanceof Part && !$this->part->useFloatAmount()) {
             return round($this->amount);
         }
 
@@ -328,7 +328,7 @@ 
 
     public function __clone()
     {
-        if($this->id) {
+        if ($this->id) {
             $this->addedDate = null;
         }
         parent::__clone();