documentation #182

src/Spout/Writer/Common/Cell.php

@@ -50,7 +50,7 @@ class Cell
 
     /**
      * The cell type
-     * @var int
+     * @var int|null
      */
     protected $type = null;
 
@@ -81,7 +81,7 @@ class Cell
     }
 
     /**
-     * @return mixed|null
+     * @return int|null
      */
     public function getType()
     {

src/Spout/Writer/ODS/Internal/Worksheet.php

@@ -192,6 +192,7 @@ class Worksheet implements WorksheetInterface
             $data .= ' table:number-columns-repeated="' . $numTimesValueRepeated . '"';
         }
 
+        /** @TODO Remove code duplication with XLSX writer: https://github.com/box/spout/pull/383#discussion_r113292746 */
         if ($cellValue instanceof Cell) {
             $cell = $cellValue;
         } else {

src/Spout/Writer/XLSX/Internal/Worksheet.php

@@ -214,6 +214,7 @@ EOD;
         $cellXML = '<c r="' . $columnIndex . $rowIndex . '"';
         $cellXML .= ' s="' . $styleId . '"';
 
+        /** @TODO Remove code duplication with ODS writer: https://github.com/box/spout/pull/383#discussion_r113292746 */
         if ($cellValue instanceof Cell) {
             $cell = $cellValue;
         } else {