Merge pull request #186 from dododedodonl/change

Rewind iterator after use

Author: freekmurze

src/SimpleExcelReader.php

@@ -244,6 +244,12 @@ public function getRows(): LazyCollection
 
                 $this->rowIterator->next();
             }
+            $this->rowIterator->rewind();
+
+            if ($this->processHeader) {
+                $this->getHeaders();
+                $this->rowIterator->next();
+            }
         });
     }
 

tests/SimpleExcelReaderTest.php

@@ -643,3 +643,18 @@ function () {
     expect($reader->getRows()->count())->toBe(2);
     expect($reader->preserveEmptyRows()->getRows()->count())->toBe(3);
 });
+
+it('can count and take rows', function () {
+    $reader = SimpleExcelReader::create(getStubPath('header-and-rows.csv'));
+
+    $lazyCollection = $reader->getRows();
+
+    expect($lazyCollection->count())->toBe(2);
+    expect($lazyCollection->take(1)->all())->toEqual([
+        [
+            'email' => '[email protected]',
+            'first_name' => 'john',
+            'last_name' => 'doe',
+        ],
+    ]);
+});