Skip to content
Snippets Groups Projects
Commit 7b47e553 authored by Tim van Dijen's avatar Tim van Dijen
Browse files

Fix: Do not mix several configuration styles

parent 8195d203
No related branches found
No related tags found
No related merge requests found
...@@ -39,10 +39,10 @@ Parameters ...@@ -39,10 +39,10 @@ Parameters
If no other values exist, the attribute will be removed completely. If no other values exist, the attribute will be removed completely.
This parameter is OPTIONAL. This parameter is OPTIONAL.
`merge` `%merge`
: A boolean indicating whether the altered values must be merged with the : Indicates whether the altered values must be merged with the target attribute values. The default
target attribute values, or if the target attribute should be overwritten. behaviour is to overwrite the target attribute completely.
Defaults to `false`. This parameter is OPTIONAL.
Examples Examples
-------- --------
......
...@@ -79,6 +79,8 @@ class AttributeAlter extends Auth\ProcessingFilter ...@@ -79,6 +79,8 @@ class AttributeAlter extends Auth\ProcessingFilter
$this->replace = true; $this->replace = true;
} elseif ($value === '%remove') { } elseif ($value === '%remove') {
$this->remove = true; $this->remove = true;
} elseif ($value === '%merge') {
$this->merge = true;
} else { } else {
throw new Error\Exception('Unknown flag : ' . var_export($value, true)); throw new Error\Exception('Unknown flag : ' . var_export($value, true));
} }
...@@ -95,9 +97,6 @@ class AttributeAlter extends Auth\ProcessingFilter ...@@ -95,9 +97,6 @@ class AttributeAlter extends Auth\ProcessingFilter
} elseif ($name === 'target') { } elseif ($name === 'target') {
// Set target // Set target
$this->target = $value; $this->target = $value;
} elseif ($name === 'merge') {
// Set the merging strategy
$this->merge = $value;
} }
} }
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment