feat: annotation changes (index, delete, etc.)

Server/DB/Documents/Account.cs

@@ -23,14 +23,13 @@ public class Account
     [DatabaseGenerated(DatabaseGeneratedOption.Identity)]
     public Guid Id { get; set; }
 
-    [Column(TypeName = "varchar(20)")]
-    [MaxLength(20)]
-    public string Username { get; set; }
+    [MaxLength(20)] public string Username { get; set; }
 
     [Column(TypeName = "bytea")] public byte[] Password { get; set; }
 
     [EmailAddress] public string Email { get; set; }
     public byte PermissionLevel { get; set; }
     [Column(TypeName = "bytea")] public byte[] Salt { get; set; }
+
     public virtual ICollection<Character> Characters { get; } = new List<Character>();
 }

Server/DB/Documents/Character.cs

@@ -12,6 +12,7 @@ namespace Server.DB.Documents;
 [Index(nameof(Id), IsUnique = true)]
 public class Character
 {
+    [DeleteBehavior(DeleteBehavior.Restrict)]
     public virtual Account Account { get; set; }
 
     [Key]
@@ -20,7 +21,6 @@ public class Character
 
     public ushort MapId { get; set; }
 
-    [Column(TypeName = "varchar(20)")]
     [MaxLength(20)]
     public string Name { get; set; }
 
@@ -38,5 +38,7 @@ public class Character
     public JobData JobData { get; set; }
     public int Health { get; set; }
     public int Mana { get; set; }
+
+    [DeleteBehavior(DeleteBehavior.Restrict)]
     public virtual Guild Guild { get; set; }
 }

Server/DB/Documents/Guild.cs

@@ -2,16 +2,21 @@
 
 using System.ComponentModel.DataAnnotations;
 using System.ComponentModel.DataAnnotations.Schema;
+using Microsoft.EntityFrameworkCore;
 
 namespace Server.DB.Documents;
 
+[Index(nameof(Name), IsUnique = true)]
+[Index(nameof(Id), IsUnique = true)]
 public class Guild
 {
     [Key]
     [DatabaseGenerated(DatabaseGeneratedOption.Identity)]
     public Guid Id { get; set; }
 
-    public string Name { get; set; }
+    [MaxLength(16)] public string Name { get; set; }
     public string Notice { get; set; }
+
+    [DeleteBehavior(DeleteBehavior.Cascade)]
     public virtual ICollection<GuildMember> GuildMembers { get; set; }
 }

Server/DB/Documents/GuildMember.cs

@@ -2,16 +2,19 @@
 
 using System.ComponentModel.DataAnnotations;
 using System.ComponentModel.DataAnnotations.Schema;
+using Microsoft.EntityFrameworkCore;
 
 namespace Server.DB.Documents;
 
+[Index(nameof(Id), IsUnique = true)]
 public class GuildMember
 {
     [Key]
     [DatabaseGenerated(DatabaseGeneratedOption.Identity)]
     public Guid Id { get; set; }
-
+    [DeleteBehavior(DeleteBehavior.Restrict)]
     public virtual Character Character { get; set; }
+    [DeleteBehavior(DeleteBehavior.Restrict)]
     public virtual Guild Guild { get; set; }
     public GuildRank Rank { get; set; }
 }

Server/DB/Migrations/20231122064508_IndexingAndVariousOtherAnnotations.Designer.cs

