|
@@ -183,13 +183,13 @@ |
|
|
block discarded – undo |
|
183
|
183
|
// array is empty or not set then an empty array will be returned. |
|
184
|
184
|
$attributesFromQueryConfiguration = $this->parseQueryAttributes($dom); |
|
185
|
185
|
if (!empty($attributesFromQueryConfiguration)) { |
|
186
|
|
- // Overwrite attributes from parseAuthenticationSuccess with configured |
|
187
|
|
- // XPath-based attributes, instead of combining them. |
|
|
186
|
+ // Overwrite attributes from parseAuthenticationSuccess with configured |
|
|
187
|
+ // XPath-based attributes, instead of combining them. |
|
188
|
188
|
foreach ($attributesFromQueryConfiguration as $name => $values) { |
|
189
|
|
- // Ensure a clean, unique list of string values |
|
|
189
|
+ // Ensure a clean, unique list of string values |
|
190
|
190
|
$values = array_values(array_unique(array_map('strval', $values))); |
|
191
|
191
|
|
|
192
|
|
- // Configuration wins: replace any existing attribute with the same name |
|
|
192
|
+ // Configuration wins: replace any existing attribute with the same name |
|
193
|
193
|
$attributes[$name] = $values; |
|
194
|
194
|
} |
|
195
|
195
|
} |
Please login to merge, or discard this patch.