adds missing abstract function to Import abstract
authorMichael Spahn <m.spahn@metaways.de>
Thu, 29 Jun 2017 13:32:56 +0000 (15:32 +0200)
committerPhilipp Schüle <p.schuele@metaways.de>
Fri, 30 Jun 2017 06:31:38 +0000 (08:31 +0200)
* and clarifies phpdoc params

Change-Id: I48e8b95b202f0159acf0ebd605d120e69a2e9bce
Reviewed-on: http://gerrit.tine20.com/customers/4978
Reviewed-by: Michael Spahn <m.spahn@metaways.de>
Tested-by: Michael Spahn <m.spahn@metaways.de>
Tested-by: Jenkins CI (http://ci.tine20.com/)
tine20/Calendar/Import/Abstract.php
tine20/Tinebase/Import/Abstract.php
tine20/Tinebase/Import/Csv/Abstract.php

index 8341755..9ccbe52 100644 (file)
@@ -268,4 +268,6 @@ abstract class Calendar_Import_Abstract extends Tinebase_Import_Abstract
             Calendar_Controller_Event::getInstance()->delete($missingEvents->id);
         }
     }
+
+    protected function _getRawData(&$_resource) {}
 }
index 834c199..9a45f41 100644 (file)
@@ -129,7 +129,7 @@ abstract class Tinebase_Import_Abstract implements Tinebase_Import_Interface
     /**
      * import the data
      *
-     * @param resource|string $_resource (if $_filename is a stream)
+     * @param mixed $_resource (if $_filename is a stream)
      * @param array $_clientRecordData
      * @return array with import data (imported records, failures, duplicates and totalcount)
      */
@@ -153,11 +153,11 @@ abstract class Tinebase_Import_Abstract implements Tinebase_Import_Interface
      * - iconv with IGNORE
      * - replace linebreaks
      * 
-     * @param resource $resource
+     * @param mixed $resource
      */
     protected function _appendStreamFilters($resource)
     {
-        if (! $resource || ! isset($this->_options['useStreamFilter']) || ! $this->_options['useStreamFilter']) {
+        if (! is_resource($resource) || ! isset($this->_options['useStreamFilter']) || ! $this->_options['useStreamFilter']) {
             return;
         }
 
@@ -198,7 +198,7 @@ abstract class Tinebase_Import_Abstract implements Tinebase_Import_Interface
     /**
      * do something before the import
      * 
-     * @param resource $_resource
+     * @param mixed $_resource
      */
     protected function _beforeImport($_resource = NULL)
     {
@@ -212,6 +212,14 @@ abstract class Tinebase_Import_Abstract implements Tinebase_Import_Interface
     }
 
     /**
+     * get raw data of a single record
+     *
+     * @param  mixed $_resource
+     * @return array
+     */
+    abstract protected function _getRawData(&$_resource);
+
+    /**
      * do import: loop data -> convert to records -> import records
      * 
      * @param mixed $_resource
index df6cc19..1c005ed 100644 (file)
@@ -85,7 +85,7 @@ abstract class Tinebase_Import_Csv_Abstract extends Tinebase_Import_Abstract
      * @param  resource $_resource
      * @return array
      */
-    protected function _getRawData($_resource)
+    protected function _getRawData(&$_resource)
     {
         $delimiter = ((isset($this->_specialDelimiter[$this->_options['delimiter']]) || array_key_exists($this->_options['delimiter'], $this->_specialDelimiter))) ? $this->_specialDelimiter[$this->_options['delimiter']] : $this->_options['delimiter'];
         $lineData = fgetcsv(