GermaniaKG / Salesmen

src/ApplySalesman.php

Spacing +4 added lines, -4 removed lines

@@ -18,7 +18,7 @@ 
     /**
      * @param callable $salesman_factory Callable that takes a Salesman ID that should return a SalesmanInterface instance.
      */
-    public function __construct( callable $salesman_factory, SalesmanInterface $salesman_fallback = null )
+    public function __construct(callable $salesman_factory, SalesmanInterface $salesman_fallback = null)
     {
         $this->salesman_factory  = $salesman_factory;
         $this->salesman_fallback = $salesman_fallback;
@@ -29,7 +29,7 @@ 
      * @param  object $aggregator Object that
      * @return bool
      */
-    public function __invoke( $aggregator )
+    public function __invoke($aggregator)
     {
 
         // Default value
@@ -44,10 +44,10 @@ 
 
         // If salesman_id is available
         if ($aggregator instanceOf SalesmanIdProviderInterface):
-            $aggregator->salesman = $salesman_factory( $aggregator->getSalesmanId(), $this->salesman_fallback );
+            $aggregator->salesman = $salesman_factory($aggregator->getSalesmanId(), $this->salesman_fallback);
 
         elseif (isset($aggregator->salesman_id)):
-            $aggregator->salesman = $salesman_factory( $aggregator->salesman_id, $this->salesman_fallback );
+            $aggregator->salesman = $salesman_factory($aggregator->salesman_id, $this->salesman_fallback);
 
         // Salesman can not be set
         else:

Braces +3 added lines, -1 removed lines

@@ -50,8 +50,10 @@
             $aggregator->salesman = $salesman_factory( $aggregator->salesman_id, $this->salesman_fallback );
 
         // Salesman can not be set
-        else:
+        else {
+            :
             return false;
+        }
         endif;
 
         return true;

src/IteratorApplySalesman.php

Spacing +4 added lines, -4 removed lines

@@ -32,10 +32,10 @@ 
      * @param SalesmanInterface $salesman_fallback Optional SalesmanInterface fallback instance
      * @param callable          $apply             Optional Callable that takes the current item on iteration.
      */
-    public function __construct( callable $salesman_factory, SalesmanInterface $salesman_fallback = null, callable $apply = null )
+    public function __construct(callable $salesman_factory, SalesmanInterface $salesman_fallback = null, callable $apply = null)
     {
         $this->salesman_factory = $salesman_factory;
-        $this->apply = $apply ?: new ApplySalesman( $salesman_factory, $salesman_fallback );
+        $this->apply = $apply ?: new ApplySalesman($salesman_factory, $salesman_fallback);
     }
 
 
@@ -44,7 +44,7 @@ 
      *
      * @see  http://php.net/manual/de/function.iterator-apply.php
      */
-    public function __invoke( \Traversable $iterator )
+    public function __invoke(\Traversable $iterator)
     {
 
         $current = ($iterator instanceOf \IteratorAggregate)
@@ -53,7 +53,7 @@ 
 
         // Perform
         $apply = $this->apply;
-        $apply( $current );
+        $apply($current);
 
         // Must return true for iterator_apply
         return true;