Skip to content

Add author and nid field to import from Drupal 7.#237

Merged
parkr merged 1 commit intojekyll:masterfrom
geerlingguy:drupal7
Jan 6, 2016
Merged

Add author and nid field to import from Drupal 7.#237
parkr merged 1 commit intojekyll:masterfrom
geerlingguy:drupal7

Conversation

@geerlingguy
Copy link
Contributor

With the current Drupal 7 import script, there is no author data (very helpful when there are multi-author blogs, which is often the case with Drupal sites) nor nid attached to the imported posts. The nid can be extremely helpful in cases where you use a customized template to show Disqus comments or need to print other legacy-powered data on particular posts.

Copy link
Member

Choose a reason for hiding this comment

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

please add a space between u and \

@parkr
Copy link
Member

parkr commented Jan 6, 2016

Wonderful! One nit above, then I'll merge 👍

@geerlingguy
Copy link
Contributor Author

@parkr - Sorry about that, it's fixed now!

parkr added a commit that referenced this pull request Jan 6, 2016
@parkr parkr merged commit 9afec94 into jekyll:master Jan 6, 2016
parkr added a commit that referenced this pull request Jan 6, 2016
@parkr
Copy link
Member

parkr commented Jan 6, 2016

Can this be backported to Drupal 6?

Thanks for your contributions!! 🎉

@geerlingguy
Copy link
Contributor Author

@parkr - The structure of the users table and uid/name fields is exactly the same between D6/D7... however, the field_data_body table and it's relationship to the node table isn't the same, so these particular changes could be backported, but the exact commit couldn't be cherry-picked into the drupal6 file/backported without rejiggering a little.

Unfortunately, I don't have any Drupal 6 sites I can use to test it, otherwise I'd do that really quick and backport it myself :/

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants