ellumilel / php-excel-writer

src/ExcelWriter.php

Doc Comments +9 added lines, -9 removed lines

@@ -227,7 +227,7 @@ 
     }
 
     /**
-     * @param $cellFormat
+     * @param string $cellFormat
      *
      * @return string
      */
@@ -271,7 +271,7 @@ 
     /**
      * @todo  check escaping
      *
-     * @param $cellFormat
+     * @param string $cellFormat
      *
      * @return string
      */
@@ -310,7 +310,7 @@ 
      *
      * @param $cellFormat
      *
-     * @return int|mixed
+     * @return integer
      */
     private function addCellFormat($cellFormat)
     {
@@ -517,8 +517,8 @@ 
 
     /**
      * @param Writer $file
-     * @param $rowNumber
-     * @param $columnNumber
+     * @param integer $rowNumber
+     * @param integer $columnNumber
      * @param $value
      * @param $cellIndex
      */
@@ -647,7 +647,7 @@ 
     }
 
     /**
-     * @param $v
+     * @param string $v
      *
      * @return int|mixed
      */
@@ -792,7 +792,7 @@ 
     }
 
     /**
-     * @param $string
+     * @param string $string
      */
     public static function log($string)
     {
@@ -807,7 +807,7 @@ 
      *
      * @param string $filename
      *
-     * @return mixed
+     * @return string
      */
     public static function checkFilename($filename)
     {
@@ -822,7 +822,7 @@ 
     /**
      * @param $val
      *
-     * @return mixed
+     * @return string
      */
     public static function xmlspecialchars($val)
     {

Spacing +14 added lines, -14 removed lines

@@ -23,19 +23,19 @@ 
     private $urlSchemaFormat = 'http://schemas.openxmlformats.org/officeDocument/2006';
 
     /** @var string */
-    protected $author ='Unknown Author';
+    protected $author = 'Unknown Author';
     /** @var array */
     protected $sheets = [];
     /** @var array */
-    protected $sharedStrings = [];//unique set
+    protected $sharedStrings = []; //unique set
     /** @var int */
-    protected $shared_string_count = 0;//count of non-unique references to the unique set
+    protected $shared_string_count = 0; //count of non-unique references to the unique set
     /** @var array */
     protected $tempFiles = [];
     /** @var array */
-    protected $cellFormats = [];//contains excel format like YYYY-MM-DD HH:MM:SS
+    protected $cellFormats = []; //contains excel format like YYYY-MM-DD HH:MM:SS
     /** @var array */
-    protected $cellTypes = [];//contains friendly format like datetime
+    protected $cellTypes = []; //contains friendly format like datetime
     /** @var string  */
     protected $currentSheet = '';
     /** @var null */
@@ -123,7 +123,7 @@ 
     public function writeToFile($filename)
     {
         foreach ($this->sheets as $sheetName => $sheet) {
-            $this->finalizeSheet($sheetName);//making sure all footers have been written
+            $this->finalizeSheet($sheetName); //making sure all footers have been written
         }
         if (file_exists($filename)) {
             if (is_writable($filename)) {
@@ -186,7 +186,7 @@ 
             return;
         }
         $sheetFilename = $this->tempFilename();
-        $sheetXmlName = 'sheet' . (count($this->sheets) + 1).".xml";
+        $sheetXmlName = 'sheet'.(count($this->sheets) + 1).".xml";
         $sheetObj = new Sheet();
         $sheetObj
             ->setFilename($sheetFilename)
@@ -197,8 +197,8 @@ 
         $this->sheets[$sheetName] = $sheetObj;
         /** @var Sheet $sheet */
         $sheet = &$this->sheets[$sheetName];
-        $selectedTab = count($this->sheets) == 1 ? 'true' : 'false';//only first sheet is selected
-        $maxCell = ExcelWriter::xlsCell(self::EXCEL_MAX_ROW, self::EXCEL_MAX_COL);//XFE1048577
+        $selectedTab = count($this->sheets) == 1 ? 'true' : 'false'; //only first sheet is selected
+        $maxCell = ExcelWriter::xlsCell(self::EXCEL_MAX_ROW, self::EXCEL_MAX_COL); //XFE1048577
         $sheet->getWriter()->write('<?xml version="1.0" encoding="UTF-8" standalone="yes"?>'."\n");
         $sheet->getWriter()->write(
             '<worksheet xmlns="http://schemas.openxmlformats.org/spreadsheetml/2006/main" 
@@ -405,7 +405,7 @@ 
         $sheet = &$this->sheets[$sheetName];
         $columns = $sheet->getColumns();
         if (empty($columns)) {
-            $sheet->setColumns(array_fill($from = 0, $until = count($row), '0'));//'0'=>'string'
+            $sheet->setColumns(array_fill($from = 0, $until = count($row), '0')); //'0'=>'string'
         }
         $sheet->getWriter()->write(
             '<row collapsed="false" customFormat="false" customHeight="false" 
@@ -624,7 +624,7 @@ 
         $file->write('<cellXfs count="'.count($this->cellFormats).'">');
         foreach ($this->cellFormats as $i => $v) {
             $file->write('<xf applyAlignment="false" applyBorder="false" applyFont="false" 
-            applyProtection="false" borderId="0" fillId="0" fontId="0" numFmtId="'.(164+$i).'" xfId="0"/>');
+            applyProtection="false" borderId="0" fillId="0" fontId="0" numFmtId="'.(164 + $i).'" xfId="0"/>');
         }
         $file->write('</cellXfs>');
         //$file->write(	'<cellXfs count="4">');
@@ -659,7 +659,7 @@ 
             $stringValue = count($this->sharedStrings);
             $this->sharedStrings[$v] = $stringValue;
         }
-        $this->shared_string_count++;//non-unique count
+        $this->shared_string_count++; //non-unique count
 
         return $stringValue;
     }
@@ -870,8 +870,8 @@ 
         $norm   = 300;
         $range  = $year - $epoch;
         # Set month days and check for leap year.
-        $leap = (($year % 400 == 0) || (($year % 4 == 0) && ($year % 100)) ) ? 1 : 0;
-        $mdays = array( 31, ($leap ? 29 : 28), 31, 30, 31, 30, 31, 31, 30, 31, 30, 31 );
+        $leap = (($year % 400 == 0) || (($year % 4 == 0) && ($year % 100))) ? 1 : 0;
+        $mdays = array(31, ($leap ? 29 : 28), 31, 30, 31, 30, 31, 31, 30, 31, 30, 31);
         # Some boundary checks
         if ($year < $epoch || $year > 9999) {
             return 0;