ENUM column type on PostgreSQL adapter

[bhean]

Hi,

I'm tryin to migrate some of my projects to Phinx (its great!), but I cannot complete the migration because Phinx don't support ENUM column types on PostgreSQL adapter...

Are there any reason for that, or I'm doing something wrong?

Regards!!

[dereuromark]

any update here from the team?

[dereuromark]

Side note: Afaik only few DBs (like MySQL) have enum support.
That is also why CakePHP does not have native support for it, as it is a snowflake kind of thing.
I recommend using integer columns and some low level framework or project PHP abstraction to simulate enums. This is then also DB agostic.

[bhean]

Sure, but in Phinx there are a lot of things only applies for one DB adapter (signedcolumns for some data types in MySQL, timezoneon timestamp data types in PostgreSQL, etc) so I think "DB agnostic" is not a good reason to don't support some interesting and usefull features (like enum types, in my opinion)

[dereuromark]

Fair enough
Are you able to provide a pull request with your suggested changes?

[bhean]

I'd love to do it, but I'm not sure if I can.
I'll try it...

[acanthis]

@bhean Did you manage to do this?

[MasterOdin]

I'm not sure how one would integrate this into Phinx. To use enums in PostgreSQL, you have to do CREATE TYPE identifier AS ENUM ('value1', 'value2', 'value3', ...);

The example they give on their docs is:

CREATE TYPE mood AS ENUM ('sad', 'ok', 'happy');
CREATE TABLE person (
    name text,
    current_mood mood
);

I'm guessing it'd be something like when in createTable(), loop through all columns and if any are enums, generate the CREATE TYPE statement for it, issue that, and then continue to generating the create table statement, substituting out the 'enum' type for the name we used in the CREATE TYPE (and in changeColumn, if the new type is enum, then do the CREATE TYPE business like above). Would you want an additional option name so that people could name their enums instead of having Phinx auto-generate the name?

[acanthis]

@MasterOdin Yes I need it

[othercorey]

enums have tricky requirements.

Closing due to lack of activity or out of scope. If there is new interest, please open a PR.

[MasterOdin]

@othercorey can this be re-opened. It's annoying to do, but something phinx should probably support at some point.