From 4ee95eca49089061c2f1e84c2578111a28efee9f Mon Sep 17 00:00:00 2001 From: Semyon Knyazev Date: Fri, 20 Dec 2024 01:05:33 +0400 Subject: [PATCH] Reduce gpo tree gui interaction time GPO permission syncronization is removed for fetched policy tree items --- src/admc/console_impls/policy_impl.cpp | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/admc/console_impls/policy_impl.cpp b/src/admc/console_impls/policy_impl.cpp index 630649eb..897cbf90 100755 --- a/src/admc/console_impls/policy_impl.cpp +++ b/src/admc/console_impls/policy_impl.cpp @@ -110,7 +110,12 @@ void PolicyImpl::selected_as_scope(const QModelIndex &index) { // If they don't, offer to update GPT permissions. const QString selected_gpo = index.data(PolicyRole_DN).toString(); bool ok = true; - const bool perms_ok = ad.gpo_check_perms(selected_gpo, &ok); + bool perms_ok = true; + const bool was_fetched = console_item_get_was_fetched(index); + // Dont check perms if item was already fetched. It helps to avoid excessive freezes. + if (!was_fetched) { + perms_ok = ad.gpo_check_perms(selected_gpo, &ok); + } if (!perms_ok && ok) { const QString title = tr("Incorrect permissions detected");