Merge 220be6c into 0bf7a40

hughgrigg · Apr 5, 2018 · 4449e99 · 4449e99
2 parents 0bf7a40 + 220be6c
commit 4449e99
Show file tree

Hide file tree

Showing 7 changed files with 8 additions and 9 deletions.
diff --git a/src/BusinessTime.php b/src/BusinessTime.php
@@ -659,7 +659,7 @@ private function diffInBusinessTime(
      */
     private function canonicalNarrator(): BusinessTimeNarrator
     {
-        /** @var BusinessTimeConstraint $constraint */
+        /* @var BusinessTimeConstraint $constraint */
         if (!$this->isBusinessTime()) {
             foreach ($this->businessTimeConstraints() as $constraint) {
                 if (!$constraint->isBusinessTime($this)) {

diff --git a/src/BusinessTimePeriod.php b/src/BusinessTimePeriod.php
@@ -7,8 +7,6 @@
 use DatePeriod;
 
 /**
- * @package BusinessTime
- *
  * @property-read BusinessTime $start
  * @property-read BusinessTime $end
  */
@@ -33,7 +31,7 @@ public function __construct(
     }
 
     /**
-     * TODO
+     * TODO.
      *
      * Get this business time period separated into consecutive business and
      * non-business times.

diff --git a/src/Constraint/AnyTime.php b/src/Constraint/AnyTime.php
@@ -3,8 +3,8 @@
 namespace BusinessTime\Constraint;
 
 use BusinessTime\Constraint\Composite\Combinations;
-use BusinessTime\Constraint\Narration\BusinessTimeNarrator;
 use BusinessTime\Constraint\Narration\BasicNarration;
+use BusinessTime\Constraint\Narration\BusinessTimeNarrator;
 use DateTimeInterface;
 
 /**

diff --git a/src/Constraint/Composite/All.php b/src/Constraint/Composite/All.php
@@ -51,7 +51,9 @@ public function isBusinessTime(DateTimeInterface $time): bool
      * Retrieve an external iterator.
      *
      * @link  http://php.net/manual/en/iteratoraggregate.getiterator.php
+     *
      * @return Traversable
+     *
      * @since 5.0.0
      */
     public function getIterator(): Traversable

diff --git a/src/Constraint/Narration/BusinessTimeNarrator.php b/src/Constraint/Narration/BusinessTimeNarrator.php
@@ -6,7 +6,7 @@
 
 interface BusinessTimeNarrator
 {
-    public const DEFAULT_BUSINESS     = 'business hours';
+    public const DEFAULT_BUSINESS = 'business hours';
     public const DEFAULT_NON_BUSINESS = 'outside business hours';
 
     /**

diff --git a/src/Constraint/Weekends.php b/src/Constraint/Weekends.php
@@ -29,7 +29,7 @@ public function __construct()
      */
     public function narrate(DateTimeInterface $time): string
     {
-        /**
+        /*
          * The Weekends constraint matches weekends as business time.
          *
          * @see WeekDays is the opposite.

diff --git a/tests/Unit/Constraint/Narration/DefaultNarratorTest.php b/tests/Unit/Constraint/Narration/DefaultNarratorTest.php
@@ -47,8 +47,7 @@ public function testOffersDefault(
         string $expectedNarration
     ): void {
         // Given we have a constraint without narration;
-        $constraint = new class implements BusinessTimeConstraint
-        {
+        $constraint = new class() implements BusinessTimeConstraint {
             public function isBusinessTime(DateTimeInterface $time): bool
             {
                 return true;