Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate Beans term-meta from options table to termmeta table #316

Open
christophherr opened this issue Jul 28, 2018 · 3 comments
Open

Migrate Beans term-meta from options table to termmeta table #316

christophherr opened this issue Jul 28, 2018 · 3 comments

Comments

@christophherr
Copy link
Member

WordPress 4.4 introduced term meta and a termmeta DB table in Core at the end of 2015:
https://codex.wordpress.org/Version_4.4
https://make.wordpress.org/core/2015/09/04/taxonomy-term-metadata-proposal/
https://make.wordpress.org/core/2015/09/22/preparing-your-plugins-and-your-client-sites-for-termmeta/
https://make.wordpress.org/core/2015/10/23/4-4-taxonomy-roundup/

It would be great if we could migrate Beans term-meta from the options table to the termmeta table.

@christophherr
Copy link
Member Author

@ibes
Copy link
Contributor

ibes commented Aug 22, 2018

So that would require to rewrite the term-meta API
and to offer a migration path.

So rather rewrite the database or offer a fall back for old term-meta or rewrite term-metas on the go?

From my point of view: whatever is offered by the core should be used from the core.
And that will be some work.

@christophherr
Copy link
Member Author

@ibes

So that would require to rewrite the term-meta API and to offer a migration path.

Yes, I would say so.
Rewrite custom Beans functions with WP Core equivalents and/or throw deprecated notices for old Beans functions and run new functions...

So rather rewrite the database or offer a fall back for old term-meta or rewrite term-metas on the go?

Ideally, we'd run a migration script/function moving the options to termmeta.

From my point of view: whatever is offered by the core should be used from the core.
And that will be some work.

👍

@christophherr christophherr self-assigned this Sep 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants