Part-DB / Part-DB-symfony

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

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

src/Entity/User.php

Spacing +5 added lines, -5 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()
@@ -221,7 +221,7 @@ 
      */
     public function getIDString(): string
     {
-        return 'U' . sprintf('%06d', $this->getID());
+        return 'U'.sprintf('%06d', $this->getID());
     }
 
 
@@ -242,9 +242,9 @@ 
      */
     public function getFullName(bool $including_username = false)
     {
-        $str = $this->getFirstName() . ' ' . $this->getLastName();
+        $str = $this->getFirstName().' '.$this->getLastName();
         if ($including_username) {
-            $str .= ' (' . $this->getName() . ')';
+            $str .= ' ('.$this->getName().')';
         }
 
         return $str;
@@ -254,7 +254,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/Security/Annotations/ColumnSecurity.php

Spacing +6 added lines, -6 removed lines

@@ -74,16 +74,16 @@ 
 
     public function getReadOperationName() : string
     {
-        if($this->prefix !== '') {
-            return $this->prefix . '.' . $this->read;
+        if ($this->prefix !== '') {
+            return $this->prefix.'.'.$this->read;
         }
         return $this->read;
     }
 
     public function getEditOperationName() : string
     {
-        if($this->prefix !== '') {
-            return $this->prefix . '.' . $this->edit;
+        if ($this->prefix !== '') {
+            return $this->prefix.'.'.$this->edit;
         }
 
         return $this->edit;
@@ -91,9 +91,9 @@ 
 
     public function getPlaceholder()
     {
-        if($this->placeholder === null)
+        if ($this->placeholder === null)
         {
-            switch($this->type)
+            switch ($this->type)
             {
                 case 'integer':
                     return 0;

src/Security/EntityListeners/ElementPermissionListener.php

Spacing +6 added lines, -6 removed lines

@@ -75,7 +75,7 @@ 
         $properties = $reflectionClass->getProperties();
         $reader = new AnnotationReader();
 
-        foreach($properties as $property)
+        foreach ($properties as $property)
         {
             /**
              * @var ColumnSecurity $annotation
@@ -83,10 +83,10 @@ 
             $annotation = $reader->getPropertyAnnotation($property,
                 ColumnSecurity::class);
 
-            if($annotation !== null)
+            if ($annotation !== null)
             {
                 //Check if user is allowed to read info, otherwise apply placeholder
-                if(!$this->security->isGranted($annotation->getReadOperationName(), $element))
+                if (!$this->security->isGranted($annotation->getReadOperationName(), $element))
                 {
                     $property->setAccessible(true);
                     $property->setValue($element, $annotation->getPlaceholder());
@@ -106,7 +106,7 @@ 
         $properties = $reflectionClass->getProperties();
         $reader = new AnnotationReader();
 
-        foreach($properties as $property)
+        foreach ($properties as $property)
         {
             /**
              * @var ColumnSecurity $annotation
@@ -114,13 +114,13 @@ 
             $annotation = $reader->getPropertyAnnotation($property,
                 ColumnSecurity::class);
 
-            if($annotation !== null)
+            if ($annotation !== null)
             {
                 $field_name = $property->getName();
 
                 //Check if user is allowed to edit info, otherwise overwrite the new value
                 // so that nothing is changed in the DB.
-                if($event->hasChangedField($field_name) &&
+                if ($event->hasChangedField($field_name) &&
                     !$this->security->isGranted($annotation->getEditOperationName(), $element))
                 {
                     $event->setNewValue($field_name, $event->getOldValue($field_name));