Extension development: Unsupported field types in migrations

Discuss general development subjects that are not specific to a particular version like the versioning control system we use or other infrastructure.
Locked
shaim
Registered User
Posts: 1
Joined: Sat Dec 02, 2017 11:41 am

Extension development: Unsupported field types in migrations

Post by shaim » Sat Dec 02, 2017 12:10 pm

Hi all,
I'd like to use the update_schema() method to update table structure, but regretfully, supported data types do not include set, enum, or date types - as supported in mysql.

Will that be acceptable for when the extension is reviewed - if I run an "alter table" sql statement using $db->sql_query()?

Or is there any other way it should be done?

Thanks!

User avatar
david63 Online
Registered User
Posts: 217
Joined: Mon Feb 07, 2005 7:23 am
Location: Lancashire, UK

Re: Extension development: Unsupported field types in migrations

Post by david63 » Sun Dec 03, 2017 3:30 pm

That has already been answered here - https://www.phpbb.com/community/viewtop ... &t=2448586
David
Remember: You only know what you know -
and you do not know what you do not know!

User avatar
canonknipser
Registered User
Posts: 60
Joined: Mon Sep 19, 2011 4:42 am
Location: Germany

Re: Extension development: Unsupported field types in migrations

Post by canonknipser » Mon Dec 04, 2017 4:30 pm

If you have an idea (or better: a solution) how to implement set, enum and date types in the other database systems currently supported by phpBB by using the dbal, maybe you can create a pull request for the core ;)
Greetings, Frank

english is not my native language

User avatar
DavidIQ
Customisations Team Leader
Customisations Team Leader
Posts: 1732
Joined: Thu Mar 02, 2006 4:29 pm
Location: Earth
Contact:

Re: Extension development: Unsupported field types in migrations

Post by DavidIQ » Mon Dec 04, 2017 10:09 pm

Since this was answered on phpbb.com, where it should have been, I'll close this now so the conversation can continue in the right place.
Image

Locked