EFCore.NamingConventions not working with Abp

• Abp package version: 5.10.1
• base framework: .Net Core
• Steps needed to reproduce the problem:

• Download a aspnetboilerplate template
• Integrate postgres
• Use UseSnakeCaseNamingConvention in *DbContextConfigurer
• In *DbContext add modelBuilder.ChangeAbpTablePrefix<Tenant, Role, User>("", "abp"); in OnModelCreating method
• Then add-migration Initial_Migration in package-manager

The ModelSnapshot is something like it:

The columns, index, pk and fk has applied nameconvention but table names not.

using System;
using Microsoft.EntityFrameworkCore.Migrations;
using Npgsql.EntityFrameworkCore.PostgreSQL.Metadata;

namespace Coders.SSO.Migrations
{
    public partial class Initial_Migration : Migration
    {
        protected override void Up(MigrationBuilder migrationBuilder)
        {
            migrationBuilder.EnsureSchema(
                name: "abp");

            migrationBuilder.CreateTable(
                name: "AuditLogs",
                schema: "abp",
                columns: table => new
                {
                    id = table.Column<long>(nullable: false)
                        .Annotation("Npgsql:ValueGenerationStrategy", NpgsqlValueGenerationStrategy.IdentityByDefaultColumn),
                    tenant_id = table.Column<int>(nullable: true),
                    user_id = table.Column<long>(nullable: true),
                    service_name = table.Column<string>(maxLength: 256, nullable: true),
                    method_name = table.Column<string>(maxLength: 256, nullable: true),
                    parameters = table.Column<string>(maxLength: 1024, nullable: true),
                    return_value = table.Column<string>(nullable: true),
                    execution_time = table.Column<DateTime>(nullable: false),
                    execution_duration = table.Column<int>(nullable: false),
                    client_ip_address = table.Column<string>(maxLength: 64, nullable: true),
                    client_name = table.Column<string>(maxLength: 128, nullable: true),
                    browser_info = table.Column<string>(maxLength: 512, nullable: true),
                    exception = table.Column<string>(maxLength: 2000, nullable: true),
                    impersonator_user_id = table.Column<long>(nullable: true),
                    impersonator_tenant_id = table.Column<int>(nullable: true),
                    custom_data = table.Column<string>(maxLength: 2000, nullable: true)
                },
                constraints: table =>
                {
                    table.PrimaryKey("pk_abp_audit_logs", x => x.id);
                });

            migrationBuilder.CreateTable(
                name: "BackgroundJobs",
                schema: "abp",
                columns: table => new
                {
                    id = table.Column<long>(nullable: false)
                        .Annotation("Npgsql:ValueGenerationStrategy", NpgsqlValueGenerationStrategy.IdentityByDefaultColumn),
                    creation_time = table.Column<DateTime>(nullable: false),
                    creator_user_id = table.Column<long>(nullable: true),
                    job_type = table.Column<string>(maxLength: 512, nullable: false),
                    job_args = table.Column<string>(maxLength: 1048576, nullable: false),
                    try_count = table.Column<short>(nullable: false),
                    next_try_time = table.Column<DateTime>(nullable: false),
                    last_try_time = table.Column<DateTime>(nullable: true),
                    is_abandoned = table.Column<bool>(nullable: false),
                    priority = table.Column<byte>(nullable: false)
                },
                constraints: table =>
                {
                    table.PrimaryKey("pk_abp_background_jobs", x => x.id);
                });

          //......
}

[maliming]

hi

I think it’s maybe we specify the table name. You can refer to AbpZeroDbModelBuilderExtensions.cs.

https://github.com/aspnetboilerplate/aspnetboilerplate/blob/671d25597251358d2218a80f198edac6f43624c4/src/Abp.ZeroCore.EntityFrameworkCore/Zero/EntityFrameworkCore/AbpZeroDbModelBuilderExtensions.cs#L42

[mirusky]

@princefishthrower I don’t remember what we did in this case, but I’m pretty sure we ended up changing table by table. Because we didn’t have time to create a proper PR with tests and stuff…

My last comment was what we are intended to change in the followin method: https://github.com/aspnetboilerplate/aspnetboilerplate/blob/671d25597251358d2218a80f198edac6f43624c4/src/Abp.ZeroCore.EntityFrameworkCore/Zero/EntityFrameworkCore/AbpZeroDbModelBuilderExtensions.cs#L35

Maybe you could create a new issue with suggested changes.