From 1c2b82dedd73d9b8161b68f9f152a3664551b8dc Mon Sep 17 00:00:00 2001 From: Jeremy Stretch Date: Thu, 18 Jul 2024 16:02:24 -0400 Subject: [PATCH] Use MultipleChoiceField for event_types --- netbox/extras/forms/bulk_edit.py | 10 +++++----- netbox/extras/forms/model_forms.py | 11 +++++------ 2 files changed, 10 insertions(+), 11 deletions(-) diff --git a/netbox/extras/forms/bulk_edit.py b/netbox/extras/forms/bulk_edit.py index 444b12732..74cf65c32 100644 --- a/netbox/extras/forms/bulk_edit.py +++ b/netbox/extras/forms/bulk_edit.py @@ -1,9 +1,9 @@ from django import forms -from django.contrib.postgres.forms import SimpleArrayField from django.utils.translation import gettext_lazy as _ from extras.choices import * from extras.models import * +from netbox.events import get_event_type_choices from netbox.forms import NetBoxModelBulkEditForm from utilities.forms import BulkEditForm, add_blank_choice from utilities.forms.fields import ColorField, CommentField, DynamicModelChoiceField @@ -249,10 +249,10 @@ class EventRuleBulkEditForm(NetBoxModelBulkEditForm): required=False, widget=BulkEditNullBooleanSelect() ) - event_types = SimpleArrayField( - label=_('Event types'), - base_field=forms.CharField(), - required=False + event_types = forms.MultipleChoiceField( + choices=get_event_type_choices(), + required=False, + label=_('Event types') ) description = forms.CharField( label=_('Description'), diff --git a/netbox/extras/forms/model_forms.py b/netbox/extras/forms/model_forms.py index 34b72d247..b4221b960 100644 --- a/netbox/extras/forms/model_forms.py +++ b/netbox/extras/forms/model_forms.py @@ -2,7 +2,6 @@ import json import re from django import forms -from django.contrib.postgres.forms import SimpleArrayField from django.utils.safestring import mark_safe from django.utils.translation import gettext_lazy as _ @@ -11,6 +10,7 @@ from core.models import ObjectType from dcim.models import DeviceRole, DeviceType, Location, Platform, Region, Site, SiteGroup from extras.choices import * from extras.models import * +from netbox.events import get_event_type_choices from netbox.forms import NetBoxModelForm from tenancy.models import Tenant, TenantGroup from users.models import Group, User @@ -304,9 +304,9 @@ class EventRuleForm(NetBoxModelForm): label=_('Object types'), queryset=ObjectType.objects.with_feature('event_rules'), ) - event_types = SimpleArrayField( - label=_('Event types'), - base_field=forms.CharField() + event_types = forms.MultipleChoiceField( + choices=get_event_type_choices(), + label=_('Event types') ) action_choice = forms.ChoiceField( label=_('Action choice'), @@ -324,8 +324,7 @@ class EventRuleForm(NetBoxModelForm): fieldsets = ( FieldSet('name', 'description', 'object_types', 'enabled', 'tags', name=_('Event Rule')), - FieldSet('event_types', name=_('Event Types')), - FieldSet('conditions', name=_('Conditions')), + FieldSet('event_types', 'conditions', name=_('Triggers')), FieldSet('action_type', 'action_choice', 'action_data', name=_('Action')), )