diff options
Diffstat (limited to 'src/components/policy/policy_external/src/policy_helper.cc')
-rw-r--r-- | src/components/policy/policy_external/src/policy_helper.cc | 28 |
1 files changed, 3 insertions, 25 deletions
diff --git a/src/components/policy/policy_external/src/policy_helper.cc b/src/components/policy/policy_external/src/policy_helper.cc index 90d61320ad..39b9399c24 100644 --- a/src/components/policy/policy_external/src/policy_helper.cc +++ b/src/components/policy/policy_external/src/policy_helper.cc @@ -325,20 +325,13 @@ bool CheckAppPolicy::operator()(const AppPoliciesValueType& app_policy) { if (!IsPredefinedApp(app_policy)) { const bool is_request_type_changed = IsRequestTypeChanged(app_policy); const bool is_request_subtype_changed = IsRequestSubTypeChanged(app_policy); - if (is_request_type_changed && is_request_subtype_changed) { - LOG4CXX_TRACE( - logger_, - "Both request types & subtypes were changed for application: " - << app_id); - SetPendingPermissions(app_policy, - RESULT_REQUEST_TYPE_AND_SUBTYPE_CHANGED); - AddResult(app_id, RESULT_REQUEST_TYPE_AND_SUBTYPE_CHANGED); - } else if (is_request_type_changed) { + if (is_request_type_changed) { LOG4CXX_TRACE(logger_, "Request types were changed for application: " << app_id); SetPendingPermissions(app_policy, RESULT_REQUEST_TYPE_CHANGED); AddResult(app_id, RESULT_REQUEST_TYPE_CHANGED); - } else if (is_request_subtype_changed) { + } + if (is_request_subtype_changed) { LOG4CXX_TRACE( logger_, "Request subtypes were changed for application: " << app_id); SetPendingPermissions(app_policy, RESULT_REQUEST_SUBTYPE_CHANGED); @@ -421,20 +414,6 @@ void policy::CheckAppPolicy::SetPendingPermissions( permissions_diff.requestSubType.push_back(request_subtype); } break; - case RESULT_REQUEST_TYPE_AND_SUBTYPE_CHANGED: - permissions_diff.requestTypeChanged = true; - permissions_diff.requestSubTypeChanged = true; - - // Getting Request Types from PTU (not from cache) - for (const auto& request_type : *app_policy.second.RequestType) { - permissions_diff.requestType.push_back(EnumToJsonString(request_type)); - } - - // Getting Request SubTypes from PTU (not from cache) - for (const auto& request_subtype : *app_policy.second.RequestSubType) { - permissions_diff.requestSubType.push_back(request_subtype); - } - break; default: return; } @@ -571,7 +550,6 @@ void FillActionsForAppPolicies::operator()( case RESULT_PERMISSIONS_REVOKED: case RESULT_REQUEST_TYPE_CHANGED: case RESULT_REQUEST_SUBTYPE_CHANGED: - case RESULT_REQUEST_TYPE_AND_SUBTYPE_CHANGED: break; case RESULT_NO_CHANGES: default: |