Commit c92b6887 authored by Hugo LEVY-FALK's avatar Hugo LEVY-FALK Committed by chirac
Browse files

Fix radius options, force to set a vlan when adding a policy which requires it.

parent 3443965e
...@@ -231,6 +231,27 @@ class EditRadiusOptionForm(ModelForm): ...@@ -231,6 +231,27 @@ class EditRadiusOptionForm(ModelForm):
model = RadiusOption model = RadiusOption
fields = '__all__' fields = '__all__'
def clean(self):
cleaned_data = super().clean()
ignored=('radius_general_policy', 'vlan_decision_ok')
fields = (
f for f in self.fields.keys()
if 'vlan' not in f and f not in ignored
for f in fields:
choice = cleaned_data.get(f)
vlan = cleaned_data.get(f+'_vlan')
if choice == RadiusOption.SET_VLAN and vlan is None:
_("You chose to set vlan but did not set any VLAN."),
_("Please, choose a VLAN."),
return cleaned_data
class ServiceForm(ModelForm): class ServiceForm(ModelForm):
"""Edition, ajout de services sur la page d'accueil""" """Edition, ajout de services sur la page d'accueil"""
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment