Part-DB / Part-DB-symfony

src/Twig/AppExtension.php

Spacing +4 added lines, -4 removed lines

@@ -61,7 +61,7 @@ 
 
     public function generateEntityURL(DBElement $entity, string $method = 'info') : string
     {
-        switch($method) {
+        switch ($method) {
             case 'info':
                 return $this->entityURLGenerator->infoURL($entity);
             case 'edit':
@@ -75,10 +75,10 @@ 
 
     public function parseBBCode(string $bbcode) : string
     {
-        if($bbcode === '') return '';
+        if ($bbcode === '') return '';
 
-        $item = $this->cache->getItem('bbcode_' . md5($bbcode));
-        if(!$item->isHit()) {
+        $item = $this->cache->getItem('bbcode_'.md5($bbcode));
+        if (!$item->isHit()) {
             $xml = TextFormatter::parse($bbcode);
             $item->set(TextFormatter::render($xml));
             $this->cache->save($item);

Braces +3 added lines, -1 removed lines

@@ -75,7 +75,9 @@
 
     public function parseBBCode(string $bbcode) : string
     {
-        if($bbcode === '') return '';
+        if($bbcode === '') {
+         return '';
+        }
 
         $item = $this->cache->getItem('bbcode_' . md5($bbcode));
         if(!$item->isHit()) {

src/Services/PermissionResolver.php

Spacing +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
     {
         //Read the permission config file...
         $config = Yaml::parse(
-            file_get_contents(__DIR__ . '/../../config/permissions.yaml')
+            file_get_contents(__DIR__.'/../../config/permissions.yaml')
         );
 
 
@@ -121,7 +121,7 @@ 
         }
 
         $parent = $user->getGroup();
-        while($parent != null){ //The top group, has parent == null
+        while ($parent != null) { //The top group, has parent == null
             //Check if our current element gives a info about disallow/allow
             $allowed = $this->dontInherit($parent, $permission, $operation);
             if ($allowed !== null) {

src/Entity/Group.php

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
      */
     public function getIDString(): string
     {
-        return 'G' . sprintf('%06d', $this->getID());
+        return 'G'.sprintf('%06d', $this->getID());
     }
 
     public function getPermissions(): PermissionsEmbed

src/Security/Voter/ExtendedVoter.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
         // if the user is anonymous, we use the anonymous user.
         if (!$user instanceof User) {
             $user = $this->entityManager->find(User::class, User::ID_ANONYMOUS);
-            if($user === null) {
+            if ($user === null) {
                 return false;
             }
         }

src/Security/Voter/UserVoter.php

Spacing +4 added lines, -4 removed lines

@@ -47,7 +47,7 @@ 
      */
     protected function supports($attribute, $subject)
     {
-        if($subject instanceof User)
+        if ($subject instanceof User)
         {
             return in_array($attribute, array_merge(
                 $this->resolver->listOperationsForPermission('users'),
@@ -69,15 +69,15 @@ 
      */
     protected function voteOnUser($attribute, $subject, User $user): bool
     {
-        if($subject instanceof User)
+        if ($subject instanceof User)
         {
             //Check if the checked user is the user itself
-            if($subject->getID() === $user->getID() &&
+            if ($subject->getID() === $user->getID() &&
                 $this->resolver->isValidOperation('self', $attribute)) {
                 //Then we also need to check the self permission
                 $tmp = $this->resolver->inherit($user, 'self', $attribute) ?? false;
                 //But if the self value is not allowed then use just the user value:
-                if($tmp)
+                if ($tmp)
                     return $tmp;
             }
             //Else just check users permission:

Braces +3 added lines, -2 removed lines

@@ -77,8 +77,9 @@
                 //Then we also need to check the self permission
                 $tmp = $this->resolver->inherit($user, 'self', $attribute) ?? false;
                 //But if the self value is not allowed then use just the user value:
-                if($tmp)
-                    return $tmp;
+                if($tmp) {
+                                    return $tmp;
+                }
             }
             //Else just check users permission:
             return $this->resolver->inherit($user, 'users', $attribute) ?? false;

src/Security/Voter/PartVoter.php

Spacing +4 added lines, -4 removed lines

@@ -33,7 +33,7 @@ 
         {
 
             //Check if a sub permission should be checked -> $attribute has format name.edit
-            if(strpos($attribute, '.') !== false) {
+            if (strpos($attribute, '.') !== false) {
                 [$perm, $op] = explode('.', $attribute);
                 return in_array($op, $this->resolver->listOperationsForPermission('parts_'.$perm), false);
             }
@@ -48,12 +48,12 @@ 
 
     protected function voteOnUser($attribute, $subject, User $user): bool
     {
-        if($subject instanceof Part) {
+        if ($subject instanceof Part) {
 
             //Check for sub permissions
-            if(strpos($attribute, '.') !== false) {
+            if (strpos($attribute, '.') !== false) {
                 [$perm, $op] = explode('.', $attribute);
-                return $this->resolver->inherit($user, 'parts_'. $perm, $op) ?? false;
+                return $this->resolver->inherit($user, 'parts_'.$perm, $op) ?? false;
             }
 
             //Null concealing operator means, that no

src/Controller/PartController.php

Spacing +2 added lines, -2 removed lines

@@ -117,7 +117,7 @@ 
             $em->persist($new_part);
             $em->flush();
             $this->addFlash('success', $translator->trans('part.created_flash'));
-            return $this->redirectToRoute('part_edit',['id' => $new_part->getID()]);
+            return $this->redirectToRoute('part_edit', ['id' => $new_part->getID()]);
         }
 
 
@@ -148,7 +148,7 @@ 
             $em->persist($new_part);
             $em->flush();
             $this->addFlash('success', $translator->trans('part.created_flash'));
-            return $this->redirectToRoute('part_edit',['id' => $new_part->getID()]);
+            return $this->redirectToRoute('part_edit', ['id' => $new_part->getID()]);
         }
 
 

src/Services/EntityURLGenerator.php

Spacing +5 added lines, -5 removed lines

@@ -60,7 +60,7 @@ 
      */
     public function infoURL($entity) : string
     {
-        if($entity instanceof Part)
+        if ($entity instanceof Part)
         {
             return $this->urlGenerator->generate('part_info', ['id' => $entity->getID()]);
         }
@@ -71,7 +71,7 @@ 
 
     public function editURL($entity) : string
     {
-        if($entity instanceof Part)
+        if ($entity instanceof Part)
         {
             return $this->urlGenerator->generate('part_edit', ['id' => $entity->getID()]);
         }
@@ -82,7 +82,7 @@ 
 
     public function createURL($entity) : string
     {
-        if($entity instanceof Part)
+        if ($entity instanceof Part)
         {
             return $this->urlGenerator->generate('part_new');
         }
@@ -92,7 +92,7 @@ 
 
     public function cloneURL($entity) : string
     {
-        if($entity instanceof Part)
+        if ($entity instanceof Part)
         {
             return $this->urlGenerator->generate('part_clone', ['id' => $entity->getID()]);
         }
@@ -111,7 +111,7 @@ 
     {
         $href = $this->infoURL($entity);
 
-        if($entity instanceof NamedDBElement)
+        if ($entity instanceof NamedDBElement)
         {
             return sprintf('<a href="%s">%s</a>', $href, $entity->getName());
         }

src/Command/SetPasswordCommand.php

Spacing +4 added lines, -4 removed lines

@@ -50,7 +50,7 @@ 
         $user = $users[0];
 
 
-        if($user == null)
+        if ($user == null)
         {
             $io->error(sprintf('No user with the given username %s found in the database!', $user_name));
             return;
@@ -62,7 +62,7 @@ 
             sprintf('You are going to change the password of %s with ID %d. Proceed?',
                 $user->getFullName(true), $user->getID()));
 
-        if(!$proceed)
+        if (!$proceed)
         {
             return;
         }
@@ -70,10 +70,10 @@ 
         $success = false;
         $new_password = "";
 
-        while(!$success) {
+        while (!$success) {
             $pw1 = $io->askHidden("Please enter new password:");
             $pw2 = $io->askHidden('Please confirm:');
-            if($pw1 !== $pw2) {
+            if ($pw1 !== $pw2) {
                 $io->error('The entered password did not match! Please try again.');
             } else {
                 //Exit loop