dompdf / dompdf

src/Frame/Factory.php

Indentation +5 added lines, -5 removed lines

@@ -31,11 +31,11 @@
 class Factory
 {
 
-     /**
-     * Array of positioners for specific frame types
-     *
-     * @var AbstractPositioner[]
-     */
+        /**
+         * Array of positioners for specific frame types
+         *
+         * @var AbstractPositioner[]
+         */
     protected static $_positioners;
     
     /**

Spacing +1 added lines, -1 removed lines

@@ -276,7 +276,7 @@
     protected static function getPositionerInstance($type)
     {
         if (!isset(self::$_positioners[$type])) {
-            $class = '\\Dompdf\\Positioner\\'.$type;
+            $class = '\\Dompdf\\Positioner\\' . $type;
             self::$_positioners[$type] = new $class();
         }
         return self::$_positioners[$type];

lib/Cpdf.php

Braces +6 added lines, -2 removed lines

@@ -3810,8 +3810,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;
+        }
 
         $debug = !$options['compress'];
         $tmp = ltrim($this->output($debug));

Indentation +22 added lines, -22 removed lines

@@ -2409,8 +2409,8 @@ 
 
             case 'font':
                 $this->objects[$id]['fonts'][$options['fontNum']] = [
-                  'objNum' => $options['objNum'],
-                  'fontNum' => $options['fontNum']
+                    'objNum' => $options['objNum'],
+                    'fontNum' => $options['fontNum']
                 ];
                 break;
 
@@ -2504,8 +2504,8 @@ 
 
             case 'font':
                 $this->objects[$id]['fonts'][$options['fontNum']] = [
-                  'objNum' => $options['objNum'],
-                  'fontNum' => $options['fontNum']
+                    'objNum' => $options['objNum'],
+                    'fontNum' => $options['fontNum']
                 ];
                 break;
 
@@ -2732,7 +2732,7 @@ 
                         case 'ContactInfo':
                             if ($v !== null && $v !== '') {
                                 $res .= "/$k (" .
-                                  ($encrypted ? $this->filterText($this->ARC4($v), false, false) : $v) . ") \n";
+                                    ($encrypted ? $this->filterText($this->ARC4($v), false, false) : $v) . ") \n";
                             }
                             break;
                     }
@@ -4299,13 +4299,13 @@ 
     function addSignature($signcert, $privkey, $password = '', $name = null, $location = null, $reason = null, $contactinfo = null) {
         $sigId = ++$this->numObj;
         $this->o_sig($sigId, 'new', [
-          'SignCert' => $signcert,
-          'PrivKey' => $privkey,
-          'Password' => $password,
-          'Name' => $name,
-          'Location' => $location,
-          'Reason' => $reason,
-          'ContactInfo' => $contactinfo
+            'SignCert' => $signcert,
+            'PrivKey' => $privkey,
+            'Password' => $password,
+            'Name' => $name,
+            'Location' => $location,
+            'Reason' => $reason,
+            'ContactInfo' => $contactinfo
         ]);
 
         return $sigId;
@@ -4340,17 +4340,17 @@ 
         );
 
         $this->o_acroform($this->acroFormId, 'font',
-          ['objNum' => $font[0]['objNum'], 'fontNum' => $font[0]['fontNum']]);
+            ['objNum' => $font[0]['objNum'], 'fontNum' => $font[0]['fontNum']]);
 
         $fieldId = ++$this->numObj;
         $this->o_field($fieldId, 'new', [
-          'rect' => [$x0, $y0, $x1, $y1],
-          'F' => 4,
-          'FT' => "/$type",
-          'T' => $name,
-          'Ff' => $ff,
-          'pageid' => $this->currentPage,
-          'da' => "$color /F$this->currentFontNum " . sprintf('%.1F Tf ', $size)
+            'rect' => [$x0, $y0, $x1, $y1],
+            'F' => 4,
+            'FT' => "/$type",
+            'T' => $name,
+            'Ff' => $ff,
+            'pageid' => $this->currentPage,
+            'da' => "$color /F$this->currentFontNum " . sprintf('%.1F Tf ', $size)
         ]);
 
         return $fieldId;
@@ -4422,8 +4422,8 @@ 
     {
         $this->acroFormId = ++$this->numObj;
         $this->o_acroform($this->acroFormId, 'new', [
-          'NeedAppearances' => $needAppearances ? 'true' : 'false',
-          'SigFlags' => $sigFlags
+            'NeedAppearances' => $needAppearances ? 'true' : 'false',
+            'SigFlags' => $sigFlags
         ]);
     }
 

Spacing +21 added lines, -21 removed lines

@@ -27,24 +27,24 @@ 
     const PDF_VERSION = '1.7';
 
     const ACROFORM_SIG_SIGNATURESEXISTS = 0x0001;
-    const ACROFORM_SIG_APPENDONLY =       0x0002;
+    const ACROFORM_SIG_APPENDONLY = 0x0002;
 
-    const ACROFORM_FIELD_BUTTON =   'Btn';
-    const ACROFORM_FIELD_TEXT =     'Tx';
-    const ACROFORM_FIELD_CHOICE =   'Ch';
-    const ACROFORM_FIELD_SIG =      'Sig';
+    const ACROFORM_FIELD_BUTTON = 'Btn';
+    const ACROFORM_FIELD_TEXT = 'Tx';
+    const ACROFORM_FIELD_CHOICE = 'Ch';
+    const ACROFORM_FIELD_SIG = 'Sig';
 
-    const ACROFORM_FIELD_READONLY =               0x0001;
-    const ACROFORM_FIELD_REQUIRED =               0x0002;
+    const ACROFORM_FIELD_READONLY = 0x0001;
+    const ACROFORM_FIELD_REQUIRED = 0x0002;
 
-    const ACROFORM_FIELD_TEXT_MULTILINE =         0x1000;
-    const ACROFORM_FIELD_TEXT_PASSWORD =          0x2000;
-    const ACROFORM_FIELD_TEXT_RICHTEXT =         0x10000;
+    const ACROFORM_FIELD_TEXT_MULTILINE = 0x1000;
+    const ACROFORM_FIELD_TEXT_PASSWORD = 0x2000;
+    const ACROFORM_FIELD_TEXT_RICHTEXT = 0x10000;
 
-    const ACROFORM_FIELD_CHOICE_COMBO =          0x20000;
-    const ACROFORM_FIELD_CHOICE_EDIT =           0x40000;
-    const ACROFORM_FIELD_CHOICE_SORT =           0x80000;
-    const ACROFORM_FIELD_CHOICE_MULTISELECT =   0x200000;
+    const ACROFORM_FIELD_CHOICE_COMBO = 0x20000;
+    const ACROFORM_FIELD_CHOICE_EDIT = 0x40000;
+    const ACROFORM_FIELD_CHOICE_SORT = 0x80000;
+    const ACROFORM_FIELD_CHOICE_MULTISELECT = 0x200000;
 
     const XOBJECT_SUBTYPE_FORM = 'Form';
 
@@ -407,7 +407,7 @@ 
     function __construct($pageSize = [0, 0, 612, 792], $isUnicode = false, $fontcache = '', $tmp = '')
     {
         $this->isUnicode = $isUnicode;
-        $this->fontcache = rtrim($fontcache, DIRECTORY_SEPARATOR."/\\");
+        $this->fontcache = rtrim($fontcache, DIRECTORY_SEPARATOR . "/\\");
         $this->tmp = ($tmp !== '' ? $tmp : sys_get_temp_dir());
         $this->newDocument($pageSize);
 
@@ -583,7 +583,7 @@ 
                     if (is_string($v)) {
                         $v = '/' . $v;
                     } elseif (is_int($v)) {
-                        $v = (string) $v;
+                        $v = (string)$v;
                     } elseif (is_bool($v)) {
                         $v = ($v ? 'true' : 'false');
                     } elseif (is_array($v)) {
@@ -2196,7 +2196,7 @@ 
                 // make the new object
                 $this->objects[$id] = ['t' => 'image', 'data' => &$options['data'], 'info' => []];
 
-                $info =& $this->objects[$id]['info'];
+                $info = & $this->objects[$id]['info'];
 
                 $info['Type'] = '/XObject';
                 $info['Subtype'] = '/Image';
@@ -2656,7 +2656,7 @@ 
 
             case 'byterange':
                 $o = &$this->objects[$id];
-                $content =& $options['content'];
+                $content = & $options['content'];
                 $content_len = strlen($content);
                 $pos = strpos($content, sprintf("/ByteRange [ %'.010d", $id));
                 $len = strlen('/ByteRange [ ********** ********** ********** ********** ]');
@@ -2710,7 +2710,7 @@ 
                     $this->encryptInit($id);
                 }
 
-                $res .= "/ByteRange " .sprintf("[ %'.010d ********** ********** ********** ]\n", $id);
+                $res .= "/ByteRange " . sprintf("[ %'.010d ********** ********** ********** ]\n", $id);
                 $res .= "/Contents <" . str_pad('', $sign_maxlen, '0') . ">\n";
                 $res .= "/Filter/Adobe.PPKLite\n"; //PPKMS \n";
                 $res .= "/Type/Sig/SubFilter/adbe.pkcs7.detached \n";
@@ -4336,7 +4336,7 @@ 
         $currentFontNum = $this->currentFontNum;
         $font = array_filter(
             $this->objects[$this->currentNode]['info']['fonts'],
-            function ($item) use ($currentFontNum) { return $item['fontNum'] == $currentFontNum; }
+            function($item) use ($currentFontNum) { return $item['fontNum'] == $currentFontNum; }
         );
 
         $this->o_acroform($this->acroFormId, 'font',
@@ -5508,7 +5508,7 @@ 
         // else assume that they are both scalar, anything else will probably error
         if (is_array($label)) {
             foreach ($label as $l => $v) {
-                $this->o_info($this->infoObject, $l, (string) $v);
+                $this->o_info($this->infoObject, $l, (string)$v);
             }
         } else {
             $this->o_info($this->infoObject, $label, $value);

src/Frame/FrameTree.php

Spacing +1 added lines, -1 removed lines

@@ -210,7 +210,7 @@
         if (isset($previousChild, $nextChild)) {
             if ($previousChild->nodeName === "#text" && $nextChild->nodeName === "#text") {
                 $previousChild->nodeValue .= $nextChild->nodeValue;
-                $this->_remove_node($node, $children, $index+1);
+                $this->_remove_node($node, $children, $index + 1);
             }
         }
         array_splice($children, $index, 1);

src/CanvasFactory.php

Braces +1 added lines, -3 removed lines

@@ -43,9 +43,7 @@
                 class_exists("PDFLib", false)
             ) {
                 $class = "Dompdf\\Adapter\\PDFLib";
-            }
-
-            else {
+            } else {
                 if ($backend === "gd" && extension_loaded('gd')) {
                     $class = "Dompdf\\Adapter\\GD";
                 } else {

src/Frame.php

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 = [
             "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/Adapter/GD.php

Braces +12 added lines, -4 removed lines

@@ -999,8 +999,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) {
@@ -1051,8 +1055,12 @@ 
      */
     protected function _output($options = [])
     {
-        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])) {

Spacing +3 added lines, -3 removed lines

@@ -875,7 +875,7 @@ 
         $text = mb_encode_numericentity($text, [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;
@@ -891,7 +891,7 @@ 
             $font = "";
         }
 
-        if ( stripos($font, ".ttf") === false ) {
+        if (stripos($font, ".ttf") === false) {
             $font .= ".ttf";
         }
 
@@ -936,7 +936,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;
     }
 

src/Css/Stylesheet.php

Spacing +4 added lines, -4 removed lines

@@ -665,8 +665,8 @@ 
                         case "nth-of-type":
                             //FIXME: this fix-up is pretty ugly, would parsing the selector in reverse work better generally?
                             $descendant_delimeter = strrpos($query, "::");
-                            $isChild = substr($query, $descendant_delimeter-5, 5) == "child";
-                            $el = substr($query, $descendant_delimeter+2);
+                            $isChild = substr($query, $descendant_delimeter - 5, 5) == "child";
+                            $el = substr($query, $descendant_delimeter + 2);
                             $query = substr($query, 0, strrpos($query, "/")) . ($isChild ? "/" : "//") . $el;
 
                             $pseudo_classes[$tok] = true;
@@ -696,8 +696,8 @@ 
                         case "nth-child":
                             //FIXME: this fix-up is pretty ugly, would parsing the selector in reverse work better generally?
                             $descendant_delimeter = strrpos($query, "::");
-                            $isChild = substr($query, $descendant_delimeter-5, 5) == "child";
-                            $el = substr($query, $descendant_delimeter+2);
+                            $isChild = substr($query, $descendant_delimeter - 5, 5) == "child";
+                            $el = substr($query, $descendant_delimeter + 2);
                             $query = substr($query, 0, strrpos($query, "/")) . ($isChild ? "/" : "//") . "*";
 
                             $pseudo_classes[$tok] = true;

Braces +27 added lines, -9 removed lines

@@ -1591,7 +1591,9 @@ 
             }
             $prop = trim($prop);
             */
-            if ($DEBUGCSS) print '(';
+            if ($DEBUGCSS) {
+                print '(';
+            }
 
             $important = false;
             $prop = trim($prop);
@@ -1606,24 +1608,32 @@ 
             }
 
             if ($prop === "") {
-                if ($DEBUGCSS) print 'empty)';
+                if ($DEBUGCSS) {
+                    print 'empty)';
+                }
                 continue;
             }
 
             $i = mb_strpos($prop, ":");
             if ($i === false) {
-                if ($DEBUGCSS) print 'novalue' . $prop . ')';
+                if ($DEBUGCSS) {
+                    print 'novalue' . $prop . ')';
+                }
                 continue;
             }
 
             $prop_name = rtrim(mb_strtolower(mb_substr($prop, 0, $i)));
             $value = ltrim(mb_substr($prop, $i + 1));
 
