NeoAnd/netbox
1
0
Fork 0
mirror of https://github.com/netbox-community/netbox.git synced 2025-07-28 19:36:26 -06:00
Code Issues Actions 8 Packages Projects Releases Wiki Activity

18417 add rack outer height

Browse Source
This commit is contained in:
Arthur 2025-03-18 08:16:41 -07:00
parent d4f8cb72aa
commit 2283f6fdf6
2 changed files with 43 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
netbox/dcim/migrations/0202_add_rack_outer_height.py Normal file
View File

@ -0,0 +1,23 @@
# Generated by Django 5.2b1 on 2025-03-18 15:15
from django.db import migrations, models
class Migration(migrations.Migration):
dependencies = [
('dcim', '0201_add_power_outlet_status'),
]
operations = [
migrations.AddField(
model_name='rack',
name='outer_height',
field=models.PositiveSmallIntegerField(blank=True, null=True),
),
migrations.AddField(
model_name='racktype',
name='outer_height',
field=models.PositiveSmallIntegerField(blank=True, null=True),
),
]

29
netbox/dcim/models/racks.py
View File

@ -73,6 +73,12 @@ class RackBase(WeightMixin, PrimaryModel):
null=True, null=True,
help_text=_('Outer dimension of rack (width)') help_text=_('Outer dimension of rack (width)')
) )
outer_height = models.PositiveSmallIntegerField(
verbose_name=_('outer height'),
blank=True,
null=True,
help_text=_('Outer dimension of rack (height)')
)
outer_depth = models.PositiveSmallIntegerField( outer_depth = models.PositiveSmallIntegerField(
verbose_name=_('outer depth'), verbose_name=_('outer depth'),
blank=True, blank=True,
@ -140,7 +146,7 @@ class RackType(RackBase):
) )
clone_fields = ( clone_fields = (
'manufacturer', 'form_factor', 'width', 'u_height', 'desc_units', 'outer_width', 'outer_depth', 'manufacturer', 'form_factor', 'width', 'u_height', 'desc_units', 'outer_width', 'outer_height', 'outer_depth',
'outer_unit', 'mounting_depth', 'weight', 'max_weight', 'weight_unit', 'outer_unit', 'mounting_depth', 'weight', 'max_weight', 'weight_unit',
) )
prerequisite_models = ( prerequisite_models = (
@ -173,8 +179,10 @@ class RackType(RackBase):
super().clean() super().clean()
# Validate outer dimensions and unit # Validate outer dimensions and unit
if (self.outer_width is not None or self.outer_depth is not None) and not self.outer_unit: if (
raise ValidationError(_("Must specify a unit when setting an outer width/depth")) self.outer_width is not None or self.outer_depth is not None or self.outer_height is not None
) and not self.outer_unit:
raise ValidationError(_("Must specify a unit when setting an outer width/depth/height"))
# Validate max_weight and weight_unit # Validate max_weight and weight_unit
if self.max_weight and not self.weight_unit: if self.max_weight and not self.weight_unit:
@ -188,7 +196,7 @@ class RackType(RackBase):
self._abs_max_weight = None self._abs_max_weight = None
# Clear unit if outer width & depth are not set # Clear unit if outer width & depth are not set
if self.outer_width is None and self.outer_depth is None: if self.outer_width is None and self.outer_depth is None and self.outer_height is None:
self.outer_unit = None self.outer_unit = None
super().save(*args, **kwargs) super().save(*args, **kwargs)
@ -235,8 +243,8 @@ class Rack(ContactsMixin, ImageAttachmentsMixin, RackBase):
""" """
# Fields which cannot be set locally if a RackType is assigned # Fields which cannot be set locally if a RackType is assigned
RACKTYPE_FIELDS = ( RACKTYPE_FIELDS = (
'form_factor', 'width', 'u_height', 'starting_unit', 'desc_units', 'outer_width', 'outer_depth', 'form_factor', 'width', 'u_height', 'starting_unit', 'desc_units', 'outer_width', 'outer_height',
'outer_unit', 'mounting_depth', 'weight', 'weight_unit', 'max_weight', 'outer_depth', 'outer_unit', 'mounting_depth', 'weight', 'weight_unit', 'max_weight',
) )
form_factor = models.CharField( form_factor = models.CharField(
@ -329,7 +337,8 @@ class Rack(ContactsMixin, ImageAttachmentsMixin, RackBase):
clone_fields = ( clone_fields = (
'site', 'location', 'tenant', 'status', 'role', 'form_factor', 'width', 'airflow', 'u_height', 'desc_units', 'site', 'location', 'tenant', 'status', 'role', 'form_factor', 'width', 'airflow', 'u_height', 'desc_units',
'outer_width', 'outer_depth', 'outer_unit', 'mounting_depth', 'weight', 'max_weight', 'weight_unit', 'outer_width', 'outer_height', 'outer_depth', 'outer_unit', 'mounting_depth',
'weight', 'max_weight', 'weight_unit',
) )
prerequisite_models = ( prerequisite_models = (
'dcim.Site', 'dcim.Site',
@ -364,7 +373,9 @@ class Rack(ContactsMixin, ImageAttachmentsMixin, RackBase):
raise ValidationError(_("Assigned location must belong to parent site ({site}).").format(site=self.site)) raise ValidationError(_("Assigned location must belong to parent site ({site}).").format(site=self.site))
# Validate outer dimensions and unit # Validate outer dimensions and unit
if (self.outer_width is not None or self.outer_depth is not None) and not self.outer_unit: if (
self.outer_width is not None or self.outer_depth is not None or self.outer_height is not None
) and not self.outer_unit:
raise ValidationError(_("Must specify a unit when setting an outer width/depth")) raise ValidationError(_("Must specify a unit when setting an outer width/depth"))
# Validate max_weight and weight_unit # Validate max_weight and weight_unit
@ -414,7 +425,7 @@ class Rack(ContactsMixin, ImageAttachmentsMixin, RackBase):
self._abs_max_weight = None self._abs_max_weight = None
# Clear unit if outer width & depth are not set # Clear unit if outer width & depth are not set
if self.outer_width is None and self.outer_depth is None: if self.outer_width is None and self.outer_depth is None and self.outer_height is None:
self.outer_unit = None self.outer_unit = None
super().save(*args, **kwargs) super().save(*args, **kwargs)