Jelle-S / TheSportsDb

src/Entity/Entity.php

Doc Comments +4 added lines

@@ -90,6 +90,10 @@
     return $reversedEntities;
   }
 
+  /**
+   * @param string $entityType
+   * @param string $idName
+   */
   public static function transform($value, $context, EntityManagerInterface $entityManager, $entityType, $idName, array $contextPropertyMap = array()) {
     $data = static::transformHelper($value, $context, $idName, $contextPropertyMap);
     $entity = $entityManager->repository($entityType)->byId($data['id']);

Braces +1 added lines, -2 removed lines

@@ -47,8 +47,7 @@
           $this->_raw->{$prop} = $val;
           if (method_exists($val, 'raw')) {
             $this->_raw->{$prop} = $val->raw();
-          }
-          elseif (is_array($val)) {
+          } elseif (is_array($val)) {
             $this->_raw->{$prop} = array();
             foreach ($val as $v) {
               $this->_raw->{$prop}[] = method_exists($v, 'raw') ? $v->raw() : $v;

src/Entity/EntityManager.php

Doc Comments +2 added lines, -1 removed lines

@@ -220,7 +220,8 @@
   /**
    * Gets the property map.
    *
-   * @return FastNorth\PropertyMapper\Map
+   * @param string $entityType
+   * @return \FastNorth\PropertyMapper\MapInterface
    *   The property map.
    */
   protected function getPropertyMap($entityType) {

Spacing +2 added lines, -2 removed lines

@@ -199,13 +199,13 @@
         $transform = $args[2][0];
         $reverse = $args[2][1];
         $args[2] = new Callback(
-          function ($value, $context) use ($entityManager, $transform) {
+          function($value, $context) use ($entityManager, $transform) {
             if ($entityManager->isEmptyValue($value)) {
               return $value;
             }
             return call_user_func_array($transform, array($value, $context, $entityManager));
           },
-          function ($value, $context) use ($entityManager, $reverse) {
+          function($value, $context) use ($entityManager, $reverse) {
             if ($entityManager->isEmptyValue($value)) {
               return $value;
             }

Braces +1 added lines, -2 removed lines

@@ -77,8 +77,7 @@
     if (isset($this->repository[$mapped->id])) {
       $entity = $this->repository[$mapped->id];
       $entity->update($mapped);
-    }
-    else {
+    } else {
       $factory =  $this->entityManager->factory($this->getEntityTypeName());
       $this->repository[$mapped->id] = $factory->create($mapped, $this->getEntityTypeName(), FALSE);
     }

Unused Use Statements +2 added lines, -2 removed lines

@@ -6,10 +6,10 @@
 
 namespace TheSportsDb\Entity;
 
+use FastNorth\PropertyMapper\Map;
+use FastNorth\PropertyMapper\MapperInterface;
 use TheSportsDb\Entity\Factory\FactoryContainerInterface;
 use TheSportsDb\Entity\Repository\RepositoryContainerInterface;
-use FastNorth\PropertyMapper\MapperInterface;
-use FastNorth\PropertyMapper\Map;
 use TheSportsDb\PropertyMapper\Transformer\Callback;
 
 /**

src/Entity/Factory/Factory.php

Doc Comments +5 added lines, -8 removed lines

@@ -32,16 +32,10 @@ 
   /**
    * Creates a \TheSportsDb\Facotory\Factory object.
    *
-   * @param TheSportsDb\Http\TheSportsDbClientInterface $sportsDbClient
+   * @param TheSportsDbClientInterface $sportsDbClient
    *   The sports db client to make the requests.
-   * @param string $realClass
-   *   The fully qualified classname of the entity to create.
-   * @param string $proxyClass
-   *   The fully qualified classname of the proxy entity to create.
-   * @param TheSportsDb\Entity\EntityManagerInterface $entityManager
+   * @param EntityManagerInterface $entityManager
    *   The factory container.
-   * @param MapperInterface $propertyMapper
-   *   The property mapper.
    */
   public function __construct(TheSportsDbClientInterface $sportsDbClient, EntityManagerInterface $entityManager = NULL) {
     $this->sportsDbClient = $sportsDbClient;
@@ -76,6 +70,9 @@ 
     return $entity;
   }
 
+  /**
+   * @param string $entityType
+   */
   public function isFullObject(\stdClass $object, $entityType) {
     $reflection = new \ReflectionClass($this->entityManager->getClass($entityType));
     $defaultProperties = $reflection->getDefaultProperties();

Braces +2 added lines, -4 removed lines

@@ -57,16 +57,14 @@
     $reflection = new \ReflectionClass($this->entityManager->getClass($entityType));
     if ($mapProperties) {
       $givenProperties = $this->entityManager->mapProperties($values, $entityType);
-    }
-    else {
+    } else {
       $givenProperties = $values;
     }
     // Not all properties are loaded, return a proxy.
     if (!$this->isFullObject($givenProperties, $entityType)) {
       $proxyReflection = new \ReflectionClass($this->entityManager->getClass($entityType, 'proxy'));
       $entity = $proxyReflection->newInstance($givenProperties);
-    }
-    else {
+    } else {
       // All properties are loaded, return a full object.
       $entity = $reflection->newInstance($givenProperties);
     }

Unused Use Statements +4 added lines, -4 removed lines

@@ -6,12 +6,12 @@
 
 namespace TheSportsDb\Entity\Factory;
 
-use TheSportsDb\Http\TheSportsDbClientInterface;
-use TheSportsDb\Entity\EntityInterface;
-use TheSportsDb\Entity\Proxy\ProxyInterface;
 use FastNorth\PropertyMapper\MapperInterface;
-use TheSportsDb\Entity\EntityManagerInterface;
+use TheSportsDb\Entity\EntityInterface;
 use TheSportsDb\Entity\EntityManagerConsumerTrait;
+use TheSportsDb\Entity\EntityManagerInterface;
+use TheSportsDb\Entity\Proxy\ProxyInterface;
+use TheSportsDb\Http\TheSportsDbClientInterface;
 
 /**
  * Default implementation of factories.

src/Entity/Proxy/Proxy.php

Doc Comments -4 removed lines

@@ -44,10 +44,6 @@
    *
    * @param \stdClass $values
    *   The sport data.
-   * @param \TheSportsDb\Http\TheSportsDbClientInterface $sportsDbClient
-   *   A sports db client
-   * @param \TheSportsDb\Entity\Factory\FactoryInterface $factory
-   *   The sport factory.
    */
   public function __construct(\stdClass $values) {
     $this->properties = new \stdClass();

Braces +2 added lines, -4 removed lines

@@ -100,8 +100,7 @@ 
     // full yet, so load it first and repeat the operation.
     if (method_exists($this, 'load' . ucfirst($name))) {
       $this->{'load' . ucfirst($name)}();
-    }
-    else {
+    } else {
       $this->load();
     }
     return $this->get($name);
@@ -136,8 +135,7 @@ 
           $this->_raw->{$prop} = $val;
           if (method_exists($val, 'raw')) {
             $this->_raw->{$prop} = $val->raw();
-          }
-          elseif (is_array($val)) {
+          } elseif (is_array($val)) {
             $this->_raw->{$prop} = array();
             foreach ($val as $v) {
               $this->_raw->{$prop}[] = method_exists($v, 'raw') ? $v->raw() : $v;

Unused Use Statements +2 added lines, -2 removed lines

@@ -6,10 +6,10 @@
 
 namespace TheSportsDb\Entity\Proxy;
 
-use TheSportsDb\Http\TheSportsDbClientInterface;
-use TheSportsDb\Entity\EntityManagerInterface;
 use TheSportsDb\Entity\EntityInterface;
 use TheSportsDb\Entity\EntityManagerConsumerTrait;
+use TheSportsDb\Entity\EntityManagerInterface;
+use TheSportsDb\Http\TheSportsDbClientInterface;
 
 /**
  * Default implementation of proxy objects.

src/TheSportsDb.php

Doc Comments +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
   /**
    * Get a league by id.
    *
-   * @param int $league_id
+   * @param int $leagueId
    *   The league id.
    *
    * @return \TheSportsDb\Entity\LeagueInterface

src/Entity/Event.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
     )),
     array('strDescriptionEN', 'description'),
     array('intHomeScore', 'homeScore'),
-    array('intRound' , 'round'),
+    array('intRound', 'round'),
     array('intAwayScore', 'awayScore'),
     array('intSpectators', 'spectators'),
     array('strHomeGoalDetails', 'homeGoalDetails'),

src/Entity/Team.php

Indentation +29 added lines, -29 removed lines

@@ -31,47 +31,47 @@
     )),
     array('strDivision', 'division'),
     array('strManager', 'manager'),
-	array('strStadium', 'stadium'),
+  array('strStadium', 'stadium'),
     array('strKeywords', 'keywords'),
-	array('strRSS', 'rss'),
+  array('strRSS', 'rss'),
     array('strStadiumThumb', 'stadiumThumb'),
     array('strStadiumDescription', 'stadiumDescription'),
-	array('strStadiumLocation', 'stadiumLocation'),
-	array('intStadiumCapacity', 'stadiumCapacity'),
-	array('strWebsite', 'website'),
+  array('strStadiumLocation', 'stadiumLocation'),
+  array('intStadiumCapacity', 'stadiumCapacity'),
+  array('strWebsite', 'website'),
     array('strFacebook', 'facebook'),
-	array('strTwitter', 'twitter'),
-	array('strInstagram', 'instagram'),
-	array('strDescriptionEN', 'description'),
-	array('strGender', 'gender'),
+  array('strTwitter', 'twitter'),
+  array('strInstagram', 'instagram'),
+  array('strDescriptionEN', 'description'),
+  array('strGender', 'gender'),
     array('strCountry', 'country'),
-	array('strTeamBadge', 'badge'),
+  array('strTeamBadge', 'badge'),
     array('strTeamJersey', 'jersey'),
-	array('strTeamLogo', 'logo'),
-	array('strTeamBanner', 'banner'),
-	array('strYoutube', 'youtube'),
+  array('strTeamLogo', 'logo'),
+  array('strTeamBanner', 'banner'),
+  array('strYoutube', 'youtube'),
     array('strLocked', 'locked'),
     // idSoccerXML
     // intLoved
     // strLeague
     // strDescriptionDE
-	// strDescriptionFR
-	// strDescriptionCN
-	// strDescriptionIT
-	// strDescriptionJP
-	// strDescriptionRU
-	// strDescriptionES
-	// strDescriptionPT
-	// strDescriptionSE
-	// strDescriptionNL
-	// strDescriptionHU
-	// strDescriptionNO
-	// strDescriptionIL
-	// strDescriptionPL
+  // strDescriptionFR
+  // strDescriptionCN
+  // strDescriptionIT
+  // strDescriptionJP
+  // strDescriptionRU
+  // strDescriptionES
+  // strDescriptionPT
+  // strDescriptionSE
+  // strDescriptionNL
+  // strDescriptionHU
+  // strDescriptionNO
+  // strDescriptionIL
+  // strDescriptionPL
     // strTeamFanart1
-	// strTeamFanart2
-	// strTeamFanart3
-	// strTeamFanart4
+  // strTeamFanart2
+  // strTeamFanart3
+  // strTeamFanart4
   );
 
   protected $id;

src/Entity/Repository/Repository.php

Spacing +2 added lines, -2 removed lines

@@ -48,7 +48,7 @@ 
    */
   public function byId($id) {
     if (!isset($this->repository[$id])) {
-      $factory =  $this->entityManager->factory($this->getEntityTypeName());
+      $factory = $this->entityManager->factory($this->getEntityTypeName());
       $this->repository[$id] = $factory->create(
         (object) array('id' => $id),
         $this->getEntityTypeName(),
@@ -79,7 +79,7 @@ 
       $entity->update($mapped);
     }
     else {
-      $factory =  $this->entityManager->factory($this->getEntityTypeName());
+      $factory = $this->entityManager->factory($this->getEntityTypeName());
       $this->repository[$mapped->id] = $factory->create($mapped, $this->getEntityTypeName(), FALSE);
     }
     return $this->repository[$mapped->id];

Braces +1 added lines, -2 removed lines

@@ -77,8 +77,7 @@
     if (isset($this->repository[$mapped->id])) {
       $entity = $this->repository[$mapped->id];
       $entity->update($mapped);
-    }
-    else {
+    } else {
       $factory =  $this->entityManager->factory($this->getEntityTypeName());
       $this->repository[$mapped->id] = $factory->create($mapped, $this->getEntityTypeName(), FALSE);
     }

Unused Use Statements +1 added lines, -1 removed lines

@@ -6,8 +6,8 @@
 
 namespace TheSportsDb\Entity\Repository;
 
-use TheSportsDb\Entity\EntityManagerInterface;
 use TheSportsDb\Entity\EntityManagerConsumerTrait;
+use TheSportsDb\Entity\EntityManagerInterface;
 use TheSportsDb\Http\TheSportsDbClientInterface;
 
 /**

src/Entity/Repository/SeasonRepository.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
   public function byId($id) {
     if (!isset($this->repository[$id])) {
       list($name, $league) = explode('|', $id);
-      $factory =  $this->entityManager->factory($this->getEntityTypeName());
+      $factory = $this->entityManager->factory($this->getEntityTypeName());
       $this->repository[$id] = $factory->create(
         $this->entityManager->reverseMapProperties(
           (object) array('id' => $id, 'name' => $name, 'league' => (object) array('id' => $league)),