Part-DB / Part-DB-symfony

src/Security/Voter/PermissionVoter.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
             $valid = $this->resolver->isValidOperation($perm, $op);
 
             //if an invalid operation is encountered, throw an exception so the developer knows it
-            if(!$valid) {
+            if (!$valid) {
                 throw new \RuntimeException('Encountered invalid permission operation "'.$op.'" for permission "'.$perm.'"!');
             }
 

src/Security/Voter/AttachmentVoter.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
 
         //If the attachment has no element (which should not happen), we deny access, as we can not determine if the user is allowed to access the associated element
         $target_element = $subject->getElement();
-        if (! $subject instanceof Attachment || null === $target_element) {
+        if (!$subject instanceof Attachment || null === $target_element) {
             return false;
         }
 

src/Security/Voter/ParameterVoter.php

Spacing +1 added lines, -1 removed lines

@@ -112,7 +112,7 @@
         } elseif ($subject instanceof SupplierParameter) {
             $param = 'suppliers';
         } else {
-            throw new RuntimeException('Encountered unknown Parameter type: ' . get_class($subject));
+            throw new RuntimeException('Encountered unknown Parameter type: '.get_class($subject));
         }
 
         return $this->resolver->inherit($user, $param, $attribute) ?? false;

src/Security/Voter/OrderdetailVoter.php

Spacing +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
 
     protected function voteOnUser(string $attribute, $subject, User $user): bool
     {
-        if (! is_a($subject, Orderdetail::class, true)) {
+        if (!is_a($subject, Orderdetail::class, true)) {
             throw new \RuntimeException('This voter can only handle Orderdetail objects!');
         }
 

src/Security/Voter/PartLotVoter.php

Spacing +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
 
     protected function voteOnUser(string $attribute, $subject, User $user): bool
     {
-        if (! is_a($subject, PartLot::class, true)) {
+        if (!is_a($subject, PartLot::class, true)) {
             throw new \RuntimeException('This voter can only handle PartLot objects!');
         }
 

src/Command/User/UserListCommand.php

Spacing +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
                 $user->getUsername(),
                 $user->getFullName(),
                 $user->getEmail(),
-                $user->getGroup() !== null ? $user->getGroup()->getName() . ' (ID: ' . $user->getGroup()->getID() . ')' : 'No group',
+                $user->getGroup() !== null ? $user->getGroup()->getName().' (ID: '.$user->getGroup()->getID().')' : 'No group',
                 $user->isDisabled() ? 'Yes' : 'No',
             ]);
         }

src/Command/User/UpgradePermissionsSchemaCommand.php

Spacing +9 added lines, -9 removed lines

@@ -62,7 +62,7 @@ 
     {
         $io = new SymfonyStyle($input, $output);
 
-        $io->info('Target schema version number: '. PermissionData::CURRENT_SCHEMA_VERSION);
+        $io->info('Target schema version number: '.PermissionData::CURRENT_SCHEMA_VERSION);
 
         //Retrieve all users and groups
         $users = $this->em->getRepository(User::class)->findAll();
@@ -82,7 +82,7 @@ 
             }
         }
 
-        $io->info('Found '. count($groups_to_upgrade) .' groups and '. count($users_to_upgrade) .' users that need an update.');
+        $io->info('Found '.count($groups_to_upgrade).' groups and '.count($users_to_upgrade).' users that need an update.');
         if (empty($groups_to_upgrade) && empty($users_to_upgrade)) {
             $io->success('All users and group permissions schemas are up-to-date. No update needed.');
 
@@ -91,27 +91,27 @@ 
 
         //List all users and groups that need an update
         $io->section('Groups that need an update:');
-        $io->listing(array_map(function (Group $group) {
-            return $group->getName() . ' (ID: '. $group->getID() .', Current version: ' . $group->getPermissions()->getSchemaVersion() . ')';
+        $io->listing(array_map(function(Group $group) {
+            return $group->getName().' (ID: '.$group->getID().', Current version: '.$group->getPermissions()->getSchemaVersion().')';
         }, $groups_to_upgrade));
 
         $io->section('Users that need an update:');
-        $io->listing(array_map(function (User $user) {
-            return $user->getUsername() . ' (ID: '. $user->getID() .', Current version: ' . $user->getPermissions()->getSchemaVersion() . ')';
+        $io->listing(array_map(function(User $user) {
+            return $user->getUsername().' (ID: '.$user->getID().', Current version: '.$user->getPermissions()->getSchemaVersion().')';
         }, $users_to_upgrade));
 
-        if(!$io->confirm('Continue with the update?', false)) {
+        if (!$io->confirm('Continue with the update?', false)) {
             $io->warning('Update aborted.');
             return 0;
         }
 
         //Update all users and groups
         foreach ($groups_to_upgrade as $group) {
-            $io->writeln('Updating group '. $group->getName() .' (ID: '. $group->getID() .') to schema version '. PermissionData::CURRENT_SCHEMA_VERSION .'...', OutputInterface::VERBOSITY_VERBOSE);
+            $io->writeln('Updating group '.$group->getName().' (ID: '.$group->getID().') to schema version '.PermissionData::CURRENT_SCHEMA_VERSION.'...', OutputInterface::VERBOSITY_VERBOSE);
             $this->permissionSchemaUpdater->upgradeSchema($group);
         }
         foreach ($users_to_upgrade as $user) {
-            $io->writeln('Updating user '. $user->getUsername() .' (ID: '. $user->getID() .') to schema version '. PermissionData::CURRENT_SCHEMA_VERSION .'...', OutputInterface::VERBOSITY_VERBOSE);
+            $io->writeln('Updating user '.$user->getUsername().' (ID: '.$user->getID().') to schema version '.PermissionData::CURRENT_SCHEMA_VERSION.'...', OutputInterface::VERBOSITY_VERBOSE);
             $this->permissionSchemaUpdater->upgradeSchema($user);
         }
 

src/Command/User/UserEnableCommand.php

Spacing +3 added lines, -3 removed lines

@@ -60,7 +60,7 @@ 
         $all_users = $input->getOption('all');
         $disabling = $input->getOption('disable');
 
-        if(!$all_users && empty($usernames)) {
+        if (!$all_users && empty($usernames)) {
             $io->error('No users given! You have to pass atleast one username or use the --all option to use all users!');
             return self::FAILURE;
         }
@@ -68,7 +68,7 @@ 
         $repo = $this->entityManager->getRepository(User::class);
 
         $users = [];
-        if($all_users) { //If we requested to change all users at once, then get all users from repo
+        if ($all_users) { //If we requested to change all users at once, then get all users from repo
             $users = $repo->findAll();
         } else { //Otherwise, fetch the users from DB
             foreach ($usernames as $username) {
@@ -91,7 +91,7 @@ 
             return [$user->getFullName(true), $user->isDisabled() ? 'Disabled' : 'Enabled'];
         }, $users));
 
-        if(!$io->confirm('Do you want to continue?')) {
+        if (!$io->confirm('Do you want to continue?')) {
             $io->warning('Aborting!');
             return self::SUCCESS;
         }

src/Command/User/UsersPermissionsCommand.php

Spacing +4 added lines, -4 removed lines

@@ -71,10 +71,10 @@ 
         $inherit = !$input->getOption('noInherit') && !$edit_mode; //Show the non inherited perms in edit mode
 
         //Find user
-        $io->note('Finding user with username: ' . $username);
+        $io->note('Finding user with username: '.$username);
         $user = $this->userRepository->findByEmailOrName($username);
         if ($user === null) {
-            $io->error('No user found with username: ' . $username);
+            $io->error('No user found with username: '.$username);
             return Command::FAILURE;
         }
 
@@ -82,7 +82,7 @@ 
 
         $edit_mapping = $this->renderPermissionTable($output, $user, $inherit);
 
-        while($edit_mode) {
+        while ($edit_mode) {
             $index_to_edit = $io->ask('Which permission do you want to edit? Enter the index (e.g. 2-4) to edit, * for all permissions or "q" to quit', 'q');
             if ($index_to_edit === 'q') {
                 break;
@@ -97,7 +97,7 @@ 
                 $io->warning('You are about to edit all permissions. This will overwrite all permissions!');
             } else {
                 [$perm_to_edit, $op_to_edit] = $edit_mapping[$index_to_edit];
-                $io->note('Editing permission ' . $perm_to_edit . ' with operation <options=bold>' . $op_to_edit);
+                $io->note('Editing permission '.$perm_to_edit.' with operation <options=bold>'.$op_to_edit);
             }