dvdoug / BoxPacker

src/VolumePacker.php

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 
 use Psr\Log\LoggerAwareInterface;
 use Psr\Log\LoggerAwareTrait;
-use Psr\Log\LoggerInterface;
 use Psr\Log\NullLogger;
 
 /**

Doc Comments +1 added lines, -1 removed lines

@@ -205,7 +205,7 @@
      * Used when we've packed a tall item, and have just put a shorter one next to it
      *
      * @param PackedItemList $packedItems
-     * @param ?PackedItem $prevItem
+     * @param null|PackedItem $prevItem
      * @param int $maxWidth
      * @param int $maxLength
      * @param int $maxDepth

Spacing +3 added lines, -3 removed lines

@@ -4,7 +4,7 @@ 
  * @package BoxPacker
  * @author Doug Wright
  */
-declare(strict_types=1);
+declare(strict_types = 1);
 namespace DVDoug\BoxPacker;
 
 use Psr\Log\LoggerAwareInterface;
@@ -180,7 +180,7 @@ 
         int $maxWidth,
         int $maxLength,
         int $maxDepth
-    ): ?OrientatedItem {
+    ) : ?OrientatedItem {
         $this->logger->debug(
             "evaluating item {$itemToPack->getDescription()} for fit",
             [
@@ -222,7 +222,7 @@ 
         int $x,
         int $y,
         int $z
-    ): void {
+    ) : void {
         while (!$this->items->isEmpty() && $this->checkNonDimensionalConstraints($this->items->top(), $packedItems)) {
             $stackedItem = $this->getOrientationForItem(
                 $this->items->top(),

src/OrientatedItem.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
  * @package BoxPacker
  * @author Doug Wright
  */
-declare(strict_types=1);
+declare(strict_types = 1);
 namespace DVDoug\BoxPacker;
 
 /**

src/WeightRedistributor.php

Spacing +5 added lines, -5 removed lines

@@ -4,7 +4,7 @@ 
  * @package BoxPacker
  * @author Doug Wright
  */
-declare(strict_types=1);
+declare(strict_types = 1);
 namespace DVDoug\BoxPacker;
 
 use Psr\Log\LoggerAwareInterface;
@@ -68,18 +68,18 @@ 
 
         do { //Keep moving items from most overweight box to most underweight box
             $tryRepack = false;
-            $this->logger->log(LogLevel::DEBUG, 'boxes under/over target: ' . count($underWeightBoxes) . '/' . count($overWeightBoxes));
+            $this->logger->log(LogLevel::DEBUG, 'boxes under/over target: '.count($underWeightBoxes).'/'.count($overWeightBoxes));
 
             foreach ($underWeightBoxes as $u => $underWeightBox) {
-                $this->logger->log(LogLevel::DEBUG, 'Underweight Box ' . $u);
+                $this->logger->log(LogLevel::DEBUG, 'Underweight Box '.$u);
                 foreach ($overWeightBoxes as $o => $overWeightBox) {
-                    $this->logger->log(LogLevel::DEBUG, 'Overweight Box ' . $o);
+                    $this->logger->log(LogLevel::DEBUG, 'Overweight Box '.$o);
                     $overWeightBoxItems = $overWeightBox->getItems()->asItemArray();
 
                     //For each item in the heavier box, try and move it to the lighter one
                     /** @var Item $overWeightBoxItem */
                     foreach ($overWeightBoxItems as $oi => $overWeightBoxItem) {
-                        $this->logger->log(LogLevel::DEBUG, 'Overweight Item ' . $oi);
+                        $this->logger->log(LogLevel::DEBUG, 'Overweight Item '.$oi);
                         if ($underWeightBox->getWeight() + $overWeightBoxItem->getWeight() > $targetWeight) {
                             $this->logger->log(LogLevel::DEBUG, 'Skipping item for hindering weight distribution');
                             continue; //skip if moving this item would hinder rather than help weight distribution

src/Packer.php

Spacing +2 added lines, -2 removed lines

@@ -4,7 +4,7 @@ 
  * @package BoxPacker
  * @author Doug Wright
  */
-declare(strict_types=1);
+declare(strict_types = 1);
 namespace DVDoug\BoxPacker;
 
 use Psr\Log\LoggerAwareInterface;
@@ -149,7 +149,7 @@ 
 
             //Check iteration was productive
             if ($packedBoxesIteration->isEmpty()) {
-                throw new ItemTooLargeException('Item ' . $this->items->top()->getDescription() . ' is too large to fit into any box', $this->items->top());
+                throw new ItemTooLargeException('Item '.$this->items->top()->getDescription().' is too large to fit into any box', $this->items->top());
             }
 
             //Find best box of iteration, and remove packed items from unpacked list

src/OrientatedItemFactory.php

Spacing +15 added lines, -15 removed lines

@@ -4,7 +4,7 @@ 
  * @package BoxPacker
  * @author Doug Wright
  */
-declare(strict_types=1);
+declare(strict_types = 1);
 namespace DVDoug\BoxPacker;
 
 use Psr\Log\LoggerAwareInterface;
@@ -43,7 +43,7 @@ 
         int $widthLeft,
         int $lengthLeft,
         int $depthLeft
-    ): ?OrientatedItem {
+    ) : ?OrientatedItem {
 
         $possibleOrientations = $this->getPossibleOrientations($item, $prevItem, $widthLeft, $lengthLeft, $depthLeft);
         $usableOrientations = $this->getUsableOrientations($possibleOrientations, $box, $item, $isLastItem);
@@ -81,7 +81,7 @@ 
         int $widthLeft,
         int $lengthLeft,
         int $depthLeft
-    ): array {
+    ) : array {
 
         $orientations = [];
 
@@ -117,18 +117,18 @@ 
      */
     public function getPossibleOrientationsInEmptyBox(Item $item, Box $box): array
     {
-        $cacheKey = $item->getWidth() .
-            '|' .
-            $item->getLength() .
-            '|' .
-            $item->getDepth() .
-            '|' .
-            ($item->getKeepFlat() ? '2D' : '3D') .
-            '|' .
-            $box->getInnerWidth() .
-            '|' .
-            $box->getInnerLength() .
-            '|' .
+        $cacheKey = $item->getWidth().
+            '|'.
+            $item->getLength().
+            '|'.
+            $item->getDepth().
+            '|'.
+            ($item->getKeepFlat() ? '2D' : '3D').
+            '|'.
+            $box->getInnerWidth().
+            '|'.
+            $box->getInnerLength().
+            '|'.
             $box->getInnerDepth();
 
         if (isset(static::$emptyBoxCache[$cacheKey])) {

src/ConstrainedItem.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
  * @package BoxPacker
  * @author Doug Wright
  */
-declare(strict_types=1);
+declare(strict_types = 1);
 namespace DVDoug\BoxPacker;
 
 /**

src/PackedBox.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
  * @package BoxPacker
  * @author Doug Wright
  */
-declare(strict_types=1);
+declare(strict_types = 1);
 namespace DVDoug\BoxPacker;
 
 /**

src/ItemList.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
  * @package BoxPacker
  * @author Doug Wright
  */
-declare(strict_types=1);
+declare(strict_types = 1);
 namespace DVDoug\BoxPacker;
 
 /**

src/Item.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
  * @package BoxPacker
  * @author Doug Wright
  */
-declare(strict_types=1);
+declare(strict_types = 1);
 namespace DVDoug\BoxPacker;
 
 /**