servo-php / fluidxml

specs/FluidXml.php

Spacing +37 added lines, -37 removed lines

@@ -24,10 +24,10 @@ 
                 $expected = $xml->xml();
                 \assert($actual === $expected, __($actual, $expected));
 
-                $options = [ 'root'       => 'root',
+                $options = ['root'       => 'root',
                              'version'    => '1.2',
                              'encoding'   => 'UTF-16',
-                             'stylesheet' => 'stylesheet.xsl' ];
+                             'stylesheet' => 'stylesheet.xsl'];
 
                 $xml   = new FluidXml($options);
                 $alias = fluidxml($options);
@@ -132,7 +132,7 @@ 
 describe('domnodes_to_string', function() {
         it('should convert an array of DOMNode instances to an XML string', function() {
                 $xml   = new FluidXml();
-                $nodes = [ $xml->dom()->documentElement ];
+                $nodes = [$xml->dom()->documentElement];
 
                 $actual   = domnodes_to_string($nodes);
                 $expected = "<doc/>";
@@ -237,7 +237,7 @@ 
                         try {
                                 $xml = FluidXml::load('.impossible.xml');
                         } catch (\Exception $e) {
-                                $actual   = $e;
+                                $actual = $e;
                         }
 
                         assert_is_a($actual, \Exception::class);
@@ -247,7 +247,7 @@ 
                         try {
                                 $xml = FluidXml::load(0);
                         } catch (\Exception $e) {
-                                $actual   = $e;
+                                $actual = $e;
                         }
 
                         assert_is_a($actual, \Exception::class);
@@ -257,19 +257,19 @@ 
         if (\version_compare(\phpversion(), '7', '>=')) {
         describe(':new', function() {
                 it('should behave like FluidXml::__construct', function() {
-                        $xml   = new FluidXml();
+                        $xml = new FluidXml();
                         eval('$alias = \FluidXml\FluidXml::new();');
 
                         $actual   = $alias->xml();
                         $expected = $xml->xml();
                         \assert($actual === $expected, __($actual, $expected));
 
-                        $options = [ 'root'       => 'root',
+                        $options = ['root'       => 'root',
                                      'version'    => '1.2',
                                      'encoding'   => 'UTF-16',
-                                     'stylesheet' => 'stylesheet.xsl' ];
+                                     'stylesheet' => 'stylesheet.xsl'];
 
-                        $xml   = new FluidXml($options);
+                        $xml = new FluidXml($options);
                         eval('$alias = \FluidXml\FluidXml::new($options);');
 
                         $actual   = $alias->xml();
@@ -425,7 +425,7 @@ 
                         $x_ns  = new FluidNamespace('x', 'x.com');
                         $xx_ns = fluidns('xx', 'xx.com', FluidNamespace::MODE_IMPLICIT);
 
-                        $nss = $xml->namespace([ $x_ns, $xx_ns ])
+                        $nss = $xml->namespace([$x_ns, $xx_ns])
                                    ->namespaces();
 
                         $actual   = $nss[$x_ns->id()];
@@ -467,7 +467,7 @@ 
                 it('should accept an array of queries', function() {
                         $xml = new FluidXml();
                         $xml->appendChild('html', true)
-                            ->appendChild(['head','body']);
+                            ->appendChild(['head', 'body']);
                         $xml->query(['//html', '//head', '//body'])
                             ->setAttribute('lang', 'en');
 
@@ -481,7 +481,7 @@ 
 
                         $xml = new FluidXml();
                         $xml->appendChild('html', true)
-                            ->appendChild(['head','body'])
+                            ->appendChild(['head', 'body'])
                             ->query(['.', 'head', 'body'])
                             ->setAttribute('lang', 'en');
 
@@ -497,7 +497,7 @@ 
                 it('should accept variable queries arguments', function() {
                         $xml = new FluidXml();
                         $xml->appendChild('html', true)
-                            ->appendChild(['head','body']);
+                            ->appendChild(['head', 'body']);
                         $xml->query('//html', '//head', '//body')
                             ->setAttribute('lang', 'en');
 
@@ -511,7 +511,7 @@ 
 
                         $xml = new FluidXml();
                         $xml->appendChild('html', true)
-                            ->appendChild(['head','body'])
+                            ->appendChild(['head', 'body'])
                             ->query('.', 'head', 'body')
                             ->setAttribute('lang', 'en');
 
@@ -528,7 +528,7 @@ 
                         // XPATH: //child subchild
                         $xml = new FluidXml();
                         $cx = $xml->appendChild('html', true);
-                        $cx->appendChild(['head','body']);
+                        $cx->appendChild(['head', 'body']);
                         $cx = $cx->query('body');
 
                         $actual   = $cx[0]->nodeName;
@@ -536,7 +536,7 @@ 
                         \assert($actual === $expected, __($actual, $expected));
 
                         $xml = new FluidXml();
-                        $xml->appendChild('html', true)->appendChild(['head','body']);
+                        $xml->appendChild('html', true)->appendChild(['head', 'body']);
                         $cx = $xml->query('/doc/html')->query('head');
 
                         $actual   = $cx[0]->nodeName;
@@ -548,7 +548,7 @@ 
                         // XPATH: //child/subchild //child
                         $xml = new FluidXml();
                         $xml->appendChild('html', true)
-                            ->appendChild(['head','body']);
+                            ->appendChild(['head', 'body']);
                         $cx = $xml->query('/doc/html/body')
                                   ->appendChild('h1')
                                   ->query('/doc/html/head');
@@ -562,7 +562,7 @@ 
                         // XPATH: //child/subchild ../..
                         $xml = new FluidXml();
                         $xml->appendChild('html', true)
-                            ->appendChild(['head','body'], true)
+                            ->appendChild(['head', 'body'], true)
                             ->query('../body')
                             ->appendChild('h1')
                             ->query('../..')
@@ -587,12 +587,12 @@ 
 
                         $xml->namespace($x_ns, $xx_ns);
 
-                        $xml->appendChild('x:a',  true)
-                            ->appendChild('x:b',  true)
+                        $xml->appendChild('x:a', true)
+                            ->appendChild('x:b', true)
                             ->appendChild('xx:c', true)
                             ->appendChild('xx:d', true)
-                            ->appendChild('e',    true)
-                            ->appendChild('x:f',  true)
+                            ->appendChild('e', true)
+                            ->appendChild('x:f', true)
                             ->appendChild('g');
 
                         $r = $xml->query('/doc/a');
@@ -659,7 +659,7 @@ 
 
         describe('.each', function() {
                 it('should be fluid', function() {
-                        assert_is_fluid('each', function(){});
+                        assert_is_fluid('each', function() {});
                 });
 
                 it('should iterate the nodes inside the context', function() {
@@ -887,9 +887,9 @@ 
                         assert_equal_xml($xml, $expected);
 
                         $xml = new FluidXml();
-                        $xml->appendChild([ 'child', ['child'], ['child' => 'value1'], ['child' => 'value2'] ])
+                        $xml->appendChild(['child', ['child'], ['child' => 'value1'], ['child' => 'value2']])
                              ->appendChild('parent', true)
-                             ->appendChild([ 'child', ['child'], ['child' => 'value3'], ['child' => 'value4'] ]);
+                             ->appendChild(['child', ['child'], ['child' => 'value3'], ['child' => 'value4']]);
 
                         $expected = "<doc>\n"
                                   . "  <child/>\n"
@@ -940,12 +940,12 @@ 
 
                 it('should add children with some attributes and text using the @ syntax', function() {
                         $xml = new FluidXml();
-                        $attrs = [ '@class' => 'Class attr',
+                        $attrs = ['@class' => 'Class attr',
                                    '@'      => 'Text content',
-                                   '@id'    => 'Id attr' ];
-                        $xml->appendChild(['child1' => $attrs ])
-                            ->appendChild(['child2' => $attrs ], true)
-                                ->appendChild(['child3' => $attrs ]);
+                                   '@id'    => 'Id attr'];
+                        $xml->appendChild(['child1' => $attrs])
+                            ->appendChild(['child2' => $attrs], true)
+                                ->appendChild(['child3' => $attrs]);
 
                         $expected = "<doc>\n"
                                   . "  <child1 class=\"Class attr\" id=\"Id attr\">Text content</child1>\n"
@@ -1109,8 +1109,8 @@ 
                 it('should add many instances', function() use ($doc, $dom) {
                         $fxml = FluidXml::load($dom)->query('/doc/parent');
                         $xml  = new FluidXml();
-                        $xml->appendChild([ $fxml,
-                                            'imported' => $fxml ]);
+                        $xml->appendChild([$fxml,
+                                            'imported' => $fxml]);
 
                         $expected = "<doc>\n"
                                   . "  <parent>content</parent>\n"
@@ -1122,11 +1122,11 @@ 
                 });
 
                 it('should throw for not supported input', function() {
-                        $xml  = new FluidXml();
+                        $xml = new FluidXml();
                         try {
                                 $xml->appendChild(0);
                         } catch (\Exception $e) {
-                                $actual   = $e;
+                                $actual = $e;
                         }
 
                         assert_is_a($actual, \Exception::class);
@@ -1872,7 +1872,7 @@ 
                         try {
                                 $cx[] = "value";
                         } catch (\Exception $e) {
-                                $actual   = $e;
+                                $actual = $e;
                         }
                         assert_is_a($actual, \Exception::class);
 
@@ -1985,8 +1985,8 @@ 
                         $ns_id   = 'x';
                         $ns_uri  = 'x.com';
                         $ns_mode = FluidNamespace::MODE_EXPLICIT;
-                        $args    = [ FluidNamespace::ID  => $ns_id,
-                                     FluidNamespace::URI => $ns_uri ];
+                        $args    = [FluidNamespace::ID  => $ns_id,
+                                     FluidNamespace::URI => $ns_uri];
                         $ns      = new FluidNamespace($args);
 
                         $actual   = $ns->id();

source/FluidXml.php

Spacing +30 added lines, -30 removed lines

@@ -210,7 +210,7 @@ 
                 $namespaces = [];
 
                 if (\is_string($arguments[0])) {
-                        $args = [ $arguments[0], $arguments[1] ];
+                        $args = [$arguments[0], $arguments[1]];
 
                         if (isset($arguments[2])) {
                                 $args[] = $arguments[2];
@@ -234,8 +234,8 @@ 
 class FluidXml implements FluidInterface
 {
         use FluidNamespaceTrait,
-            FluidXmlShadowTrait,        // For compatibility with PHP 5.6.
-            FluidContextShadowTrait;    // For compatibility with PHP 5.6.
+            FluidXmlShadowTrait, // For compatibility with PHP 5.6.
+            FluidContextShadowTrait; // For compatibility with PHP 5.6.
 
         const ROOT_NODE = 'doc';
 
@@ -243,7 +243,7 @@ 
 
         public static function load($document)
         {
-                if (\is_string($document) && ! \FluidXml\is_an_xml_string($document)) {
+                if (\is_string($document) && !\FluidXml\is_an_xml_string($document)) {
                         // Removes any empty new line at the beginning,
                         // otherwise the first character check fails.
 
@@ -255,7 +255,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.");
                         }
                 }
@@ -265,10 +265,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
@@ -462,14 +462,14 @@ 
 
         protected function newContext($context = null)
         {
-                if (! $context) {
+                if (!$context) {
                         $context = $this->dom->documentElement;
                 }
 
                 // If the user has requested ['root' => null] at construction time
                 // the 'documentElement' property is null because we have not created
                 // a root node yet.
-                if (! $context) {
+                if (!$context) {
                         // Whether there is not a root node, the DOMDocument is
                         // promoted as root node.
                         $context = $this->dom;
@@ -493,7 +493,7 @@ 
 class FluidContext implements FluidInterface, \ArrayAccess, \Iterator
 {
         use FluidNamespaceTrait,
-            FluidContextShadowTrait;    // For compatibility with PHP 5.6.
+            FluidContextShadowTrait; // For compatibility with PHP 5.6.
 
         private $dom;
         private $nodes = [];
@@ -501,8 +501,8 @@ 
 
         public function __construct($context, array $namespaces = [])
         {
-                if (! \is_array($context)) {
-                        $context = [ $context ];
+                if (!\is_array($context)) {
+                        $context = [$context];
                 }
 
                 foreach ($context as $n) {
@@ -523,7 +523,7 @@ 
                         }
                 }
 
-                if (! empty($namespaces)) {
+                if (!empty($namespaces)) {
                         $this->namespace(...\array_values($namespaces));
                 }
         }
@@ -652,7 +652,7 @@ 
                                 }
                         }
 
-                        if (! $found) {
+                        if (!$found) {
                                 $unique_results[] = $r;
                         }
                 }
@@ -754,7 +754,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]];
                 }
 
@@ -938,11 +938,11 @@ 
 
                                 return [];
                         } else {
-                                return [ $fn($parent, $this->createElement($k, $v)) ];
+                                return [$fn($parent, $this->createElement($k, $v))];
                         }
                 }
 
-                $v_isnt_string = ! $v_is_string;
+                $v_isnt_string = !$v_is_string;
 
                 if ($k_is_string && $v_isnt_string) {
                         // The user has passed one of these two cases:
@@ -955,7 +955,7 @@ 
                         // they are supplied, so 'appendChild' is the perfect operation.
                         $this->newContext($el)->appendChild($v, ...$optionals);
 
-                        return [ $el ];
+                        return [$el];
                 }
 
                 $k_is_int   = \is_int($k);
@@ -977,13 +977,13 @@ 
                 }
 
                 $v_is_xml   = $v_is_string && is_an_xml_string($v);
-                $v_isnt_xml = ! $v_is_xml;
+                $v_isnt_xml = !$v_is_xml;
 
                 if ($k_is_int && $v_is_string && $v_isnt_xml) {
                         // The user has passed a node name without a node value:
                         // [ 'element', ... ]
 
-                        return [ $fn($parent, $this->createElement($v)) ];
+                        return [$fn($parent, $this->createElement($v))];
                 }
 
                 // The user has passed an XML document instance:
@@ -1046,19 +1046,19 @@ 
                 $v_is_domnode = $v instanceof \DOMNode;
 
                 if ($k_is_int && $v_is_domnode) {
-                        return $attach_nodes([ $v ]);
+                        return $attach_nodes([$v]);
                 }
 
                 $v_is_simplexml = $v instanceof \SimpleXMLElement;
 
                 if ($k_is_int && $v_is_simplexml) {
-                        return $attach_nodes([ dom_import_simplexml($v) ]);
+                        return $attach_nodes([dom_import_simplexml($v)]);
                 }
 
                 $v_is_fluidxml = $v instanceof FluidXml;
 
                 if ($k_is_int && $v_is_fluidxml) {
-                        return $attach_nodes([ $v->dom()->documentElement ]);
+                        return $attach_nodes([$v->dom()->documentElement]);
                 }
 
                 $v_is_fluidcontext = $v instanceof FluidContext;
@@ -1080,8 +1080,8 @@ 
 
         protected function insertElement(\Closure $fn, $element, ...$optionals)
         {
-                if (! \is_array($element)) {
-                        $element = [ $element ];
+                if (!\is_array($element)) {
+                        $element = [$element];
                 }
 
                 $switchContext = false;
@@ -1091,7 +1091,7 @@ 
                         if (\is_array($opt)) {
                                 $attributes = $opt;
 
-                        } else if (\is_bool($opt)){
+                        } else if (\is_bool($opt)) {
                                 $switchContext = $opt;
 
                         } else if (\is_string($opt)) {
@@ -1134,16 +1134,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)
         {

support/peridot.php

Spacing +2 added lines, -2 removed lines

@@ -12,14 +12,14 @@
 use Peridot\Reporter\CodeCoverage\AbstractCodeCoverageReporter;
 use Peridot\Reporter\CodeCoverageReporters;
 
-return function (EventEmitterInterface $eventEmitter) {
+return function(EventEmitterInterface $eventEmitter) {
         (new CodeCoverageReporters($eventEmitter))->register();
 
         // $eventEmitter->on('peridot.start', function (\Peridot\Console\Environment $environment) {
         //         $environment->getDefinition()->getArgument('path')->setDefault(__DIR__ . '/../specs');
         // });
 
-        $eventEmitter->on('code-coverage.start', function (AbstractCodeCoverageReporter $reporter) {
+        $eventEmitter->on('code-coverage.start', function(AbstractCodeCoverageReporter $reporter) {
                 /*
                 $reporter->addDirectoryToWhitelist(__DIR__ . '/../source')
                          ->addFilesToBlacklist([__DIR__ . '/../source/FluidXml.php56.php',

support/speedtest.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 $machine = new Codevelox();
 
 $fluidxml = 'fluidxml';
-if (! function_exists($fluidxml)) {
+if (!function_exists($fluidxml)) {
         $fluidxml = '\FluidXml\fluidxml';
 }
 

documents/Examples.php

Spacing +24 added lines, -24 removed lines

@@ -35,10 +35,10 @@ 
 // $booksheet = FluidXml::new('book', ['stylesheet' => 'http://domain.com/style.xsl']);
 
 $book->setAttribute('type', 'science')                  // It sets an attribute of the root node ('book').
-     ->appendChild([ 'title'  => 'The Theory Of Everything',
-                     'author' => 'S. Hawking' ]);       // It creates two nodes, each one with some text inside.
+     ->appendChild(['title'  => 'The Theory Of Everything',
+                     'author' => 'S. Hawking']); // It creates two nodes, each one with some text inside.
 
-echo $book->xml();                                      // Exports the xml document as a string.
+echo $book->xml(); // Exports the xml document as a string.
 echo "————————————————————————————————————————————————————————————————————————————————\n";
 
 
@@ -59,9 +59,9 @@ 
 */
 
 $book->appendChild('chapters', true)                     // true forces the return of the 'chapters' node.
-        ->appendChild('chapter', 'Ideas About The Universe',    ['id' => 123, 'first' => ''])
-        ->appendChild('chapter', 'The Expanding Universe',      ['id' => 321])
-        ->appendChild('chapter', 'Black Holes',                 ['id' => 432])
+        ->appendChild('chapter', 'Ideas About The Universe', ['id' => 123, 'first' => ''])
+        ->appendChild('chapter', 'The Expanding Universe', ['id' => 321])
+        ->appendChild('chapter', 'Black Holes', ['id' => 432])
         ->appendChild('chapter', 'Black Holes Ain\'t So Black', ['id' =>234]);
 
 
@@ -83,26 +83,26 @@ 
 $food = fluidxml('food');
 
 $food->add('fruit')               // A 'fruit' node with an empty content.
-     ->add('fruit', 'orange');    // A 'fruit' node with 'orange' as content.
+     ->add('fruit', 'orange'); // A 'fruit' node with 'orange' as content.
 
 // A node can have even a bunch of attributes.
-$food->add('fruit', 'apple', [ 'price' => 'expensive',
-                               'color' => 'red' ]);
+$food->add('fruit', 'apple', ['price' => 'expensive',
+                               'color' => 'red']);
 
 // Batch insertion of nodes.
-$food->add([ 'cake'  => 'Tiramisu',
-             'pizza' => 'Margherita' ]);
+$food->add(['cake'  => 'Tiramisu',
+             'pizza' => 'Margherita']);
 
 // PHP arrays can't contain identical keys.
 // But it's still possible to create, in a batch operation, nodes with the same tag.
-$food->add([ [ 'pasta' => 'Carbonara' ],
-             [ 'pasta' => 'Matriciana' ] ]);
+$food->add([['pasta' => 'Carbonara'],
+             ['pasta' => 'Matriciana']]);
 
 // A bunch of egg's all with the same price.
-$food->add([ ['egg'], ['egg'], ['egg'] ], ['price' => '0.25']);
+$food->add([['egg'], ['egg'], ['egg']], ['price' => '0.25']);
 
 // Complex array structures are supported too.
-$food->add([ 'menu' => [
+$food->add(['menu' => [
                  'pasta' => [
                      'spaghetti' => [
                          '@id'      => '123',
@@ -110,8 +110,8 @@ 
                          '@'        => 'Spaghetti are an Italian dish...',
 
                          'variants' => [
-                             'tomato' => [ '@type' => 'vegan' ],
-                             'egg'    => [ '@type' => 'vegetarian' ] ]]]]]);
+                             'tomato' => ['@type' => 'vegan'],
+                             'egg'    => ['@type' => 'vegetarian']]]]]]);
 
 echo $food->xml();
 echo "————————————————————————————————————————————————————————————————————————————————\n";
@@ -188,10 +188,10 @@ 
 $fluid = fluidxml();
 
 $fluid->add($other)                                     // Imports a FluidXml instance.
-      ->add([ 'query'     => $fluid->query('//meta'),   // Imports a FluidXml query.
-              'dom'       => $dom,                      // Imports a DOMDocument.
-              'domnodes'  => $dom->childNodes,          // Imports a DOMNodeList.
-              'simplexml' => $simplexml ]);             // Imports a SimpleXMLElement.
+      ->add(['query'     => $fluid->query('//meta'), // Imports a FluidXml query.
+              'dom'       => $dom, // Imports a DOMDocument.
+              'domnodes'  => $dom->childNodes, // Imports a DOMNodeList.
+              'simplexml' => $simplexml]); // Imports a SimpleXMLElement.
 
 
 
@@ -278,7 +278,7 @@ 
 /*
 * To retrieve all DOMNode in one operation there is the ->asArray() method.
 */
-$chapters_nodes = $chapters->asArray();          // Returns an array of DOMNode.
+$chapters_nodes = $chapters->asArray(); // Returns an array of DOMNode.
 
 echo "————————————————————————————————————————————————————————————————————————————————\n";
 
@@ -298,7 +298,7 @@ 
      ->namespace('svg', 'http://www.w3.org/2000/svg')
      ->namespace('xsl', 'http://www.w3.org/TR/xsl', FluidNamespace::MODE_IMPLICIT)
      ->add('xhtml:h1')
-     ->add([ 'xsl:template'  => [ 'xsl:variable' ] ])
+     ->add(['xsl:template'  => ['xsl:variable']])
      ->query('//xhtml:h1')
      ->add('svg:shape');
 
@@ -311,7 +311,7 @@ 
  * Removing Nodes. *
  *******************/
 
-$food->remove('//egg');     // Removes all the eggs.
+$food->remove('//egg'); // Removes all the eggs.
 
 // Which is the same of
 // $food->query('//egg')->remove();     // Removes all the eggs using a query.