Merge pull request DesignPatternsPHP#130 from nueckman/codestyle-1

Dominik Liebler · Dominik Liebler · commit 8eed6f9c212b · 2015-04-09T21:40:48.000+02:00
codestyle - remove trailing whitespaces
diff --git a/Behavioral/Command/HelloCommand.php b/Behavioral/Command/HelloCommand.php
@@ -4,7 +4,7 @@
 
 /**
  * This concrete command calls "print" on the Receiver, but an external
- * invoker just know he can call "execute" 
+ * invoker just know he can call "execute"
  */
 class HelloCommand implements CommandInterface
 {
diff --git a/Behavioral/NullObject/LoggerInterface.php b/Behavioral/NullObject/LoggerInterface.php
@@ -4,7 +4,7 @@
 
 /**
  * LoggerInterface is a contract for logging something
- * 
+ *
  * Key feature: NullLogger MUST inherit from this interface like any other Loggers
  */
 interface LoggerInterface
diff --git a/Behavioral/NullObject/NullLogger.php b/Behavioral/NullObject/NullLogger.php
@@ -5,7 +5,7 @@
 /**
  * Performance concerns : ok there is a call for nothing but we spare an "if is_null"
  * I didn't run a benchmark but I think it's equivalent.
- * 
+ *
  * Key feature : of course this logger MUST implement the same interface (or abstract)
  * like the other loggers.
  */
diff --git a/Behavioral/Observer/User.php b/Behavioral/Observer/User.php
@@ -4,7 +4,7 @@
 
 /**
  * Observer pattern : The observed object (the subject)
- * 
+ *
  * The subject maintains a list of Observers and sends notifications.
  *
  */
diff --git a/Behavioral/Observer/UserObserver.php b/Behavioral/Observer/UserObserver.php
@@ -10,7 +10,7 @@ class UserObserver implements \SplObserver
     /**
      * This is the only method to implement as an observer.
      * It is called by the Subject (usually by SplSubject::notify() )
-     * 
+     *
      * @param \SplSubject $subject
      */
     public function update(\SplSubject $subject)
diff --git a/Behavioral/Specification/Either.php b/Behavioral/Specification/Either.php
@@ -12,7 +12,7 @@ class Either extends AbstractSpecification
 
     /**
      * A composite wrapper of two specifications
-     * 
+     *
      * @param SpecificationInterface $left
      * @param SpecificationInterface $right
      */
@@ -24,9 +24,9 @@ public function __construct(SpecificationInterface $left, SpecificationInterface
 
     /**
      * Returns the evaluation of both wrapped specifications as a logical OR
-     * 
+     *
      * @param Item $item
-     * 
+     *
      * @return bool
      */
     public function isSatisfiedBy(Item $item)
diff --git a/Behavioral/Specification/Item.php b/Behavioral/Specification/Item.php
@@ -10,7 +10,7 @@ class Item
 
     /**
      * An item must have a price
-     * 
+     *
      * @param int $price
      */
     public function __construct($price)
@@ -20,7 +20,7 @@ public function __construct($price)
 
     /**
      * Get the items price
-     * 
+     *
      * @return int
      */
     public function getPrice()
diff --git a/Behavioral/Specification/Not.php b/Behavioral/Specification/Not.php
@@ -11,7 +11,7 @@ class Not extends AbstractSpecification
 
     /**
      * Creates a new specification wrapping another
-     * 
+     *
      * @param SpecificationInterface $spec
      */
     public function __construct(SpecificationInterface $spec)
@@ -21,9 +21,9 @@ public function __construct(SpecificationInterface $spec)
 
     /**
      * Returns the negated result of the wrapped specification
-     * 
+     *
      * @param Item $item
-     * 
+     *
      * @return bool
      */
     public function isSatisfiedBy(Item $item)
diff --git a/Behavioral/TemplateMethod/Journey.php b/Behavioral/TemplateMethod/Journey.php
@@ -10,7 +10,7 @@ abstract class Journey
     /**
      * This is the public service provided by this class and its subclasses.
      * Notice it is final to "freeze" the global behavior of algorithm.
-     * If you want to override this contract, make an interface with only takeATrip() 
+     * If you want to override this contract, make an interface with only takeATrip()
      * and subclass it.
      */
     final public function takeATrip()
diff --git a/Behavioral/Visitor/RoleVisitorInterface.php b/Behavioral/Visitor/RoleVisitorInterface.php
@@ -6,25 +6,25 @@
  * Visitor Pattern
  *
  * The contract for the visitor.
- * 
+ *
  * Note 1 : in C++ or java, with method polymorphism based on type-hint, there are many
  * methods visit() with different type for the 'role' parameter.
- * 
+ *
  * Note 2 : the visitor must not choose itself which method to
  * invoke, it is the Visitee that make this decision.
  */
 interface RoleVisitorInterface
 {
     /**
      * Visit a User object
-     * 
+     *
      * @param \DesignPatterns\Behavioral\Visitor\User $role
      */
     public function visitUser(User $role);
 
     /**
      * Visit a Group object
-     * 
+     *
      * @param \DesignPatterns\Behavioral\Visitor\Group $role
      */
     public function visitGroup(Group $role);
diff --git a/Creational/AbstractFactory/AbstractFactory.php b/Creational/AbstractFactory/AbstractFactory.php
@@ -21,7 +21,7 @@ abstract class AbstractFactory
 {
     /**
      * Creates a text component
-     * 
+     *
      * @param string $content
      *
      * @return Text
@@ -30,7 +30,7 @@ abstract public function createText($content);
 
     /**
      * Creates a picture component
-     * 
+     *
      * @param string $path
      * @param string $name
      *
diff --git a/Creational/AbstractFactory/Tests/AbstractFactoryTest.php b/Creational/AbstractFactory/Tests/AbstractFactoryTest.php
@@ -21,9 +21,9 @@ public function getFactories()
 
     /**
      * This is the client of factories. Note that the client does not
-     * care which factory is given to him, he can create any component he 
+     * care which factory is given to him, he can create any component he
      * wants and render how he wants.
-     * 
+     *
      * @dataProvider getFactories
      */
     public function testComponentCreation(AbstractFactory $factory)
diff --git a/Creational/Builder/BikeBuilder.php b/Creational/Builder/BikeBuilder.php
@@ -17,7 +17,6 @@ class BikeBuilder implements BuilderInterface
      */
     public function addDoors()
     {
-
     }
 
     /**
diff --git a/Creational/Builder/Director.php b/Creational/Builder/Director.php
@@ -4,8 +4,8 @@
 
 /**
  * Director is part of the builder pattern. It knows the interface of the builder
- * and builds a complex object with the help of the builder. 
- * 
+ * and builds a complex object with the help of the builder.
+ *
  * You can also inject many builders instead of one to build more complex objects
  */
 class Director
diff --git a/Creational/Builder/Parts/Bike.php b/Creational/Builder/Parts/Bike.php
@@ -7,5 +7,4 @@
  */
 class Bike extends Vehicle
 {
-
 }
diff --git a/Creational/Builder/Parts/Car.php b/Creational/Builder/Parts/Car.php
@@ -7,5 +7,4 @@
  */
 class Car extends Vehicle
 {
-
 }
diff --git a/Creational/Builder/Parts/Door.php b/Creational/Builder/Parts/Door.php
@@ -7,5 +7,4 @@
  */
 class Door
 {
-
 }
diff --git a/Creational/Builder/Parts/Engine.php b/Creational/Builder/Parts/Engine.php
@@ -7,5 +7,4 @@
  */
 class Engine
 {
-
 }
diff --git a/Creational/Builder/Parts/Wheel.php b/Creational/Builder/Parts/Wheel.php
@@ -7,5 +7,4 @@
  */
 class Wheel
 {
-
 }
diff --git a/Creational/Builder/Tests/DirectorTest.php b/Creational/Builder/Tests/DirectorTest.php
@@ -31,7 +31,7 @@ public function getBuilder()
     /**
      * Here we test the build process. Notice that the client don't know
      * anything about the concrete builder.
-     * 
+     *
      * @dataProvider getBuilder
      */
     public function testBuild(BuilderInterface $builder)
diff --git a/Creational/FactoryMethod/FactoryMethod.php b/Creational/FactoryMethod/FactoryMethod.php
@@ -13,20 +13,20 @@ abstract class FactoryMethod
 
     /**
      * The children of the class must implement this method
-     * 
+     *
      * Sometimes this method can be public to get "raw" object
-     * 
+     *
      * @param string $type a generic type
-     * 
+     *
      * @return VehicleInterface a new vehicle
      */
     abstract protected function createVehicle($type);
 
     /**
      * Creates a new vehicle
-     * 
+     *
      * @param int $type
-     * 
+     *
      * @return VehicleInterface a new vehicle
      */
     public function create($type)
diff --git a/Creational/FactoryMethod/Porsche.php b/Creational/FactoryMethod/Porsche.php
@@ -26,6 +26,5 @@ public function setColor($rgb)
      */
     public function addTuningAMG()
     {
-
     }
 }
diff --git a/Creational/Multiton/Multiton.php b/Creational/Multiton/Multiton.php
@@ -32,13 +32,12 @@ class Multiton
      */
     private function __construct()
     {
-
     }
 
     /**
      * gets the instance with the given name, e.g. Multiton::INSTANCE_1
      * uses lazy initialization
-     * 
+     *
      * @param string $instanceName
      *
      * @return Multiton
@@ -59,7 +58,6 @@ public static function getInstance($instanceName)
      */
     private function __clone()
     {
-
     }
 
     /**
@@ -69,6 +67,5 @@ private function __clone()
      */
     private function __wakeup()
     {
-
     }
 }
diff --git a/Creational/Pool/Tests/TestWorker.php b/Creational/Pool/Tests/TestWorker.php
@@ -1,4 +1,4 @@
-<?php 
+<?php
 
 namespace DesignPatterns\Creational\Pool\Tests;
 
diff --git a/Creational/Prototype/BarBookPrototype.php b/Creational/Prototype/BarBookPrototype.php
@@ -17,6 +17,5 @@ class BarBookPrototype extends BookPrototype
      */
     public function __clone()
     {
-
     }
 }
diff --git a/Creational/Prototype/FooBookPrototype.php b/Creational/Prototype/FooBookPrototype.php
@@ -14,6 +14,5 @@ class FooBookPrototype extends BookPrototype
      */
     public function __clone()
     {
-
     }
 }
