WorkbookManagerAbstract::__construct() - Code Metrics - Inspection of "Not just another custom column widths PR" - box/spout - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#715)

unknown

created 2021-05-26 18:04 UTC

WorkbookManagerAbstract::__construct() A

↳ Parent: WorkbookManagerAbstract

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	19
ccs	10
cts	10
cp	1
rs	9.6333
c	0
b	0
f	0
cc	1
nc	1
nop	8
crap	1

How to fix Many Parameters

<?php

namespace Box\Spout\Writer\Common\Manager;

use Box\Spout\Common\Entity\Row;
use Box\Spout\Common\Exception\IOException;
use Box\Spout\Common\Manager\OptionsManagerInterface;
use Box\Spout\Writer\Common\Creator\InternalEntityFactory;
use Box\Spout\Writer\Common\Creator\ManagerFactoryInterface;
use Box\Spout\Writer\Common\Entity\Options;
use Box\Spout\Writer\Common\Entity\Sheet;
use Box\Spout\Writer\Common\Entity\Workbook;
use Box\Spout\Writer\Common\Entity\Worksheet;
use Box\Spout\Writer\Common\Helper\FileSystemWithRootFolderHelperInterface;
use Box\Spout\Writer\Common\Manager\Style\StyleManagerInterface;
use Box\Spout\Writer\Common\Manager\Style\StyleMerger;
use Box\Spout\Writer\Exception\SheetNotFoundException;

/**
 * Class WorkbookManagerAbstract
 * Abstract workbook manager, providing the generic interfaces to work with workbook.
 */
abstract class WorkbookManagerAbstract implements WorkbookManagerInterface
{
    /** @var Workbook The workbook to manage */
    protected $workbook;

    /** @var OptionsManagerInterface */
    protected $optionsManager;

    /** @var WorksheetManagerInterface */
    protected $worksheetManager;

    /** @var StyleManagerInterface Manages styles */
    protected $styleManager;

    /** @var StyleMerger Helper to merge styles */
    protected $styleMerger;

    /** @var FileSystemWithRootFolderHelperInterface Helper to perform file system operations */
    protected $fileSystemHelper;

    /** @var InternalEntityFactory Factory to create entities */
    protected $entityFactory;

    /** @var ManagerFactoryInterface Factory to create managers */
    protected $managerFactory;

    /** @var Worksheet The worksheet where data will be written to */
    protected $currentWorksheet;

    /**
     * @param Workbook $workbook
     * @param OptionsManagerInterface $optionsManager
     * @param WorksheetManagerInterface $worksheetManager
     * @param StyleManagerInterface $styleManager
     * @param StyleMerger $styleMerger
     * @param FileSystemWithRootFolderHelperInterface $fileSystemHelper
     * @param InternalEntityFactory $entityFactory
     * @param ManagerFactoryInterface $managerFactory
     */
    public function __construct(
        Workbook $workbook,
        OptionsManagerInterface $optionsManager,
        WorksheetManagerInterface $worksheetManager,
        StyleManagerInterface $styleManager,
        StyleMerger $styleMerger,
        FileSystemWithRootFolderHelperInterface $fileSystemHelper,
        InternalEntityFactory $entityFactory,
        ManagerFactoryInterface $managerFactory
    ) {
        $this->workbook = $workbook;
        $this->optionsManager = $optionsManager;
        $this->worksheetManager = $worksheetManager;
        $this->styleManager = $styleManager;
        $this->styleMerger = $styleMerger;
        $this->fileSystemHelper = $fileSystemHelper;
        $this->entityFactory = $entityFactory;
        $this->managerFactory = $managerFactory;
    }

    /**
     * @return int Maximum number of rows/columns a sheet can contain
     */
    abstract protected function getMaxRowsPerWorksheet();

    /**
     * @param Sheet $sheet
     * @return string The file path where the data for the given sheet will be stored
     */
    abstract protected function getWorksheetFilePath(Sheet $sheet);

    /**
     * @return Workbook
     */
    public function getWorkbook()
    {
        return $this->workbook;
    }

    /**
     * Creates a new sheet in the workbook and make it the current sheet.
     * The writing will resume where it stopped (i.e. data won't be truncated).
     *
     * @return Worksheet The created sheet
     */
    public function addNewSheetAndMakeItCurrent()
    {
        $worksheet = $this->addNewSheet();
        $this->setCurrentWorksheet($worksheet);

        return $worksheet;
    }

