[DependencyInjection] fixed FrozenParameterBag and improved Parameter…

Author: Tomaz Ahlin

ParameterBag/FrozenParameterBag.php

@@ -69,4 +69,12 @@ public function set($name, $value)
     {
         throw new LogicException('Impossible to call set() on a frozen ParameterBag.');
     }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function remove($name)
+    {
+        throw new LogicException('Impossible to call remove() on a frozen ParameterBag.');
+    }
 }

ParameterBag/ParameterBagInterface.php

@@ -11,6 +11,7 @@
 
 namespace Symfony\Component\DependencyInjection\ParameterBag;
 
+use Symfony\Component\DependencyInjection\Exception\LogicException;
 use Symfony\Component\DependencyInjection\Exception\ParameterNotFoundException;
 
 /**
@@ -25,6 +26,8 @@ interface ParameterBagInterface
     /**
      * Clears all parameters.
      *
+     * @throws LogicException if the ParameterBagInterface can not be cleared
+     *
      * @api
      */
     public function clear();
@@ -34,6 +37,8 @@ public function clear();
      *
      * @param array $parameters An array of parameters
      *
+     * @throws LogicException if the parameter can not be added
+     *
      * @api
      */
     public function add(array $parameters);
@@ -66,6 +71,8 @@ public function get($name);
      * @param string $name  The parameter name
      * @param mixed  $value The parameter value
      *
+     * @throws LogicException if the parameter can not be set
+     *
      * @api
      */
     public function set($name, $value);

Tests/ParameterBag/FrozenParameterBagTest.php

@@ -57,4 +57,13 @@ public function testAdd()
         $bag = new FrozenParameterBag(array());
         $bag->add(array());
     }
+
+    /**
+     * @expectedException \LogicException
+     */
+    public function testRemove()
+    {
+        $bag = new FrozenParameterBag(array('foo' => 'bar'));
+        $bag->remove('foo');
+    }
 }