@@ -0,0 +1,352 @@
+// <auto-generated />
+using System;
+using Microsoft.EntityFrameworkCore;
+using Microsoft.EntityFrameworkCore.Infrastructure;
+using Microsoft.EntityFrameworkCore.Migrations;
+using Microsoft.EntityFrameworkCore.Storage.ValueConversion;
+using Npgsql.EntityFrameworkCore.PostgreSQL.Metadata;
+using Server.DB;
+
+#nullable disable
+
+namespace Server.DB.Migrations
+{
+    [DbContext(typeof(WonderkingContext))]
+    [Migration("20231122064508_IndexingAndVariousOtherAnnotations")]
+    partial class IndexingAndVariousOtherAnnotations
+    {
+        /// <inheritdoc />
+        protected override void BuildTargetModel(ModelBuilder modelBuilder)
+        {
+#pragma warning disable 612, 618
+            modelBuilder
+                .HasAnnotation("ProductVersion", "8.0.0")
+                .HasAnnotation("Proxies:ChangeTracking", false)
+                .HasAnnotation("Proxies:CheckEquality", false)
+                .HasAnnotation("Proxies:LazyLoading", true)
+                .HasAnnotation("Relational:MaxIdentifierLength", 63);
+
+            NpgsqlModelBuilderExtensions.UseIdentityByDefaultColumns(modelBuilder);
+
+            modelBuilder.Entity("Server.DB.Documents.Account", b =>
+                {
+                    b.Property<Guid>("Id")
+                        .ValueGeneratedOnAdd()
+                        .HasColumnType("uuid");
+
+                    b.Property<string>("Email")
+                        .HasColumnType("text");
+
+                    b.Property<byte[]>("Password")
+                        .HasColumnType("bytea");
+
+                    b.Property<byte>("PermissionLevel")
+                        .HasColumnType("smallint");
+
+                    b.Property<byte[]>("Salt")
+                        .HasColumnType("bytea");
+
+                    b.Property<string>("Username")
+                        .HasMaxLength(20)
+                        .HasColumnType("character varying(20)");
+
+                    b.HasKey("Id");
+
+                    b.HasIndex("Id")
+                        .IsUnique();
+
+                    b.HasIndex("Username")
+                        .IsUnique();
+
+                    b.ToTable("Accounts");
+                });
+
+            modelBuilder.Entity("Server.DB.Documents.Character", b =>
+                {
+                    b.Property<Guid>("Id")
+                        .ValueGeneratedOnAdd()
+                        .HasColumnType("uuid");
+
+                    b.Property<Guid?>("AccountId")
+                        .HasColumnType("uuid");
+
+                    b.Property<long>("Experience")
+                        .HasColumnType("bigint");
+
+                    b.Property<byte>("Gender")
+                        .HasColumnType("smallint");
+
+                    b.Property<Guid?>("GuildId")
+                        .HasColumnType("uuid");
+
+                    b.Property<int>("Health")
+                        .HasColumnType("integer");
+
+                    b.Property<short>("LastXCoordinate")
+                        .HasColumnType("smallint");
+
+                    b.Property<short>("LastYCoordinate")
+                        .HasColumnType("smallint");
+
+                    b.Property<byte>("Level")
+                        .HasColumnType("smallint");
+
+                    b.Property<int>("Mana")
+                        .HasColumnType("integer");
+
+                    b.Property<int>("MapId")
+                        .HasColumnType("integer");
+
+                    b.Property<string>("Name")
+                        .HasMaxLength(20)
+                        .HasColumnType("character varying(20)");
+
+                    b.Property<byte>("PvPLevel")
+                        .HasColumnType("smallint");
+
+                    b.HasKey("Id");
+
+                    b.HasIndex("AccountId");
+
+                    b.HasIndex("GuildId");
+
+                    b.HasIndex("Id")
+                        .IsUnique();
+
+                    b.HasIndex("Name")
+                        .IsUnique();
+
+                    b.ToTable("Characters");
+                });
+
+            modelBuilder.Entity("Server.DB.Documents.Guild", b =>
+                {
+                    b.Property<Guid>("Id")
+                        .ValueGeneratedOnAdd()
+                        .HasColumnType("uuid");
+
+                    b.Property<string>("Name")
+                        .HasMaxLength(16)
+                        .HasColumnType("character varying(16)");
+
+                    b.Property<string>("Notice")
+                        .HasColumnType("text");
+
+                    b.HasKey("Id");
+
+                    b.HasIndex("Id")
+                        .IsUnique();
+
+                    b.HasIndex("Name")
+                        .IsUnique();
+
+                    b.ToTable("Guilds");
+                });
+
+            modelBuilder.Entity("Server.DB.Documents.GuildMember", b =>
+                {
+                    b.Property<Guid>("Id")
+                        .ValueGeneratedOnAdd()
+                        .HasColumnType("uuid");
+
+                    b.Property<Guid?>("CharacterId")
+                        .HasColumnType("uuid");
+
+                    b.Property<Guid?>("GuildId")
+                        .HasColumnType("uuid");
+
+                    b.Property<byte>("Rank")
+                        .HasColumnType("smallint");
+
+                    b.HasKey("Id");
+
+                    b.HasIndex("CharacterId");
+
+                    b.HasIndex("GuildId");
+
+                    b.HasIndex("Id")
+                        .IsUnique();
+
+                    b.ToTable("GuildMember");
+                });
+
+            modelBuilder.Entity("Server.DB.Documents.InventoryItem", b =>
+                {
+                    b.Property<Guid>("Id")
+                        .ValueGeneratedOnAdd()
+                        .HasColumnType("uuid");
+
+                    b.Property<byte>("AddOption")
+                        .HasColumnType("smallint");
+
+                    b.Property<byte>("AddOption2")
+                        .HasColumnType("smallint");
+
+                    b.Property<byte>("AddOption3")
+                        .HasColumnType("smallint");
+
+                    b.Property<Guid?>("CharacterId")
+                        .HasColumnType("uuid");
+
+                    b.Property<int>("Count")
+                        .HasColumnType("integer");
+
+                    b.Property<byte>("InventoryTab")
+                        .HasColumnType("smallint");
+
+                    b.Property<int>("ItemId")
+                        .HasColumnType("integer");
+
+                    b.Property<byte>("Level")
+                        .HasColumnType("smallint");
+
+                    b.Property<short>("Option")
+                        .HasColumnType("smallint");
+
+                    b.Property<short>("Option2")
+                        .HasColumnType("smallint");
+
+                    b.Property<short>("Option3")
+                        .HasColumnType("smallint");
+
+                    b.Property<byte>("Rarity")
+                        .HasColumnType("smallint");
+
+                    b.Property<byte>("Slot")
+                        .HasColumnType("smallint");
+
+                    b.HasKey("Id");
+
+                    b.HasIndex("CharacterId");
+
+                    b.ToTable("InventoryItems");
+                });
+
+            modelBuilder.Entity("Server.DB.Documents.Character", b =>
+                {
+                    b.HasOne("Server.DB.Documents.Account", "Account")
+                        .WithMany("Characters")
+                        .HasForeignKey("AccountId")
+                        .OnDelete(DeleteBehavior.Restrict);
+
+                    b.HasOne("Server.DB.Documents.Guild", "Guild")
+                        .WithMany()
+                        .HasForeignKey("GuildId")
+                        .OnDelete(DeleteBehavior.Restrict);
+
+                    b.OwnsOne("Wonderking.Packets.Outgoing.Data.BaseStats", "BaseStats", b1 =>
+                        {
+                            b1.Property<Guid>("CharacterId")
+                                .HasColumnType("uuid");
+
+                            b1.Property<short>("Dexterity")
+                                .HasColumnType("smallint")
+                                .HasAnnotation("Relational:JsonPropertyName", "dexterity");
+
+                            b1.Property<short>("Intelligence")
+                                .HasColumnType("smallint")
+                                .HasAnnotation("Relational:JsonPropertyName", "intelligence");
+
+                            b1.Property<short>("Luck")
+                                .HasColumnType("smallint")
+                                .HasAnnotation("Relational:JsonPropertyName", "luck");
+
+                            b1.Property<short>("Strength")
+                                .HasColumnType("smallint")
+                                .HasAnnotation("Relational:JsonPropertyName", "strength");
+
+                            b1.Property<short>("Vitality")
+                                .HasColumnType("smallint")
+                                .HasAnnotation("Relational:JsonPropertyName", "vitality");
+
+                            b1.Property<short>("Wisdom")
+                                .HasColumnType("smallint")
+                                .HasAnnotation("Relational:JsonPropertyName", "wisdom");
+
+                            b1.HasKey("CharacterId");
+
+                            b1.ToTable("Characters");
+
+                            b1.WithOwner()
+                                .HasForeignKey("CharacterId");
+                        });
+
+                    b.OwnsOne("Wonderking.Packets.Outgoing.Data.JobData", "JobData", b1 =>
+                        {
+                            b1.Property<Guid>("CharacterId")
+                                .HasColumnType("uuid");
+
+                            b1.Property<byte>("FirstJob")
+                                .HasColumnType("smallint");
+
+                            b1.Property<byte>("FourthJob")
+                                .HasColumnType("smallint");
+
+                            b1.Property<byte>("SecondJob")
+                                .HasColumnType("smallint");
+
+                            b1.Property<byte>("ThirdJob")
+                                .HasColumnType("smallint");
+
+                            b1.HasKey("CharacterId");
+
+                            b1.ToTable("Characters");
+
+                            b1.WithOwner()
+                                .HasForeignKey("CharacterId");
+                        });
+
+                    b.Navigation("Account");
+
+                    b.Navigation("BaseStats");
+
+                    b.Navigation("Guild");
+
+                    b.Navigation("JobData");
+                });
+
+            modelBuilder.Entity("Server.DB.Documents.GuildMember", b =>
+                {
+                    b.HasOne("Server.DB.Documents.Character", "Character")
+                        .WithMany()
+                        .HasForeignKey("CharacterId")
+                        .OnDelete(DeleteBehavior.Restrict);
+
+                    b.HasOne("Server.DB.Documents.Guild", "Guild")
+                        .WithMany("GuildMembers")
+                        .HasForeignKey("GuildId")
+                        .OnDelete(DeleteBehavior.Cascade);
+
+                    b.Navigation("Character");
+
+                    b.Navigation("Guild");
+                });
+
+            modelBuilder.Entity("Server.DB.Documents.InventoryItem", b =>
+                {
+                    b.HasOne("Server.DB.Documents.Character", "Character")
+                        .WithMany("InventoryItems")
+                        .HasForeignKey("CharacterId")
+                        .OnDelete(DeleteBehavior.Cascade);
+
+                    b.Navigation("Character");
+                });
+
+            modelBuilder.Entity("Server.DB.Documents.Account", b =>
+                {
+                    b.Navigation("Characters");
+                });
+
+            modelBuilder.Entity("Server.DB.Documents.Character", b =>
+                {
+                    b.Navigation("InventoryItems");
+                });
+
+            modelBuilder.Entity("Server.DB.Documents.Guild", b =>
+                {
+                    b.Navigation("GuildMembers");
+                });
+#pragma warning restore 612, 618
+        }
+    }
+}

