twhiston / simplexml_debug

src/SxmlDebug.php

Spacing +35 added lines, -36 removed lines

@@ -39,7 +39,7 @@ 
 
     $dump = '';
     // Note that the header is added at the end, so we can add stats
-    $dump .= '[' . PHP_EOL;
+    $dump .= '['.PHP_EOL;
 
     // SimpleXML objects can be either a single node, or (more commonly) a list of 0 or more nodes
     // I haven't found a reliable way of distinguishing between the two cases
@@ -61,10 +61,10 @@ 
         $dump .= self::dumpAddElement($item);
       }
     }
-    $dump .= ']' . PHP_EOL;
+    $dump .= ']'.PHP_EOL;
 
     // Add on the header line, with the total number of items output
-    return self::getHeaderLine($item_index) . $dump;
+    return self::getHeaderLine($item_index).$dump;
   }
 
   /**
@@ -77,15 +77,15 @@ 
     // To what namespace does this attribute belong? Returns array( alias => URI )
     $ns = $item->getNamespaces(FALSE);
     if ($ns) {
-      $dump .= self::INDENT . self::INDENT . 'Namespace: \'' . reset($ns) .
-               '\'' .
+      $dump .= self::INDENT.self::INDENT.'Namespace: \''.reset($ns).
+               '\''.
                PHP_EOL;
       if (key($ns) == '') {
-        $dump .= self::INDENT . self::INDENT . '(Default Namespace)' . PHP_EOL;
+        $dump .= self::INDENT.self::INDENT.'(Default Namespace)'.PHP_EOL;
       } else {
-        $dump .= self::INDENT . self::INDENT . 'Namespace Alias: \'' .
-                 key($ns) .
-                 '\'' .
+        $dump .= self::INDENT.self::INDENT.'Namespace Alias: \''.
+                 key($ns).
+                 '\''.
                  PHP_EOL;
       }
     }
@@ -104,9 +104,9 @@ 
                                       $data,
                                       $indent = 1,
                                       $backtick = TRUE): string {
-    return str_repeat(self::INDENT, $indent) . $title . ': ' .
-           ($backtick ? '\'' : '') . $data .
-           ($backtick ? '\'' : '') . PHP_EOL;
+    return str_repeat(self::INDENT, $indent).$title.': '.
+           ($backtick ? '\'' : '').$data.
+           ($backtick ? '\'' : '').PHP_EOL;
   }
 
   /**
@@ -115,14 +115,14 @@ 
    */
   private static function dumpAddAttribute(\SimpleXMLElement $item): string {
 
-    $dump = self::INDENT . 'Attribute {' . PHP_EOL;
+    $dump = self::INDENT.'Attribute {'.PHP_EOL;
 
     $dump .= self::dumpAddNamespace($item);
 
     $dump .= self::dumpGetLine('Name', $item->getName(), 2);
     $dump .= self::dumpGetLine('Value', (string) $item, 2);
 
-    $dump .= self::INDENT . '}' . PHP_EOL;
+    $dump .= self::INDENT.'}'.PHP_EOL;
     return $dump;
 
   }
@@ -133,7 +133,7 @@ 
    */
   private static function dumpAddElement(\SimpleXMLElement $item): string {
 
-    $dump = self::INDENT . 'Element {' . PHP_EOL;
+    $dump = self::INDENT.'Element {'.PHP_EOL;
 
     $dump .= self::dumpAddNamespace($item);
 
@@ -176,10 +176,9 @@ 
         continue;
       }
 
-      $ns_label = (($ns_alias === '') ? 'Default Namespace' :
-        "Namespace $ns_alias");
+      $ns_label = (($ns_alias === '') ? 'Default Namespace' : "Namespace $ns_alias");
 
-      $dump .= self::INDENT . self::INDENT . 'Content in ' . $ns_label .
+      $dump .= self::INDENT.self::INDENT.'Content in '.$ns_label.
                PHP_EOL;
 
       if (NULL !== $ns_uri) {
@@ -199,7 +198,7 @@ 
                                  FALSE);
     }
 
