jinja2: rename revision*-filters to phabricator_revision*

Author: shtrom

src/lando/jinja.py

@@ -255,13 +255,13 @@ def linkify_bug_numbers(text: str) -> str:
     return re.sub(search, replace, str(text), flags=re.IGNORECASE)
 
 
-def linkify_revision_urls(text: str) -> str:
+def linkify_phabricator_revision_urls(text: str) -> str:
     search = r"(?=\b)(" + re.escape(settings.PHABRICATOR_URL) + r"/D\d+)(?=\b)"
     replace = r'<a href="\g<1>">\g<1></a>'
     return re.sub(search, replace, str(text), flags=re.IGNORECASE)
 
 
-def linkify_revision_ids(text: str) -> str:
+def linkify_phabricator_revision_ids(text: str) -> str:
     """Linkify revision IDs to proper Phabricator URLs."""
     search = r"\b(D\d+)\b"
     replace = (
@@ -321,7 +321,7 @@ def pull_request_link(repo: Repo, revision: Revision) -> str:
     return f"{repo.normalized_url}/pull/{revision.pull_number}"
 
 
-def revision_url(revision_id: int | str, diff_id: str | None = None) -> str:
+def phabricator_revision_url(revision_id: int | str, diff_id: str | None = None) -> str:
     if isinstance(revision_id, int):
         path = f"D{revision_id}"
     elif isinstance(revision_id, str) and not revision_id.startswith("D"):
@@ -461,8 +461,8 @@ def environment(**options):  # noqa: ANN201
             "graph_x_pos": graph_x_pos,
             "linkify_bug_numbers": linkify_bug_numbers,
             "linkify_faq": linkify_faq,
-            "linkify_revision_ids": linkify_revision_ids,
-            "linkify_revision_urls": linkify_revision_urls,
+            "linkify_phabricator_revision_ids": linkify_phabricator_revision_ids,
+            "linkify_phabricator_revision_urls": linkify_phabricator_revision_urls,
             "linkify_sec_bug_docs": linkify_sec_bug_docs,
             "linkify_transplant_details": linkify_transplant_details,
             "message_type_to_notification_class": message_type_to_notification_class,
@@ -472,7 +472,7 @@ def environment(**options):  # noqa: ANN201
             "reviewer_to_action_text": reviewer_to_action_text,
             "reviewer_to_status_badge_class": reviewer_to_status_badge_class,
             "revision_status_to_badge_class": revision_status_to_badge_class,
-            "revision_url": revision_url,
+            "phabricator_revision_url": phabricator_revision_url,
             "static": static,
             "tostatusbadgeclass": tostatusbadgeclass,
             "tostatusbadgename": tostatusbadgename,

src/lando/ui/jinja2/partials/error-breakdown.html

@@ -12,7 +12,7 @@
             While applying <a href="{{ pull_request_link(job.target_repo, top_revision) }}">pull request #{{ top_revision.pull_number }}</a> to
             <code>{{ tree }}</code>, the following files had conflicts:
         {% else %}
-            While applying <a href="{{ job.error_breakdown.revision_id|revision_url() }}">revision D{{
+            While applying <a href="{{ job.error_breakdown.revision_id|phabricator_revision_url() }}">revision D{{
             job.error_breakdown.revision_id }}</a> to <code>{{ tree }}</code>, the following files had
             conflicts:
         {% endif %}

src/lando/ui/jinja2/partials/job.html

@@ -33,7 +33,7 @@
                 <strong>Revisions:</strong>
                 {% for i in job.serialized_landing_path -%}
                     {{- "" if loop.first else " ← " -}}
-                    <a href="{{ i['revision_id']|revision_url(diff_id=i['diff_id']) }}">
+                    <a href="{{ i['revision_id']|phabricator_revision_url(diff_id=i['diff_id']) }}">
                         {{ i['revision_id'] }}
                         {% if i['diff_id'] %}diff {{ i['diff_id'] }}{% endif %}
                     </a>

src/lando/ui/jinja2/stack/partials/landing-preview.html

@@ -49,7 +49,7 @@ <h3 class="StackPage-landingPreview-sectionLabel">Commits:</h3>
                 </div>
                 <div class="StackPage-landingPreview-displayMessagePanel">
                     <pre class="StackPage-landingPreview-commitMessage">{{
-        revision['commit_message']|escape_html|linkify_bug_numbers|linkify_revision_urls|safe
+        revision['commit_message']|escape_html|linkify_bug_numbers|linkify_phabricator_revision_urls|safe
       }}</pre>
                     <div class="StackPage-landingPreview-seeMore"></div>
                 </div>
@@ -87,8 +87,8 @@ <h3 class="StackPage-landingPreview-sectionLabel">Warnings:</h3>
                                 <li class="StackPage-landingPreview-warning">
                                     <label>
                                         <input type="checkbox" name="warnings[]" value="1" />
-                                        {{ dw.message|escape_html|linkify_bug_numbers|linkify_revision_urls|linkify_faq|linkify_sec_bug_docs|safe }}
-                                        [{{ w.revision_id | linkify_revision_ids | safe }}]
+                                        {{ dw.message|escape_html|linkify_bug_numbers|linkify_phabricator_revision_urls|linkify_faq|linkify_sec_bug_docs|safe }}
+                                        [{{ w.revision_id | linkify_phabricator_revision_ids | safe }}]
                                     </label>
                                 </li>
                             {% endfor %}
@@ -97,12 +97,12 @@ <h3 class="StackPage-landingPreview-sectionLabel">Warnings:</h3>
                         <li class="StackPage-landingPreview-warning">
                             <label>
                                 <input type="checkbox" name="warnings[]" value="1" />
-                                {{ warning['display']|escape_html|linkify_bug_numbers|linkify_revision_urls|linkify_faq|linkify_sec_bug_docs|safe }}
+                                {{ warning['display']|escape_html|linkify_bug_numbers|linkify_phabricator_revision_urls|linkify_faq|linkify_sec_bug_docs|safe }}
                                 [
                                 {% for instance in warning['instances'] %}
                                     {{
                                     ", " if not loop.first else ""
-                                    }}{{ instance['revision_id'] | linkify_revision_ids | safe }}
+                                    }}{{ instance['revision_id'] | linkify_phabricator_revision_ids | safe }}
                                 {% endfor %}
                                 ]
                             </label>

src/lando/ui/tests/test_template_helpers.py

@@ -8,13 +8,13 @@
     build_manual_uplift_instructions,
     linkify_bug_numbers,
     linkify_faq,
-    linkify_revision_ids,
-    linkify_revision_urls,
+    linkify_phabricator_revision_ids,
+    linkify_phabricator_revision_urls,
     linkify_sec_bug_docs,
     linkify_transplant_details,
+    phabricator_revision_url,
     repo_branch_url,
     repo_path,
-    revision_url,
 )
 from lando.main.models import SCM_TYPE_GIT, SCM_TYPE_HG, JobStatus, LandingJob, Repo
 from lando.main.models.revision import Revision
