tfboe / fm-lib

src/Helpers/BasicEnum.php

Spacing +15 added lines, -15 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 /**
  * Created by PhpStorm.
  * User: benedikt
@@ -32,9 +32,9 @@ 
    * @param bool $strict if yes type checks are performed
    * @throws ValueNotValid if the value is not valid
    */
-  public static function ensureValidValue($value, bool $strict = true): void
+  public static function ensureValidValue( $value, bool $strict = true ): void
   {
-    if (!self::isValidValue($value, $strict)) {
+    if ( !self::isValidValue($value, $strict) ) {
       throw new ValueNotValid($value, get_called_class());
     }
   }
@@ -55,17 +55,17 @@ 
    * @return mixed the corresponding value
    * @throws ValueNotValid if the name is not valid
    */
-  public static function getValue(string $name, bool $strict = false)
+  public static function getValue( string $name, bool $strict = false )
   {
     $constants = self::getConstants();
-    if ($strict) {
-      if (array_key_exists($name, $constants)) {
+    if ( $strict ) {
+      if ( array_key_exists($name, $constants) ) {
         return $constants[$name];
       }
     } else {
       $mapping = self::getCaseMapping();
       $key = strtolower($name);
-      if (array_key_exists($key, $mapping)) {
+      if ( array_key_exists($key, $mapping) ) {
         return $constants[$mapping[$key]];
       }
     }
@@ -88,11 +88,11 @@ 
    * @param bool $strict if yes check is done case sensitive and otherwise case insensitive
    * @return bool true if the name is part of the enum and false otherwise
    */
-  public static function isValidName(string $name, bool $strict = false): bool
+  public static function isValidName( string $name, bool $strict = false ): bool
   {
     $constants = self::getConstants();
 
-    if ($strict) {
+    if ( $strict ) {
       return array_key_exists($name, $constants);
     }
 
@@ -106,7 +106,7 @@ 
    * @param bool $strict if yes type checks are performed
    * @return bool true if the value is part of the enum and false otherwise
    */
-  public static function isValidValue($value, bool $strict = true): bool
+  public static function isValidValue( $value, bool $strict = true ): bool
   {
     $values = self::getValues();
     return in_array($value, $values, $strict);
@@ -121,13 +121,13 @@ 
    */
   private static function getCaseMapping(): array
   {
-    if (self::$constCacheArrayCaseMapping == NULL) {
+    if ( self::$constCacheArrayCaseMapping == NULL ) {
       self::$constCacheArrayCaseMapping = [];
     }
     $calledClass = get_called_class();
-    if (!array_key_exists($calledClass, self::$constCacheArrayCaseMapping)) {
+    if ( !array_key_exists($calledClass, self::$constCacheArrayCaseMapping) ) {
       self::$constCacheArrayCaseMapping[$calledClass] = [];
-      foreach (self::getNames() as $name) {
+      foreach ( self::getNames() as $name ) {
         self::$constCacheArrayCaseMapping[$calledClass][strtolower($name)] = $name;
       }
     }
@@ -141,11 +141,11 @@ 
    */
   private static function getConstants(): array
   {
-    if (self::$constCacheArray == NULL) {
+    if ( self::$constCacheArray == NULL ) {
       self::$constCacheArray = [];
     }
     $calledClass = get_called_class();
-    if (!array_key_exists($calledClass, self::$constCacheArray)) {
+    if ( !array_key_exists($calledClass, self::$constCacheArray) ) {
       // ReflectionException => whe know that calledClass is a valid class since it is the result of get_called_class
       /** @noinspection PhpUnhandledExceptionInspection */
       $reflect = new \ReflectionClass($calledClass);

src/Providers/FmLibServiceProvider.php

Spacing +7 added lines, -7 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 /**
  * Created by PhpStorm.
  * User: benedikt
@@ -44,7 +44,7 @@ 
    */
   public function boot()
   {
-    include __DIR__ . '/../routes.php';
+    include __DIR__.'/../routes.php';
   }
 
   /** @noinspection PhpDocMissingThrowsInspection */ //\Doctrine\DBAL\DBALException
@@ -63,8 +63,8 @@ 
       Handler::class
     );
 
-    if ($this->app->environment() !== 'production') {
-      if (class_exists('\Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider')) {
+    if ( $this->app->environment() !== 'production' ) {
+      if ( class_exists('\Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider') ) {
         $this->app->register('\Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider');
       }
     }
@@ -77,16 +77,16 @@ 
     $this->app->register(GedmoExtensionsServiceProvider::class);
     $this->app->register(JwtAuthGuardServiceProvider::class);
 
-    $this->app->singleton(DynamicServiceLoadingServiceInterface::class, function (Container $app) {
+    $this->app->singleton(DynamicServiceLoadingServiceInterface::class, function( Container $app ) {
       return new DynamicServiceLoadingService($app);
     });
 
-    $this->app->singleton(RankingSystemServiceInterface::class, function (Container $app) {
+    $this->app->singleton(RankingSystemServiceInterface::class, function( Container $app ) {
       return new RankingSystemService($app->make(DynamicServiceLoadingServiceInterface::class),
         $app->make(EntityManagerInterface::class));
     });
 
-    $this->app->singleton(EloRankingInterface::class, function (Container $app) {
+    $this->app->singleton(EloRankingInterface::class, function( Container $app ) {
       $timeService = new RecursiveEndStartTimeService();
       $entityComparer = new EntityComparerByTimeStartTimeAndLocalIdentifier($timeService);
       return new EloRanking($app->make(EntityManagerInterface::class), $timeService, $entityComparer);

src/TestHelpers/LumenTestCase.php

Spacing +12 added lines, -12 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Class TestCase
@@ -28,21 +28,21 @@ 
    * @param mixed[] $enumProperties the enum properties to check, property name maps to an info array, which contains
    *                                the enum name and the default value
    */
-  protected function checkProperties(array $data, $object, array $properties, array $enumProperties = [])
+  protected function checkProperties( array $data, $object, array $properties, array $enumProperties = [] )
   {
-    foreach ($properties as $property => $default) {
-      $getter = 'get' . ucfirst($property);
-      if (!method_exists($object, $getter)) {
-        $getter = 'is' . ucfirst($property);
+    foreach ( $properties as $property => $default ) {
+      $getter = 'get'.ucfirst($property);
+      if ( !method_exists($object, $getter) ) {
+        $getter = 'is'.ucfirst($property);
       }
       $transformer = null;
-      if (is_array($default) && array_key_exists('transformer', $default)) {
+      if ( is_array($default) && array_key_exists('transformer', $default) ) {
         $transformer = $default['transformer'];
         $default = $default['default'];
       }
-      if (array_key_exists($property, $data)) {
+      if ( array_key_exists($property, $data) ) {
         $value = $data[$property];
-        if ($transformer != null) {
+        if ( $transformer != null ) {
           $value = $transformer($value);
         }
         self::assertEquals($value, $object->$getter());
@@ -51,11 +51,11 @@ 
       }
     }
 
-    foreach ($enumProperties as $property => $info) {
+    foreach ( $enumProperties as $property => $info ) {
       $enumClass = $info['enum'];
       $default = $info['default'];
-      $getter = 'get' . ucfirst($property);
-      if (array_key_exists($property, $data)) {
+      $getter = 'get'.ucfirst($property);
+      if ( array_key_exists($property, $data) ) {
         $name = $data[$property];
         /** @noinspection PhpUndefinedMethodInspection */
         self::assertEquals($enumClass::getValue($name), $object->$getter());

src/TestHelpers/AuthenticatedTestCase.php

Spacing +3 added lines, -3 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Created by PhpStorm.
@@ -44,9 +44,9 @@ 
    * @param array $headers the request headers
    * @return $this
    */
-  protected function jsonAuth(string $method, string $uri, array $data = [], array $headers = [])
+  protected function jsonAuth( string $method, string $uri, array $data = [], array $headers = [] )
   {
-    $headers['Authorization'] = 'Bearer ' . $this->token;
+    $headers['Authorization'] = 'Bearer '.$this->token;
     return $this->json($method, $uri, $data, $headers);
   }
 

src/TestHelpers/DatabaseTestCase.php

Spacing +11 added lines, -11 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Class DatabaseTestCase
@@ -39,7 +39,7 @@ 
    * @param string $dataName test data name
    * @param bool $clear
    */
-  public function __construct($name = null, array $data = [], $dataName = '', $clear = false)
+  public function __construct( $name = null, array $data = [], $dataName = '', $clear = false )
   {
     parent::__construct($name, $data, $dataName);
     srand(3); //always use the same faker values to get reproducibility
@@ -61,7 +61,7 @@ 
     $connection = EntityManager::getConnection();
     $connection->query(sprintf('SET FOREIGN_KEY_CHECKS = 0;'));
     $tables = $connection->getSchemaManager()->listTables();
-    foreach ($tables as $table) {
+    foreach ( $tables as $table ) {
       $sql = sprintf('TRUNCATE TABLE %s', $table->getName());
       $connection->query($sql);
     }
@@ -73,10 +73,10 @@ 
    * @param int $number the number of players
    * @return Player[] the created player array
    */
-  protected function createPlayers(int $number = 1): array
+  protected function createPlayers( int $number = 1 ): array
   {
     $result = [];
-    for ($i = 0; $i < $number; $i++) {
+    for ( $i = 0; $i < $number; $i++ ) {
       $result[] = entity(Player::class)->create();
     }
     return $result;
@@ -88,13 +88,13 @@ 
    * @param int $playerPerTeam the number of players per team
    * @return Team[] the created team array
    */
-  protected function createTeams(int $number, $playerPerTeam = 1): array
+  protected function createTeams( int $number, $playerPerTeam = 1 ): array
   {
     $result = [];
-    for ($i = 0; $i < $number; $i++) {
+    for ( $i = 0; $i < $number; $i++ ) {
       /** @var Team $team */
       $team = entity(Team::class)->create(['startNumber' => $i + 1, 'rank' => $number - $i]);
-      foreach ($this->createPlayers($playerPerTeam) as $player) {
+      foreach ( $this->createPlayers($playerPerTeam) as $player ) {
         $team->getPlayers()->add($player);
       }
       $result[] = $team;
@@ -137,7 +137,7 @@ 
     srand(3); //always use the same faker values to get reproducibility
     $clear = $this->clear;
     parent::setUpTraits();
-    if ($clear) {
+    if ( $clear ) {
       $this->clearDatabase();
       $this->workOnDatabaseSetUp();
     } else {
@@ -146,8 +146,8 @@ 
       EntityManager::beginTransaction();
     }
 
-    $this->beforeApplicationDestroyed(function () use ($clear) {
-      if ($clear) {
+    $this->beforeApplicationDestroyed(function() use ($clear) {
+      if ( $clear ) {
         $this->workOnDatabaseDestroy();
         $this->clearDatabase();
       } else {

src/TestHelpers/ReflectionMethods.php

Spacing +5 added lines, -5 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 /**
  * Created by PhpStorm.
  * User: benedikt
@@ -24,7 +24,7 @@ 
    * @param mixed[] $args the arguments for the method
    * @return mixed the return value of the method
    */
-  protected static function callProtectedMethod($object, string $method, array $args = [])
+  protected static function callProtectedMethod( $object, string $method, array $args = [] )
   {
     // ReflectionException => get_class is a valid class
     /** @noinspection PhpUnhandledExceptionInspection */
@@ -38,7 +38,7 @@ 
    * @return \ReflectionMethod the accessible method object
    * @throws \ReflectionException the given class does not exist
    */
-  protected static function getMethod(string $class, string $name): \ReflectionMethod
+  protected static function getMethod( string $class, string $name ): \ReflectionMethod
   {
     $class = new \ReflectionClass($class);
     $method = $class->getMethod($name);
@@ -53,11 +53,11 @@ 
    * @return \ReflectionProperty the accessible property object
    * @throws \ReflectionException the given class does not exist
    */
-  protected static function getProperty(string $class, string $name): \ReflectionProperty
+  protected static function getProperty( string $class, string $name ): \ReflectionProperty
   {
     $class = new \ReflectionClass($class);
     /** @noinspection PhpStatementHasEmptyBodyInspection */
-    while (!$class->hasProperty($name) && ($class = $class->getParentClass()) !== null) {
+    while ( !$class->hasProperty($name) && ($class = $class->getParentClass()) !== null ) {
     }
     $property = $class->getProperty($name);
     $property->setAccessible(true);

src/TestHelpers/UnitTestCase.php

Spacing +13 added lines, -13 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 /**
  * Created by PhpStorm.
  * User: benedikt
@@ -32,10 +32,10 @@ 
    * @param array $methodResults a dictionary mapping method names to results of this methods
    * @return MockObject the configured stub
    */
-  protected function createStub(string $class, array $methodResults): MockObject
+  protected function createStub( string $class, array $methodResults ): MockObject
   {
     $entity = $this->createMock($class);
-    foreach ($methodResults as $method => $result) {
+    foreach ( $methodResults as $method => $result ) {
       $entity->method($method)->willReturn($result);
     }
     return $entity;
@@ -48,7 +48,7 @@ 
    * @param string $getterMethod the name of the getter method
    * @return \PHPUnit\Framework\MockObject\MockObject the mocked instance
    */
-  protected function createStubWithId(string $class, $entityId = "entity-id", $getterMethod = 'getId')
+  protected function createStubWithId( string $class, $entityId = "entity-id", $getterMethod = 'getId' )
   {
     return $this->createStub($class, [$getterMethod => $entityId]);
   }
@@ -61,8 +61,8 @@ 
    * @param string[] $otherMockedMethods list of other methods to mock
    * @return MockObject the mocked entity manager
    */
-  protected function getEntityManagerMockForQuery(array $result, ?string $expectedQuery = null,
-                                                  array $otherMockedMethods = [])
+  protected function getEntityManagerMockForQuery( array $result, ?string $expectedQuery = null,
+                                                  array $otherMockedMethods = [] )
   {
     $entityManager = $this->getMockForAbstractClass(EntityManager::class, [], '',
       false, true, true, array_merge($otherMockedMethods, ['createQueryBuilder']));
@@ -70,9 +70,9 @@ 
       '', true, true, true, ['getQuery']);
     $query = $this->createMock(AbstractQuery::class);
     $query->expects(static::once())->method('getResult')->willReturn($result);
-    if ($expectedQuery !== null) {
+    if ( $expectedQuery !== null ) {
       $queryBuilder->expects(static::once())->method('getQuery')->willReturnCallback(
-        function () use ($queryBuilder, $query, $expectedQuery) {
+        function() use ($queryBuilder, $query, $expectedQuery) {
           /** @var QueryBuilder $queryBuilder */
           self::assertEquals($expectedQuery, $queryBuilder->getDQL());
           return $query;
@@ -93,14 +93,14 @@ 
    * @param string[] $mockedMethods the methods to mock in the class
    * @return MockObject the mocked object
    */
-  protected final function getMockWithMockedArguments(string $className, array $arguments = [],
-                                                      array $mockedMethods = []): MockObject
+  protected final function getMockWithMockedArguments( string $className, array $arguments = [],
+                                                      array $mockedMethods = [] ): MockObject
   {
     /** @noinspection PhpUnhandledExceptionInspection */
     $reflection = new \ReflectionClass($className);
     $params = $reflection->getConstructor()->getParameters();
     $allArguments = $arguments;
-    for ($i = count($arguments); $i < count($params); $i++) {
+    for ( $i = count($arguments); $i < count($params); $i++ ) {
       $allArguments[] = $this->createMock($params[$i]->getClass()->name);
     }
     return $this->getMockForAbstractClass($className, $allArguments, '', true, true, true, $mockedMethods);
@@ -114,13 +114,13 @@ 
    * @param array $arguments the arguments to use for the constructor
    * @return mixed an instance of the given class
    */
-  protected final function getObjectWithMockedArguments($className, array $arguments = [])
+  protected final function getObjectWithMockedArguments( $className, array $arguments = [] )
   {
     /** @noinspection PhpUnhandledExceptionInspection */
     $reflection = new \ReflectionClass($className);
     $params = $reflection->getConstructor()->getParameters();
     $allArguments = $arguments;
-    for ($i = count($arguments); $i < count($params); $i++) {
+    for ( $i = count($arguments); $i < count($params); $i++ ) {
       $allArguments[$i] = $this->createMock($params[$i]->getClass()->name);
     }
     return new $className(...$allArguments);

src/TestHelpers/OnlyTestLogging.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 /**
  * Created by PhpStorm.
  * User: benedikt

src/TestHelpers/TestEnum.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 /**
  * Created by PhpStorm.
  * User: benedikt