-    return $dump . self::INDENT . '}' . PHP_EOL;
+    return $dump.self::INDENT.'}'.PHP_EOL;
   }
 
   /**
@@ -228,7 +227,7 @@ 
     $childrenString = count($children);
     // Don't output a trailing " - " if there are no children
     if (count($children) > 0) {
-      $childrenString .= ' - ' . implode(', ', $child_name_output);
+      $childrenString .= ' - '.implode(', ', $child_name_output);
     }
     return $childrenString;
   }
@@ -241,14 +240,14 @@ 
 // Attributes can't be duplicated, but I'm going to put them in alphabetical order
     $attribute_names = [];
     foreach ($attributes as $sx_attribute) {
-      $attribute_names[] = "'" . $sx_attribute->getName() . "'";
+      $attribute_names[] = "'".$sx_attribute->getName()."'";
     }
     ksort($attribute_names);
 
     $attString = count($attributes);
     // Don't output a trailing " - " if there are no attributes
     if (count($attributes) > 0) {
-      $attString .= ' - ' . implode(', ', $attribute_names);
+      $attString .= ' - '.implode(', ', $attribute_names);
     }
     return $attString;
   }
@@ -259,8 +258,8 @@ 
    */
   private static function getHeaderLine($index): string {
 
-    return 'SimpleXML object (' . $index . ' item' .
-           ($index > 1 ? 's' : '') . ')' . PHP_EOL;
+    return 'SimpleXML object ('.$index.' item'.
+           ($index > 1 ? 's' : '').')'.PHP_EOL;
   }
 
   /**
@@ -304,9 +303,9 @@ 
         // To what namespace does this attribute belong? Returns array( alias => URI )
         $ns = $root_item->getNamespaces(FALSE);
         if (key($ns)) {
-          $dump .= key($ns) . ':';
+          $dump .= key($ns).':';
         }
-        $dump .= $root_item->getName() . '="' . (string) $root_item . '"' .
+        $dump .= $root_item->getName().'="'.(string) $root_item.'"'.
                  PHP_EOL;
       } else {
         // Display the root node as a numeric key reference, plus a hint as to its tag name
@@ -315,11 +314,11 @@ 
         // To what namespace does this attribute belong? Returns array( alias => URI )
         $ns = $root_item->getNamespaces(FALSE);
         if (key($ns)) {
-          $root_node_name = key($ns) . ':' . $root_item->getName();
+          $root_node_name = key($ns).':'.$root_item->getName();
         } else {
           $root_node_name = $root_item->getName();
         }
-        $dump .= "[$root_item_index] // <$root_node_name>" . PHP_EOL;
+        $dump .= "[$root_item_index] // <$root_node_name>".PHP_EOL;
 
         // This function is effectively recursing depth-first through the tree,
         // but this is managed manually using a stack rather than actual recursion
@@ -335,7 +334,7 @@ 
     }
 
     // Add on the header line, with the total number of items output
-    $dump = self::getHeaderLine($root_item_index) . $dump;
+    $dump = self::getHeaderLine($root_item_index).$dump;
 
     return $dump;
 
@@ -358,7 +357,7 @@ 
       str_repeat(SxmlDebug::INDENT, $depth)
       . '(string) '
       . "'$string_extract'"
-      . ' (' . strlen($stringContent) . ' chars)'
+      . ' ('.strlen($stringContent).' chars)'
       . PHP_EOL : '';
 
   }
@@ -405,7 +404,7 @@ 
     if (count($attributes) > 0) {
       if (!$isCurrentNamespace) {
         $dump .= str_repeat(self::INDENT, $depth)
-                 . "->attributes('$nsAlias', true)" . PHP_EOL;
+                 . "->attributes('$nsAlias', true)".PHP_EOL;
       }
 
       foreach ($attributes as $sx_attribute) {
@@ -414,14 +413,14 @@ 
           // In current namespace
           // e.g. ['attribName']
           $dump .= str_repeat(self::INDENT, $depth)
-                   . "['" . $sx_attribute->getName() . "']"
+                   . "['".$sx_attribute->getName()."']"
                    . PHP_EOL;
           $string_display_depth = $depth + 1;
         } else {
           // After a call to ->attributes()
           // e.g. ->attribName
           $dump .= str_repeat(self::INDENT, $depth + 1)
-                   . '->' . $sx_attribute->getName()
+                   . '->'.$sx_attribute->getName()
                    . PHP_EOL;
           $string_display_depth = $depth + 2;
         }
@@ -456,7 +455,7 @@ 
         $display_depth = $depth;
       } else {
         $dump .= str_repeat(self::INDENT, $depth)
-                 . "->children('$nsAlias', true)" . PHP_EOL;
+                 . "->children('$nsAlias', true)".PHP_EOL;
         $display_depth = $depth + 1;
       }
 
@@ -474,8 +473,8 @@ 
 
         // e.g. ->Foo[0]
         $dump .= str_repeat(self::INDENT, $display_depth)
-                 . '->' . $sx_child->getName()
-                 . '[' . ($child_names[$child_node_name] - 1) . ']'
+                 . '->'.$sx_child->getName()
+                 . '['.($child_names[$child_node_name] - 1).']'
                  . PHP_EOL;
 
         $dump .= self::recursivelyProcessNode(