Part-DB / Part-DB-symfony

src/Services/LogSystem/TimeTravel.php

Spacing +4 added lines, -4 removed lines

@@ -79,7 +79,7 @@ 
      */
     public function revertEntityToTimestamp(AbstractDBElement $element, \DateTime $timestamp, array $reverted_elements = []): void
     {
-        if (! $element instanceof TimeStampableInterface) {
+        if (!$element instanceof TimeStampableInterface) {
             throw new \InvalidArgumentException('$element must have a Timestamp!');
         }
 
@@ -153,7 +153,7 @@ 
                 foreach ($target_elements as $target_element) {
                     if (null !== $target_element && $element->getLastModified() >= $timestamp) {
                         //Remove the element from collection, if it did not existed at $timestamp
-                        if (! $this->repo->getElementExistedAtTimestamp($target_element, $timestamp)) {
+                        if (!$this->repo->getElementExistedAtTimestamp($target_element, $timestamp)) {
                             if ($target_elements instanceof Collection) {
                                 $target_elements->removeElement($target_element);
                             }
@@ -173,10 +173,10 @@ 
     public function applyEntry(AbstractDBElement $element, TimeTravelInterface $logEntry): void
     {
         //Skip if this does not provide any info...
-        if (! $logEntry->hasOldDataInformations()) {
+        if (!$logEntry->hasOldDataInformations()) {
             return;
         }
-        if (! $element instanceof TimeStampableInterface) {
+        if (!$element instanceof TimeStampableInterface) {
             return;
         }
         $metadata = $this->em->getClassMetadata(get_class($element));

src/Services/LogSystem/LogEntryExtraFormatter.php

Spacing +3 added lines, -3 removed lines

@@ -91,7 +91,7 @@ 
                 $str .= '<error>'.$this->translator->trans($key).'</error>: ';
             }
             $str .= $value;
-            if (! empty($str)) {
+            if (!empty($str)) {
                 $tmp[] = $str;
             }
         }
@@ -116,7 +116,7 @@ 
                 $str .= '<b>'.$this->translator->trans($key).'</b>: ';
             }
             $str .= $value;
-            if (! empty($str)) {
+            if (!empty($str)) {
                 $tmp[] = $str;
             }
         }
@@ -186,7 +186,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());
         }

src/Form/Type/CurrencyEntityType.php

Spacing +3 added lines, -3 removed lines

@@ -72,7 +72,7 @@ 
         // This options allows you to override the currency shown for the null value
         $resolver->setDefault('base_currency', null);
 
-        $resolver->setDefault('empty_message', function (Options $options) {
+        $resolver->setDefault('empty_message', function(Options $options) {
             //By default we use the global base currency:
             $iso_code = $this->base_currency;
 
@@ -88,7 +88,7 @@ 
     {
         //Similar to StructuralEntityType, but we use the currency symbol instead if available
 
-        if (! $choice instanceof Currency) {
+        if (!$choice instanceof Currency) {
             throw new \InvalidArgumentException('$choice must be an currency object!');
         }
 
@@ -117,7 +117,7 @@ 
         /** @var Currency $choice */
         $tmp = [];
 
-        if (! empty($choice->getIsoCode())) {
+        if (!empty($choice->getIsoCode())) {
             //Show the name of the currency
             $tmp += ['data-subtext' => $choice->getName()];
         }

src/Form/Part/PartBaseType.php

Spacing +20 added lines, -20 removed lines

@@ -105,7 +105,7 @@ 
                 'attr' => [
                     'placeholder' => 'part.edit.name.placeholder',
                 ],
-                'disabled' => ! $this->security->isGranted('name.edit', $part),
+                'disabled' => !$this->security->isGranted('name.edit', $part),
             ])
             ->add('description', CKEditorType::class, [
                 'required' => false,
@@ -116,7 +116,7 @@ 
                     'placeholder' => 'part.edit.description.placeholder',
                     'rows' => 2,
                 ],
-                'disabled' => ! $this->security->isGranted('description.edit', $part),
+                'disabled' => !$this->security->isGranted('description.edit', $part),
             ])
             ->add('minAmount', SIUnitType::class, [
                 'attr' => [
@@ -125,20 +125,20 @@ 
                 ],
                 'label' => 'part.edit.mininstock',
                 'measurement_unit' => $part->getPartUnit(),
-                'disabled' => ! $this->security->isGranted('minamount.edit', $part),
+                'disabled' => !$this->security->isGranted('minamount.edit', $part),
             ])
             ->add('category', StructuralEntityType::class, [
                 'class' => Category::class,
                 'label' => 'part.edit.category',
                 'disable_not_selectable' => true,
-                'disabled' => ! $this->security->isGranted('category.edit', $part),
+                'disabled' => !$this->security->isGranted('category.edit', $part),
             ])
             ->add('footprint', StructuralEntityType::class, [
                 'class' => Footprint::class,
                 'required' => false,
                 'label' => 'part.edit.footprint',
                 'disable_not_selectable' => true,
-                'disabled' => ! $this->security->isGranted('footprint.edit', $part),
+                'disabled' => !$this->security->isGranted('footprint.edit', $part),
             ])
             ->add('tags', TextType::class, [
                 'required' => false,
@@ -148,7 +148,7 @@ 
                     'class' => 'tagsinput',
                     'data-autocomplete' => $this->urlGenerator->generate('typeahead_tags', ['query' => 'QUERY']),
                 ],
-                'disabled' => ! $this->security->isGranted('tags.edit', $part),
+                'disabled' => !$this->security->isGranted('tags.edit', $part),
             ]);
 
         //Manufacturer section
@@ -157,25 +157,25 @@ 
             'required' => false,
             'label' => 'part.edit.manufacturer.label',
             'disable_not_selectable' => true,
-            'disabled' => ! $this->security->isGranted('manufacturer.edit', $part),
+            'disabled' => !$this->security->isGranted('manufacturer.edit', $part),
         ])
             ->add('manufacturer_product_url', UrlType::class, [
                 'required' => false,
                 'empty_data' => '',
                 'label' => 'part.edit.manufacturer_url.label',
-                'disabled' => ! $this->security->isGranted('mpn.edit', $part),
+                'disabled' => !$this->security->isGranted('mpn.edit', $part),
             ])
             ->add('manufacturer_product_number', TextType::class, [
                 'required' => false,
                 'empty_data' => '',
                 'label' => 'part.edit.mpn',
-                'disabled' => ! $this->security->isGranted('mpn.edit', $part),
+                'disabled' => !$this->security->isGranted('mpn.edit', $part),
             ])
             ->add('manufacturing_status', ChoiceType::class, [
                 'label' => 'part.edit.manufacturing_status',
                 'choices' => $status_choices,
                 'required' => false,
-                'disabled' => ! $this->security->isGranted('status.edit', $part),
+                'disabled' => !$this->security->isGranted('status.edit', $part),
             ]);
 
         //Advanced section
@@ -185,7 +185,7 @@ 
             ],
             'required' => false,
             'label' => 'part.edit.needs_review',
-            'disabled' => ! $this->security->isGranted('edit', $part),
+            'disabled' => !$this->security->isGranted('edit', $part),
         ])
             ->add('favorite', CheckboxType::class, [
                 'label_attr' => [
@@ -193,20 +193,20 @@ 
                 ],
                 'required' => false,
                 'label' => 'part.edit.is_favorite',
-                'disabled' => ! $this->security->isGranted('change_favorite', $part),
+                'disabled' => !$this->security->isGranted('change_favorite', $part),
             ])
             ->add('mass', SIUnitType::class, [
                 'unit' => 'g',
                 'label' => 'part.edit.mass',
                 'required' => false,
-                'disabled' => ! $this->security->isGranted('mass.edit', $part),
+                'disabled' => !$this->security->isGranted('mass.edit', $part),
             ])
             ->add('partUnit', StructuralEntityType::class, [
                 'class' => MeasurementUnit::class,
                 'required' => false,
                 'disable_not_selectable' => true,
                 'label' => 'part.edit.partUnit',
-                'disabled' => ! $this->security->isGranted('unit.edit', $part),
+                'disabled' => !$this->security->isGranted('unit.edit', $part),
             ]);
 
         //Comment section
@@ -216,7 +216,7 @@ 
             'attr' => [
                 'rows' => 4,
             ],
-            'disabled' => ! $this->security->isGranted('comment.edit', $part),
+            'disabled' => !$this->security->isGranted('comment.edit', $part),
             'empty_data' => '',
         ]);
 
@@ -228,7 +228,7 @@ 
             'label' => false,
             'entry_options' => [
                 'measurement_unit' => $part->getPartUnit(),
-                'disabled' => ! $this->security->isGranted('lots.edit', $part),
+                'disabled' => !$this->security->isGranted('lots.edit', $part),
             ],
             'by_reference' => false,
         ]);
