PHPOffice / PhpSpreadsheet

src/PhpSpreadsheet/Helper/HTML.php

Doc Comments +4 added lines

@@ -802,6 +802,10 @@
         $this->buildTextRun();
     }
 
+    /**
+     * @param DOMElement $element
+     * @param string $callbackTag
+     */
     protected function handleCallback($element, $callbackTag, $callbacks)
     {
         if (isset($callbacks[$callbackTag])) {

src/PhpSpreadsheet/Reader/HTML.php

Doc Comments +6 added lines

@@ -222,6 +222,12 @@
         $cellContent = (string) '';
     }
 
+    /**
+     * @param \PhpSpreadsheet\Worksheet $sheet
+     * @param integer $row
+     * @param string $column
+     * @param string $cellContent
+     */
     protected function processDomElement(DOMNode $element, $sheet, &$row, &$column, &$cellContent, $format = null)
     {
         foreach ($element->childNodes as $child) {

src/PhpSpreadsheet/Shared/Date.php

Doc Comments +5 added lines, -5 removed lines

@@ -164,7 +164,7 @@ 
     /**
      * Convert a MS serialized datetime value from Excel to a PHP Date/Time object
      *
-     * @param     int|float                      $excelTimestamp      MS Excel serialized date/time value
+     * @param     integer                      $excelTimestamp      MS Excel serialized date/time value
      * @param     \DateTimeZone|string|null          $timeZone            The timezone to assume for the Excel timestamp,
      *                                                                        if you don't want to treat it as a UTC value
      *                                                                    Use the default (UST) unless you absolutely need a conversion
@@ -203,7 +203,7 @@ 
     /**
      * Convert a MS serialized datetime value from Excel to a unix timestamp
      *
-     * @param     int|float                      $excelTimestamp        MS Excel serialized date/time value
+     * @param     integer                      $excelTimestamp        MS Excel serialized date/time value
      * @param     \DateTimeZone|string|null          $timeZone            The timezone to assume for the Excel timestamp,
      *                                                                        if you don't want to treat it as a UTC value
      *                                                                    Use the default (UST) unless you absolutely need a conversion
@@ -219,7 +219,7 @@ 
     /**
      *    Convert a date from PHP to an MS Excel serialized date/time value
      *
-     *    @param    mixed            $dateValue            Unix Timestamp or PHP DateTime object or a string
+     *    @param    integer            $dateValue            Unix Timestamp or PHP DateTime object or a string
      *    @return   float|bool    Excel date/time value
      *                                  or boolean FALSE on failure
      */
@@ -257,7 +257,7 @@ 
     /**
      *    Convert a Unix timestamp to an MS Excel serialized date/time value
      *
-     *    @param    \DateTimeInterface    $dateValue       Unix Timestamp
+     *    @param    integer    $dateValue       Unix Timestamp
      *    @return   float                 MS Excel serialized date/time value
      */
     public static function timestampToExcel($dateValue = 0)
@@ -278,7 +278,7 @@ 
      * @param    int    $hours
      * @param    int    $minutes
      * @param    int    $seconds
-     * @return   int    Excel date/time value
+     * @return   double    Excel date/time value
      */
     public static function formattedPHPToExcel($year, $month, $day, $hours = 0, $minutes = 0, $seconds = 0)
     {