Skip to content
Snippets Groups Projects

Using variables instead of literals in logs in PerunEntitlement.php

Merged Pavel Vyskočil requested to merge github/fork/vyskocilpavel/literal_to_variable into master
1 file
+ 2
2
Compare changes
  • Side-by-side
  • Inline
@@ -186,7 +186,7 @@ class PerunEntitlement extends ProcessingFilter
@@ -186,7 +186,7 @@ class PerunEntitlement extends ProcessingFilter
isset($request['SPMetadata']['groupMapping']) &&
isset($request['SPMetadata']['groupMapping']) &&
isset($request['SPMetadata']['groupMapping'][$groupName])) {
isset($request['SPMetadata']['groupMapping'][$groupName])) {
Logger::debug(
Logger::debug(
'Mapping $groupName to ' . $request['SPMetadata']['groupMapping'][$groupName] .
'Mapping ' . $groupName . ' to ' . $request['SPMetadata']['groupMapping'][$groupName] .
' for SP ' . $request['SPMetadata']['entityid']
' for SP ' . $request['SPMetadata']['entityid']
);
);
return $request['SPMetadata']['groupMapping'][$groupName];
return $request['SPMetadata']['groupMapping'][$groupName];
@@ -199,7 +199,7 @@ class PerunEntitlement extends ProcessingFilter
@@ -199,7 +199,7 @@ class PerunEntitlement extends ProcessingFilter
} else {
} else {
# No mapping defined, so just put groupNamePrefix in front of the group
# No mapping defined, so just put groupNamePrefix in front of the group
Logger::debug(
Logger::debug(
'No mapping found for group $groupName for SP ' . $request['SPMetadata']['entityid']
'No mapping found for group ' . $groupName . ' for SP ' . $request['SPMetadata']['entityid']
);
);
return $this->entitlementPrefix . 'group:' . $groupName;
return $this->entitlementPrefix . 'group:' . $groupName;
}
}
Loading