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

Spacing +3 added lines, -3 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');
@@ -69,7 +69,7 @@ 
      */
     public function content()
     {
-        if($this->content == null){
+        if ($this->content == null) {
             $this->content = $this->repository->getContentService()->loadContent($this->location->contentId);
         }
         return $this->content;
@@ -81,7 +81,7 @@ 
      */
     public function location()
     {
-        if($this->location == null){
+        if ($this->location == null) {
             $this->location = $this->repository->getLocationService()->loadLocation($this->content->contentInfo->mainLocationId);
         }
         return $this->location;

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;

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/Repository.php

Spacing +4 added lines, -4 removed lines

@@ -58,7 +58,7 @@ 
     protected $settings;
     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;
@@ -78,7 +78,7 @@ 
         return $this;
     }
 
-    public function setLogger(LoggerInterface $logger=null)
+    public function setLogger(LoggerInterface $logger = null)
     {
         $this->logger = $logger;
         return $this;
@@ -109,7 +109,7 @@ 
      */
     public function __call($method, $args)
     {
-        switch($method) {
+        switch ($method) {
             case 'emergency':
             case 'alert':
             case 'critical':
@@ -244,7 +244,7 @@ 
     protected function loadEntitiesFromSearchResults(SearchResult $searchResult)
     {
         $entities = array();
-        foreach($searchResult->searchHits as $searchHit) {
+        foreach ($searchResult->searchHits as $searchHit) {
             // let's hope that in the future eZPublish does not add new types of results to SearchResult... :-P
             if ($searchHit->valueObject instanceof \eZ\Publish\API\Repository\Values\Content\Location) {
                 $entities[] = $this->loadEntityFromLocation($searchHit->valueObject);

WrapperBundle/Core/EntityManager.php

Spacing +7 added lines, -7 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,22 +168,22 @@ 
      */
     public function loadMany($contents)
     {
-        switch(true)
+        switch (true)
         {
             case $contents instanceof SearchResult:
                 $entities = array();
-                foreach($contents->searchHits as $searchHit) {
+                foreach ($contents->searchHits as $searchHit) {
                     $entities[] = $this->load($searchHit->valueObject);
                 }
                 return $entities;
             case is_array($contents):
                 $entities = array();
-                foreach($contents as $key => $value) {
+                foreach ($contents as $key => $value) {
                     $entities[$key] = $this->load($value);
                 }
                 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));
     }
 
     /**

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->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");
         }