Move ReaderCommonOptions class to Common folder (#343)

box · Oct 18, 2016 · 3a330de · 3a330de
1 parent a19231f
commit 3a330de
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 16 deletions.
diff --git a/src/Spout/Reader/AbstractReader.php b/src/Spout/Reader/AbstractReader.php
@@ -19,13 +19,13 @@ abstract class AbstractReader implements ReaderInterface
  /** @var \Box\Spout\Common\Helper\GlobalFunctionsHelper Helper to work with global functions */
  protected $globalFunctionsHelper;
 
- /** @var \Box\Spout\Reader\ReaderOptionsCommon Reader's customized options */
+ /** @var \Box\Spout\Reader\Common\ReaderOptions Reader's customized options */
  protected $options;
 
  /**
  * Returns the reader's current options
  *
- * @return \Box\Spout\Reader\ReaderOptionsCommon
+ * @return \Box\Spout\Reader\Common\ReaderOptions
  */
  abstract protected function getOptions();
 

diff --git a/src/Spout/Reader/CSV/ReaderOptions.php b/src/Spout/Reader/CSV/ReaderOptions.php
@@ -3,15 +3,14 @@
 namespace Box\Spout\Reader\CSV;
 
 use Box\Spout\Common\Helper\EncodingHelper;
-use Box\Spout\Reader\ReaderOptionsCommon;
 
 /**
  * Class ReaderOptions
  * This class is used to customize the reader's behavior
  *
  * @package Box\Spout\Reader\CSV
  */
-class ReaderOptions extends ReaderOptionsCommon
+class ReaderOptions extends \Box\Spout\Reader\Common\ReaderOptions
 {
  /** @var string Defines the character used to delimit fields (one character only) */
  protected $fieldDelimiter = ',';

diff --git a/src/Spout/Reader/ReaderOptionsCommon.php → src/Spout/Reader/Common/ReaderOptions.php b/src/Spout/Reader/ReaderOptionsCommon.php → src/Spout/Reader/Common/ReaderOptions.php
@@ -1,14 +1,14 @@
 <?php
 
-namespace Box\Spout\Reader;
+namespace Box\Spout\Reader\Common;
 
 /**
- * Class ReaderOptionsCommon
+ * Class ReaderOptions
  * Readers' common options
  *
- * @package Box\Spout\Reader
+ * @package Box\Spout\Reader\Common
  */
-class ReaderOptionsCommon
+class ReaderOptions
 {
  /** @var bool Whether date/time values should be returned as PHP objects or be formatted as strings */
  protected $shouldFormatDates = false;
@@ -28,7 +28,7 @@ public function shouldFormatDates()
  * Sets whether date/time values should be returned as PHP objects or be formatted as strings.
  *
  * @param bool $shouldFormatDates
- * @return ReaderOptionsCommon
+ * @return ReaderOptions
  */
  public function setShouldFormatDates($shouldFormatDates)
  {
@@ -48,7 +48,7 @@ public function shouldPreserveEmptyRows()
  * Sets whether empty rows should be returned or skipped.
  *
  * @param bool $shouldPreserveEmptyRows
- * @return ReaderOptionsCommon
+ * @return ReaderOptions
  */
  public function setShouldPreserveEmptyRows($shouldPreserveEmptyRows)
  {

diff --git a/src/Spout/Reader/ODS/ReaderOptions.php b/src/Spout/Reader/ODS/ReaderOptions.php
@@ -2,15 +2,13 @@
 
 namespace Box\Spout\Reader\ODS;
 
-use Box\Spout\Reader\ReaderOptionsCommon;
-
 /**
  * Class ReaderOptions
  * This class is used to customize the reader's behavior
  *
  * @package Box\Spout\Reader\ODS
  */
-class ReaderOptions extends ReaderOptionsCommon
+class ReaderOptions extends \Box\Spout\Reader\Common\ReaderOptions
 {
  // No extra options
 }
diff --git a/src/Spout/Reader/XLSX/ReaderOptions.php b/src/Spout/Reader/XLSX/ReaderOptions.php
@@ -2,15 +2,13 @@
 
 namespace Box\Spout\Reader\XLSX;
 
-use Box\Spout\Reader\ReaderOptionsCommon;
-
 /**
  * Class ReaderOptions
  * This class is used to customize the reader's behavior
  *
  * @package Box\Spout\Reader\XLSX
  */
-class ReaderOptions extends ReaderOptionsCommon
+class ReaderOptions extends \Box\Spout\Reader\Common\ReaderOptions
 {
  /** @var string|null Temporary folder where the temporary files will be created */
  protected $tempFolder = null;