Ejegg has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/242588

Change subject: Allow skipping rows before CSV headers
......................................................................

Allow skipping rows before CSV headers

Because sometimes people put junk in the preamble.

Change-Id: I19bf6ddd5914f21a67ac30300c813e387702ee28
---
M Core/DataFiles/HeadedCsvReader.php
1 file changed, 5 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/SmashPig 
refs/changes/88/242588/1

diff --git a/Core/DataFiles/HeadedCsvReader.php 
b/Core/DataFiles/HeadedCsvReader.php
index 22ff008..bf0babb 100644
--- a/Core/DataFiles/HeadedCsvReader.php
+++ b/Core/DataFiles/HeadedCsvReader.php
@@ -10,9 +10,13 @@
        protected $colNames;
        protected $colIndexes;
 
-       public function __construct( $file, $delimiter = ',', $maxRowLength = 
4098 ) {
+       public function __construct( $file, $delimiter = ',', $maxRowLength = 
4098, $skipRows = 0 ) {
                parent::__construct( $file, $delimiter, $maxRowLength );
 
+               while ( $skipRows > 0 ) {
+                       parent::next();
+                       $skipRows--;
+               }
                // Extract the header information
                $this->colNames = parent::current();
                foreach( $this->colNames as $index => $name ) {

-- 
To view, visit https://gerrit.wikimedia.org/r/242588
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I19bf6ddd5914f21a67ac30300c813e387702ee28
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/SmashPig
Gerrit-Branch: master
Gerrit-Owner: Ejegg <eeggles...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to