servo-php / fluidxml

source/FluidXml.php

Indentation +12 added lines, -14 removed lines

@@ -133,8 +133,8 @@ 
 class FluidXml implements FluidInterface
 {
         use NewableTrait,
-            ReservedCallTrait,          // For compatibility with PHP 5.6.
-            ReservedCallStaticTrait;    // For compatibility with PHP 5.6.
+                ReservedCallTrait,          // For compatibility with PHP 5.6.
+                ReservedCallStaticTrait;    // For compatibility with PHP 5.6.
 
         const ROOT_NODE = 'doc';
 
@@ -168,9 +168,9 @@ 
                 $doc = $this->document;
 
                 $defaults = [ 'root'       => self::ROOT_NODE,
-                              'version'    => '1.0',
-                              'encoding'   => 'UTF-8',
-                              'stylesheet' => null ];
+                                'version'    => '1.0',
+                                'encoding'   => 'UTF-8',
+                                'stylesheet' => null ];
 
                 if (\is_string($root)) {
                         // The root option can be specified as first argument
@@ -196,9 +196,9 @@ 
 
                 if (! empty($opts['stylesheet'])) {
                         $attrs = 'type="text/xsl" '
-                               . "encoding=\"{$opts['encoding']}\" "
-                               . 'indent="yes" '
-                               . "href=\"{$opts['stylesheet']}\"";
+                                . "encoding=\"{$opts['encoding']}\" "
+                                . 'indent="yes" '
+                                . "href=\"{$opts['stylesheet']}\"";
                         $stylesheet = new \DOMProcessingInstruction('xml-stylesheet', $attrs);
 
                         $doc->dom->insertBefore($stylesheet, $doc->dom->documentElement);
@@ -447,8 +447,8 @@ 
         const MODE_EXPLICIT = 1;
 
         private $config = [ self::ID   => '',
-                            self::URI  => '',
-                            self::MODE => self::MODE_EXPLICIT ];
+                                self::URI  => '',
+                                self::MODE => self::MODE_EXPLICIT ];
 
         public function __construct($id, $uri, $mode = 1)
         {
@@ -529,7 +529,6 @@ 
          *
          * ```php
          * $xml = fluidxml();
-
          * $xml->query("/doc/book[@id='123']");
          *
          * // Relative queries are valid.
@@ -549,7 +548,6 @@ 
          *
          * ```php
          * $xml = fluidxml();
-
          * $xml->appendChild('title', 'The Theory Of Everything');
          * $xml->appendChild([ 'author' => 'S. Hawking' ]);
          *
@@ -664,8 +662,8 @@ 
 class FluidContext implements FluidInterface, \ArrayAccess, \Iterator
 {
         use NewableTrait,
-            ReservedCallTrait,          // For compatibility with PHP 5.6.
-            ReservedCallStaticTrait;    // For compatibility with PHP 5.6.
+                ReservedCallTrait,          // For compatibility with PHP 5.6.
+                ReservedCallStaticTrait;    // For compatibility with PHP 5.6.
 
         private $document;
         private $nodes = [];

Spacing +36 added lines, -36 removed lines

@@ -137,8 +137,8 @@ 
 class FluidXml implements FluidInterface
 {
         use NewableTrait,
-            ReservedCallTrait,          // For compatibility with PHP 5.6.
-            ReservedCallStaticTrait;    // For compatibility with PHP 5.6.
+            ReservedCallTrait, // For compatibility with PHP 5.6.
+            ReservedCallStaticTrait; // For compatibility with PHP 5.6.
 
         const ROOT_NODE = 'doc';
 
@@ -147,7 +147,7 @@ 
 
         public static function load($document)
         {
-                if (\is_string($document) && ! is_an_xml_string($document)) {
+                if (\is_string($document) && !is_an_xml_string($document)) {
                         // Removes any empty new line at the beginning,
                         // otherwise the first character check fails.
 
@@ -159,7 +159,7 @@ 
                                 $document = \file_get_contents($file);
                         }
 
-                        if (! $is_file || ! $is_readable || ! $document) {
+                        if (!$is_file || !$is_readable || !$document) {
                                 throw new \Exception("File '$file' not accessible.");
                         }
                 }
@@ -169,10 +169,10 @@ 
 
         public function __construct($root = null, $options = [])
         {
-                $defaults = [ 'root'       => self::ROOT_NODE,
+                $defaults = ['root'       => self::ROOT_NODE,
                               'version'    => '1.0',
                               'encoding'   => 'UTF-8',
-                              'stylesheet' => null ];
+                              'stylesheet' => null];
 
                 if (\is_string($root)) {
                         // The root option can be specified as first argument
@@ -197,11 +197,11 @@ 
 
                 $this->handler = new FluidInsertionHandler($doc);
 
-                if (! empty($opts['root'])) {
+                if (!empty($opts['root'])) {
                         $this->appendSibling($opts['root']);
                 }
 
-                if (! empty($opts['stylesheet'])) {
+                if (!empty($opts['stylesheet'])) {
                         $attrs = 'type="text/xsl" '
                                . "encoding=\"{$opts['encoding']}\" "
                                . 'indent="yes" '
@@ -239,7 +239,7 @@ 
                 $namespaces = [];
 
                 if (\is_string($arguments[0])) {
-                        $args = [ $arguments[0], $arguments[1] ];
+                        $args = [$arguments[0], $arguments[1]];
 
                         if (isset($arguments[2])) {
                                 $args[] = $arguments[2];
@@ -446,16 +446,16 @@ 
 
 class FluidNamespace
 {
-        const ID   = 'id'  ;
-        const URI  = 'uri' ;
+        const ID   = 'id';
+        const URI  = 'uri';
         const MODE = 'mode';
 
         const MODE_IMPLICIT = 0;
         const MODE_EXPLICIT = 1;
 
-        private $config = [ self::ID   => '',
+        private $config = [self::ID   => '',
                             self::URI  => '',
-                            self::MODE => self::MODE_EXPLICIT ];
+                            self::MODE => self::MODE_EXPLICIT];
 
         public function __construct($id, $uri, $mode = 1)
         {
@@ -493,7 +493,7 @@ 
         {
                 $id = $this->id();
 
-                if (! empty($id)) {
+                if (!empty($id)) {
                         $id .= ':';
                 }
 
@@ -681,7 +681,7 @@ 
         {
                 $this->document   = $document;
                 $this->dom        = $document->dom;
-                $this->namespaces =& $document->namespaces;
+                $this->namespaces = & $document->namespaces;
         }
 
         public function insertElement(&$nodes, $element, &$optionals, $fn, $orig_context)
@@ -704,7 +704,7 @@ 
                 // offers to the user and is the same of:
                 // 1. appending a child switching the context
                 // 2. setting the attributes over the new context.
-                if (! empty($attributes)) {
+                if (!empty($attributes)) {
                         $new_context->setAttribute($attributes);
                 }
 
@@ -722,8 +722,8 @@ 
 
         protected function handleOptionals($element, &$optionals)
         {
-                if (! \is_array($element)) {
-                        $element = [ $element ];
+                if (!\is_array($element)) {
+                        $element = [$element];
                 }
 
                 $switch_context = false;
@@ -746,7 +746,7 @@ 
                         }
                 }
 
-                return [ $element, $attributes, $switch_context ];
+                return [$element, $attributes, $switch_context];
         }
 
 
@@ -767,9 +767,9 @@ 
                 $v_is_string    = \is_string($v);
                 $v_is_xml       = $v_is_string && is_an_xml_string($v);
                 $k_is_special   = $k_is_string && $k[0] === '@';
-                $k_isnt_special = ! $k_is_special;
-                $v_isnt_string  = ! $v_is_string;
-                $v_isnt_xml     = ! $v_is_xml;
+                $k_isnt_special = !$k_is_special;
+                $v_isnt_string  = !$v_is_string;
+                $v_isnt_xml     = !$v_is_xml;
                 ///////////////////////////////////////////////////////
 
                 if ($k_is_string && $k_isnt_special && $v_is_string && $v_isnt_xml) {
@@ -789,7 +789,7 @@ 
                 }
 
                 /////////////////////////////////////////////////////
-                $k_is_special_a = $k_is_special && ! $k_is_special_c;
+                $k_is_special_a = $k_is_special && !$k_is_special_c;
                 /////////////////////////////////////////////////////
 
                 if ($k_is_special_a && $v_is_string) {
@@ -841,7 +841,7 @@ 
                 $v_is_domnode = $v instanceof \DOMNode;
                 ///////////////////////////////////////
 
-                if ($k_is_integer && ! $v_is_domdoc && $v_is_domnode) {
+                if ($k_is_integer && !$v_is_domdoc && $v_is_domnode) {
                         return $this->insertIntegerDomnode($parent, $k, $v, $fn, $optionals);
                 }
 
@@ -909,8 +909,8 @@ 
 
         protected function attachNodes($parent, $nodes, $fn)
         {
-                if (! \is_array($nodes) && ! $nodes instanceof \Traversable) {
-                        $nodes = [ $nodes ];
+                if (!\is_array($nodes) && !$nodes instanceof \Traversable) {
+                        $nodes = [$nodes];
                 }
 
                 $context = [];
@@ -959,7 +959,7 @@ 
                 $el = $this->createElement($k, $v);
                 $el = $fn($parent, $el);
 
-                return [ $el ];
+                return [$el];
         }
 
         protected function insertStringMixed($parent, $k, $v, $fn, &$optionals)
@@ -975,7 +975,7 @@ 
                 // they are supplied, so 'appendChild' is the perfect operation.
                 $this->newContext($el)->appendChild($v, ...$optionals);
 
-                return [ $el ];
+                return [$el];
         }
 
         protected function insertIntegerArray($parent, $k, $v, $fn, &$optionals)
@@ -1002,7 +1002,7 @@ 
                 $el = $this->createElement($v);
                 $el = $fn($parent, $el);
 
-                return [ $el ];
+                return [$el];
         }
 
         protected function insertIntegerXml($parent, $k, $v, $fn)
@@ -1074,8 +1074,8 @@ 
 class FluidContext implements FluidInterface, \ArrayAccess, \Iterator
 {
         use NewableTrait,
-            ReservedCallTrait,          // For compatibility with PHP 5.6.
-            ReservedCallStaticTrait;    // For compatibility with PHP 5.6.
+            ReservedCallTrait, // For compatibility with PHP 5.6.
+            ReservedCallStaticTrait; // For compatibility with PHP 5.6.
 
         private $document;
         private $handler;
@@ -1087,14 +1087,14 @@ 
                 $this->document = $document;
                 $this->handler  = $handler;
 
-                if (! \is_array($context) && ! $context instanceof \Traversable) {
+                if (!\is_array($context) && !$context instanceof \Traversable) {
                         // DOMDocument, DOMElement and DOMNode are not iterable.
                         // DOMNodeList and FluidContext are iterable.
-                        $context = [ $context ];
+                        $context = [$context];
                 }
 
                 foreach ($context as $n) {
-                        if (! $n instanceof \DOMNode) {
+                        if (!$n instanceof \DOMNode) {
                                 throw new \Exception('Node type not recognized.');
                         }
 
@@ -1328,7 +1328,7 @@ 
                 // the user has passed two arguments:
                 // 1. is the attribute name
                 // 2. is the attribute value
-                if (! \is_array($arguments[0])) {
+                if (!\is_array($arguments[0])) {
                         $attrs = [$arguments[0] => $arguments[1]];
                 }
 
@@ -1463,7 +1463,7 @@ 
                                 }
                         }
 
-                        if (! $found) {
+                        if (!$found) {
                                 $set[] = $r;
                         }
                 }

Doc Comments +68 added lines, -5 removed lines

@@ -70,7 +70,7 @@ 
  *
  * @param array $arguments Options that influence the construction of the XML document.
  *
- * @return FluidXml A new FluidXml instance.
+ * @return FluidContext A new FluidXml instance.
  */
 function fluidify(...$arguments)
 {
@@ -87,6 +87,9 @@ 
         return new FluidNamespace(...$arguments);
 }
 
+/**
+ * @param string $string
+ */
 function is_an_xml_string($string)
 {
         // Removes any empty new line at the beginning,
@@ -543,7 +546,7 @@ 
          * $xml->query("/doc")->query("book[@id='123']");
          * ```
          *
-         * @param string $xpath The XPath to execute.
+         * @param string[] $xpath The XPath to execute.
          *
          * @return FluidContext The context associated to the DOMNodeList.
          */
@@ -565,9 +568,6 @@ 
          * ```
          *
          * @param string|array $child The child/children to add.
-         * @param string $value The child text content.
-         * @param bool $switchContext Whether to return the current context
-         *                            or the context of the created node.
          *
          * @return FluidContext The context associated to the DOMNodeList.
          */
@@ -582,6 +582,10 @@ 
         public function remove(...$xpath);
         public function xml($strip = false);
         // Aliases:
+
+        /**
+         * @return FluidContext
+         */
         public function add($child, ...$optionals);
         public function prepend($sibling, ...$optionals);
         public function insertSiblingBefore($sibling, ...$optionals);
@@ -645,6 +649,9 @@ 
         private $context;
         private $times;
 
+        /**
+         * @param FluidContext $context
+         */
         public function __construct($document, $handler, $context, $times)
         {
                 $this->document = $document;
@@ -677,6 +684,9 @@ 
         private $dom;
         private $namespaces;
 
+        /**
+         * @param FluidDocument $document
+         */
         public function __construct($document)
         {
                 $this->document   = $document;
@@ -923,6 +933,10 @@ 
                 return $context;
         }
 
+        /**
+         * @param string $k
+         * @param string $v
+         */
         protected function insertSpecialContent($parent, $k, $v)
         {
                 // The user has passed an element text content:
@@ -940,6 +954,10 @@ 
                 return [];
         }
 
+        /**
+         * @param string $k
+         * @param string $v
+         */
         protected function insertSpecialAttribute($parent, $k, $v)
         {
                 // The user has passed an attribute name and an attribute value:
@@ -951,6 +969,10 @@ 
                 return [];
         }
 
+        /**
+         * @param string $k
+         * @param string $v
+         */
         protected function insertStringString($parent, $k, $v, $fn)
         {
                 // The user has passed an element name and an element value:
@@ -962,6 +984,9 @@ 
                 return [ $el ];
         }
 
+        /**
+         * @param string $k
+         */
         protected function insertStringMixed($parent, $k, $v, $fn, &$optionals)
         {
                 // The user has passed one of these two cases:
@@ -978,6 +1003,9 @@ 
                 return [ $el ];
         }
 
+        /**
+         * @param integer $k
+         */
         protected function insertIntegerArray($parent, $k, $v, $fn, &$optionals)
         {
                 // The user has passed a wrapper array:
@@ -994,6 +1022,10 @@ 
                 return $context;
         }
 
+        /**
+         * @param integer $k
+         * @param string $v
+         */
         protected function insertIntegerString($parent, $k, $v, $fn)
         {
                 // The user has passed a node name without a node value:
@@ -1005,6 +1037,10 @@ 
                 return [ $el ];
         }
 
+        /**
+         * @param integer $k
+         * @param string $v
+         */
         protected function insertIntegerXml($parent, $k, $v, $fn)
         {
                 // The user has passed an XML document instance:
@@ -1034,6 +1070,10 @@ 
                 return $this->attachNodes($parent, $nodes, $fn);
         }
 
+        /**
+         * @param integer $k
+         * @param \DOMDocument $v
+         */
         protected function insertIntegerDomdocument($parent, $k, $v, $fn)
         {
                 // A DOMDocument can have multiple root nodes.
@@ -1045,26 +1085,46 @@ 
                 // return $this->attachNodes($parent, $v->documentElement, $fn);
         }
 
+        /**
+         * @param integer $k
+         * @param \DOMNodeList $v
+         */
         protected function insertIntegerDomnodelist($parent, $k, $v, $fn)
         {
                 return $this->attachNodes($parent, $v, $fn);
         }
 
+        /**
+         * @param integer $k
+         * @param \DOMNode $v
+         */
         protected function insertIntegerDomnode($parent, $k, $v, $fn)
         {
                 return $this->attachNodes($parent, $v, $fn);
         }
 
+        /**
+         * @param integer $k
+         * @param \SimpleXMLElement $v
+         */
         protected function insertIntegerSimplexml($parent, $k, $v, $fn)
         {
                 return $this->attachNodes($parent, \dom_import_simplexml($v), $fn);
         }
 
+        /**
+         * @param integer $k
+         * @param FluidXml $v
+         */
         protected function insertIntegerFluidxml($parent, $k, $v, $fn)
         {
                 return $this->attachNodes($parent, $v->dom()->documentElement, $fn);
         }
 
+        /**
+         * @param integer $k
+         * @param FluidContext $v
+         */
         protected function insertIntegerFluidcontext($parent, $k, $v, $fn)
         {
                 return $this->attachNodes($parent, $v->asArray(), $fn);
@@ -1450,6 +1510,9 @@ 
                 return $set;
         }
 
+        /**
+         * @param callable $fn
+         */
         protected function callfn($fn, $args)
         {
                 if ($fn instanceof \Closure) {