-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
#30262 Combine terms
and term_taxonomy
tables
#8354
base: trunk
Are you sure you want to change the base?
#30262 Combine terms
and term_taxonomy
tables
#8354
Conversation
Test using WordPress PlaygroundThe changes in this pull request can previewed and tested using a WordPress Playground instance. WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser. Some things to be aware of
For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation. |
…ed to make any transformations itself.
…DATE` on `wp_terms`.
# Conflicts: # src/wp-includes/default-filters.php # tests/phpunit/tests/term/meta.php
I am very interested in being involved in this ticket. Please ping me for code review. |
Cheers Jonny, will do. I'm going to try an alternative approach (converting |
The approach of using a view has it's own problems. The biggest one to me is permissions. Do all database users have access to create a views? How will that work in Multisite? |
This is some initial investigation into the feasibility of doing this. Thar be dragons.
Trac ticket: https://core.trac.wordpress.org/ticket/30262