PHPOffice / PhpSpreadsheet

src/PhpSpreadsheet/Shared/XMLWriter.php

Doc Comments +1 added lines, -1 removed lines

@@ -109,7 +109,7 @@
      * Fallback method for writeRaw, introduced in PHP 5.2
      *
      * @param string $text
-     * @return string
+     * @return boolean
      */
     public function writeRawData($text)
     {

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
      *
      * @var string
      */
-    private $tempFileName  = '';
+    private $tempFileName = '';
 
     /**
      * Create a new XMLWriter instance

src/PhpSpreadsheet/Shared/ZipArchive.php

Doc Comments +3 added lines

@@ -155,6 +155,9 @@
         return $contents;
     }
 
+    /**
+     * @param integer $index
+     */
     public function getFromIndex($index)
     {
         $extracted = $this->zip->extractByIndex($index, PCLZIP_OPT_EXTRACT_AS_STRING);

Indentation +1 added lines, -1 removed lines

@@ -160,7 +160,7 @@
         $extracted = $this->zip->extractByIndex($index, PCLZIP_OPT_EXTRACT_AS_STRING);
         $contents = '';
         if ((is_array($extracted)) && ($extracted != 0)) {
-             $contents = $extracted[0]["content"];
+                $contents = $extracted[0]["content"];
         }
     }
 }

Spacing +3 added lines, -3 removed lines

