Merge pull request #190 from vyskocilpavel/fix_ecs

Fix ecs

Author: vyskocilpavel

composer.json

@@ -20,6 +20,18 @@
         {
             "name": "Pavel Brousek",
             "email": "[email protected]"
+        },
+        {
+            "name": "Dominik Frantisek Bucik",
+            "email": "[email protected]"
+        },
+        {
+            "name": "Dominik Baranek",
+            "email": "[email protected]"
+        },
+        {
+            "name": "Dominik Baranek",
+            "email": "[email protected]"
         }
     ],
     "require": {

config-templates/module_perun_idpListsServiceDB.php

@@ -5,8 +5,6 @@
 /**
  * This is example configuration for connection to database with whitelist/greylist Copy this file to default config
  * directory and edit the properties.
- *
- * @author Pavel Vyskočil <[email protected]>
  */
 $config = [
     /*

ecs.php

@@ -27,15 +27,15 @@
         __DIR__ . '/composer.json',
     ]);
 
-    $containerConfigurator->import(SetList::CLEAN_CODE);
-    $containerConfigurator->import(SetList::SYMPLIFY);
     $containerConfigurator->import(SetList::ARRAY);
+    $containerConfigurator->import(SetList::CLEAN_CODE);
     $containerConfigurator->import(SetList::COMMON);
     $containerConfigurator->import(SetList::COMMENTS);
     $containerConfigurator->import(SetList::CONTROL_STRUCTURES);
     $containerConfigurator->import(SetList::DOCBLOCK);
     $containerConfigurator->import(SetList::NAMESPACES);
     $containerConfigurator->import(SetList::PHPUNIT);
-    $containerConfigurator->import(SetList::SPACES);
     $containerConfigurator->import(SetList::PSR_12);
+    $containerConfigurator->import(SetList::SPACES);
+    $containerConfigurator->import(SetList::SYMPLIFY);
 };

lib/Adapter.php

@@ -33,7 +33,6 @@ public function getConnector()
     /**
      * @param string $interface code of interface. Check constants of this class.
      * @return Adapter instance of this class. note it is NOT singleton.
-     * @throws Exception thrown if interface does not match any supported interface
      */
     public static function getInstance($interface)
     {
@@ -56,21 +55,18 @@ abstract public function getPerunUser($idpEntityId, $uids);
      * @param model\Vo $vo
      * @param string $name group name. Note that name of group is without VO name prefix.
      * @return Group
-     * @throws Exception if does not exists
      */
     abstract public function getGroupByName($vo, $name);
 
     /**
      * @param string $voShortName
      * @return Vo
-     * @throws Exception
      */
     abstract public function getVoByShortName($voShortName);
 
     /**
      * @param integer $id
      * @return Vo
-     * @throws Exception if does not exists
      */
     abstract public function getVoById($id);
 

lib/AdapterLdap.php

@@ -21,10 +21,6 @@
  * folder.
  *
  * Perun adapter which uses Perun LDAP interface
- *
- * @author Ondrej Velisek <[email protected]>
- * @author Michal Prochazka <[email protected]>
- * @author Pavel Vyskocil <[email protected]>
  */
 class AdapterLdap extends Adapter
 {

lib/AdapterRpc.php

@@ -19,9 +19,6 @@
  * Class sspmod_perun_AdapterRpc
  *
  * Perun adapter which uses Perun RPC interface
- *
- * @author Michal Prochazka <[email protected]>
- * @author Pavel Vyskocil <[email protected]>
  */
 class AdapterRpc extends Adapter
 {

lib/AttributeTransformer.php

@@ -2,32 +2,6 @@
 
 declare(strict_types=1);
 
-/**
- * @author Pavel Brousek <[email protected]>
- * ───────────▄▄▄▄▄▄▄▄▄───────────
- * ────────▄█████████████▄────────
- * █████──█████████████████──█████
- * ▐████▌─▀███▄───────▄███▀─▐████▌
- * ─█████▄──▀███▄───▄███▀──▄█████─
- * ─▐██▀███▄──▀███▄███▀──▄███▀██▌─
- * ──███▄▀███▄──▀███▀──▄███▀▄███──
- * ──▐█▄▀█▄▀███─▄─▀─▄─███▀▄█▀▄█▌──
- * ───███▄▀█▄██─██▄██─██▄█▀▄███───
- * ────▀███▄▀██─█████─██▀▄███▀────
- * ───█▄─▀█████─█████─█████▀─▄█───
- * ───███────────███────────███───
- * ───███▄────▄█─███─█▄────▄███───
- * ───█████─▄███─███─███▄─█████───
- * ───█████─████─███─████─█████───
- * ───█████─████─███─████─█████───
- * ───█████─████─███─████─█████───
- * ───█████─████▄▄▄▄▄████─█████───
- * ────▀███─█████████████─███▀────
- * ──────▀█─███─▄▄▄▄▄─███─█▀──────
- * ─────────▀█▌▐█████▌▐█▀─────────
- * ────────────███████────────────
- */
-
 namespace SimpleSAML\Module\perun;
 
 abstract class AttributeTransformer

lib/Auth/Process/IdPAttribute.php

@@ -11,8 +11,6 @@
  * Class IdPAttribute
  *
  * This class for each line in $attrMAp search the $key in IdP Metadata and save it to $request['Attributes'][$value]
- *
- * @author Pavel Vyskocil <[email protected]>
  */
 class IdPAttribute extends \SimpleSAML\Auth\ProcessingFilter
 {

lib/Auth/Process/PerunAttributes.php

@@ -19,8 +19,6 @@
  * It strongly relays on PerunIdentity filter to obtain perun user id. Configure it before this filter properly.
  *
  * if attribute in Perun value is null or is not set at all SSP attribute is set to empty array.
- *
- * @author Ondrej Velisek <[email protected]>
  */
 class PerunAttributes extends ProcessingFilter
 {
@@ -126,7 +124,6 @@ private function hasStringKeys($array): bool
      *
      * @param $user
      * @param array $attributes List of attributes which will be loaded from Perun system
-     * @throws Exception
      */
     private function processAttributes(User $user, array $attributes): array
     {

lib/Auth/Process/PerunEntitlement.php

@@ -16,9 +16,6 @@
  *
  * This filter joins eduPersonEntitlement, forwardedEduPersonEntitlement, resource capabilities and facility
  * capabilities
- *
- * @author Dominik Baránek <[email protected]>
- * @author Pavel Vyskočil <[email protected]>
  */
 class PerunEntitlement extends ProcessingFilter
 {