Server/DB/Migrations/20231122064508_IndexingAndVariousOtherAnnotations.cs

@@ -0,0 +1,205 @@
+// Copyright (c) 2023 Timothy Schenk. Subject to the GNU AGPL Version 3 License.
+
+using Microsoft.EntityFrameworkCore.Migrations;
+
+#nullable disable
+
+namespace Server.DB.Migrations;
+
+/// <inheritdoc />
+public partial class IndexingAndVariousOtherAnnotations : Migration
+{
+    /// <inheritdoc />
+    protected override void Up(MigrationBuilder migrationBuilder)
+    {
+        migrationBuilder.DropForeignKey(
+            name: "FK_Characters_Accounts_AccountId",
+            table: "Characters");
+
+        migrationBuilder.DropForeignKey(
+            name: "FK_Characters_Guilds_GuildId",
+            table: "Characters");
+
+        migrationBuilder.DropForeignKey(
+            name: "FK_GuildMember_Characters_CharacterId",
+            table: "GuildMember");
+
+        migrationBuilder.DropForeignKey(
+            name: "FK_GuildMember_Guilds_GuildId",
+            table: "GuildMember");
+
+        migrationBuilder.AlterColumn<string>(
+            name: "Name",
+            table: "Guilds",
+            type: "character varying(16)",
+            maxLength: 16,
+            nullable: true,
+            oldClrType: typeof(string),
+            oldType: "text",
+            oldNullable: true);
+
+        migrationBuilder.AlterColumn<string>(
+            name: "Name",
+            table: "Characters",
+            type: "character varying(20)",
+            maxLength: 20,
+            nullable: true,
+            oldClrType: typeof(string),
+            oldType: "varchar(20)",
+            oldMaxLength: 20,
+            oldNullable: true);
+
+        migrationBuilder.AlterColumn<string>(
+            name: "Username",
+            table: "Accounts",
+            type: "character varying(20)",
+            maxLength: 20,
+            nullable: true,
+            oldClrType: typeof(string),
+            oldType: "varchar(20)",
+            oldMaxLength: 20,
+            oldNullable: true);
+
+        migrationBuilder.CreateIndex(
+            name: "IX_Guilds_Id",
+            table: "Guilds",
+            column: "Id",
+            unique: true);
+
+        migrationBuilder.CreateIndex(
+            name: "IX_Guilds_Name",
+            table: "Guilds",
+            column: "Name",
+            unique: true);
+
+        migrationBuilder.CreateIndex(
+            name: "IX_GuildMember_Id",
+            table: "GuildMember",
+            column: "Id",
+            unique: true);
+
+        migrationBuilder.AddForeignKey(
+            name: "FK_Characters_Accounts_AccountId",
+            table: "Characters",
+            column: "AccountId",
+            principalTable: "Accounts",
+            principalColumn: "Id",
+            onDelete: ReferentialAction.Restrict);
+
+        migrationBuilder.AddForeignKey(
+            name: "FK_Characters_Guilds_GuildId",
+            table: "Characters",
+            column: "GuildId",
+            principalTable: "Guilds",
+            principalColumn: "Id",
+            onDelete: ReferentialAction.Restrict);
+
+        migrationBuilder.AddForeignKey(
+            name: "FK_GuildMember_Characters_CharacterId",
+            table: "GuildMember",
+            column: "CharacterId",
+            principalTable: "Characters",
+            principalColumn: "Id",
+            onDelete: ReferentialAction.Restrict);
+
+        migrationBuilder.AddForeignKey(
+            name: "FK_GuildMember_Guilds_GuildId",
+            table: "GuildMember",
+            column: "GuildId",
+            principalTable: "Guilds",
+            principalColumn: "Id",
+            onDelete: ReferentialAction.Cascade);
+    }
+
+    /// <inheritdoc />
+    protected override void Down(MigrationBuilder migrationBuilder)
+    {
+        migrationBuilder.DropForeignKey(
+            name: "FK_Characters_Accounts_AccountId",
+            table: "Characters");
+
+        migrationBuilder.DropForeignKey(
+            name: "FK_Characters_Guilds_GuildId",
+            table: "Characters");
+
+        migrationBuilder.DropForeignKey(
+            name: "FK_GuildMember_Characters_CharacterId",
+            table: "GuildMember");
+
+        migrationBuilder.DropForeignKey(
+            name: "FK_GuildMember_Guilds_GuildId",
+            table: "GuildMember");
+
+        migrationBuilder.DropIndex(
+            name: "IX_Guilds_Id",
+            table: "Guilds");
+
+        migrationBuilder.DropIndex(
+            name: "IX_Guilds_Name",
+            table: "Guilds");
+
+        migrationBuilder.DropIndex(
+            name: "IX_GuildMember_Id",
+            table: "GuildMember");
+
+        migrationBuilder.AlterColumn<string>(
+            name: "Name",
+            table: "Guilds",
+            type: "text",
+            nullable: true,
+            oldClrType: typeof(string),
+            oldType: "character varying(16)",
+            oldMaxLength: 16,
+            oldNullable: true);
+
+        migrationBuilder.AlterColumn<string>(
+            name: "Name",
+            table: "Characters",
+            type: "varchar(20)",
+            maxLength: 20,
+            nullable: true,
+            oldClrType: typeof(string),
+            oldType: "character varying(20)",
+            oldMaxLength: 20,
+            oldNullable: true);
+
+        migrationBuilder.AlterColumn<string>(
+            name: "Username",
+            table: "Accounts",
+            type: "varchar(20)",
+            maxLength: 20,
+            nullable: true,
+            oldClrType: typeof(string),
+            oldType: "character varying(20)",
+            oldMaxLength: 20,
+            oldNullable: true);
+
+        migrationBuilder.AddForeignKey(
+            name: "FK_Characters_Accounts_AccountId",
+            table: "Characters",
+            column: "AccountId",
+            principalTable: "Accounts",
+            principalColumn: "Id");
+
+        migrationBuilder.AddForeignKey(
+            name: "FK_Characters_Guilds_GuildId",
+            table: "Characters",
+            column: "GuildId",
+            principalTable: "Guilds",
+            principalColumn: "Id");
+
+        migrationBuilder.AddForeignKey(
+            name: "FK_GuildMember_Characters_CharacterId",
+            table: "GuildMember",
+            column: "CharacterId",
+            principalTable: "Characters",
+            principalColumn: "Id");
+
+        migrationBuilder.AddForeignKey(
+            name: "FK_GuildMember_Guilds_GuildId",
+            table: "GuildMember",
+            column: "GuildId",
+            principalTable: "Guilds",
+            principalColumn: "Id");
+    }
+}

