Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

BUG FOUND wordpress migrator, can migrate when title has SLASH '/' in linux. #680

Closed
sunuslee opened this Issue Dec 4, 2012 · 4 comments

Comments

Projects
None yet
4 participants

sunuslee commented Dec 4, 2012

in will treat the '/' in title as a part of path, so get file not found error.
eg: <title>a post has tow part, part 1/2</title>
this, will create file with path "a post has tow part, part 1/2" witch is wrong, because the '/' isn't escaped correctly.
thank you!

Owner

mattr- commented Dec 5, 2012

there are two wordpress migrators. are you referring to the local wordpress migrator or the wordpress.com migrator? The test I wrote to reproduce this bug is not failing. the test is for the local wordpress migrator

sunuslee commented Dec 5, 2012

hi, i used the one in https://github.com/mojombo/jekyll/wiki/blog-migrations

Using Jekyll + Wordpress export file (works with wordpress.com)

ruby -rubygems -e 'require "jekyll/migrators/wordpressdotcom"; Jekyll::WordpressDotCom.process("wordpress.xml")'

after i manually changed '/' in title to something else, it can be imported.
but after the importation. i opened 0.0.0.0:4000
it says in page:
REXML could not parse this XML/HTML: blah blah..

Owner

parkr commented Dec 8, 2012

+1.

Owner

parkr commented Jan 18, 2013

Fixed with merge of #684.

@parkr parkr closed this Jan 18, 2013

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