initial work to render hierarchical region #13735

netbox/dcim/templatetags/display_region.py

@@ -0,0 +1,39 @@
+from django import template
+from django.utils.safestring import mark_safe
+
+from dcim.models import Site
+
+register = template.Library()
+
+
+@register.simple_tag(takes_context=True)
+def display_region(context, obj):
+    """
+    Renders hierarchical region data for a given object.
+    """
+    # Check if the obj is an instance of Site
+    if isinstance(obj, Site):
+        if not obj.region:
+            return mark_safe('—')
+
+        # If so, retrieve the Site's Region
+        region = obj.region
+    else:
+        if not hasattr(obj, 'site'):
+            return mark_safe('—')
+
+        # Otherwise, retrieve the Region from the Site associated with the object
+        region = obj.site.region
+
+    # Retrieve all regions in the hierarchy
+    regions = region.get_ancestors(include_self=True)
+
+    # Render the hierarchy as a list of links
+    return mark_safe(
+        ' / '.join([
+            '<a href="{}">{}</a>'.format(
+                context['request'].build_absolute_uri(region.get_absolute_url()),
+                region
+            ) for region in regions
+        ])
+    )

netbox/templates/dcim/device.html

@@ -5,6 +5,7 @@
 {% load helpers %}
 {% load plugins %}
 {% load i18n %}
+{% load display_region %}
 
 {% block content %}
     <div class="row">
@@ -16,14 +17,7 @@
                         <tr>
                             <th scope="row">{% trans "Region" %}</th>
                             <td>
-                                {% if object.site.region %}
+                                {% display_region object %}
-                                    {% for region in object.site.region.get_ancestors %}
-                                        {{ region|linkify }} /
-                                    {% endfor %}
-                                    {{ object.site.region|linkify }}
-                                {% else %}
-                                    {{ ''|placeholder }}
-                                {% endif %}
                             </td>
                         </tr>
                         <tr>

netbox/templates/dcim/site.html

@@ -3,6 +3,7 @@
 {% load plugins %}
 {% load tz %}
 {% load i18n %}
+{% load display_region %}
 
 {% block breadcrumbs %}
   {{ block.super }}
@@ -29,14 +30,7 @@
           <tr>
             <th scope="row">{% trans "Region" %}</th>
             <td>
-              {% if object.region %}
+              {% display_region object %}
-                {% for region in object.region.get_ancestors %}
-                  {{ region|linkify }} /
-                {% endfor %}
-                {{ object.region|linkify }}
-              {% else %}
-                {{ ''|placeholder }}
-              {% endif %}
             </td>
           </tr>
           <tr>