ellumilel / php-excel-writer

src/ExcelWriter.php

Doc Comments +8 added lines, -8 removed lines

@@ -214,7 +214,7 @@ 
     }
 
     /**
-     * @param $cellFormat
+     * @param string $cellFormat
      *
      * @return string
      */
@@ -256,7 +256,7 @@ 
     }
 
     /**
-     * @param $cellFormat
+     * @param string $cellFormat
      *
      * @return string
      */
@@ -502,8 +502,8 @@ 
 
     /**
      * @param Writer $file
-     * @param $rowNumber
-     * @param $columnNumber
+     * @param integer $rowNumber
+     * @param integer $columnNumber
      * @param $value
      * @param $cellIndex
      */
@@ -632,7 +632,7 @@ 
     }
 
     /**
-     * @param $v
+     * @param string $v
      *
      * @return int|mixed
      */
@@ -832,7 +832,7 @@ 
     }
 
     /**
-     * @param $string
+     * @param string $string
      */
     public static function log($string)
     {
@@ -847,7 +847,7 @@ 
      *
      * @param string $filename
      *
-     * @return mixed
+     * @return string
      */
     public static function checkFilename($filename)
     {
@@ -862,7 +862,7 @@ 
     /**
      * @param $val
      *
-     * @return mixed
+     * @return string
      */
     public static function xmlspecialchars($val)
     {

Spacing +19 added lines, -19 removed lines

@@ -17,19 +17,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 */
@@ -115,7 +115,7 @@ 
     public function writeToFile($filename)
     {
         foreach ($this->sheets as $sheetName => $sheet) {
-            self::finalizeSheet($sheetName);//making sure all footers have been written
+            self::finalizeSheet($sheetName); //making sure all footers have been written
         }
         if (file_exists($filename)) {
             if (is_writable($filename)) {
@@ -151,13 +151,13 @@ 
             $zip->addFile(
                 $this->writeSharedStringsXML(),
                 "xl/sharedStrings.xml"
-            );  //$zip->addFromString("xl/sharedStrings.xml",     self::buildSharedStringsXML() );
+            ); //$zip->addFromString("xl/sharedStrings.xml",     self::buildSharedStringsXML() );
         }
         $zip->addFromString("xl/workbook.xml", self::buildWorkbookXML());
         $zip->addFile(
             $this->writeStylesXML(),
             "xl/styles.xml"
-        );  //$zip->addFromString("xl/styles.xml"           , self::buildStylesXML() );
+        ); //$zip->addFromString("xl/styles.xml"           , self::buildStylesXML() );
         $zip->addFromString("[Content_Types].xml", self::buildContentTypesXML());
         $zip->addEmptyDir("xl/_rels/");
         $zip->addFromString("xl/_rels/workbook.xml.rels", self::buildWorkbookRelXML());
@@ -173,7 +173,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)
@@ -184,8 +184,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" 
@@ -390,7 +390,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" 
@@ -533,7 +533,7 @@ 
         } elseif ($cellType == 'currency' || $cellType == 'percent' || $cellType == 'numeric') {
             $file->write(
                 '<c r="'.$cellName.'" s="'.$cellIndex.'" t="n"><v>'.self::xmlspecialchars($value).'</v></c>'
-            );//int,float,currency
+            ); //int,float,currency
         } else {
             if (!is_string($value)) {
                 $file->write('<c r="'.$cellName.'" s="'.$cellIndex.'" t="n"><v>'.($value * 1).'</v></c>');
@@ -609,7 +609,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">');
@@ -644,7 +644,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;
     }
@@ -876,8 +876,8 @@ 
      */
     public static function convertDateTime($dateInput) //thanks to Excel::Writer::XLSX::Worksheet.pm (perl)
     {
-        $days = 0;    # Number of days since epoch
-        $seconds = 0;    # Time expressed as fraction of 24h hours in seconds
+        $days = 0; # Number of days since epoch
+        $seconds = 0; # Time expressed as fraction of 24h hours in seconds
         $year = $month = $day = 0;
         $hour = $min = $sec = 0;
         $date_time = $dateInput;
@@ -908,8 +908,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;