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

Baseurl for xml #7

Closed
wants to merge 2 commits into from
Closed

Baseurl for xml #7

wants to merge 2 commits into from

Conversation

Klortho
Copy link

@Klortho Klortho commented Feb 7, 2016

This PR comprises two changes. I'll describe them as line comments below.

The main purpose of this PR is to add the baseUrl option to FromXml. I noticed that this (and a few other options) were already added to FromHtml a long time ago, so I added them in the same way.

I think the two methods should be refactored to move the common code (99% of it) to a common function. If it had been done that way originally, then users of FromXml would have benefited from these extra options added to FromHtml.

looks like a typo, and I was hoping it would fix the problem, but it did
not. The symptoms are simple: even when I use the `xinclude` option, the
library still does not expand the <xi:include href='...'/> element,
whereas xmllint does.
@Klortho
Copy link
Author

Klortho commented Feb 10, 2016

-> libxmljs#388

@Klortho Klortho closed this Feb 10, 2016
@Klortho Klortho deleted the baseurl-for-xml branch February 29, 2016 04:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant