Part-DB / Part-DB-symfony

src/Controller/TypeaheadController.php

Spacing +1 added lines, -1 removed lines

@@ -149,7 +149,7 @@
         foreach ($parts as $part) {
             //Determine the picture to show:
             $preview_attachment = $previewGenerator->getTablePreviewAttachment($part);
-            if($preview_attachment !== null) {
+            if ($preview_attachment !== null) {
                 $preview_url = $attachmentURLGenerator->getThumbnailURL($preview_attachment, 'thumbnail_sm');
             } else {
                 $preview_url = '';

src/Controller/UserController.php

Spacing +2 added lines, -2 removed lines

@@ -83,7 +83,7 @@ 
      *
      * @throws Exception
      */
-    public function edit(User $entity, Request $request, EntityManagerInterface $em,  PermissionPresetsHelper $permissionPresetsHelper, PermissionSchemaUpdater $permissionSchemaUpdater, ?string $timestamp = null): Response
+    public function edit(User $entity, Request $request, EntityManagerInterface $em, PermissionPresetsHelper $permissionPresetsHelper, PermissionSchemaUpdater $permissionSchemaUpdater, ?string $timestamp = null): Response
     {
         //Do an upgrade of the permission schema if needed (so the user can see the permissions a user get on next request (even if it was not done yet)
         $permissionSchemaUpdater->userUpgradeSchemaRecursively($entity);
@@ -155,7 +155,7 @@ 
      * @Route("/{id}/clone", name="user_clone")
      * @Route("/")
      */
-    public function new(Request $request, EntityManagerInterface $em, EntityImporter $importer, ?User $entity = null): Response
+    public function new(Request $request, EntityManagerInterface $em, EntityImporter $importer, ?User $entity = null) : Response
     {
         return $this->_new($request, $em, $importer, $entity);
     }

src/Controller/PartController.php

Spacing +5 added lines, -5 removed lines

@@ -215,7 +215,7 @@ 
      */
     public function new(Request $request, EntityManagerInterface $em, TranslatorInterface $translator,
         AttachmentSubmitHandler $attachmentSubmitHandler, ProjectBuildPartHelper $projectBuildPartHelper,
-        ?Part $part = null, ?Project $project = null): Response
+        ?Part $part = null, ?Project $project = null) : Response
     {
 
         if ($part) { //Clone part
@@ -328,14 +328,14 @@ 
      */
     public function withdrawAddHandler(Part $part, Request $request, EntityManagerInterface $em, PartLotWithdrawAddHelper $withdrawAddHelper): Response
     {
-        if ($this->isCsrfTokenValid('part_withraw' . $part->getID(), $request->request->get('_csfr'))) {
+        if ($this->isCsrfTokenValid('part_withraw'.$part->getID(), $request->request->get('_csfr'))) {
             //Retrieve partlot from the request
             $partLot = $em->find(PartLot::class, $request->request->get('lot_id'));
-            if($partLot === null) {
+            if ($partLot === null) {
                 throw new \RuntimeException('Part lot not found!');
             }
             //Ensure that the partlot belongs to the part
-            if($partLot->getPart() !== $part) {
+            if ($partLot->getPart() !== $part) {
                 throw new \RuntimeException("The origin partlot does not belong to the part!");
             }
             //Try to determine the target lot (used for move actions)
@@ -378,7 +378,7 @@ 
         }
 
         //If an redirect was passed, then redirect there
-        if($request->request->get('_redirect')) {
+        if ($request->request->get('_redirect')) {
             return $this->redirect($request->request->get('_redirect'));
         }
         //Otherwise just redirect to the part page

src/Controller/ProjectController.php

Spacing +2 added lines, -2 removed lines

@@ -77,7 +77,7 @@ 
      */
     public function addPart(Request $request, EntityManagerInterface $entityManager, ?Project $project): Response
     {
-        if($project) {
+        if ($project) {
             $this->denyAccessUnlessGranted('edit', $project);
         } else {
             $this->denyAccessUnlessGranted('@projects.edit');
@@ -128,7 +128,7 @@ 
 
             $data = $form->getData();
             $bom_entries = $data['bom_entries'];
-            foreach ($bom_entries as $bom_entry){
+            foreach ($bom_entries as $bom_entry) {
                 $target_project->addBOMEntry($bom_entry);
             }
             $entityManager->flush();

src/Controller/WebauthnKeyRegistrationController.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
             //Retrieve other data from the form, that you want to store with the key
             $keyName = $request->request->get('keyName');
             if (empty($keyName)) {
-                $keyName = 'Key ' . date('Y-m-d H:i:s');
+                $keyName = 'Key '.date('Y-m-d H:i:s');
             }
 
             //Check the response

src/Controller/PartListsController.php

Spacing +15 added lines, -15 removed lines

@@ -129,12 +129,12 @@ 
         $formRequest = clone $request;
         $formRequest->setMethod('GET');
         $filter = new PartFilter($this->nodesListBuilder);
-        if($filter_changer !== null){
+        if ($filter_changer !== null) {
             $filter_changer($filter);
         }
 
         $filterForm = $this->createForm(PartFilterType::class, $filter, ['method' => 'GET']);
-        if($form_changer !== null) {
+        if ($form_changer !== null) {
             $form_changer($filterForm);
         }
 
@@ -164,9 +164,9 @@ 
 
         return $this->showListWithFilter($request,
             'Parts/lists/category_list.html.twig',
-            function (PartFilter $filter) use ($category) {
+            function(PartFilter $filter) use ($category) {
                 $filter->getCategory()->setOperator('INCLUDING_CHILDREN')->setValue($category);
-            }, function (FormInterface $filterForm) {
+            }, function(FormInterface $filterForm) {
                 $this->disableFormFieldAfterCreation($filterForm->get('category')->get('value'));
             }, [
                 'entity' => $category,
@@ -186,9 +186,9 @@ 
 
         return $this->showListWithFilter($request,
             'Parts/lists/footprint_list.html.twig',
-            function (PartFilter $filter) use ($footprint) {
+            function(PartFilter $filter) use ($footprint) {
                 $filter->getFootprint()->setOperator('INCLUDING_CHILDREN')->setValue($footprint);
-            }, function (FormInterface $filterForm) {
+            }, function(FormInterface $filterForm) {
                 $this->disableFormFieldAfterCreation($filterForm->get('footprint')->get('value'));
             }, [
                 'entity' => $footprint,
@@ -208,9 +208,9 @@ 
 
         return $this->showListWithFilter($request,
             'Parts/lists/manufacturer_list.html.twig',
-            function (PartFilter $filter) use ($manufacturer) {
+            function(PartFilter $filter) use ($manufacturer) {
                 $filter->getManufacturer()->setOperator('INCLUDING_CHILDREN')->setValue($manufacturer);
-            }, function (FormInterface $filterForm) {
+            }, function(FormInterface $filterForm) {
                 $this->disableFormFieldAfterCreation($filterForm->get('manufacturer')->get('value'));
             }, [
                 'entity' => $manufacturer,
@@ -230,9 +230,9 @@ 
 
         return $this->showListWithFilter($request,
             'Parts/lists/store_location_list.html.twig',
-            function (PartFilter $filter) use ($storelocation) {
+            function(PartFilter $filter) use ($storelocation) {
                 $filter->getStorelocation()->setOperator('INCLUDING_CHILDREN')->setValue($storelocation);
-            }, function (FormInterface $filterForm) {
+            }, function(FormInterface $filterForm) {
                 $this->disableFormFieldAfterCreation($filterForm->get('storelocation')->get('value'));
             }, [
                 'entity' => $storelocation,
@@ -252,9 +252,9 @@ 
 
         return $this->showListWithFilter($request,
             'Parts/lists/supplier_list.html.twig',
-            function (PartFilter $filter) use ($supplier) {
+            function(PartFilter $filter) use ($supplier) {
                 $filter->getSupplier()->setOperator('INCLUDING_CHILDREN')->setValue($supplier);
-            }, function (FormInterface $filterForm) {
+            }, function(FormInterface $filterForm) {
                 $this->disableFormFieldAfterCreation($filterForm->get('supplier')->get('value'));
             }, [
                 'entity' => $supplier,
@@ -274,9 +274,9 @@ 
 
         return $this->showListWithFilter($request,
             'Parts/lists/tags_list.html.twig',
-            function (PartFilter $filter) use ($tag) {
+            function(PartFilter $filter) use ($tag) {
                 $filter->getTags()->setOperator('ANY')->setValue($tag);
-            }, function (FormInterface $filterForm) {
+            }, function(FormInterface $filterForm) {
                 $this->disableFormFieldAfterCreation($filterForm->get('tags')->get('value'));
             }, [
                 'tag' => $tag,
@@ -334,6 +334,6 @@ 
      */
     public function showAll(Request $request, DataTableFactory $dataTable)
     {
-        return $this->showListWithFilter($request,'Parts/lists/all_list.html.twig');
+        return $this->showListWithFilter($request, 'Parts/lists/all_list.html.twig');
     }
 }

src/Services/UserSystem/PermissionManager.php

Spacing +3 added lines, -3 removed lines

@@ -78,7 +78,7 @@ 
     public function dontInherit(HasPermissionsInterface $user, string $permission, string $operation): ?bool
     {
         //Check that the permission/operation combination is valid
-        if (! $this->isValidOperation($permission, $operation)) {
+        if (!$this->isValidOperation($permission, $operation)) {
             throw new InvalidArgumentException('The permission/operation combination "'.$permission.'/'.$operation.'" is not valid!');
         }
 
@@ -140,7 +140,7 @@ 
         $perm_list = $user->getPermissions();
 
         //Check if the permission/operation combination is valid
-        if (! $this->isValidOperation($permission, $operation)) {
+        if (!$this->isValidOperation($permission, $operation)) {
             throw new InvalidArgumentException(sprintf('The permission/operation combination "%s.%s" is not valid!', $permission, $operation));
         }
 
@@ -233,7 +233,7 @@ 
                     }
                 }
             }
-        } while($anything_changed);
+        } while ($anything_changed);
     }
 
     /**

src/Services/UserSystem/PermissionPresetsHelper.php

Spacing +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
         //We need to reset the permission data first (afterwards all values are inherit)
         $perm_holder->getPermissions()->resetPermissions();
 
-        switch($preset_name) {
+        switch ($preset_name) {
             case self::PRESET_ALL_INHERIT:
                 //Do nothing, all values are inherit after reset
                 break;
@@ -118,7 +118,7 @@ 
 
         //Labels permissions (allow all except use twig)
         $this->permissionResolver->setAllOperationsOfPermission($permHolder, 'labels', PermissionData::ALLOW);
-        $this->permissionResolver->setPermission($permHolder,'labels', 'use_twig', PermissionData::INHERIT);
+        $this->permissionResolver->setPermission($permHolder, 'labels', 'use_twig', PermissionData::INHERIT);
 
         //Self permissions
         $this->permissionResolver->setPermission($permHolder, 'self', 'edit_infos', PermissionData::ALLOW);

src/Services/Parts/PartLotWithdrawAddHelper.php

Spacing +7 added lines, -7 removed lines

@@ -28,17 +28,17 @@ 
     public function canAdd(PartLot $partLot): bool
     {
         //We cannot add or withdraw parts from lots with unknown instock value.
-        if($partLot->isInstockUnknown()) {
+        if ($partLot->isInstockUnknown()) {
             return false;
         }
 
         //So far all other restrictions are defined at the storelocation level
-        if($partLot->getStorageLocation() === null) {
+        if ($partLot->getStorageLocation() === null) {
             return true;
         }
 
         //We can not add parts if the storage location of the lot is marked as full
-        if($partLot->getStorageLocation()->isFull()) {
+        if ($partLot->getStorageLocation()->isFull()) {
             return false;
         }
 
@@ -96,7 +96,7 @@ 
         $oldAmount = $partLot->getAmount();
         $partLot->setAmount($oldAmount - $amount);
 
-        $event = PartStockChangedLogEntry::withdraw($partLot, $oldAmount, $partLot->getAmount(), $part->getAmountSum() , $comment);
+        $event = PartStockChangedLogEntry::withdraw($partLot, $oldAmount, $partLot->getAmount(), $part->getAmountSum(), $comment);
         $this->eventLogger->log($event);
 
         //Apply the comment also to global events, so it gets associated with the elementChanged log entry
@@ -136,7 +136,7 @@ 
         $oldAmount = $partLot->getAmount();
         $partLot->setAmount($oldAmount + $amount);
 
-        $event = PartStockChangedLogEntry::add($partLot, $oldAmount, $partLot->getAmount(), $part->getAmountSum() , $comment);
+        $event = PartStockChangedLogEntry::add($partLot, $oldAmount, $partLot->getAmount(), $part->getAmountSum(), $comment);
         $this->eventLogger->log($event);
 
         //Apply the comment also to global events, so it gets associated with the elementChanged log entry
@@ -165,7 +165,7 @@ 
         $part = $origin->getPart();
 
         //Ensure that both part lots belong to the same part
-        if($origin->getPart() !== $target->getPart()) {
+        if ($origin->getPart() !== $target->getPart()) {
             throw new \RuntimeException("Cannot move instock between different parts!");
         }
 
@@ -191,7 +191,7 @@ 
         //And add it to the target
         $target->setAmount($target->getAmount() + $amount);
 
-        $event = PartStockChangedLogEntry::move($origin, $oldOriginAmount, $origin->getAmount(), $part->getAmountSum() , $comment, $target);
+        $event = PartStockChangedLogEntry::move($origin, $oldOriginAmount, $origin->getAmount(), $part->getAmountSum(), $comment, $target);
         $this->eventLogger->log($event);
 
         //Apply the comment also to global events, so it gets associated with the elementChanged log entry