mirror/netbox-community-netbox
1
0
Fork 0
mirror of https://github.com/netbox-community/netbox.git synced 2024-05-10 07:54:54 +00:00
Code Activity

Apply suggestions from code review

Browse Source 
Co-authored-by: Jeremy Stretch <jstretch@netboxlabs.com>
This commit is contained in:
Daniel Sheppard
2024-03-21 22:18:34 -05:00
committed by GitHub
parent 25a4e9448c
commit f257f4aad4
4 changed files with 10 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
netbox/netbox/views/generic/bulk_views.py
View File

@@ -162,9 +162,8 @@ class ObjectListView(BaseMultiObjectView, ActionsMixin, TableMixin):
table = self.get_table(self.queryset, request, has_bulk_actions)
# Check for filterset_form on this view, if a form exists, apply to context and table, otherwise set to None
filterset_form = None
if hasattr(self, 'filterset_form') and self.filterset_form:
filterset_form = self.filterset_form(request.GET, label_suffix='')
if self.filterset_form:
filterset_form = self.filterset_form(request.GET)
table.filterset_form = filterset_form
# If this is an HTMX request, return only the rendered table HTML

3
netbox/templates/inc/table_header_filter_dropdown.html
View File

@@ -1,7 +1,8 @@
{% load form_helpers %}
{% if form_field %}
<div class="column-filter dropdown">
<a href="#" classs="btn dropdown-toggle" data-bs-toggle="dropdown" data-bs-auto-close="outside">
<a href="#" class="btn dropdown-toggle" data-bs-toggle="dropdown" data-bs-auto-close="outside">
<i class="mdi mdi-filter-settings" style="font-size: 1.25rem;"> </i>
</a>
<div class="dropdown-menu">

8
netbox/templates/inc/table_htmx.html
View File

@@ -17,8 +17,8 @@
><i class="mdi mdi-close"></i></a>
</div>
{% endif %}
{% if table.filterset_form %}
{% include 'inc/table_header_filter_dropdown.html' with form_field=table.filterset_form|getfilterfield:column.name %}
{% if table.filterset_form %}
{% include 'inc/table_header_filter_dropdown.html' with form_field=table.filterset_form|get_filter_field:column.name %}
{% endif %}
<a href="#"
hx-get="{{ table.htmx_url }}{% querystring table.prefixed_order_by_field=column.order_by_alias.next %}"
@@ -28,8 +28,8 @@
</th>
{% else %}
<th {{ column.attrs.th.as_html }}>
{% if table.filterset_form %}
{% include 'inc/table_header_filter_dropdown.html' with form_field=table.filterset_form|getfilterfield:column.name %}
{% if table.filterset_form %}
{% include 'inc/table_header_filter_dropdown.html' with form_field=table.filterset_form|get_filter_field:column.name %}
{% endif %}
{{ column.header }}
</th>

4
netbox/utilities/templatetags/form_helpers.py
View File

@@ -6,7 +6,7 @@ from utilities.forms.rendering import FieldSet, InlineFields, ObjectAttribute, T
__all__ = (
'getfield',
'getfilterfield',
'get_filter_field',
'render_custom_fields',
'render_errors',
'render_field',
@@ -35,7 +35,7 @@ def getfield(form, fieldname):
@register.filter()
def getfilterfield(form, fieldname):
def get_filter_field(form, fieldname):
field = getfield(form, f'{fieldname}')
if field is not None:
return field