bug #15220 [DependencyInjection] Freeze also FrozenParameterBag::remove (lyrixx)

fabpot · fabpot · commit 973f53c32bf0 · 2015-07-07T20:27:32.000+02:00
This PR was merged into the 2.3 branch.

Discussion
----------

[DependencyInjection] Freeze also FrozenParameterBag::remove

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

Commits
-------

4a72c44 [DependencyInjection] Freeze also FrozenParameterBag::remove
diff --git a/ParameterBag/FrozenParameterBag.php b/ParameterBag/FrozenParameterBag.php
@@ -69,4 +69,14 @@ public function set($name, $value)
     {
         throw new LogicException('Impossible to call set() on a frozen ParameterBag.');
     }
+
+    /**
+     * {@inheritdoc}
+     *
+     * @api
+     */
+    public function remove($name)
+    {
+        throw new LogicException('Impossible to call remove() on a frozen ParameterBag.');
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -69,4 +69,14 @@ public function set($name, $value)`
`69`	`69`	`{`
`70`	`70`	`throw new LogicException('Impossible to call set() on a frozen ParameterBag.');`
`71`	`71`	`}`
	`72`	`+`
	`73`	`+ /**`
	`74`	`+ * {@inheritdoc}`
	`75`	`+ *`
	`76`	`+ * @api`
	`77`	`+ */`
	`78`	`+ public function remove($name)`
	`79`	`+ {`
	`80`	`+ throw new LogicException('Impossible to call remove() on a frozen ParameterBag.');`
	`81`	`+ }`
`72`	`82`	`}`