mcndave/spout
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix incompatible return types in XLSX\SheetIterator

Browse Source
This commit is contained in:
Fabian Wiget 2022-01-06 12:14:57 +01:00
parent e22369596f
commit 09b42171e5
1 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/Spout/Reader/XLSX/SheetIterator.php
View File

@ -38,7 +38,7 @@ class SheetIterator implements IteratorInterface
* *
* @return void * @return void
*/ */
public function rewind() public function rewind() : void
{ {
$this->currentSheetIndex = 0; $this->currentSheetIndex = 0;
} }
@ -49,7 +49,7 @@ class SheetIterator implements IteratorInterface
* *
* @return bool * @return bool
*/ */
public function valid() public function valid() : bool
{ {
return ($this->currentSheetIndex < \count($this->sheets)); return ($this->currentSheetIndex < \count($this->sheets));
} }
@ -60,7 +60,7 @@ class SheetIterator implements IteratorInterface
* *
* @return void * @return void
*/ */
public function next() public function next() : void
{ {
// Using isset here because it is way faster than array_key_exists... // Using isset here because it is way faster than array_key_exists...
if (isset($this->sheets[$this->currentSheetIndex])) { if (isset($this->sheets[$this->currentSheetIndex])) {
@ -75,9 +75,9 @@ class SheetIterator implements IteratorInterface
* Return the current element * Return the current element
* @see http://php.net/manual/en/iterator.current.php * @see http://php.net/manual/en/iterator.current.php
* *
* @return \Box\Spout\Reader\XLSX\Sheet * @return Sheet
*/ */
public function current() public function current() : Sheet
{ {
return $this->sheets[$this->currentSheetIndex]; return $this->sheets[$this->currentSheetIndex];
} }
@ -88,7 +88,7 @@ class SheetIterator implements IteratorInterface
* *
* @return int * @return int
*/ */
public function key() public function key() : int
{ {
return $this->currentSheetIndex + 1; return $this->currentSheetIndex + 1;
} }
@ -98,7 +98,7 @@ class SheetIterator implements IteratorInterface
* *
* @return void * @return void
*/ */
public function end() public function end() : void
{ {
// make sure we are not leaking memory in case the iteration stopped before the end // make sure we are not leaking memory in case the iteration stopped before the end
foreach ($this->sheets as $sheet) { foreach ($this->sheets as $sheet) {