Composite primary key with enum causes generated entity to fail to compile

[Pure-Peace]

Description

Composite primary key with enum causes generated entity to fail to compile.

Entities generated from the database:

/* `sea_orm_active_enums.rs */

//! `SeaORM` Entity. Generated by sea-orm-codegen 0.10.5

#[derive(Debug, Clone, PartialEq, Eq, EnumIter, DeriveActiveEnum)]
#[sea_orm(
    rs_type = "String",
    db_type = "Enum",
    enum_name = "version"
)]
pub enum Version {
    #[sea_orm(string_value = "v1")]
    V1,
    #[sea_orm(string_value = "v2")]
    V2,
}

/* `report.rs */

//! `SeaORM` Entity. Generated by sea-orm-codegen 0.10.5

use super::sea_orm_active_enums::Version;
use sea_orm::entity::prelude::*;

#[derive(Clone, Debug, PartialEq, DeriveEntityModel, Eq)]
#[sea_orm(table_name = "report")]
pub struct Model {
    #[sea_orm(primary_key, auto_increment = false)]
    pub id: i64,
    #[sea_orm(primary_key, auto_increment = false)]
    pub version: Version
}

The generated entity fails to compile with the following error:

error[E0277]: the trait bound `sea_orm_active_enums::Version: TryFromU64` is not satisfied
  --> demo.rs
   |
   | #[derive(Clone, Debug, PartialEq, DeriveEntityModel, Eq)]
   |                                   ^^^^^^^^^^^^^^^^^ the trait `TryFromU64` is not implemented for `sea_orm_active_enums::Version`
   |
   = help: the following other types implement trait `TryFromU64`:
             (A, B)
             (A, B, C)
             (A, B, C, D)
             (A, B, C, D, E)
             (A, B, C, D, E, F)
             JsonValue
             Vec<u8>
             bool
           and 23 others
   = note: required for `(i64, sea_orm_active_enums::Version)` to implement `TryFromU64`
note: required by a bound in `sea_orm::PrimaryKeyTrait::ValueType`
  --> sea-orm-0.10.5\src\entity\primary_key.rs:49:11
   |
49 |         + TryFromU64;
   |           ^^^^^^^^^^ required by this bound in `sea_orm::PrimaryKeyTrait::ValueType`
   = note: this error originates in the derive macro `DeriveEntityModel` (in Nightly builds, run with -Z macro-backtrace for more info)

Steps to Reproduce

Create a table and execute migrate up

use sea_orm_migration::prelude::*;

#[derive(Iden)]
pub enum Report {
    Table,
    Id,
    Version,
}

#[derive(Iden)]
pub enum Version {
    #[iden = "version"]
    Enum = -1,
    #[iden = "v1"]
    V1,
    #[iden = "v2"]
    V2,
}

pub fn create_table() -> TableCreateStatement {
    Table::create()
        .table(Report::Table)
        .if_not_exists()
        .col(ColumnDef::new(Report::Id).big_integer().not_null())
        .col(
            ColumnDef::new(Report::Version)
                .enumeration(Version::Enum, [Version::V1, Version::V2])
                .not_null()
                .default(Version::V1.to_string()),
        )
        .primary_key(
            // Form composite primary key with enum
            sea_query::Index::create().col(Report::Id).col(Report::Version),
        )
        .to_owned()
}

Expected Behavior

Actual Behavior

Reproduces How Often

Versions

Additional Information

[billy1624]

Hey @Pure-Peace, welcome to SeaQL. Thanks for the report!!

I just made a PR which provides a blanket implementation of TryFromU64 for all ActiveEnum

• Use ActiveEnum field as composite primary key #1414

impl<T> TryFromU64 for T
where
    T: ActiveEnum,
{
    fn try_from_u64(_: u64) -> Result<Self, DbErr> {
        Err(DbErr::ConvertFromU64(
            "Fail to construct ActiveEnum from a u64, if your primary key consist of a ActiveEnum field, its auto increment should be set to false."
        ))
    }
}