    /**
     * Creates a new sheet in the workbook. The current sheet remains unchanged.
     *
     * @throws IOException
     * @return Worksheet The created sheet
     */
    private function addNewSheet()
    {
        $worksheets = $this->getWorksheets();

        $newSheetIndex = \count($worksheets);
        $sheetManager = $this->managerFactory->createSheetManager();
        $sheet = $this->entityFactory->createSheet($newSheetIndex, $this->workbook->getInternalId(), $sheetManager);

        $worksheetFilePath = $this->getWorksheetFilePath($sheet);
        $worksheet = $this->entityFactory->createWorksheet($worksheetFilePath, $sheet);

        $this->worksheetManager->startSheet($worksheet);

        $worksheets[] = $worksheet;
        $this->workbook->setWorksheets($worksheets);

        return $worksheet;
    }

    /**
     * @return Worksheet[] All the workbook's sheets
     */
    public function getWorksheets()
    {
        return $this->workbook->getWorksheets();
    }

    /**
     * Returns the current sheet
     *
     * @return Worksheet The current sheet
     */
    public function getCurrentWorksheet()
    {
        return $this->currentWorksheet;
    }

    /**
     * Starts the current sheet and opens the file pointer
     *
     * @throws IOException
     */
    public function startCurrentSheet()
    {
        $this->worksheetManager->startSheet($this->getCurrentWorksheet());
    }

    /**
     * Sets the given sheet as the current one. New data will be written to this sheet.
     * The writing will resume where it stopped (i.e. data won't be truncated).
     *
     * @param Sheet $sheet The "external" sheet to set as current
     * @throws SheetNotFoundException If the given sheet does not exist in the workbook
     * @return void
     */
    public function setCurrentSheet(Sheet $sheet)
    {
        $worksheet = $this->getWorksheetFromExternalSheet($sheet);
        if ($worksheet !== null) {
            $this->currentWorksheet = $worksheet;
        } else {
            throw new SheetNotFoundException('The given sheet does not exist in the workbook.');
        }
    }

    /**
     * @param Worksheet $worksheet
     * @return void
     */
    private function setCurrentWorksheet($worksheet)
    {
        $this->currentWorksheet = $worksheet;
    }

    /**
     * Returns the worksheet associated to the given external sheet.
     *
     * @param Sheet $sheet
     * @return Worksheet|null The worksheet associated to the given external sheet or null if not found.
     */
    private function getWorksheetFromExternalSheet($sheet)
    {
        $worksheetFound = null;

        foreach ($this->getWorksheets() as $worksheet) {
            if ($worksheet->getExternalSheet() === $sheet) {
                $worksheetFound = $worksheet;
                break;
            }
        }

        return $worksheetFound;
    }

    /**
     * Adds a row to the current sheet.
     * If shouldCreateNewSheetsAutomatically option is set to true, it will handle pagination
     * with the creation of new worksheets if one worksheet has reached its maximum capicity.
     *
     * @param Row $row The row to be added
     *
     * @throws IOException If trying to create a new sheet and unable to open the sheet for writing
     * @throws \Box\Spout\Common\Exception\InvalidArgumentException
     * @return void
     */
    public function addRowToCurrentWorksheet(Row $row)
    {
        $currentWorksheet = $this->getCurrentWorksheet();
        $hasReachedMaxRows = $this->hasCurrentWorksheetReachedMaxRows();

        // if we reached the maximum number of rows for the current sheet...
        if ($hasReachedMaxRows) {
            // ... continue writing in a new sheet if option set
            if ($this->optionsManager->getOption(Options::SHOULD_CREATE_NEW_SHEETS_AUTOMATICALLY)) {
                $currentWorksheet = $this->addNewSheetAndMakeItCurrent();

                $this->addRowToWorksheet($currentWorksheet, $row);
            } else {
                // otherwise, do nothing as the data won't be written anyways
            }
        } else {
            $this->addRowToWorksheet($currentWorksheet, $row);
        }
    }

    /**
     * @return bool Whether the current worksheet has reached the maximum number of rows per sheet.
     */
    private function hasCurrentWorksheetReachedMaxRows()
    {
        $currentWorksheet = $this->getCurrentWorksheet();

        return ($currentWorksheet->getLastWrittenRowIndex() >= $this->getMaxRowsPerWorksheet());
    }

