Closes #17166: Remove obsolete limit_choices_to argument from ForeignKey & M2M fields

This commit is contained in:
Jeremy Stretch
2025-04-03 09:17:20 -04:00
parent 6a966ee6c1
commit d93d398afa
30 changed files with 2 additions and 224 deletions

View File

@@ -4,7 +4,6 @@ import taggit.managers
from django.db import migrations, models
import utilities.json
import wireless.models
class Migration(migrations.Migration):
@@ -149,7 +148,6 @@ class Migration(migrations.Migration):
(
'interface_a',
models.ForeignKey(
limit_choices_to=wireless.models.get_wireless_interface_types,
on_delete=django.db.models.deletion.PROTECT,
related_name='+',
to='dcim.interface',
@@ -158,7 +156,6 @@ class Migration(migrations.Migration):
(
'interface_b',
models.ForeignKey(
limit_choices_to=wireless.models.get_wireless_interface_types,
on_delete=django.db.models.deletion.PROTECT,
related_name='+',
to='dcim.interface',

View File

@@ -66,7 +66,6 @@ class Migration(migrations.Migration):
name='scope_type',
field=models.ForeignKey(
blank=True,
limit_choices_to=models.Q(('model__in', ('region', 'sitegroup', 'site', 'location'))),
null=True,
on_delete=django.db.models.deletion.PROTECT,
related_name='+',

View File

@@ -123,26 +123,18 @@ class WirelessLAN(WirelessAuthenticationBase, CachedScopeMixin, PrimaryModel):
return WirelessLANStatusChoices.colors.get(self.status)
def get_wireless_interface_types():
# Wrap choices in a callable to avoid generating dummy migrations
# when the choices are updated.
return {'type__in': WIRELESS_IFACE_TYPES}
class WirelessLink(WirelessAuthenticationBase, DistanceMixin, PrimaryModel):
"""
A point-to-point connection between two wireless Interfaces.
"""
interface_a = models.ForeignKey(
to='dcim.Interface',
limit_choices_to=get_wireless_interface_types,
on_delete=models.PROTECT,
related_name='+',
verbose_name=_('interface A'),
)
interface_b = models.ForeignKey(
to='dcim.Interface',
limit_choices_to=get_wireless_interface_types,
on_delete=models.PROTECT,
related_name='+',
verbose_name=_('interface B'),