diff --git a/Creational/SimpleFactory/Bicycle.php b/Creational/SimpleFactory/Bicycle.php
@@ -14,6 +14,5 @@ class Bicycle implements VehicleInterface
      */
     public function driveTo($destination)
     {
-
     }
 }
diff --git a/Creational/SimpleFactory/ConcreteFactory.php b/Creational/SimpleFactory/ConcreteFactory.php
@@ -26,7 +26,7 @@ public function __construct()
 
     /**
      * Creates a vehicle
-     * 
+     *
      * @param string $type a known type key
      *
      * @return VehicleInterface a new instance of VehicleInterface
diff --git a/Creational/SimpleFactory/Scooter.php b/Creational/SimpleFactory/Scooter.php
@@ -12,6 +12,5 @@ class Scooter implements VehicleInterface
      */
     public function driveTo($destination)
     {
-
     }
 }
diff --git a/Creational/Singleton/Singleton.php b/Creational/Singleton/Singleton.php
@@ -19,7 +19,6 @@ class Singleton
      */
     public static function getInstance()
     {
-        
         if (null === static::$instance) {
             static::$instance = new static;
         }
@@ -33,7 +32,6 @@ public static function getInstance()
      */
     private function __construct()
     {
-
     }
 
     /**
@@ -43,7 +41,6 @@ private function __construct()
      */
     private function __clone()
     {
-
     }
 
     /**
@@ -53,6 +50,5 @@ private function __clone()
      */
     private function __wakeup()
     {
-
     }
 }