@@ -122,8 +122,8 @@ def test_linkify_bug_numbers(input_text, output_text):
         ),
     ],
 )
-def test_linkify_revision_urls(input_text, output_text):
-    assert output_text == linkify_revision_urls(input_text)
+def test_linkify_phabricator_revision_urls(input_text, output_text):
+    assert output_text == linkify_phabricator_revision_urls(input_text)
 
 
 @pytest.mark.parametrize(
@@ -150,8 +150,8 @@ def test_linkify_revision_urls(input_text, output_text):
         ),
     ],
 )
-def test_linkify_revision_ids(input_text, output_text):
-    assert output_text == linkify_revision_ids(input_text)
+def test_linkify_phabricator_revision_ids(input_text, output_text):
+    assert output_text == linkify_phabricator_revision_ids(input_text)
 
 
 @pytest.mark.django_db
@@ -280,29 +280,29 @@ def test_repo_branch_url(repo, path):
 def test_revision_url__integer():
     revision_id = 1234
     expected_result = "http://phabricator.test/D1234"
-    actual_result = revision_url(revision_id)
+    actual_result = phabricator_revision_url(revision_id)
     assert expected_result == actual_result
 
 
 def test_revision_url__prepended_string():
     revision_id = "D1234"
     expected_result = "http://phabricator.test/D1234"
-    actual_result = revision_url(revision_id)
+    actual_result = phabricator_revision_url(revision_id)
     assert expected_result == actual_result
 
 
 def test_revision_url__string():
     revision_id = "1234"
     expected_result = "http://phabricator.test/D1234"
-    actual_result = revision_url(revision_id)
+    actual_result = phabricator_revision_url(revision_id)
     assert expected_result == actual_result
 
 
 def test_revision_url__general_case_with_diff():
     revision_id = 123
     diff_id = 456
     expected_result = "http://phabricator.test/D123?id=456"
-    actual_result = revision_url(revision_id, diff_id)
+    actual_result = phabricator_revision_url(revision_id, diff_id)
     assert expected_result == actual_result