Skip to content

Commit baa4a4f

Browse files
authored
fix: google#3515 Move upstream field in frontend and sort alphabetically (google#3583)
1 parent e286032 commit baa4a4f

File tree

2 files changed

+12
-10
lines changed

2 files changed

+12
-10
lines changed

gcp/website/frontend3/src/templates/vulnerability.html

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,14 @@ <h1 class="title">
6363
</ul>
6464
</dd>
6565
{% endif -%}
66+
{%- if vulnerability.upstream_hierarchy -%}
67+
<dt>Upstream</dt>
68+
<dd class="upstream expandible-list">{{ vulnerability.upstream_hierarchy | safe }}</dd>
69+
{%- endif -%}
70+
{%- if vulnerability.downstream_hierarchy -%}
71+
<dt>Downstream</dt>
72+
<dd class="downstream expandible-list">{{ vulnerability.downstream_hierarchy | safe }}</dd>
73+
{%- endif -%}
6674
{% if vulnerability.related -%}
6775
<dt>Related</dt>
6876
<dd>
@@ -90,14 +98,6 @@ <h1 class="title">
9098
<dd>{{ vulnerability.published }}</dd>
9199
<dt>Modified</dt>
92100
<dd>{{ vulnerability.modified }}</dd>
93-
{%- if vulnerability.upstream_hierarchy -%}
94-
<dt>Upstream</dt>
95-
<dd class="upstream expandible-list">{{ vulnerability.upstream_hierarchy | safe }}</dd>
96-
{%- endif -%}
97-
{%- if vulnerability.downstream_hierarchy -%}
98-
<dt>Downstream</dt>
99-
<dd class="downstream expandible-list">{{ vulnerability.downstream_hierarchy | safe }}</dd>
100-
{%- endif -%}
101101
{%- if vulnerability.severity -%}
102102
<dt>Severity</dt>
103103
<dd>

gcp/website/frontend_handlers.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -974,13 +974,15 @@ def print_subtree(vuln_id: str) -> None:
974974
output_lines.append("<li>" + vuln_id + "</li>")
975975

976976
if vuln_id in graph:
977-
for child in graph[vuln_id]:
977+
sorted_children = sorted(graph[vuln_id])
978+
for child in sorted_children:
978979
if child != target_bug_id:
979980
output_lines.append("<ul class=\"substream\">")
980981
print_subtree(child)
981982
output_lines.append("</ul>")
982983

983-
for root in root_nodes:
984+
sorted_root_nodes = sorted(root_nodes)
985+
for root in sorted_root_nodes:
984986
output_lines.append("<ul class=\"aliases\">")
985987
print_subtree(root)
986988
output_lines.append("</ul>")

0 commit comments

Comments
 (0)