Changeset 841:e1c6ec0a0736 for plugins/importExport/inc/flat/class.flat.backup.php

Timestamp:

07/09/12 16:53:05 (13 years ago)

Author:

JcDenis

Branch:

default

Message:

Flat import errors are now more verbose, fixes #888

File:

• plugins/importExport/inc/flat/class.flat.backup.php (modified) (7 diffs)

plugins/importExport/inc/flat/class.flat.backup.php

@@ -17 +17 @@
      private $line_cols = array();
      private $line_name;
+     private $line_num;
      
      private $replacement = array(
@@ -29 +30 @@
           if (file_exists($file) && is_readable($file)) {
                $this->fp = fopen($file,'rb');
+               $this->line_num = 1;
           } else {
                throw new Exception(__('No file to read.'));
@@ -43 +45 @@
      public function getLine()
      {
-          if (feof($this->fp)) {
+          if (($line = $this->nextLine()) === false) {
                return false;
           }
-          
-          $line = trim(fgets($this->fp));
           
           if (substr($line,0,1) == '[')
@@ -64 +64 @@
                
                if (count($this->line_cols) != count($line)) {
-                    throw new Exception('Invalid row count');
+                    throw new Exception(sprintf('Invalid row count at line %s',$this->line_num));
                }
                
@@ -74 +74 @@
                }
                
-               return new flatBackupItem($this->line_name,$res);
+               return new flatBackupItem($this->line_name,$res,$this->line_num);
           }
           else
@@ -81 +81 @@
           }
      }
+     
+     private function nextLine()
+     {
+          if (feof($this->fp)) {
+               return false;
+          }
+          $this->line_num++;
+          
+          $line = fgets($this->fp);
+          $line = trim($line);
+          
+          return empty($line) ? $this->nextLine() : $line;
+     }
 }
 
@@ -86 +99 @@
 {
      public $__name;
+     public $__line;
      private $__data = array();
      
-     public function __construct($name,$data)
+     public function __construct($name,$data,$line)
      {
           $this->__name = $name;
           $this->__data = $data;
+          $this->__line = $line;
      }