Server/DB/Migrations/WonderkingContextModelSnapshot.cs

@@ -45,7 +45,7 @@ namespace Server.DB.Migrations
 
                     b.Property<string>("Username")
                         .HasMaxLength(20)
-                        .HasColumnType("varchar(20)");
+                        .HasColumnType("character varying(20)");
 
                     b.HasKey("Id");
 
@@ -96,7 +96,7 @@ namespace Server.DB.Migrations
 
                     b.Property<string>("Name")
                         .HasMaxLength(20)
-                        .HasColumnType("varchar(20)");
+                        .HasColumnType("character varying(20)");
 
                     b.Property<byte>("PvPLevel")
                         .HasColumnType("smallint");
@@ -123,13 +123,20 @@ namespace Server.DB.Migrations
                         .HasColumnType("uuid");
 
                     b.Property<string>("Name")
-                        .HasColumnType("text");
+                        .HasMaxLength(16)
+                        .HasColumnType("character varying(16)");
 
                     b.Property<string>("Notice")
                         .HasColumnType("text");
 
                     b.HasKey("Id");
 
+                    b.HasIndex("Id")
+                        .IsUnique();
+
+                    b.HasIndex("Name")
+                        .IsUnique();
+
                     b.ToTable("Guilds");
                 });
 
@@ -154,6 +161,9 @@ namespace Server.DB.Migrations
 
                     b.HasIndex("GuildId");
 