@@ -91,16 +91,16 @@
     {
         $filenameParts = pathinfo($localname);
 
-        $handle = fopen($this->tempDir.'/'.$filenameParts["basename"], "wb");
+        $handle = fopen($this->tempDir . '/' . $filenameParts["basename"], "wb");
         fwrite($handle, $contents);
         fclose($handle);
 
-        $res = $this->zip->add($this->tempDir.'/'.$filenameParts["basename"], PCLZIP_OPT_REMOVE_PATH, $this->tempDir, PCLZIP_OPT_ADD_PATH, $filenameParts["dirname"]);
+        $res = $this->zip->add($this->tempDir . '/' . $filenameParts["basename"], PCLZIP_OPT_REMOVE_PATH, $this->tempDir, PCLZIP_OPT_ADD_PATH, $filenameParts["dirname"]);
         if ($res == 0) {
             throw new \PHPExcel\Writer\Exception("Error zipping files : " . $this->zip->errorInfo(true));
         }
 
-        unlink($this->tempDir.'/'.$filenameParts["basename"]);
+        unlink($this->tempDir . '/' . $filenameParts["basename"]);
     }
 
     /**

src/PhpSpreadsheet/Shared/ZipStreamWrapper.php

Doc Comments +3 added lines, -3 removed lines

@@ -101,7 +101,7 @@ 
     /**
      * Implements support for fstat().
      *
-     * @return  boolean
+     * @return  string
      */
     public function statName()
     {
@@ -111,7 +111,7 @@ 
     /**
      * Implements support for fstat().
      *
-     * @return  boolean
+     * @return  string
      */
     public function url_stat() // @codingStandardsIgnoreLine
     {
@@ -167,7 +167,7 @@ 
      *
      * @param    int        $offset    byte offset
      * @param    int        $whence    SEEK_SET, SEEK_CUR or SEEK_END
-     * @return    bool
+     * @return    boolean|null
      */
     public function stream_seek($offset, $whence) // @codingStandardsIgnoreLine
     {

Indentation +9 added lines, -9 removed lines

@@ -174,26 +174,26 @@
         switch ($whence) {
             case SEEK_SET:
                 if ($offset < strlen($this->data) && $offset >= 0) {
-                     $this->position = $offset;
-                     return true;
+                        $this->position = $offset;
+                        return true;
                 } else {
-                     return false;
+                        return false;
                 }
                 break;
             case SEEK_CUR:
                 if ($offset >= 0) {
-                     $this->position += $offset;
-                     return true;
+                        $this->position += $offset;
+                        return true;
                 } else {
-                     return false;
+                        return false;
                 }
                 break;
             case SEEK_END:
                 if (strlen($this->data) + $offset >= 0) {
-                     $this->position = strlen($this->data) + $offset;
-                     return true;
+                        $this->position = strlen($this->data) + $offset;
+                        return true;
                 } else {
-                     return false;
+                        return false;
                 }
                 break;
             default:

src/PhpSpreadsheet/Style/Conditional.php

Doc Comments +2 added lines, -2 removed lines

@@ -100,7 +100,7 @@ 
     /**
      * Get Condition type
      *
-     * @return string
+     * @return integer
      */
     public function getConditionType()
     {
@@ -122,7 +122,7 @@ 
     /**
      * Get Operator type
      *
-     * @return string
+     * @return integer
      */
     public function getOperatorType()
     {

Indentation +2 added lines, -2 removed lines

@@ -250,8 +250,8 @@
      */
     public function setStyle(\PHPExcel\Style $pValue = null)
     {
-           $this->style = $pValue;
-           return $this;
+            $this->style = $pValue;
+            return $this;
     }
 
     /**

src/PhpSpreadsheet/Worksheet/AutoFilter.php

Doc Comments +6 added lines

@@ -467,6 +467,7 @@ 
      *
      *    @param    string                   $dynamicRuleType
      *    @param    AutoFilter\Column        &$filterColumn
+     * @param AutoFilter\Column $filterColumn
      *    @return mixed[]
      */
     private function dynamicFilterDateRange($dynamicRuleType, &$filterColumn)
@@ -571,6 +572,11 @@ 
         return array('method' => 'filterTestInCustomDataSet', 'arguments' => array('filterRules' => $ruleValues, 'join' => AutoFilter\Column::AUTOFILTER_COLUMN_JOIN_AND));
     }
 
+    /**
+     * @param integer $columnID
+     * @param integer $startRow
+     * @param string $ruleType
+     */
     private function calculateTopTenValue($columnID, $startRow, $endRow, $ruleType, $ruleValue)
     {
         $range = $columnID.$startRow.':'.$columnID.$endRow;

Indentation +2 added lines, -2 removed lines

@@ -711,8 +711,8 @@
                                 ? AutoFilter\Column\Rule::AUTOFILTER_COLUMN_RULE_GREATERTHAN
                                 : AutoFilter\Column\Rule::AUTOFILTER_COLUMN_RULE_LESSTHAN;
                             $ruleValues[] = array('operator' => $operator,
-                                                   'value' => $average
-                                                 );
+                                                    'value' => $average
+                                                    );
                             $columnFilterTests[$columnID] = array(
                                 'method' => 'filterTestInCustomDataSet',
                                 'arguments' => array('filterRules' => $ruleValues, 'join' => AutoFilter\Column::AUTOFILTER_COLUMN_JOIN_OR)

Spacing +11 added lines, -11 removed lines

@@ -407,7 +407,7 @@ 
                 }
             } else {
                 //    String values are always tested for equality, factoring in for wildcards (hence a regexp test)
-                $retVal    = preg_match('/^'.$rule['value'].'$/i', $cellValue);
+                $retVal = preg_match('/^' . $rule['value'] . '$/i', $cellValue);
             }
             //    If there are multiple conditions, then we need to test both using the appropriate join operator
             switch ($join) {
@@ -458,7 +458,7 @@ 
      *    @var    array
      */
     private static $fromReplace = array('\*', '\?', '~~', '~.*', '~.?');
-    private static $toReplace   = array('.*', '.',  '~',  '\*',  '\?');
+    private static $toReplace   = array('.*', '.', '~', '\*', '\?');
 
 
     /**
@@ -527,9 +527,9 @@ 
             case AutoFilter\Column\Rule::AUTOFILTER_RULETYPE_DYNAMIC_NEXTQUARTER:
                 $thisMonth = date('m', $baseDate);
                 $thisQuarter = floor(--$thisMonth / 3);
-                $maxVal = (int) \PhpSpreadsheet\Shared\Date::PHPtoExcel(gmmktime(0, 0, 0, date('t', $baseDate), (1+$thisQuarter)*3, date('Y', $baseDate)));
+                $maxVal = (int) \PhpSpreadsheet\Shared\Date::PHPtoExcel(gmmktime(0, 0, 0, date('t', $baseDate), (1 + $thisQuarter) * 3, date('Y', $baseDate)));
                 ++$maxVal;
-                $val = (int) \PhpSpreadsheet\Shared\Date::PHPToExcel(gmmktime(0, 0, 0, 1, 1+$thisQuarter*3, date('Y', $baseDate)));
+                $val = (int) \PhpSpreadsheet\Shared\Date::PHPToExcel(gmmktime(0, 0, 0, 1, 1 + $thisQuarter * 3, date('Y', $baseDate)));
                 break;
             case AutoFilter\Column\Rule::AUTOFILTER_RULETYPE_DYNAMIC_THISMONTH:
             case AutoFilter\Column\Rule::AUTOFILTER_RULETYPE_DYNAMIC_LASTMONTH:
@@ -572,7 +572,7 @@ 
 
     private function calculateTopTenValue($columnID, $startRow, $endRow, $ruleType, $ruleValue)
     {
-        $range = $columnID.$startRow.':'.$columnID.$endRow;
+        $range = $columnID . $startRow . ':' . $columnID . $endRow;
         $dataValues = \PhpSpreadsheet\Calculation\Functions::flattenArray($this->workSheet->rangeToArray($range, null, true, false));
 
         $dataValues = array_filter($dataValues);
@@ -703,7 +703,7 @@ 
                             ($dynamicRuleType == AutoFilter\Column\Rule::AUTOFILTER_RULETYPE_DYNAMIC_BELOWAVERAGE)) {
                             //    Number (Average) based
                             //    Calculate the average
-                            $averageFormula = '=AVERAGE('.$columnID.($rangeStart[1]+1).':'.$columnID.$rangeEnd[1].')';
+                            $averageFormula = '=AVERAGE(' . $columnID . ($rangeStart[1] + 1) . ':' . $columnID . $rangeEnd[1] . ')';
                             $average = \PhpSpreadsheet\Calculation::getInstance()->calculateFormula($averageFormula, null, $this->workSheet->getCell('A1'));
                             //    Set above/below rule based on greaterThan or LessTan
                             $operator = ($dynamicRuleType === AutoFilter\Column\Rule::AUTOFILTER_RULETYPE_DYNAMIC_ABOVEAVERAGE)
@@ -725,8 +725,8 @@ 
                                     $ruleValues = array($period);
                                 } else {
                                     --$period;
-                                    $periodEnd = (1+$period)*3;
-                                    $periodStart = 1+$period*3;
+                                    $periodEnd = (1 + $period) * 3;
+                                    $periodStart = 1 + $period * 3;
                                     $ruleValues = range($periodStart, $periodEnd);
                                 }
                                 $columnFilterTests[$columnID] = array(
@@ -761,7 +761,7 @@ 
                         $ruleValue = 500;
                     }
 
-                    $maxVal = $this->calculateTopTenValue($columnID, $rangeStart[1]+1, $rangeEnd[1], $toptenRuleType, $ruleValue);
+                    $maxVal = $this->calculateTopTenValue($columnID, $rangeStart[1] + 1, $rangeEnd[1], $toptenRuleType, $ruleValue);
 
                     $operator = ($toptenRuleType == AutoFilter\Column\Rule::AUTOFILTER_COLUMN_RULE_TOPTEN_TOP)
                         ? AutoFilter\Column\Rule::AUTOFILTER_COLUMN_RULE_GREATERTHANOREQUAL
@@ -780,12 +780,12 @@ 
 //        var_dump($columnFilterTests);
 //
         //    Execute the column tests for each row in the autoFilter range to determine show/hide,
-        for ($row = $rangeStart[1]+1; $row <= $rangeEnd[1]; ++$row) {
+        for ($row = $rangeStart[1] + 1; $row <= $rangeEnd[1]; ++$row) {
 //            echo 'Testing Row = ', $row,PHP_EOL;
             $result = true;
             foreach ($columnFilterTests as $columnID => $columnFilterTest) {
 //                echo 'Testing cell ', $columnID.$row,PHP_EOL;
-                $cellValue = $this->workSheet->getCell($columnID.$row)->getCalculatedValue();
+                $cellValue = $this->workSheet->getCell($columnID . $row)->getCalculatedValue();
 //                echo 'Value is ', $cellValue,PHP_EOL;
                 //    Execute the filter test
                 $result = $result &&

src/PhpSpreadsheet/Worksheet/ColumnDimension.php

Doc Comments +2 added lines, -2 removed lines

@@ -69,7 +69,7 @@ 
     /**
      * Get ColumnIndex
      *
-     * @return string
+     * @return integer
      */
     public function getColumnIndex()
     {
@@ -101,7 +101,7 @@ 
     /**
      * Set Width
      *
-     * @param double $pValue
+     * @param integer $pValue
      * @return ColumnDimension
      */
     public function setWidth($pValue = -1)

src/PhpSpreadsheet/Worksheet/Dimension.php

Doc Comments +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
     /**
      * Create a new Dimension
      *
-     * @param int $pIndex Numeric row index
+     * @param integer $initialValue
      */
     public function __construct($initialValue = null)
     {

src/PhpSpreadsheet/Worksheet/PageSetup.php

Doc Comments +2 added lines, -2 removed lines

@@ -802,7 +802,7 @@ 
      * Set first page number
      *
      * @param int $value
-     * @return HeaderFooter
+     * @return PageSetup
      */
     public function setFirstPageNumber($value = null)
     {
@@ -813,7 +813,7 @@ 
     /**
      * Reset first page number
      *
-     * @return HeaderFooter
+     * @return PageSetup
      */
     public function resetFirstPageNumber()
     {

Indentation +15 added lines, -15 removed lines

@@ -207,27 +207,27 @@
     private $scale = 100;
 
     /**
-      * Fit To Page
-      * Whether scale or fitToWith / fitToHeight applies
-      *
-      * @var boolean
-      */
+     * Fit To Page
+     * Whether scale or fitToWith / fitToHeight applies
+     *
+     * @var boolean
+     */
     private $fitToPage = false;
 
     /**
-      * Fit To Height
-      * Number of vertical pages to fit on
-      *
-      * @var int?
-      */
+     * Fit To Height
+     * Number of vertical pages to fit on
+     *
+     * @var int?
+     */
     private $fitToHeight    = 1;
 
     /**
-      * Fit To Width
-      * Number of horizontal pages to fit on
-      *
-      * @var int?
-      */
+     * Fit To Width
+     * Number of horizontal pages to fit on
+     *
+     * @var int?
+     */
     private $fitToWidth    = 1;
 
     /**

Spacing +9 added lines, -9 removed lines

@@ -220,7 +220,7 @@ 
       *
       * @var int?
       */
-    private $fitToHeight    = 1;
+    private $fitToHeight = 1;
 
     /**
       * Fit To Width
@@ -228,7 +228,7 @@ 
       *
       * @var int?
       */
-    private $fitToWidth    = 1;
+    private $fitToWidth = 1;
 
     /**
      * Columns to repeat at left
@@ -599,8 +599,8 @@ 
             return $this->printArea;
         }
         $printAreas = explode(',', $this->printArea);
-        if (isset($printAreas[$index-1])) {
-            return $printAreas[$index-1];
+        if (isset($printAreas[$index - 1])) {
+            return $printAreas[$index - 1];
         }
         throw new \PHPExcel\Exception("Requested Print Area does not exist");
     }
@@ -620,7 +620,7 @@ 
             return !is_null($this->printArea);
         }
         $printAreas = explode(',', $this->printArea);
-        return isset($printAreas[$index-1]);
+        return isset($printAreas[$index - 1]);
     }
 
     /**
@@ -638,8 +638,8 @@ 
             $this->printArea = null;
         } else {
             $printAreas = explode(',', $this->printArea);
-            if (isset($printAreas[$index-1])) {
-                unset($printAreas[$index-1]);
+            if (isset($printAreas[$index - 1])) {
+                unset($printAreas[$index - 1]);
                 $this->printArea = implode(',', $printAreas);
             }
         }
@@ -689,12 +689,12 @@ 
                 if (($index <= 0) || ($index > count($printAreas))) {
                     throw new \PHPExcel\Exception('Invalid index for setting print range.');
                 }
-                $printAreas[$index-1] = $value;
+                $printAreas[$index - 1] = $value;
                 $this->printArea = implode(',', $printAreas);
             }
         } elseif ($method == self::SETPRINTRANGE_INSERT) {
             if ($index == 0) {
-                $this->printArea .= ($this->printArea == '') ? $value : ','.$value;
+                $this->printArea .= ($this->printArea == '') ? $value : ',' . $value;
             } else {
                 $printAreas = explode(',', $this->printArea);
                 if ($index < 0) {

src/PhpSpreadsheet/Worksheet/RowDimension.php

Doc Comments +1 added lines, -1 removed lines

@@ -101,7 +101,7 @@
     /**
      * Set Row Height
      *
-     * @param double $pValue
+     * @param integer $pValue
      * @return RowDimension
      */
     public function setRowHeight($pValue = -1)

Indentation +5 added lines, -5 removed lines

@@ -45,11 +45,11 @@
      */
     private $height = -1;
 
-     /**
-     * ZeroHeight for Row?
-     *
-     * @var bool
-     */
+        /**
+         * ZeroHeight for Row?
+         *
+         * @var bool
+         */
     private $zeroHeight = false;
 
     /**