Part-DB / Part-DB-symfony

src/Entity/User.php

Spacing +3 added lines, -3 removed lines

@@ -48,7 +48,7 @@ 
 class User extends NamedDBElement implements UserInterface, HasPermissionsInterface
 {
     /** The User id of the anonymous user */
-    const ID_ANONYMOUS      = 1;
+    const ID_ANONYMOUS = 1;
 
     /**
      * @ORM\Id()
@@ -215,7 +215,7 @@ 
      */
     public function getIDString(): string
     {
-        return 'U' . sprintf('%06d', $this->getID());
+        return 'U'.sprintf('%06d', $this->getID());
     }
 
 
@@ -231,7 +231,7 @@ 
     public function setName(string $new_name) : NamedDBElement
     {
         // Anonymous user is not allowed to change its username
-        if(!$this->isAnonymousUser()) {
+        if (!$this->isAnonymousUser()) {
             $this->name = $new_name;
         }
 

src/Controller/UserController.php

Spacing +5 added lines, -5 removed lines

@@ -56,14 +56,14 @@ 
     public function userInfo(?User $user, Packages $packages)
     {
         //If no user id was passed, then we show info about the current user
-        if($user == null) {
+        if ($user == null) {
             $user = $this->getUser();
         } else {
             //Else we must check, if the current user is allowed to access $user
             $this->denyAccessUnlessGranted('read', $user);
         }
 
-        if($this->getParameter("use_gravatar")) {
+        if ($this->getParameter("use_gravatar")) {
             $avatar = $this->getGravatar($user->getEmail(), 200, 'identicon');
         } else {
             $avatar = $packages->getUrl("/img/default_avatar.png");
@@ -127,7 +127,7 @@ 
         $pw_form->handleRequest($request);
 
         //Check if password if everything was correct, then save it to User and DB
-        if($pw_form->isSubmitted() && $pw_form->isValid()) {
+        if ($pw_form->isSubmitted() && $pw_form->isValid()) {
             $password = $passwordEncoder->encodePassword($user, $pw_form['new_password']->getData());
             $user->setPassword($password);
             $em->persist($user);
@@ -164,9 +164,9 @@ 
         $url .= md5(strtolower(trim($email)));
         $url .= "?s=$s&d=$d&r=$r";
         if ($img) {
-            $url = '<img src="' . $url . '"';
+            $url = '<img src="'.$url.'"';
             foreach ($atts as $key => $val) {
-                $url .= ' ' . $key . '="' . $val . '"';
+                $url .= ' '.$key.'="'.$val.'"';
             }
             $url .= ' />';
         }

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());
         }