PHPOffice / PhpSpreadsheet

src/PhpSpreadsheet/Reader/Excel2003XML.php

Doc Comments +6 added lines

@@ -231,6 +231,9 @@ 
         return $this->loadIntoExisting($pFilename, $objPHPExcel);
     }
 
+    /**
+     * @param string $styleAttributeValue
+     */
     protected static function identifyFixedStyleValue($styleList, &$styleAttributeValue)
     {
         $styleAttributeValue = strtolower($styleAttributeValue);
@@ -786,6 +789,9 @@ 
     }
 
 
+    /**
+     * @param string $charset
+     */
     protected static function convertStringEncoding($string, $charset)
     {
         if ($charset != 'UTF-8') {

Spacing +13 added lines, -13 removed lines

@@ -115,10 +115,10 @@ 
     {
         // Check if file exists
         if (!file_exists($pFilename)) {
-            throw new Exception('Could not open '.$pFilename.' for reading! File does not exist.');
+            throw new Exception('Could not open ' . $pFilename . ' for reading! File does not exist.');
         }
         if (!$this->canRead($pFilename)) {
-            throw new Exception($pFilename.' is an Invalid Spreadsheet file.');
+            throw new Exception($pFilename . ' is an Invalid Spreadsheet file.');
         }
 
         $worksheetNames = [];
@@ -150,7 +150,7 @@ 
     {
         // Check if file exists
         if (!file_exists($pFilename)) {
-            throw new Exception('Could not open '.$pFilename.' for reading! File does not exist.');
+            throw new Exception('Could not open ' . $pFilename . ' for reading! File does not exist.');
         }
 
         $worksheetInfo = [];
@@ -327,11 +327,11 @@ 
 
         // Check if file exists
         if (!file_exists($pFilename)) {
-            throw new Exception('Could not open '.$pFilename.' for reading! File does not exist.');
+            throw new Exception('Could not open ' . $pFilename . ' for reading! File does not exist.');
         }
 
         if (!$this->canRead($pFilename)) {
-            throw new Exception($pFilename.' is an Invalid Spreadsheet file.');
+            throw new Exception($pFilename . ' is an Invalid Spreadsheet file.');
         }
 
         $xml = simplexml_load_string(
@@ -578,7 +578,7 @@ 
                         if (isset($cell_ss['Index'])) {
                             $columnID = \PhpOffice\PhpSpreadsheet\Cell::stringFromColumnIndex($cell_ss['Index'] - 1);
                         }
-                        $cellRange = $columnID.$rowID;
+                        $cellRange = $columnID . $rowID;
 
                         if ($this->getReadFilter() !== null) {
                             if (!$this->getReadFilter()->readCell($columnID, $rowID, $worksheetName)) {
@@ -596,7 +596,7 @@ 
                             if (isset($cell_ss['MergeDown'])) {
                                 $rowTo = $rowTo + $cell_ss['MergeDown'];
                             }
-                            $cellRange .= ':'.$columnTo.$rowTo;
+                            $cellRange .= ':' . $columnTo . $rowTo;
                             $spreadsheet->getActiveSheet()->mergeCells($cellRange);
                         }
 
@@ -697,7 +697,7 @@ 
                                                 if ($columnReference{0} == '[') {
                                                     $columnReference = $columnNumber + trim($columnReference, '[]');
                                                 }
-                                                $A1CellReference = \PhpOffice\PhpSpreadsheet\Cell::stringFromColumnIndex($columnReference - 1).$rowReference;
+                                                $A1CellReference = \PhpOffice\PhpSpreadsheet\Cell::stringFromColumnIndex($columnReference - 1) . $rowReference;
                                                 $value = substr_replace($value, $A1CellReference, $cellReference[0][1], strlen($cellReference[0][0]));
                                             }
                                         }
@@ -708,9 +708,9 @@ 
                                 $cellDataFormula = implode('"', $temp);
                             }
 
-                            $spreadsheet->getActiveSheet()->getCell($columnID.$rowID)->setValueExplicit((($hasCalculatedValue) ? $cellDataFormula : $cellValue), $type);
+                            $spreadsheet->getActiveSheet()->getCell($columnID . $rowID)->setValueExplicit((($hasCalculatedValue) ? $cellDataFormula : $cellValue), $type);
                             if ($hasCalculatedValue) {
-                                $spreadsheet->getActiveSheet()->getCell($columnID.$rowID)->setCalculatedValue($cellValue);
+                                $spreadsheet->getActiveSheet()->getCell($columnID . $rowID)->setCalculatedValue($cellValue);
                             }
                             $cellIsSet = $rowHasData = true;
                         }
@@ -723,14 +723,14 @@ 
                             }
                             $node = $cell->Comment->Data->asXML();
                             $annotation = strip_tags($node);
-                            $spreadsheet->getActiveSheet()->getComment($columnID.$rowID)->setAuthor(self::convertStringEncoding($author, $this->charSet))->setText($this->parseRichText($annotation));
+                            $spreadsheet->getActiveSheet()->getComment($columnID . $rowID)->setAuthor(self::convertStringEncoding($author, $this->charSet))->setText($this->parseRichText($annotation));
                         }
 
                         if (($cellIsSet) && (isset($cell_ss['StyleID']))) {
                             $style = (string) $cell_ss['StyleID'];
                             if ((isset($this->styles[$style])) && (!empty($this->styles[$style]))) {
-                                if (!$spreadsheet->getActiveSheet()->cellExists($columnID.$rowID)) {
-                                    $spreadsheet->getActiveSheet()->getCell($columnID.$rowID)->setValue(null);
+                                if (!$spreadsheet->getActiveSheet()->cellExists($columnID . $rowID)) {
+                                    $spreadsheet->getActiveSheet()->getCell($columnID . $rowID)->setValue(null);
                                 }
                                 $spreadsheet->getActiveSheet()->getStyle($cellRange)->applyFromArray($this->styles[$style]);
                             }

src/PhpSpreadsheet/Shared/JAMA/EigenvalueDecomposition.php

Doc Comments +2 added lines, -2 removed lines

@@ -816,7 +816,7 @@ 
      *    Return the eigenvector matrix
      *
      *    @access public
-     *    @return V
+     *    @return Matrix
      */
     public function getV()
     {
@@ -849,7 +849,7 @@ 
      *    Return the block diagonal eigenvalue matrix
      *
      *    @access public
-     *    @return D
+     *    @return Matrix
      */
     public function getD()
     {

src/PhpSpreadsheet/Shared/JAMA/SingularValueDecomposition.php

Doc Comments +3 added lines, -3 removed lines

@@ -436,7 +436,7 @@ 
      *    Return the left singular vectors
      *
      *    @access public
-     *    @return U
+     *    @return Matrix
      */
     public function getU()
     {
@@ -448,7 +448,7 @@ 
      *    Return the right singular vectors
      *
      *    @access public
-     *    @return V
+     *    @return Matrix
      */
     public function getV()
     {
@@ -472,7 +472,7 @@ 
      *    Return the diagonal matrix of singular values
      *
      *    @access public
-     *    @return S
+     *    @return Matrix
      */
     public function getS()
     {

src/PhpSpreadsheet/Shared/TimeZone.php

Doc Comments +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
     /**
      *    Return the Timezone transition for the specified timezone and timestamp
      *
-     *    @param        DateTimeZone         $objTimezone    The timezone for finding the transitions
+     *    @param        \DateTimeZone         $objTimezone    The timezone for finding the transitions
      *    @param        integer                 $timestamp        PHP date/time value for finding the current transition
      *    @return         array                The current transition details
      */

Spacing +1 added lines, -1 removed lines

@@ -120,7 +120,7 @@
     {
         if ($timezone !== null) {
             if (!self::validateTimezone($timezone)) {
-                throw new \PhpOffice\PhpSpreadsheet\Exception('Invalid timezone '.$timezone);
+                throw new \PhpOffice\PhpSpreadsheet\Exception('Invalid timezone ' . $timezone);
             }
         } else {
             $timezone = self::$timezone;

src/PhpSpreadsheet/Shared/JAMA/utils/Error.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 
 define('POLYMORPHIC_ARGUMENT_EXCEPTION', -1);
 $error['EN'][POLYMORPHIC_ARGUMENT_EXCEPTION] = "Invalid argument pattern for polymorphic function.";
-$error['FR'][POLYMORPHIC_ARGUMENT_EXCEPTION] = "Modèle inadmissible d'argument pour la fonction polymorphe.".
+$error['FR'][POLYMORPHIC_ARGUMENT_EXCEPTION] = "Modèle inadmissible d'argument pour la fonction polymorphe." .
 $error['DE'][POLYMORPHIC_ARGUMENT_EXCEPTION] = "Unzulässiges Argumentmuster für polymorphe Funktion.";
 
 define('ARGUMENT_TYPE_EXCEPTION', -2);

src/PhpSpreadsheet/Chart/DataSeriesValues.php

Spacing +1 added lines, -1 removed lines

@@ -293,7 +293,7 @@
             $calcEngine = \PHPExcel\Calculation::getInstance($worksheet->getParent());
             $newDataValues = \PHPExcel\Calculation::unwrapResult(
                 $calcEngine->_calculateFormulaValue(
-                    '='.$this->dataSource,
+                    '=' . $this->dataSource,
                     null,
                     $worksheet->getCell('A1')
                 )

src/PhpSpreadsheet/CachedObjectStorage/CacheBase.php

Spacing +1 added lines, -1 removed lines

@@ -196,7 +196,7 @@
         foreach ($this->getCellList() as $coord) {
             sscanf($coord, '%[A-Z]%d', $c, $r);
             $row[$r] = $r;
-            $col[$c] = strlen($c).$c;
+            $col[$c] = strlen($c) . $c;
         }
         if (!empty($row)) {
             // Determine highest column and row

src/PhpSpreadsheet/Worksheet/RowDimension.php

Doc Comments +1 added lines, -1 removed lines

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

src/PhpSpreadsheet/Style/Fill.php

Doc Comments +1 added lines, -1 removed lines

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

Spacing +4 added lines, -4 removed lines

@@ -330,10 +330,10 @@
         }
 
         return md5(
-            $this->getFillType().
-            $this->getRotation().
-            $this->getStartColor()->getHashCode().
-            $this->getEndColor()->getHashCode().
+            $this->getFillType() .
+            $this->getRotation() .
+            $this->getStartColor()->getHashCode() .
+            $this->getEndColor()->getHashCode() .
             __CLASS__
         );
     }