nofusscomputing/centurion_erp
1
0
Fork 0
Code Issues Wiki Activity

feat(itam): Change model to inherit from CenturionModel for OperatingSystemVersion model

Browse Source 
ref: #823 #819 closes #588
This commit is contained in:
Jon Lockwood
2025-06-12 11:19:12 +09:30
parent 4a38f73ddf
commit 580df01cc5
2 changed files with 143 additions and 78 deletions
Download Patch File Download Diff File Expand all files Collapse all files

132
app/itam/migrations/0020_remove_operatingsystemversion_is_global_and_more.py Normal file
View File

@ -0,0 +1,132 @@
# Generated by Django 5.1.9 on 2025-06-12 01:48
import access.models.tenancy_abstract
import django.db.models.deletion
from django.db import migrations, models
class Migration(migrations.Migration):
dependencies = [
("access", "0016_remove_tenant_slug_alter_tenant_manager_and_more"),
("core", "0033_alter_ticketcommentcategory_parent_and_more"),
("itam", "0019_remove_operatingsystem_is_global_and_more"),
]
operations = [
migrations.RemoveField(
model_name="operatingsystemversion",
name="is_global",
),
migrations.AlterField(
model_name="operatingsystemversion",
name="id",
field=models.AutoField(
help_text="ID of the item",
primary_key=True,
serialize=False,
unique=True,
verbose_name="ID",
),
),
migrations.AlterField(
model_name="operatingsystemversion",
name="model_notes",
field=models.TextField(
blank=True,
help_text="Tid bits of information",
null=True,
verbose_name="Notes",
),
),
migrations.AlterField(
model_name="operatingsystemversion",
name="operating_system",
field=models.ForeignKey(
help_text="Operating system this version applies to",
on_delete=django.db.models.deletion.PROTECT,
to="itam.operatingsystem",
verbose_name="Operating System",
),
),
migrations.AlterField(
model_name="operatingsystemversion",
name="organization",
field=models.ForeignKey(
help_text="Tenant this belongs to",
on_delete=django.db.models.deletion.CASCADE,
related_name="+",
to="access.tenant",
validators=[
access.models.tenancy_abstract.TenancyAbstractModel.validatate_organization_exists
],
verbose_name="Tenant",
),
),
migrations.CreateModel(
name="OperatingSystemVersionAuditHistory",
fields=[
(
"centurionaudit_ptr",
models.OneToOneField(
auto_created=True,
on_delete=django.db.models.deletion.CASCADE,
parent_link=True,
primary_key=True,
serialize=False,
to="core.centurionaudit",
),
),
(
"model",
models.ForeignKey(
help_text="Model this history belongs to",
on_delete=django.db.models.deletion.CASCADE,
related_name="audit_history",
to="itam.operatingsystemversion",
verbose_name="Model",
),
),
],
options={
"verbose_name": "Operating System Version History",
"verbose_name_plural": "Operating System Version Histories",
"db_table": "itam_operatingsystemversion_audithistory",
"managed": True,
},
bases=("core.centurionaudit",),
),
migrations.CreateModel(
name="OperatingSystemVersionCenturionModelNote",
fields=[
(
"centurionmodelnote_ptr",
models.OneToOneField(
auto_created=True,
on_delete=django.db.models.deletion.CASCADE,
parent_link=True,
primary_key=True,
serialize=False,
to="core.centurionmodelnote",
),
),
(
"model",
models.ForeignKey(
help_text="Model this note belongs to",
on_delete=django.db.models.deletion.CASCADE,
related_name="+",
to="itam.operatingsystemversion",
verbose_name="Model",
),
),
],
options={
"verbose_name": "Operating System Version Note",
"verbose_name_plural": "Operating System Version Notes",
"db_table": "itam_operatingsystemversion_centurionmodelnote",
"managed": True,
},
bases=("core.centurionmodelnote",),
),
]

89
app/itam/models/operating_system.py
View File

@ -1,50 +1,12 @@
from django.db import models
from access.fields import AutoLastModifiedField, AutoCreatedField, AutoSlugField
from access.models.tenancy import TenancyObject
from access.fields import AutoLastModifiedField
from core.mixins.history_save import SaveHistory
from core.models.centurion import CenturionModel
from core.models.manufacturer import Manufacturer
class OperatingSystemCommonFields(TenancyObject, models.Model):
class Meta:
abstract = True
id = models.AutoField(
blank=False,
help_text = 'ID of this item',
primary_key=True,
unique=True,
verbose_name = 'ID'
)
created = AutoCreatedField()
# modified = AutoLastModifiedField()
class OperatingSystemFieldsName(OperatingSystemCommonFields):
class Meta:
abstract = True
# name = models.CharField(
# blank = False,
# help_text = 'Name of this item',
# max_length = 50,
# unique = True,
# verbose_name = 'Name'
# )
slug = AutoSlugField()
class OperatingSystem(
CenturionModel
):
@ -181,9 +143,11 @@ class OperatingSystem(
class OperatingSystemVersion(
OperatingSystemCommonFields, SaveHistory
CenturionModel
):
model_tag = 'operating_system_version'
class Meta:
@ -199,7 +163,7 @@ class OperatingSystemVersion(
operating_system = models.ForeignKey(
OperatingSystem,
help_text = 'Operating system this version applies to',
on_delete = models.CASCADE,
on_delete = models.PROTECT,
verbose_name = 'Operating System'
)
@ -261,48 +225,17 @@ class OperatingSystemVersion(
]
def get_url_kwargs(self) -> dict:
def get_url_kwargs(self, many = False ) -> dict:
return {
kwargs = super().get_url_kwargs( many = many )
kwargs.update({
'operating_system_id': self.operating_system.id,
'pk': self.id
}
})
def get_url_kwargs_notes(self) -> dict:
"""Fetch the URL kwargs for model notes
Returns:
dict: notes kwargs required for generating the URL with `reverse`
"""
return {
'operating_system_id': self.operating_system.id,
'model_id': self.id
}
# @property
# def parent_object(self):
# """ Fetch the parent object """
# return self.operating_system
return kwargs
def __str__(self):
return self.operating_system.name + ' ' + self.name
def save_history(self, before: dict, after: dict) -> bool:
from itam.models.operating_system_version_history import OperatingSystemVersionHistory
history = super().save_history(
before = before,
after = after,
history_model = OperatingSystemVersionHistory,
)
return history