nofusscomputing/centurion_erp
1
0
Fork 0
Code Issues Wiki Activity

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

Browse Source 
ref: #823 #822
This commit is contained in:
Jon Lockwood
2025-06-12 12:48:08 +09:30
parent cb0d095eb7
commit ae4b2a3c04
2 changed files with 136 additions and 62 deletions
Download Patch File Download Diff File Expand all files Collapse all files

126
app/itam/migrations/0023_remove_softwareversion_is_global_and_more.py Normal file
View File

@ -0,0 +1,126 @@
# Generated by Django 5.1.9 on 2025-06-12 03:13
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", "0022_remove_softwarecategory_is_global_and_more"),
]
operations = [
migrations.RemoveField(
model_name="softwareversion",
name="is_global",
),
migrations.RemoveField(
model_name="softwareversion",
name="slug",
),
migrations.AlterField(
model_name="softwareversion",
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="softwareversion",
name="model_notes",
field=models.TextField(
blank=True,
help_text="Tid bits of information",
null=True,
verbose_name="Notes",
),
),
migrations.AlterField(
model_name="softwareversion",
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="SoftwareVersionAuditHistory",
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.softwareversion",
verbose_name="Model",
),
),
],
options={
"verbose_name": "Software Version History",
"verbose_name_plural": "Software Version Histories",
"db_table": "itam_softwareversion_audithistory",
"managed": True,
},
bases=("core.centurionaudit",),
),
migrations.CreateModel(
name="SoftwareVersionCenturionModelNote",
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.softwareversion",
verbose_name="Model",
),
),
],
options={
"verbose_name": "Software Version Note",
"verbose_name_plural": "Software Version Notes",
"db_table": "itam_softwareversion_centurionmodelnote",
"managed": True,
},
bases=("core.centurionmodelnote",),
),
]

72
app/itam/models/software.py
View File

@ -1,33 +1,13 @@
from django.db import models
from access.fields import *
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
from settings.models.app_settings import AppSettings
class SoftwareCommonFields(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'
)
slug = AutoSlugField()
created = AutoCreatedField()
class SoftwareCategory(
CenturionModel,
@ -285,9 +265,11 @@ class Software(
class SoftwareVersion(
SoftwareCommonFields, SaveHistory
CenturionModel
):
model_tag = 'software_version'
class Meta:
@ -320,8 +302,6 @@ class SoftwareVersion(
modified = AutoLastModifiedField()
# model does not have it's own page
# as it's a secondary model.
page_layout: list = [
{
"name": "Details",
@ -379,49 +359,17 @@ class SoftwareVersion(
]
def get_url_kwargs(self) -> dict:
def get_url_kwargs(self, many = False) -> dict:
return {
kwargs = super().get_url_kwargs( many = many)
kwargs.update({
'software_id': self.software.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 {
'software_id': self.software.id,
'model_id': self.id
}
# @property
# def parent_object(self):
# """ Fetch the parent object """
# return self.software
return kwargs
def __str__(self):
return self.software.name + ' ' + self.name
def save_history(self, before: dict, after: dict) -> bool:
from itam.models.software_version_history import SoftwareVersionHistory
history = super().save_history(
before = before,
after = after,
history_model = SoftwareVersionHistory,
)
return history