Device: safe long type names directly in database
This commit is contained in:
parent
c6b1badf39
commit
832bb48983
|
@ -0,0 +1,18 @@
|
||||||
|
# Generated by Django 5.1.2 on 2024-11-09 22:38
|
||||||
|
|
||||||
|
from django.db import migrations, models
|
||||||
|
|
||||||
|
|
||||||
|
class Migration(migrations.Migration):
|
||||||
|
|
||||||
|
dependencies = [
|
||||||
|
('games', '0038_alter_purchase_price'),
|
||||||
|
]
|
||||||
|
|
||||||
|
operations = [
|
||||||
|
migrations.AlterField(
|
||||||
|
model_name='device',
|
||||||
|
name='type',
|
||||||
|
field=models.CharField(choices=[('PC', 'PC'), ('Console', 'Console'), ('Handheld', 'Handheld'), ('Mobile', 'Mobile'), ('Single-board computer', 'Single-board computer'), ('Unknown', 'Unknown')], default='Unknown', max_length=255),
|
||||||
|
),
|
||||||
|
]
|
|
@ -0,0 +1,33 @@
|
||||||
|
# Generated by Django 5.1.2 on 2024-11-09 22:39
|
||||||
|
|
||||||
|
from django.db import migrations
|
||||||
|
|
||||||
|
|
||||||
|
def update_device_types(apps, schema_editor):
|
||||||
|
Device = apps.get_model("games", "Device")
|
||||||
|
|
||||||
|
# Mapping of short names to long names
|
||||||
|
type_map = {
|
||||||
|
"pc": "PC",
|
||||||
|
"co": "Console",
|
||||||
|
"ha": "Handheld",
|
||||||
|
"mo": "Mobile",
|
||||||
|
"sbc": "Single-board computer",
|
||||||
|
"un": "Unknown",
|
||||||
|
}
|
||||||
|
|
||||||
|
# Loop through all devices and update the type field
|
||||||
|
for device in Device.objects.all():
|
||||||
|
if device.type in type_map:
|
||||||
|
device.type = type_map[device.type]
|
||||||
|
device.save()
|
||||||
|
|
||||||
|
|
||||||
|
class Migration(migrations.Migration):
|
||||||
|
dependencies = [
|
||||||
|
("games", "0039_alter_device_type"),
|
||||||
|
]
|
||||||
|
|
||||||
|
operations = [
|
||||||
|
migrations.RunPython(update_device_types),
|
||||||
|
]
|
|
@ -259,12 +259,12 @@ class Session(models.Model):
|
||||||
|
|
||||||
|
|
||||||
class Device(models.Model):
|
class Device(models.Model):
|
||||||
PC = "pc"
|
PC = "PC"
|
||||||
CONSOLE = "co"
|
CONSOLE = "Console"
|
||||||
HANDHELD = "ha"
|
HANDHELD = "Handheld"
|
||||||
MOBILE = "mo"
|
MOBILE = "Mobile"
|
||||||
SBC = "sbc"
|
SBC = "Single-board computer"
|
||||||
UNKNOWN = "un"
|
UNKNOWN = "Unknown"
|
||||||
DEVICE_TYPES = [
|
DEVICE_TYPES = [
|
||||||
(PC, "PC"),
|
(PC, "PC"),
|
||||||
(CONSOLE, "Console"),
|
(CONSOLE, "Console"),
|
||||||
|
@ -274,8 +274,8 @@ class Device(models.Model):
|
||||||
(UNKNOWN, "Unknown"),
|
(UNKNOWN, "Unknown"),
|
||||||
]
|
]
|
||||||
name = models.CharField(max_length=255)
|
name = models.CharField(max_length=255)
|
||||||
type = models.CharField(max_length=3, choices=DEVICE_TYPES, default=UNKNOWN)
|
type = models.CharField(max_length=255, choices=DEVICE_TYPES, default=UNKNOWN)
|
||||||
created_at = models.DateTimeField(auto_now_add=True)
|
created_at = models.DateTimeField(auto_now_add=True)
|
||||||
|
|
||||||
def __str__(self):
|
def __str__(self):
|
||||||
return f"{self.name} ({self.get_type_display()})"
|
return f"{self.name} ({self.type})"
|
||||||
|
|
Loading…
Reference in New Issue