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

Fix prefix replacement for drupal6 #41

Merged
merged 1 commit into from Jul 14, 2013

Conversation

Projects
None yet
3 participants
@jinghao
Contributor

jinghao commented Jul 14, 2013

I wasn't able to import from my drupal6 installation which had a custom prefix. This fixes it. With this change, the import script works.

I verified that the drupal7 migration doesn't suffer from the same issues.

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Jul 14, 2013

Member

Thanks! This doesn't change things if I have no prefix, right?

Member

parkr commented Jul 14, 2013

Thanks! This doesn't change things if I have no prefix, right?

@jinghao

This comment has been minimized.

Show comment
Hide comment
@jinghao

jinghao Jul 14, 2013

Contributor

Yup, there's no impact w/ no prefix.
I have to admit I didn't test it, but it is all in the if prefix != '' clause.

Contributor

jinghao commented Jul 14, 2013

Yup, there's no impact w/ no prefix.
I have to admit I didn't test it, but it is all in the if prefix != '' clause.

parkr added a commit that referenced this pull request Jul 14, 2013

Merge pull request #41 from jinghao/master
Fix prefix replacement for drupal6

@parkr parkr merged commit 6352087 into jekyll:master Jul 14, 2013

1 check passed

default The Travis CI build passed
Details

parkr added a commit that referenced this pull request Jul 14, 2013

@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.