-            if ($DEBUGCSS) print $prop_name . ':=' . $value . ($important ? '!IMPORTANT' : '') . ')';
+            if ($DEBUGCSS) {
+                print $prop_name . ':=' . $value . ($important ? '!IMPORTANT' : '') . ')';
+            }
 
             $style->set_prop($prop_name, $value, $important, false);
         }
-        if ($DEBUGCSS) print '_parse_properties]';
+        if ($DEBUGCSS) {
+            print '_parse_properties]';
+        }
 
         return $style;
     }
@@ -1643,12 +1653,16 @@ 
         $DEBUGCSS = $this->_dompdf->getOptions()->getDebugCss();
 
         $sections = explode("}", $str);
-        if ($DEBUGCSS) print '[_parse_sections';
+        if ($DEBUGCSS) {
+            print '[_parse_sections';
+        }
         foreach ($sections as $sect) {
             $i = mb_strpos($sect, "{");
             if ($i === false) { continue; }
 
-            if ($DEBUGCSS) print '[section';
+            if ($DEBUGCSS) {
+                print '[section';
+            }
 
             $selector_str = preg_replace($patterns, $replacements, mb_substr($sect, 0, $i));
             $selectors = preg_split("/,(?![^\(]*\))/", $selector_str, 0, PREG_SPLIT_NO_EMPTY);
@@ -1659,10 +1673,14 @@ 
                 $selector = trim($selector);
 
                 if ($selector == "") {
-                    if ($DEBUGCSS) print '#empty#';
+                    if ($DEBUGCSS) {
+                        print '#empty#';
+                    }
                     continue;
                 }
-                if ($DEBUGCSS) print '#' . $selector . '#';
+                if ($DEBUGCSS) {
+                    print '#' . $selector . '#';
+                }
                 //if ($DEBUGCSS) { if (strpos($selector,'p') !== false) print '!!!p!!!#'; }
 
                 //FIXME: tag the selector with a hash of the media query to separate it from non-conditional styles (?), xpath comments are probably not what we want to do here

src/Positioner/Absolute.php

Spacing +2 added lines, -2 removed lines

@@ -28,8 +28,8 @@
 
             // If the `top` value is `auto`, the frame will be repositioned
             // after its height has been resolved
-            $left = (float) $style->length_in_pt($style->left, $cbw);
-            $top = (float) $style->length_in_pt($style->top, $cbh);
+            $left = (float)$style->length_in_pt($style->left, $cbw);
+            $top = (float)$style->length_in_pt($style->top, $cbh);
 
             $frame->set_position($cbx + $left, $cby + $top);
         } else {

src/FrameDecorator/Text.php

Spacing +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
     {
         $style = $this->_frame->get_style();
         $pb = $this->_frame->get_padding_box();
-        $pb[3] = $pb["h"] = (float) $style->length_in_pt($style->height);
+        $pb[3] = $pb["h"] = (float)$style->length_in_pt($style->height);
         return $pb;
     }