+                    b.HasIndex("Id")
+                        .IsUnique();
+
                     b.ToTable("GuildMember");
                 });
 
@@ -213,11 +223,13 @@ namespace Server.DB.Migrations
                 {
                     b.HasOne("Server.DB.Documents.Account", "Account")
                         .WithMany("Characters")
-                        .HasForeignKey("AccountId");
+                        .HasForeignKey("AccountId")
+                        .OnDelete(DeleteBehavior.Restrict);
 
                     b.HasOne("Server.DB.Documents.Guild", "Guild")
                         .WithMany()
-                        .HasForeignKey("GuildId");
+                        .HasForeignKey("GuildId")
+                        .OnDelete(DeleteBehavior.Restrict);
 
                     b.OwnsOne("Wonderking.Packets.Outgoing.Data.BaseStats", "BaseStats", b1 =>
                         {
@@ -294,11 +306,13 @@ namespace Server.DB.Migrations
                 {
                     b.HasOne("Server.DB.Documents.Character", "Character")
                         .WithMany()
-                        .HasForeignKey("CharacterId");
+                        .HasForeignKey("CharacterId")
+                        .OnDelete(DeleteBehavior.Restrict);
 
                     b.HasOne("Server.DB.Documents.Guild", "Guild")
                         .WithMany("GuildMembers")
-                        .HasForeignKey("GuildId");
+                        .HasForeignKey("GuildId")
+                        .OnDelete(DeleteBehavior.Cascade);
 
                     b.Navigation("Character");