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

Issue #1035: [CMI] node_type table still exists on new installs. #986

Merged
merged 1 commit into from Sep 12, 2015

Conversation

hellomrcat
Copy link
Contributor

@@ -109,7 +109,6 @@ function node_schema() {
'node_frontpage' => array('promote', 'status', 'sticky', 'created'),
'node_status_type' => array('status', 'type', 'nid'),
'node_title_type' => array('title', array('type', 4)),
'node_type' => array(array('type', 4)),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This index should still be left in here, as the column in the node table for "type" will still exist.

@quicksketch
Copy link
Member

Could you add an update hook by copying node_update_1006() and adding it to the end of the update list (renaming the function to node_update_1010()) to drop this table if it exists?

@hellomrcat
Copy link
Contributor Author

@quicksketch thanks, should be good to go now.

quicksketch added a commit that referenced this pull request Sep 12, 2015
Issue #1035: [CMI] node_type table still exists on new installs.
@quicksketch quicksketch merged commit 0e93c0c into backdrop:1.x Sep 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants