kaliop / ezobjectwrapper

WrapperBundle/Twig/eZObjectWrapperExtension.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
     public function getFunctions()
     {
         return array(
-            new \Twig_SimpleFunction('render_location', array($this,'renderLocation'), array('is_safe' => array('html'))),
+            new \Twig_SimpleFunction('render_location', array($this, 'renderLocation'), array('is_safe' => array('html'))),
         );
     }
 

WrapperBundle/Core/Entity.php

Braces +3 added lines, -2 removed lines

@@ -41,8 +41,9 @@
             throw new \InvalidArgumentException('Trying to create Entity with no content or location');
         }
         if ($content != null && $location != null) {
-            if ($location->contentId != $content->id)
-                throw new \InvalidArgumentException('Trying to create Entity with mismatching content and location');
+            if ($location->contentId != $content->id) {
+                            throw new \InvalidArgumentException('Trying to create Entity with mismatching content and location');
+            }
         }
         $this->content = $content;
         $this->location = $location;

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
      * @param array $settings used to pass to the Entity instance anything that would be done via DIC if it was a service
      * @throws \InvalidArgumentException
      */
-    public function __construct(eZRepository $repository, Content $content=null, Location $location=null, array $settings = array())
+    public function __construct(eZRepository $repository, Content $content = null, Location $location = null, array $settings = array())
     {
         if ($content == null && $location == null) {
             throw new \InvalidArgumentException('Trying to create Entity with no content or location');

WrapperBundle/Core/Traits/LoggingEntity.php

Braces +27 added lines, -9 removed lines

@@ -22,46 +22,64 @@
 
     protected function emergency($message, array $context = array())
     {
-        if ($this->logger) $this->logger->emergency($message, $context);
+        if ($this->logger) {
+            $this->logger->emergency($message, $context);
+        }
     }
 
     protected function alert($message, array $context = array())
     {
-        if ($this->logger) $this->logger->alert($message, $context);
+        if ($this->logger) {
+            $this->logger->alert($message, $context);
+        }
     }
 
     protected function critical($message, array $context = array())
     {
-        if ($this->logger) $this->logger->critical($message, $context);
+        if ($this->logger) {
+            $this->logger->critical($message, $context);
+        }
     }
 
     protected function error($message, array $context = array())
     {
-        if ($this->logger) $this->logger->error($message, $context);
+        if ($this->logger) {
+            $this->logger->error($message, $context);
+        }
     }
 
     protected function warning($message, array $context = array())
     {
-        if ($this->logger) $this->logger->warning($message, $context);
+        if ($this->logger) {
+            $this->logger->warning($message, $context);
+        }
     }
 
     protected function notice($message, array $context = array())
     {
-        if ($this->logger) $this->logger->notice($message, $context);
+        if ($this->logger) {
+            $this->logger->notice($message, $context);
+        }
     }
 
     protected function info($message, array $context = array())
     {
-        if ($this->logger) $this->logger->info($message, $context);
+        if ($this->logger) {
+            $this->logger->info($message, $context);
+        }
     }
 
     protected function debug($message, array $context = array())
     {
-        if ($this->logger) $this->logger->debug($message, $context);
+        if ($this->logger) {
+            $this->logger->debug($message, $context);
+        }
     }
 
     protected function log($level, $message, array $context = array())
     {
-        if ($this->logger) $this->logger->log($level, $message, $context);
+        if ($this->logger) {
+            $this->logger->log($level, $message, $context);
+        }
     }
 }

WrapperBundle/Core/EntityInterface.php

Indentation +3 added lines, -3 removed lines

@@ -11,9 +11,9 @@
     public function __construct(eZRepository $repository, Content $content=null, Location $location=null);
 
     /**
-    * Return the content of the current location
-    * @return \eZ\Publish\API\Repository\Values\Content\Content
-    */
+     * Return the content of the current location
+     * @return \eZ\Publish\API\Repository\Values\Content\Content
+     */
     public function content();
 
     /**

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 
 interface EntityInterface
 {
-    public function __construct(eZRepository $repository, Content $content=null, Location $location=null);
+    public function __construct(eZRepository $repository, Content $content = null, Location $location = null);
 
     /**
     * Return the content of the current location

WrapperBundle/Core/Traits/RelationTraversingEntity.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Kaliop\eZObjectWrapperBundle\Core\Traits;
 
 use Kaliop\eZObjectWrapperBundle\Core\EntityInterface;
-
 use eZ\Publish\Core\FieldType\RelationList;
 use eZ\Publish\Core\FieldType\Relation;
 

Spacing +3 added lines, -3 removed lines

@@ -22,12 +22,12 @@ 
         $relatedEntities = array();
 
         $fieldValue = $this->content()->getFieldValue($fieldName);
-        if (! $fieldValue instanceof RelationList\Value) {
+        if (!$fieldValue instanceof RelationList\Value) {
             throw new \RuntimeException("Field '$fieldName' is not of type RelationList");
         }
 
         $relatedContentItems = array();
-        foreach($fieldValue->destinationContentIds as $contentId) {
+        foreach ($fieldValue->destinationContentIds as $contentId) {
             // Just in case the object has been pu tin the trash or hidden and they have not updated the related fields to remove from view.
             try {
                 $relatedContentItems[] = $this->getRepository()->getContentService()->loadContent($contentId);
@@ -49,7 +49,7 @@ 
     protected function getRelation($fieldName)
     {
         $fieldValue = $this->content()->getFieldValue($fieldName);
-        if (! $fieldValue instanceof Relation\Value) {
+        if (!$fieldValue instanceof Relation\Value) {
             throw new \RuntimeException("Field '$fieldName' is not of type Relation");
         }
 

WrapperBundle/Core/EntityManager.php

Spacing +5 added lines, -5 removed lines

@@ -29,7 +29,7 @@ 
      * @param array $classMap array of classes exposing a RepositoryInterface
      * @param RepositoryManagerInterface[] $serviceMap array of services exposing a RepositoryInterface
      */
-    public function __construct(eZRepository $repository, array $classMap=array(), array $serviceMap=array())
+    public function __construct(eZRepository $repository, array $classMap = array(), array $serviceMap = array())
     {
         $this->repository = $repository;
         foreach ($classMap as $contentTypeIdentifier => $className) {
@@ -141,7 +141,7 @@ 
      */
     public function load($content)
     {
-        switch(true)
+        switch (true)
         {
             case $content instanceof ContentInfo:
                 return $this
@@ -156,7 +156,7 @@ 
                     ->getRepositoryByContentTypeId($content->contentInfo->contentTypeId)
                     ->loadEntityFromLocation($content);
         }
-        throw new \UnexpectedValueException("Can not load an Entity for php object of class " . get_class($content));
+        throw new \UnexpectedValueException("Can not load an Entity for php object of class ".get_class($content));
     }
 
     /**
@@ -168,7 +168,7 @@ 
      */
     public function loadMany($contents)
     {
-        switch(true)
+        switch (true)
         {
             case $contents instanceof SearchResult:
                 $entities = array();
@@ -183,7 +183,7 @@ 
                 }
                 return $entities;
         }
-        throw new \UnexpectedValueException("Can not load an Entities for php object of class " . get_class($contents));
+        throw new \UnexpectedValueException("Can not load an Entities for php object of class ".get_class($contents));
     }
 
     /**

Doc Comments +2 added lines, -1 removed lines

@@ -44,6 +44,7 @@ 
      * Registers an existing service to be used as repository for a given content type
      * @var RepositoryInterface $service
      * @var string $contentTypeIdentifier when null, we will ask the Repository to see if it already has its own $contentTypeIdentifier set up
+     * @param integer $contentTypeIdentifier
      */
     public function registerService(RepositoryInterface $service, $contentTypeIdentifier)
     {
@@ -196,7 +197,7 @@ 
     }
 
     /**
-     * @param mixed $id
+     * @param integer $id
      * @return string
      * @throws \eZ\Publish\API\Repository\Exceptions\NotFoundException If a content type with the given id and status DEFINED can not be found
      */

WrapperBundle/Core/Repository.php

Spacing +3 added lines, -3 removed lines

@@ -62,7 +62,7 @@ 
     /** @var LoggerInterface $logger */
     protected $logger;
 
-    public function __construct(eZRepository $repository, $entityManager, array $settings=array(), $contentTypeIdentifier='')
+    public function __construct(eZRepository $repository, $entityManager, array $settings = array(), $contentTypeIdentifier = '')
     {
         $this->repository = $repository;
         $this->entityManager = $entityManager;
@@ -82,7 +82,7 @@ 
         return $this;
     }
 
-    public function setLogger(LoggerInterface $logger=null)
+    public function setLogger(LoggerInterface $logger = null)
     {
         $this->logger = $logger;
         return $this;
@@ -113,7 +113,7 @@ 
      */
     public function __call($method, $args)
     {
-        switch($method) {
+        switch ($method) {
             case 'emergency':
             case 'alert':
             case 'critical':