[CLI] Fix generating Enum with variant starts with number

[billy1624]

PR Info

• Closes CLI can't generate entity if enum value starts with a number #1172

Bug Fixes

• [CLI] generate entity: add prefix _ to enum variant starts with number

[Yuubari]

pub enum MediaType {
    #[sea_orm(string_value = "3D")]
    _3D,

leads to a panic in the macro when trying to use the generated entity. It looks like the solution will not be that simple?

[tyt2y3]

@Yuubari what's the error message?

[Yuubari]

The same as in the original #1172:

   Compiling entity v0.1.0 (mw-tools/entity)
error: proc-macro derive panicked
 --> entity\src\sea_orm_active_enums.rs:5:49
  |
5 | #[derive(Debug, Clone, PartialEq, Eq, EnumIter, DeriveActiveEnum)]
  |                                                 ^^^^^^^^^^^^^^^^
  |
  = help: message: `"3d"` is not a valid identifier

error: could not compile `entity` due to previous error

If I change ‘3D’ in

#[sea_orm(string_value = "3D")]

to anything else, it compiles fine.

[tyt2y3]

Ah sad we need a test case for the generated Entity as well.

[billy1624]

Oh no... we fixed one bug and another bug emerged. Now we need to fix DeriveActiveEnum.

We need test cases for generated entity. I think we can generate the entity and see if it compile.

[billy1624]

Hey @Yuubari, please check #1219.

I'll add the tests for generated entity later. That's another PR. We need to patch it first.

[tyt2y3]

https://github.com/SeaQL/sea-orm/releases/tag/0.10.4