@@ -241,14 +241,14 @@ 
             'label' => false,
             'entry_options' => [
                 'data_class' => PartAttachment::class,
-                'disabled' => ! $this->security->isGranted('attachments.edit', $part),
+                'disabled' => !$this->security->isGranted('attachments.edit', $part),
             ],
             'by_reference' => false,
         ]);
 
         $builder->add('master_picture_attachment', MasterPictureAttachmentType::class, [
             'required' => false,
-            'disabled' => ! $this->security->isGranted('attachments.edit', $part),
+            'disabled' => !$this->security->isGranted('attachments.edit', $part),
             'label' => 'part.edit.master_attachment',
             'entity' => $part,
         ]);
@@ -263,7 +263,7 @@ 
             'prototype_data' => new Orderdetail(),
             'entry_options' => [
                 'measurement_unit' => $part->getPartUnit(),
-                'disabled' => ! $this->security->isGranted('orderdetails.edit', $part),
+                'disabled' => !$this->security->isGranted('orderdetails.edit', $part),
             ],
         ]);
 
@@ -275,7 +275,7 @@ 
             'by_reference' => false,
             'prototype_data' => new PartParameter(),
             'entry_options' => [
-                'disabled' => ! $this->security->isGranted('parameters.edit', $part),
+                'disabled' => !$this->security->isGranted('parameters.edit', $part),
                 'data_class' => PartParameter::class,
             ],
         ]);

