Use file! in migration files

[tyt2y3]

sea-orm/examples/actix_example/migration/src/m20220120_000001_create_post_table.rs

Lines 6 to 10 in cdc70f4

	impl MigrationName for Migration {
	fn name(&self) -> &str {
	"m20220120_000001_create_post_table"
	}
	}

Can we use https://doc.rust-lang.org/std/macro.file.html instead of manually copying the name?

We'd probably want to change the migration template too

[smonv]

I did some testing and found out file!() return full filename with extension. So in order to use, we need extra step to remove the extension part too.

So I'm not sure it worth the effort to change for template, the string already generated by cli.

[tyt2y3]

I imagine we can use a compile time const fn to split out the file name.
I think this will reduce confusion when files got renamed.

[billy1624]

This is doable. I guess we can introduce a derive for Migration struct.

pub struct Migration;

impl MigrationName for Migration {
    fn name(&self) -> &str {
        std::path::Path::new(file!())
            .file_stem()
            .map(|f| f.to_str().unwrap())
            .unwrap()
    }
}

#[async_trait::async_trait]
impl MigrationTrait for Migration {
    ...
}

[lpotthast]

I usually do the following:

In all migration files, include:

crate::migration_file!();

I defined the macro like this:

#[macro_export]
macro_rules! migration_file {
    () => {
        pub struct Migration;
        impl MigrationName for Migration {
            fn name(&self) -> &str {
                std::path::Path::new(file!())
                    .file_name()
                    .unwrap()
                    .to_str()
                    .unwrap()
                    .split(".")
                    .next()
                    .unwrap()
            }
        }
    };
}

[tyt2y3]

Thanks. This is quite neat!