FastCrud icon indicating copy to clipboard operation
FastCrud copied to clipboard

Add schema qualifier

Open alexisread opened this issue 6 years ago • 5 comments

Add schema qualifier to allow generation on table name clashes for differing schema.

alexisread avatar Dec 10 '17 10:12 alexisread

Thought this might be useful as you can place all generated tables/views within a single namespace, even with name clashes.

alexisread avatar Dec 10 '17 10:12 alexisread

Yes, this is the first thing I saw missing in the generated code. Thank you alixisread. Please accept this merge.

dallasbeek avatar Jan 14 '18 01:01 dallasbeek

Don't merge this, needs to undo the class naming. for those peeps that have table names in multiple schemas they can use the tables["tablename"].ClassName = "newname";

but I do agree that schemas should be supported by the generator.

dallasbeek avatar Jan 14 '18 02:01 dallasbeek

Hi, apologies for taking a while on this - I've reverted the ClassName for compatibility. Hope that's ok to get this accepted? Thanks.

alexisread avatar Feb 22 '18 22:02 alexisread

3.0 has a ton of changes so I'm just gonna manage this merge manually. Thanks a lot!

MoonStorm avatar Feb 11 '22 23:02 MoonStorm

Playing around with it more, I don't believe assigning entities to schemas from the generator is a good idea. Entities can be tweaked later if needed.

MoonStorm avatar Jun 02 '24 09:06 MoonStorm

Decided to add the feature as an opt-in flag so it won't break existing generations. The flag will set the Schema property in the Table attribute. See #198

MoonStorm avatar Jun 02 '24 17:06 MoonStorm