Better fix for issue no 5 #23

Merged
merged 2 commits into from Apr 9, 2012

Conversation

Projects
None yet
2 participants
Contributor

amithgeorge commented Apr 9, 2012

As you said some wordpress exports had different wp namespace urls. Which is fine. The modified code now reads the namespaces from the provided xml file. No hard coding. What do you say?

amithgeorge added some commits Apr 7, 2012

fixed issue #5
wordpress exports the xml namespace wp with the url http://wordpress.org/export/1.1/ ...
the code was expecting http://wordpress.org/export/1.2/
better fix for issue #5
the code now parses and generates the required xml prefixes and namespaces.
no need to hard code any uris. works with the wp namespace version 1.2 and 1.1 ...
Tested against my Wordpress 3.3.1 exported xml
Owner

thomasf commented Apr 9, 2012

I guess exitwp will fail even harder and weirder when/if there are actual changes in the namespaces/document definitions that are incompatible but since as of yet that hasnt been identified as a real world problem I'll apply this.

thomasf added a commit that referenced this pull request Apr 9, 2012

@thomasf thomasf merged commit 58362ee into thomasf:master Apr 9, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment