dompdf / dompdf

src/Adapter/GD.php

Doc Comments +4 added lines

@@ -859,6 +859,10 @@
         return $this->_downscale($height);
     }
 
+    /**
+     * @param string $font
+     * @param double $size
+     */
     private function get_font_height_actual($font, $size)
     {
         $font = $this->get_ttf_file($font);

Spacing +3 added lines, -3 removed lines

@@ -866,7 +866,7 @@ 
         $text = mb_encode_numericentity($text, array(0x0080, 0xffff, 0, 0xffff), 'UTF-8');
 
         // FIXME: word spacing
-        list($x1, , $x2) = imagettfbbox($size, 0, $font, $text);
+        list($x1,, $x2) = imagettfbbox($size, 0, $font, $text);
 
         // Add additional 1pt to prevent text overflow issues
         return $this->_downscale($x2 - $x1) + 1;
@@ -878,7 +878,7 @@ 
      */
     public function get_ttf_file($font)
     {
-        if ( stripos($font, ".ttf") === false ) {
+        if (stripos($font, ".ttf") === false) {
             $font .= ".ttf";
         }
 
@@ -923,7 +923,7 @@ 
         $ratio = $this->_dompdf->getOptions()->getFontHeightRatio();
 
         // FIXME: word spacing
-        list(, $y2, , , , $y1) = imagettfbbox($size, 0, $font, "MXjpqytfhl"); // Test string with ascenders, descenders and caps
+        list(, $y2,,,, $y1) = imagettfbbox($size, 0, $font, "MXjpqytfhl"); // Test string with ascenders, descenders and caps
         return ($y2 - $y1) * $ratio;
     }
 

Braces +12 added lines, -4 removed lines

@@ -991,8 +991,12 @@ 
             die("Unable to stream image: headers already sent");
         }
 
-        if (!isset($options["type"])) $options["type"] = "png";
-        if (!isset($options["Attachment"])) $options["Attachment"] = true;
+        if (!isset($options["type"])) {
+            $options["type"] = "png";
+        }
+        if (!isset($options["Attachment"])) {
+            $options["Attachment"] = true;
+        }
         $type = strtolower($options["type"]);
 
         switch ($type) {
@@ -1043,8 +1047,12 @@ 
      */
     private function _output($options = array())
     {
-        if (!isset($options["type"])) $options["type"] = "png";
-        if (!isset($options["page"])) $options["page"] = 1;
+        if (!isset($options["type"])) {
+            $options["type"] = "png";
+        }
+        if (!isset($options["page"])) {
+            $options["page"] = 1;
+        }
         $type = strtolower($options["type"]);
 
         if (isset($this->_imgs[$options["page"] - 1])) {

src/Adapter/PDFLib.php

Unused Use Statements -2 removed lines

@@ -13,9 +13,7 @@
 use Dompdf\Dompdf;
 use Dompdf\Helpers;
 use Dompdf\Exception;
-use Dompdf\FontMetrics;
 use Dompdf\Image\Cache;
-use Dompdf\PhpEvaluator;
 
 /**
  * PDF rendering interface

Doc Comments +4 added lines, -4 removed lines

@@ -401,7 +401,7 @@ 
     }
 
     /**
-     * @return float|mixed
+     * @return double
      */
     public function get_width()
     {
@@ -409,7 +409,7 @@ 
     }
 
     /**
-     * @return float|mixed
+     * @return double
      */
     public function get_height()
     {
@@ -452,7 +452,7 @@ 
      * Sets the line style
      *
      * @param float $width
-     * @param        $cap
+     * @param        string $cap
      * @param string $join
      * @param array $dash
      *
@@ -1102,7 +1102,7 @@ 
      * @param float $size
      * @param int $word_spacing
      * @param int $letter_spacing
-     * @return mixed
+     * @return integer
      */
     public function get_text_width($text, $font, $size, $word_spacing = 0, $letter_spacing = 0)
     {

Braces +9 added lines, -3 removed lines

@@ -1282,8 +1282,12 @@ 
             die("Unable to stream pdf: headers already sent");
         }
 
-        if (!isset($options["compress"])) $options["compress"] = true;
-        if (!isset($options["Attachment"])) $options["Attachment"] = true;
+        if (!isset($options["compress"])) {
+            $options["compress"] = true;
+        }
+        if (!isset($options["Attachment"])) {
+            $options["Attachment"] = true;
+        }
 
         $this->_add_page_text();
 
@@ -1349,7 +1353,9 @@ 
      */
     public function output($options = array())
     {
-        if (!isset($options["compress"])) $options["compress"] = true;
+        if (!isset($options["compress"])) {
+            $options["compress"] = true;
+        }
 
         $this->_add_page_text();
 

src/Frame.php

Doc Comments +5 added lines, -5 removed lines

@@ -613,7 +613,7 @@ 
 
     // Set methods
     /**
-     * @param $id
+     * @param integer $id
      */
     public function set_id($id)
     {
@@ -649,10 +649,10 @@ 
     }
 
     /**
-     * @param null $x
-     * @param null $y
-     * @param null $w
-     * @param null $h
+     * @param double|null $x
+     * @param double|null $y
+     * @param double|null $w
+     * @param double|null $h
      */
     public function set_containing_block($x = null, $y = null, $w = null, $h = null)
     {

Spacing +6 added lines, -6 removed lines

@@ -195,18 +195,18 @@
             "h" => null,
         );
 
-        $this->_containing_block[0] =& $this->_containing_block["x"];
-        $this->_containing_block[1] =& $this->_containing_block["y"];
-        $this->_containing_block[2] =& $this->_containing_block["w"];
-        $this->_containing_block[3] =& $this->_containing_block["h"];
+        $this->_containing_block[0] = & $this->_containing_block["x"];
+        $this->_containing_block[1] = & $this->_containing_block["y"];
+        $this->_containing_block[2] = & $this->_containing_block["w"];
+        $this->_containing_block[3] = & $this->_containing_block["h"];
 
         $this->_position = array(
             "x" => null,
             "y" => null,
         );
 
-        $this->_position[0] =& $this->_position["x"];
-        $this->_position[1] =& $this->_position["y"];
+        $this->_position[0] = & $this->_position["x"];
+        $this->_position[1] = & $this->_position["y"];
 
         $this->_opacity = 1.0;
         $this->_decorator = null;

src/FrameDecorator/AbstractFrameDecorator.php

Doc Comments +2 added lines, -2 removed lines

@@ -128,7 +128,7 @@ 
      *
      * @param DOMNode $node
      *
-     * @return Frame
+     * @return AbstractFrameDecorator|null
      */
     function copy(DOMNode $node)
     {
@@ -141,7 +141,7 @@ 
     /**
      * Create a deep copy: copy this node and all children
      *
-     * @return Frame
+     * @return AbstractFrameDecorator|null
      */
     function deep_copy()
     {

src/FrameDecorator/Image.php

Doc Comments +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
      * Class constructor
      *
      * @param Frame $frame the frame to decorate
-     * @param DOMPDF $dompdf the document's dompdf object (required to resolve relative & remote urls)
+     * @param Dompdf $dompdf the document's dompdf object (required to resolve relative & remote urls)
      */
     function __construct(Frame $frame, Dompdf $dompdf)
     {

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 
 use Dompdf\Dompdf;
 use Dompdf\Frame;
-use Dompdf\FontMetrics;
 use Dompdf\Image\Cache;
 
 /**

src/FrameDecorator/ListBulletImage.php

Doc Comments +2 added lines, -2 removed lines

@@ -79,7 +79,7 @@ 
     /**
      * Return the bullet's width
      *
-     * @return int
+     * @return double
      */
     function get_width()
     {
@@ -133,7 +133,7 @@ 
     /**
      * Override get_margin_height()
      *
-     * @return int
+     * @return double
      */
     function get_margin_height()
     {

src/FrameDecorator/Text.php

Unused Use Statements -2 removed lines

@@ -12,8 +12,6 @@
 use Dompdf\Dompdf;
 use Dompdf\Frame;
 use Dompdf\Exception;
-use DOMText;
-use Dompdf\FontMetrics;
 
 /**
  * Decorates Frame objects for text layout

Doc Comments +3 added lines, -3 removed lines

@@ -127,7 +127,7 @@ 
     }
 
     /**
-     * @param $spacing
+     * @param double $spacing
      */
     function set_text_spacing($spacing)
     {
@@ -163,8 +163,8 @@ 
      * split the text in this frame at the offset specified.  The remaining
      * text is added a sibling frame following this one and is returned.
      *
-     * @param $offset
-     * @return Frame|null
+     * @param integer $offset
+     * @return null|AbstractFrameDecorator
      */
     function split_text($offset)
     {

src/FrameReflower/AbstractFrameReflower.php

Doc Comments +5 added lines, -1 removed lines

@@ -227,7 +227,7 @@ 
     /**
      * Parses a CSS string containing quotes and escaped hex characters
      *
-     * @param $string string The CSS string to parse
+     * @param string $string string The CSS string to parse
      * @param $single_trim
      * @return string
      */
@@ -245,6 +245,10 @@ 
 
         // Convert escaped hex characters into ascii characters (e.g. \A => newline)
         $string = preg_replace_callback("/\\\\([0-9a-fA-F]{0,6})/",
+
+            /**
+             * @param string $matches
+             */
             function ($matches) { return \Dompdf\Helpers::unichr(hexdec($matches[1])); },
             $string);
         return $string;

Spacing +12 added lines, -12 removed lines

@@ -93,7 +93,7 @@ 
 
         // Collapse vertical margins:
         $n = $frame->get_next_sibling();
-        if ( $n && !$n->is_block() & !$n->is_table() ) {
+        if ($n && !$n->is_block() & !$n->is_table()) {
             while ($n = $n->get_next_sibling()) {
                 if ($n->is_block() || $n->is_table()) {
                     break;
@@ -116,13 +116,13 @@ 
         // Collapse our first child's margin, if there is no border or padding
         if ($style->get_border_top_width() == 0 && $style->length_in_pt($style->padding_top) == 0) {
             $f = $this->_frame->get_first_child();
-            if ( $f && !$f->is_block() && !$f->is_table() ) {
-                while ( $f = $f->get_next_sibling() ) {
-                    if ( $f->is_block() || $f->is_table() ) {
+            if ($f && !$f->is_block() && !$f->is_table()) {
+                while ($f = $f->get_next_sibling()) {
+                    if ($f->is_block() || $f->is_table()) {
                         break;
                     }
 
-                    if ( !$f->get_first_child() ) {
+                    if (!$f->get_first_child()) {
                         $f = null;
                         break;
                     }
@@ -133,7 +133,7 @@ 
             if ($f) {
                 $f_style = $f->get_style();
                 $t = max($t, (float)$f_style->length_in_pt($f_style->margin_top, $cb["h"]));
-                $style->margin_top = $t."pt";
+                $style->margin_top = $t . "pt";
                 $f_style->margin_top = "0pt";
             }
         }
@@ -141,13 +141,13 @@ 
         // Collapse our last child's margin, if there is no border or padding
         if ($style->get_border_bottom_width() == 0 && $style->length_in_pt($style->padding_bottom) == 0) {
             $l = $this->_frame->get_last_child();
-            if ( $l && !$l->is_block() && !$l->is_table() ) {
-                while ( $l = $l->get_prev_sibling() ) {
-                    if ( $l->is_block() || $l->is_table() ) {
+            if ($l && !$l->is_block() && !$l->is_table()) {
+                while ($l = $l->get_prev_sibling()) {
+                    if ($l->is_block() || $l->is_table()) {
                         break;
                     }
 
-                    if ( !$l->get_last_child() ) {
+                    if (!$l->get_last_child()) {
                         $l = null;
                         break;
                     }
@@ -158,7 +158,7 @@ 
             if ($l) {
                 $l_style = $l->get_style();
                 $b = max($b, (float)$l_style->length_in_pt($l_style->margin_bottom, $cb["h"]));
-                $style->margin_bottom = $b."pt";
+                $style->margin_bottom = $b . "pt";
                 $l_style->margin_bottom = "0pt";
             }
         }
@@ -283,7 +283,7 @@ 
 
         // Convert escaped hex characters into ascii characters (e.g. \A => newline)
         $string = preg_replace_callback("/\\\\([0-9a-fA-F]{0,6})/",
-            function ($matches) { return \Dompdf\Helpers::unichr(hexdec($matches[1])); },
+            function($matches) { return \Dompdf\Helpers::unichr(hexdec($matches[1])); },
             $string);
         return $string;
     }

src/Image/Cache.php

Doc Comments +4 added lines

@@ -168,6 +168,10 @@
         self::$_cache = array();
     }
 
+    /**
+     * @param string $file
+     * @param resource $context
+     */
     static function detect_type($file, $context = null)
     {
         list(, , $type) = Helpers::dompdf_getimagesize($file, $context);

Spacing +1 added lines, -1 removed lines

@@ -170,7 +170,7 @@
 
     static function detect_type($file, $context = null)
     {
-        list(, , $type) = Helpers::dompdf_getimagesize($file, $context);
+        list(,, $type) = Helpers::dompdf_getimagesize($file, $context);
 
         return $type;
     }