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

Feature Request - Migrate to term_meta for Yoast SEO Metabox on Terms pages #6915

Closed
garretthyder opened this issue Mar 29, 2017 · 3 comments

Comments

@garretthyder
Copy link
Contributor

Hello,

With Wordpress 4.4 term meta can now be handled through add_term_meta, etc and no longer needs to be an array within the options table.

Just wanted to prompt moving towards term_meta so as to make it easier to get/update yoast meta fields through php get_term_meta, update_term_meta, etc.

Much appreciated,
Cheers

@spacedmonkey
Copy link
Contributor

👍 . Term meta has been around for a long time now. Term meta is a much better place to store this data and would be a performance win as well.

Storing it in options means that all this term meta will be autoloaded on every page. This is wasteful for sites with lots of terms. Term meta is lazy loaded in WP_Query.

@jdevalk
Copy link
Contributor

jdevalk commented Jan 20, 2018

@moorscode didn't we do this already? If not, queue it.

@moorscode
Copy link
Contributor

As we are working towards implementing an ORM and some additional Yoast-tables which will hold the meta information instead, implementing this step would be a waste of effort.

We will be removing the data from the options table in due time.

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

5 participants