Skip to content

Commit 847319d

Browse files
🎉 Add fix_available information to engagement views (#13070)
1 parent 22e6dc0 commit 847319d

File tree

4 files changed

+12
-4
lines changed

4 files changed

+12
-4
lines changed

dojo/engagement/views.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -608,6 +608,9 @@ def prefetch_for_view_tests(tests):
608608
count_findings_test_active_verified=Coalesce(
609609
count_subquery(base_findings.filter(active=True, verified=True)), Value(0),
610610
),
611+
count_findings_test_active_fix_available=Coalesce(
612+
count_subquery(base_findings.filter(active=True, fix_available=True)), Value(0),
613+
),
611614
count_findings_test_mitigated=Coalesce(count_subquery(base_findings.filter(is_mitigated=True)), Value(0)),
612615
count_findings_test_dups=Coalesce(count_subquery(base_findings.filter(duplicate=True)), Value(0)),
613616
total_reimport_count=Coalesce(

dojo/product/views.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -859,6 +859,9 @@ def prefetch_for_view_engagements(engagements, recent_test_day_count):
859859
finding_open_verified_subquery = build_count_subquery(
860860
Finding.objects.filter(test__engagement=OuterRef("pk"), active=True, verified=True), group_field="test__engagement_id",
861861
)
862+
finding_open_fix_available_subquery = build_count_subquery(
863+
Finding.objects.filter(test__engagement=OuterRef("pk"), active=True, fix_available=True), group_field="test__engagement_id",
864+
)
862865
finding_close_subquery = build_count_subquery(
863866
Finding.objects.filter(test__engagement=OuterRef("pk"), is_mitigated=True), group_field="test__engagement_id",
864867
)
@@ -874,6 +877,7 @@ def prefetch_for_view_engagements(engagements, recent_test_day_count):
874877
count_findings_all=Coalesce(finding_subquery, Value(0)),
875878
count_findings_open=Coalesce(finding_open_subquery, Value(0)),
876879
count_findings_open_verified=Coalesce(finding_open_verified_subquery, Value(0)),
880+
count_findings_fix_available=Coalesce(finding_open_fix_available_subquery, Value(0)),
877881
count_findings_close=Coalesce(finding_close_subquery, Value(0)),
878882
count_findings_duplicate=Coalesce(finding_duplicate_subquery, Value(0)),
879883
count_findings_accepted=Coalesce(finding_accepted_subquery, Value(0)),

dojo/templates/dojo/snippets/engagement_list.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ <h4> {% if status == "open" %}Active{% elif status == "paused" %}Paused {% else
4848
<th class="text-center">JIRA</th>
4949
{% endif %}
5050
<th class="text-center">Tests</th>
51-
<th class="text-center">Active (Verified)</th>
51+
<th class="text-center">Active (Verified / Fixable)</th>
5252
<th class="text-center">Mitigated</th>
5353
<th class="text-center">Accepted</th>
5454
<th class="text-center">All</th>
@@ -244,7 +244,7 @@ <h4> {% if status == "open" %}Active{% elif status == "paused" %}Paused {% else
244244
{% endif %}
245245
</div>
246246
</td>
247-
<td class="text-center" data-order="{{ eng.count_findings_open }}">{{ eng.count_findings_open }}&nbsp;({{ eng.count_findings_open_verified}})</td>
247+
<td class="text-center" data-order="{{ eng.count_findings_open }}">{{ eng.count_findings_open }}&nbsp;({{ eng.count_findings_open_verified}}/{{ eng.count_findings_fix_available}})</td>
248248
<td class="text-center">{{ eng.count_findings_close }}</td>
249249
<td class="text-center">{{ eng.count_findings_accepted }}</td>
250250
<td class="text-center">{{ eng.count_findings_all }}</td>

dojo/templates/dojo/view_eng.html

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -248,7 +248,7 @@ <h4>
248248
<th>Date</th>
249249
<th>Lead</th>
250250
<th>Total Findings</th>
251-
<th>Active (Verified)</th>
251+
<th>Active (Verified / Fixable)</th>
252252
<th>Mitigated</th>
253253
<th>Duplicates</th>
254254
<th>Notes</th>
@@ -336,7 +336,8 @@ <h4>
336336
<td><a href="{% url 'view_test' test.id %}">{{ test.count_findings_test_all }}</a></td>
337337
<td>
338338
<a href="{% url 'view_test' test.id %}?active=true">{{ test.count_findings_test_active }}</a>&nbsp;
339-
(<a href="{% url 'view_test' test.id %}?active=true&verified=true">{{ test.count_findings_test_active_verified }}</a>)
339+
(<a href="{% url 'view_test' test.id %}?active=true&verified=true">{{ test.count_findings_test_active_verified }}</a>
340+
/ <a href="{% url 'view_test' test.id %}?active=true&fix_available=true">{{ test.count_findings_test_active_fix_available }}</a>)
340341
</td>
341342
<td><a href="{% url 'view_test' test.id %}?is_mitigated=1">{{ test.count_findings_test_mitigated }}</a></td>
342343
<td><a href="{% url 'view_test' test.id %}?duplicate=1">{{ test.count_findings_test_dups }}</a></td>

0 commit comments

Comments
 (0)