diff --git a/Creational/StaticFactory/FormatNumber.php b/Creational/StaticFactory/FormatNumber.php
@@ -7,5 +7,4 @@
  */
 class FormatNumber implements FormatterInterface
 {
-
 }
diff --git a/Creational/StaticFactory/FormatString.php b/Creational/StaticFactory/FormatString.php
@@ -7,5 +7,4 @@
  */
 class FormatString implements FormatterInterface
 {
-
 }
diff --git a/Creational/StaticFactory/FormatterInterface.php b/Creational/StaticFactory/FormatterInterface.php
@@ -7,5 +7,4 @@
  */
 interface FormatterInterface
 {
-
 }
diff --git a/More/Repository/PostRepository.php b/More/Repository/PostRepository.php
@@ -5,10 +5,10 @@
 /**
  * Repository for class Post
  * This class is between Entity layer(class Post) and access object layer(interface Storage)
- * 
+ *
  * Repository encapsulates the set of objects persisted in a data store and the operations performed over them
  * providing a more object-oriented view of the persistence layer
- * 
+ *
  * Repository also supports the objective of achieving a clean separation and one-way dependency
  * between the domain and data mapping layers
  *
diff --git a/Structural/Adapter/Book.php b/Structural/Adapter/Book.php
diff --git a/Structural/Adapter/EBookAdapter.php b/Structural/Adapter/EBookAdapter.php
diff --git a/Structural/Adapter/Kindle.php b/Structural/Adapter/Kindle.php
diff --git a/Structural/Decorator/Decorator.php b/Structural/Decorator/Decorator.php
diff --git a/Structural/Decorator/Tests/DecoratorTest.php b/Structural/Decorator/Tests/DecoratorTest.php
diff --git a/Structural/Facade/Facade.php b/Structural/Facade/Facade.php
diff --git a/Structural/FluentInterface/Sql.php b/Structural/FluentInterface/Sql.php

Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`/**`
`6`	`6`	`* This concrete command calls "print" on the Receiver, but an external`
`7`		`- * invoker just know he can call "execute"`
	`7`	`+ * invoker just know he can call "execute"`
`8`	`8`	`*/`
`9`	`9`	`class HelloCommand implements CommandInterface`
`10`	`10`	`{`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`/**`
`6`	`6`	`* Observer pattern : The observed object (the subject)`
`7`		`- *`
	`7`	`+ *`
`8`	`8`	`* The subject maintains a list of Observers and sends notifications.`
`9`	`9`	`*`
`10`	`10`	`*/`