From 97f89b17264cc4e6f10d0aa8ee021acd5a179774 Mon Sep 17 00:00:00 2001 From: Adrien Loison Date: Sat, 18 Nov 2017 20:50:16 +0100 Subject: [PATCH] Remove StyleMerger from RowManager --- src/Spout/Writer/Common/Manager/RowManager.php | 14 -------------- .../Spout/Writer/Common/Manager/RowManagerTest.php | 3 +-- 2 files changed, 1 insertion(+), 16 deletions(-) diff --git a/src/Spout/Writer/Common/Manager/RowManager.php b/src/Spout/Writer/Common/Manager/RowManager.php index 7534db5..9bdf5be 100644 --- a/src/Spout/Writer/Common/Manager/RowManager.php +++ b/src/Spout/Writer/Common/Manager/RowManager.php @@ -3,23 +3,9 @@ namespace Box\Spout\Writer\Common\Manager; use Box\Spout\Writer\Common\Entity\Row; -use Box\Spout\Writer\Common\Manager\Style\StyleMerger; class RowManager { - /** - * @var StyleMerger - */ - protected $styleMerger; - - /** - * @param StyleMerger $styleMerger - */ - public function __construct(StyleMerger $styleMerger) - { - $this->styleMerger = $styleMerger; - } - /** * Detect whether a row is considered empty. * An empty row has either no cells at all - or only one empty cell diff --git a/tests/Spout/Writer/Common/Manager/RowManagerTest.php b/tests/Spout/Writer/Common/Manager/RowManagerTest.php index 56d6128..cc3ca8d 100644 --- a/tests/Spout/Writer/Common/Manager/RowManagerTest.php +++ b/tests/Spout/Writer/Common/Manager/RowManagerTest.php @@ -5,7 +5,6 @@ namespace Spout\Writer\Common\Manager; use Box\Spout\Writer\Common\Entity\Cell; use Box\Spout\Writer\Common\Entity\Row; use Box\Spout\Writer\Common\Manager\RowManager; -use Box\Spout\Writer\Common\Manager\Style\StyleMerger; use PHPUnit\Framework\TestCase; class RowManagerTest extends TestCase @@ -32,7 +31,7 @@ class RowManagerTest extends TestCase */ public function testIsEmptyRow(array $cells, $expectedIsEmpty) { - $rowManager = new RowManager(new StyleMerger()); + $rowManager = new RowManager(); $row = new Row($cells, null, $rowManager); $this->assertEquals($expectedIsEmpty, $rowManager->isEmpty($row));