    /**
     * Adds a row to the given sheet.
     *
     * @param Worksheet $worksheet Worksheet to write the row to
     * @param Row $row The row to be added
     *
     * @throws IOException
     * @throws \Box\Spout\Common\Exception\InvalidArgumentException
     * @return void
     */
    private function addRowToWorksheet(Worksheet $worksheet, Row $row)
    {
        $this->applyDefaultRowStyle($row);
        $this->worksheetManager->addRow($worksheet, $row);

        // update max num columns for the worksheet
        $currentMaxNumColumns = $worksheet->getMaxNumColumns();
        $cellsCount = $row->getNumCells();
        $worksheet->setMaxNumColumns(\max($currentMaxNumColumns, $cellsCount));
    }

    /**
     * @param Row $row
     */
    private function applyDefaultRowStyle(Row $row)
    {
        $defaultRowStyle = $this->optionsManager->getOption(Options::DEFAULT_ROW_STYLE);

        if ($defaultRowStyle !== null) {
            $mergedStyle = $this->styleMerger->merge($row->getStyle(), $defaultRowStyle);
            $row->setStyle($mergedStyle);
        }
    }

    /**
     * @param float $width
     */
    public function setDefaultColumnWidth(float $width)
    {
        $this->worksheetManager->setDefaultColumnWidth($width);
    }

    /**
     * @param float $height
     */
    public function setDefaultRowHeight(float $height)
    {
        $this->worksheetManager->setDefaultRowHeight($height);
    }

    /**
     * @param float $width
     * @param array $columns One or more columns with this width
     */
    public function setColumnWidth(float $width, ...$columns)
    {
        $this->worksheetManager->setColumnWidth($width, ...$columns);
    }

    /**
     * @param float $width The width to set
     * @param int $start First column index of the range
     * @param int $end Last column index of the range
     */
    public function setColumnWidthForRange(float $width, int $start, int $end)
    {
        $this->worksheetManager->setColumnWidthForRange($width, $start, $end);
    }

    /**
     * Closes the workbook and all its associated sheets.
     * All the necessary files are written to disk and zipped together to create the final file.
     * All the temporary files are then deleted.
     *
     * @param resource $finalFilePointer Pointer to the spreadsheet that will be created
     * @return void
     */
    public function close($finalFilePointer)
    {
        $this->closeAllWorksheets();
        $this->closeRemainingObjects();
        $this->writeAllFilesToDiskAndZipThem($finalFilePointer);
        $this->cleanupTempFolder();
    }

    /**
     * Closes custom objects that are still opened
     *
     * @return void
     */
    protected function closeRemainingObjects()
    {
        // do nothing by default
    }

    /**
     * Writes all the necessary files to disk and zip them together to create the final file.
     *
     * @param resource $finalFilePointer Pointer to the spreadsheet that will be created
     * @return void
     */
    abstract protected function writeAllFilesToDiskAndZipThem($finalFilePointer);

    /**
     * Closes all workbook's associated sheets.
     *
     * @return void
     */
    private function closeAllWorksheets()
    {
        $worksheets = $this->getWorksheets();

        foreach ($worksheets as $worksheet) {
            $this->worksheetManager->close($worksheet);
        }
    }

    /**
     * Deletes the root folder created in the temp folder and all its contents.
     *
     * @return void
     */
    protected function cleanupTempFolder()
    {
        $rootFolder = $this->fileSystemHelper->getRootFolder();
        $this->fileSystemHelper->deleteFolderRecursively($rootFolder);
    }
}