src/Repository/LogEntryRepository.php

Spacing +1 added lines, -1 removed lines

@@ -171,7 +171,7 @@
         $query = $qb->getQuery();
         $count = $query->getSingleScalarResult();
 
-        return ! ($count > 0);
+        return !($count > 0);
     }
 
     /**

src/DataTables/Column/LogEntryTargetColumn.php

Spacing +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
         $tmp = '';
 
         //The element is existing
-        if ($target instanceof NamedElementInterface && ! empty($target->getName())) {
+        if ($target instanceof NamedElementInterface && !empty($target->getName())) {
             try {
                 $tmp = sprintf(
                     '<a href="%s">%s</a>',

src/Services/EntityURLGenerator.php

Spacing +1 added lines, -1 removed lines

@@ -381,7 +381,7 @@
         $class = get_class($entity);
 
         //Check if we have an direct mapping for the given class
-        if (! array_key_exists($class, $map)) {
+        if (!array_key_exists($class, $map)) {
             //Check if we need to check inheritance by looping through our map
             foreach (array_keys($map) as $key) {
                 if (is_a($entity, $key)) {

src/Services/Parameters/ParameterExtractor.php

Spacing +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@ 
      */
     public function extractParameters(string $input, string $class = PartParameter::class): array
     {
-        if (! is_a($class, AbstractParameter::class, true)) {
+        if (!is_a($class, AbstractParameter::class, true)) {
             throw new \InvalidArgumentException('$class must be a child class of AbstractParameter!');
         }
 
@@ -68,7 +68,7 @@ 
 
         $matches = [];
         \preg_match($regex, $input, $matches);
-        if (! empty($matches)) {
+        if (!empty($matches)) {
             [, $name, $value] = $matches;
             $value = trim($value);
 

tests/Controller/AdminPages/AbstractAdminControllerTest.php

Spacing +3 added lines, -3 removed lines

@@ -90,7 +90,7 @@ 
         $client->request('GET', static::$base_path.'/new');
         $this->assertFalse($client->getResponse()->isRedirect());
         $this->assertSame($read, $client->getResponse()->isSuccessful(), 'Controller was not successful!');
-        $this->assertSame($read, ! $client->getResponse()->isForbidden(), 'Permission Checking not working!');
+        $this->assertSame($read, !$client->getResponse()->isForbidden(), 'Permission Checking not working!');
     }
 
     /**
@@ -115,7 +115,7 @@ 
         $client->request('GET', static::$base_path.'/1');
         $this->assertFalse($client->getResponse()->isRedirect());
         $this->assertSame($read, $client->getResponse()->isSuccessful(), 'Controller was not successful!');
-        $this->assertSame($read, ! $client->getResponse()->isForbidden(), 'Permission Checking not working!');
+        $this->assertSame($read, !$client->getResponse()->isForbidden(), 'Permission Checking not working!');
     }
 
     public function deleteDataProvider(): array
@@ -152,6 +152,6 @@ 
 
         //Page is redirected to '/new', when delete was successful
         $this->assertSame($delete, $client->getResponse()->isRedirect(static::$base_path.'/new'));
-        $this->assertSame($delete, ! $client->getResponse()->isForbidden(), 'Permission Checking not working!');
+        $this->assertSame($delete, !$client->getResponse()->isForbidden(), 'Permission Checking not working!');
     }
 }