[Config] replaced setInfo(), setExample() with more generic attributes

Author: schmittjoh

Definition/BaseNode.php

@@ -29,8 +29,7 @@ abstract class BaseNode implements NodeInterface
     protected $allowOverwrite;
     protected $required;
     protected $equivalentValues;
-    protected $info;
-    protected $example;
+    protected $attributes = array();
 
     /**
      * Constructor.
@@ -55,14 +54,44 @@ public function __construct($name, NodeInterface $parent = null)
         $this->equivalentValues = array();
     }
 
+    public function setAttribute($key, $value)
+    {
+        $this->attributes[$key] = $value;
+    }
+
+    public function getAttribute($key, $default = null)
+    {
+        return isset($this->attributes[$key]) ? $this->attributes[$key] : $default;
+    }
+
+    public function hasAttribute($key)
+    {
+        return isset($this->attributes[$key]);
+    }
+
+    public function getAttributes()
+    {
+        return $this->attributes;
+    }
+
+    public function setAttributes(array $attributes)
+    {
+        $this->attributes = $attributes;
+    }
+
+    public function removeAttribute($key)
+    {
+        unset($this->attributes[$key]);
+    }
+
     /**
-     * Sets info message.
+     * Sets an info message.
      *
-     * @param string $info The info text
+     * @param string $info
      */
     public function setInfo($info)
     {
-        $this->info = $info;
+        $this->setAttribute('info', $info);
     }
 
     /**
@@ -72,7 +101,7 @@ public function setInfo($info)
      */
     public function getInfo()
     {
-        return $this->info;
+        return $this->getAttribute('info');
     }
 
     /**
@@ -82,7 +111,7 @@ public function getInfo()
      */
     public function setExample($example)
     {
-        $this->example = $example;
+        $this->setAttribute('example', $example);
     }
 
     /**
@@ -92,7 +121,7 @@ public function setExample($example)
      */
     public function getExample()
     {
-        return $this->example;
+        return $this->getAttribute('example');
     }
 
     /**

Definition/Builder/NodeDefinition.php

@@ -32,8 +32,7 @@ abstract class NodeDefinition implements NodeParentInterface
     protected $trueEquivalent;
     protected $falseEquivalent;
     protected $parent;
-    protected $info;
-    protected $example;
+    protected $attributes = array();
 
     /**
      * Constructor
@@ -72,11 +71,9 @@ public function setParent(NodeParentInterface $parent)
      *
      * @return NodeDefinition
      */
-    public function setInfo($info)
+    public function info($info)
     {
-        $this->info = $info;
-
-        return $this;
+        return $this->attribute('info', $info);
     }
 
     /**
@@ -86,9 +83,22 @@ public function setInfo($info)
      *
      * @return NodeDefinition
      */
-    public function setExample($example)
+    public function example($example)
+    {
+        return $this->attribute('example', $example);
+    }
+
+    /**
+     * Sets an attribute on the node.
+     *
+     * @param string $key
+     * @param mixed $value
+     *
+     * @return NodeDefinition
+     */
+    public function attribute($key, $value)
     {
-        $this->example = $example;
+        $this->attributes[$key] = $value;
 
         return $this;
     }
@@ -125,9 +135,7 @@ public function getNode($forceRootNode = false)
         }
 
         $node = $this->createNode();
-
-        $node->setInfo($this->info);
-        $node->setExample($this->example);
+        $node->setAttributes($this->attributes);
 
         return $node;
     }

Tests/Definition/Builder/TreeBuilderTest.php

@@ -94,9 +94,9 @@ public function testDefinitionInfoGetsTransferedToNode()
     {
         $builder = new TreeBuilder();
 
-        $builder->root('test')->setInfo('root info')
+        $builder->root('test')->info('root info')
             ->children()
-                ->node('child', 'variable')->setInfo('child info')->defaultValue('default')
+                ->node('child', 'variable')->info('child info')->defaultValue('default')
             ->end()
         ->end();
 
@@ -112,9 +112,9 @@ public function testDefinitionExampleGetsTransferedToNode()
         $builder = new TreeBuilder();
 
         $builder->root('test')
-            ->setExample(array('key' => 'value'))
+            ->example(array('key' => 'value'))
             ->children()
-                ->node('child', 'variable')->setInfo('child info')->defaultValue('default')->setExample('example')
+                ->node('child', 'variable')->info('child info')->defaultValue('default')->example('example')
             ->end()
         ->end();