Skip to content

Commit dd3c053

Browse files
committed
query optimization
Signed-off-by: tdruez <tdruez@aboutcode.org>
1 parent a14a916 commit dd3c053

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

policy/tasks.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ def evaluate_all_products_rules_task(include_locked=False, product_uuids=None):
4545
"""
4646
Product = apps.get_model("product_portfolio", "product")
4747

48-
products = get_unsecured_manager(Product).select_related("dataspace")
48+
products = get_unsecured_manager(Product).select_related("dataspace__configuration")
4949
if product_uuids is not None:
5050
products = products.filter(uuid__in=product_uuids)
5151
if not include_locked:

product_portfolio/views.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2869,7 +2869,9 @@ def get_license_compliance_context(licenses, distribution_limit=10):
28692869

28702870
@staticmethod
28712871
def get_policy_compliance_context(product):
2872-
policy_violations = product.policy_violations.filter(resolved=False).order_by("rule_type")
2872+
policy_violations = list(
2873+
product.policy_violations.filter(resolved=False).order_by("rule_type")
2874+
)
28732875
violated_rule_types = {violation.rule_type for violation in policy_violations}
28742876
all_rules = [
28752877
{
@@ -2884,7 +2886,7 @@ def get_policy_compliance_context(product):
28842886
]
28852887
return {
28862888
"policy_violations": policy_violations,
2887-
"policy_violation_count": policy_violations.count(),
2889+
"policy_violation_count": len(policy_violations),
28882890
"all_rules": all_rules,
28892891
}
28902892

0 commit comments

Comments
 (0)