1		<?php
2
3		namespace Box\Spout\Writer\Common\Manager;
4
5		use Box\Spout\Common\Entity\Row;
6		use Box\Spout\Common\Exception\IOException;
7		use Box\Spout\Common\Manager\OptionsManagerInterface;
8		use Box\Spout\Writer\Common\Creator\InternalEntityFactory;
9		use Box\Spout\Writer\Common\Creator\ManagerFactoryInterface;
10		use Box\Spout\Writer\Common\Entity\Options;
11		use Box\Spout\Writer\Common\Entity\Sheet;
12		use Box\Spout\Writer\Common\Entity\Workbook;
13		use Box\Spout\Writer\Common\Entity\Worksheet;
14		use Box\Spout\Writer\Common\Helper\FileSystemWithRootFolderHelperInterface;
15		use Box\Spout\Writer\Common\Manager\Style\StyleManagerInterface;
16		use Box\Spout\Writer\Common\Manager\Style\StyleMerger;
17		use Box\Spout\Writer\Exception\SheetNotFoundException;
18
19		/**
20		* Class WorkbookManagerAbstract
21		* Abstract workbook manager, providing the generic interfaces to work with workbook.
22		*/
23		abstract class WorkbookManagerAbstract implements WorkbookManagerInterface
24		{
25		/** @var Workbook The workbook to manage */
26		protected $workbook;
27
28		/** @var OptionsManagerInterface */
29		protected $optionsManager;
30
31		/** @var WorksheetManagerInterface */
32		protected $worksheetManager;
33
34		/** @var StyleManagerInterface Manages styles */
35		protected $styleManager;
36
37		/** @var StyleMerger Helper to merge styles */
38		protected $styleMerger;
39
40		/** @var FileSystemWithRootFolderHelperInterface Helper to perform file system operations */
41		protected $fileSystemHelper;
42
43		/** @var InternalEntityFactory Factory to create entities */
44		protected $entityFactory;
45
46		/** @var ManagerFactoryInterface Factory to create managers */
47		protected $managerFactory;
48
49		/** @var Worksheet The worksheet where data will be written to */
50		protected $currentWorksheet;
51
52		/**
53		* @param Workbook $workbook
54		* @param OptionsManagerInterface $optionsManager
55		* @param WorksheetManagerInterface $worksheetManager
56		* @param StyleManagerInterface $styleManager
57		* @param StyleMerger $styleMerger
58		* @param FileSystemWithRootFolderHelperInterface $fileSystemHelper
59		* @param InternalEntityFactory $entityFactory
60		* @param ManagerFactoryInterface $managerFactory
61		*/
62	93	public function __construct(
63		Workbook $workbook,
64		OptionsManagerInterface $optionsManager,
65		WorksheetManagerInterface $worksheetManager,
66		StyleManagerInterface $styleManager,
67		StyleMerger $styleMerger,
68		FileSystemWithRootFolderHelperInterface $fileSystemHelper,
69		InternalEntityFactory $entityFactory,
70		ManagerFactoryInterface $managerFactory
71		) {
72	93	$this->workbook = $workbook;
73	93	$this->optionsManager = $optionsManager;
74	93	$this->worksheetManager = $worksheetManager;
75	93	$this->styleManager = $styleManager;
76	93	$this->styleMerger = $styleMerger;
77	93	$this->fileSystemHelper = $fileSystemHelper;
78	93	$this->entityFactory = $entityFactory;
79	93	$this->managerFactory = $managerFactory;
80	93	}
81
82		/**
83		* @return int Maximum number of rows/columns a sheet can contain
84		*/
85		abstract protected function getMaxRowsPerWorksheet();
86
87		/**
88		* @param Sheet $sheet
89		* @return string The file path where the data for the given sheet will be stored
90		*/
91		abstract protected function getWorksheetFilePath(Sheet $sheet);
92
93		/**
94		* @return Workbook
95		*/
96	87	public function getWorkbook()
97		{
98	87	return $this->workbook;
99		}
100
101		/**
102		* Creates a new sheet in the workbook and make it the current sheet.
103		* The writing will resume where it stopped (i.e. data won't be truncated).
104		*
105		* @return Worksheet The created sheet
106		*/
107	93	public function addNewSheetAndMakeItCurrent()
108		{
109	93	$worksheet = $this->addNewSheet();
110	93	$this->setCurrentWorksheet($worksheet);
111
112	93	return $worksheet;
113		}
114
115		/**
116		* Creates a new sheet in the workbook. The current sheet remains unchanged.
117		*
118		* @throws IOException
119		* @return Worksheet The created sheet
120		*/
121	93	private function addNewSheet()
122		{
123	93	$worksheets = $this->getWorksheets();
124
125	93	$newSheetIndex = \count($worksheets);
126	93	$sheetManager = $this->managerFactory->createSheetManager();
127	93	$sheet = $this->entityFactory->createSheet($newSheetIndex, $this->workbook->getInternalId(), $sheetManager);
128
129	93	$worksheetFilePath = $this->getWorksheetFilePath($sheet);
130	93	$worksheet = $this->entityFactory->createWorksheet($worksheetFilePath, $sheet);
131
132	93	$this->worksheetManager->startSheet($worksheet);
133
134	93	$worksheets[] = $worksheet;
135	93	$this->workbook->setWorksheets($worksheets);
136
137	93	return $worksheet;
138		}
139
140		/**
141		* @return Worksheet[] All the workbook's sheets
142		*/
143	93	public function getWorksheets()
144		{
145	93	return $this->workbook->getWorksheets();
146		}
147
148		/**
149		* Returns the current sheet
150		*
151		* @return Worksheet The current sheet
152		*/
153	87	public function getCurrentWorksheet()
154		{
155	87	return $this->currentWorksheet;
156		}
157
158		/**
159		* Starts the current sheet and opens the file pointer
160		*
161		* @throws IOException
162		*/
163		public function startCurrentSheet()
164		{
165		$this->worksheetManager->startSheet($this->getCurrentWorksheet());
166		}
167
168		/**
169		* Sets the given sheet as the current one. New data will be written to this sheet.
170		* The writing will resume where it stopped (i.e. data won't be truncated).
171		*
172		* @param Sheet $sheet The "external" sheet to set as current
173		* @throws SheetNotFoundException If the given sheet does not exist in the workbook
174		* @return void
175		*/
176	4	public function setCurrentSheet(Sheet $sheet)
177		{
178	4	$worksheet = $this->getWorksheetFromExternalSheet($sheet);
179	4	if ($worksheet !== null) {
180	4	$this->currentWorksheet = $worksheet;
181		} else {
182		throw new SheetNotFoundException('The given sheet does not exist in the workbook.');
183		}
184	4	}
185
186		/**
187		* @param Worksheet $worksheet
188		* @return void
189		*/
190	93	private function setCurrentWorksheet($worksheet)
191		{
192	93	$this->currentWorksheet = $worksheet;
193	93	}
194
195		/**
196		* Returns the worksheet associated to the given external sheet.
197		*
198		* @param Sheet $sheet
199		* @return Worksheet\|null The worksheet associated to the given external sheet or null if not found.
200		*/
201	4	private function getWorksheetFromExternalSheet($sheet)
202		{
203	4	$worksheetFound = null;
204
205	4	foreach ($this->getWorksheets() as $worksheet) {
206	4	if ($worksheet->getExternalSheet() === $sheet) {
207	4	$worksheetFound = $worksheet;
208	4	break;
209		}
210		}
211
212	4	return $worksheetFound;
213		}
214
215		/**
216		* Adds a row to the current sheet.
217		* If shouldCreateNewSheetsAutomatically option is set to true, it will handle pagination
218		* with the creation of new worksheets if one worksheet has reached its maximum capicity.
219		*
220		* @param Row $row The row to be added
221		*
222		* @throws IOException If trying to create a new sheet and unable to open the sheet for writing
223		* @throws \Box\Spout\Common\Exception\InvalidArgumentException
224		* @return void
225		*/
226	81	public function addRowToCurrentWorksheet(Row $row)
227		{
228	81	$currentWorksheet = $this->getCurrentWorksheet();
229	81	$hasReachedMaxRows = $this->hasCurrentWorksheetReachedMaxRows();
230
231		// if we reached the maximum number of rows for the current sheet...
232	81	if ($hasReachedMaxRows) {
233		// ... continue writing in a new sheet if option set
234	4	if ($this->optionsManager->getOption(Options::SHOULD_CREATE_NEW_SHEETS_AUTOMATICALLY)) {
235	2	$currentWorksheet = $this->addNewSheetAndMakeItCurrent();
236
237	2	$this->addRowToWorksheet($currentWorksheet, $row);
238	4	} else {
239		// otherwise, do nothing as the data won't be written anyways
240		}
241		} else {
242	81	$this->addRowToWorksheet($currentWorksheet, $row);
243		}
244	79	}
245
246		/**
247		* @return bool Whether the current worksheet has reached the maximum number of rows per sheet.
248		*/
249	81	private function hasCurrentWorksheetReachedMaxRows()
250		{
251	81	$currentWorksheet = $this->getCurrentWorksheet();
252
253	81	return ($currentWorksheet->getLastWrittenRowIndex() >= $this->getMaxRowsPerWorksheet());
254		}
255
256		/**
257		* Adds a row to the given sheet.
258		*
259		* @param Worksheet $worksheet Worksheet to write the row to
260		* @param Row $row The row to be added
261		*
262		* @throws IOException
263		* @throws \Box\Spout\Common\Exception\InvalidArgumentException
264		* @return void
265		*/
266	81	private function addRowToWorksheet(Worksheet $worksheet, Row $row)
267		{
268	81	$this->applyDefaultRowStyle($row);
269	81	$this->worksheetManager->addRow($worksheet, $row);
270
271		// update max num columns for the worksheet
272	79	$currentMaxNumColumns = $worksheet->getMaxNumColumns();
273	79	$cellsCount = $row->getNumCells();
274	79	$worksheet->setMaxNumColumns(\max($currentMaxNumColumns, $cellsCount));
275	79	}
276
277		/**
278		* @param Row $row
279		*/
280	81	private function applyDefaultRowStyle(Row $row)
281		{
282	81	$defaultRowStyle = $this->optionsManager->getOption(Options::DEFAULT_ROW_STYLE);
283
284	81	if ($defaultRowStyle !== null) {
285	81	$mergedStyle = $this->styleMerger->merge($row->getStyle(), $defaultRowStyle);
286	81	$row->setStyle($mergedStyle);
287		}
288	81	}
289
290		/**
291		* @param float $width
292		*/
293		public function setDefaultColumnWidth(float $width)
294		{
295		$this->worksheetManager->setDefaultColumnWidth($width);
296		}
297
298		/**
299		* @param float $height
300		*/
301		public function setDefaultRowHeight(float $height)
302		{
303		$this->worksheetManager->setDefaultRowHeight($height);
304		}
305
306		/**
307		* @param float $width
308		* @param array $columns One or more columns with this width
309		*/
310	6	public function setColumnWidth(float $width, ...$columns)
311		{
312	6	$this->worksheetManager->setColumnWidth($width, ...$columns);
313	6	}
314
315		/**
316		* @param float $width The width to set
317		* @param int $start First column index of the range
318		* @param int $end Last column index of the range
319		*/
320	2	public function setColumnWidthForRange(float $width, int $start, int $end)
321		{
322	2	$this->worksheetManager->setColumnWidthForRange($width, $start, $end);
323	2	}
324
325		/**
326		* Closes the workbook and all its associated sheets.
327		* All the necessary files are written to disk and zipped together to create the final file.
328		* All the temporary files are then deleted.
329		*
330		* @param resource $finalFilePointer Pointer to the spreadsheet that will be created
331		* @return void
332		*/
333	86	public function close($finalFilePointer)
334		{
335	86	$this->closeAllWorksheets();
336	86	$this->closeRemainingObjects();
337	86	$this->writeAllFilesToDiskAndZipThem($finalFilePointer);
338	86	$this->cleanupTempFolder();
339	86	}
340
341		/**
342		* Closes custom objects that are still opened
343		*
344		* @return void
345		*/
346	41	protected function closeRemainingObjects()
347		{
348		// do nothing by default
349	41	}
350
351		/**
352		* Writes all the necessary files to disk and zip them together to create the final file.
353		*
354		* @param resource $finalFilePointer Pointer to the spreadsheet that will be created
355		* @return void
356		*/
357		abstract protected function writeAllFilesToDiskAndZipThem($finalFilePointer);
358
359		/**
360		* Closes all workbook's associated sheets.
361		*
362		* @return void
363		*/
364	86	private function closeAllWorksheets()
365		{
366	86	$worksheets = $this->getWorksheets();
367
368	86	foreach ($worksheets as $worksheet) {
369	86	$this->worksheetManager->close($worksheet);
370		}
371	86	}
372
373		/**
374		* Deletes the root folder created in the temp folder and all its contents.
375		*
376		* @return void
377		*/
378	86	protected function cleanupTempFolder()
379		{
380	86	$rootFolder = $this->fileSystemHelper->getRootFolder();
381	86	$this->fileSystemHelper->deleteFolderRecursively($rootFolder);
382	86	}
383		}
384

box / spout

Pull Request — master (#715)

WorkbookManagerAbstract::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Many Parameters

Many Parameters

Duplication Side-by-Side

Filter issues like