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

invalid Chemspider html syntax #138

Closed
stanstrup opened this issue Nov 24, 2017 · 1 comment
Closed

invalid Chemspider html syntax #138

stanstrup opened this issue Nov 24, 2017 · 1 comment
Labels
bug Unexpected problem or unintended behavior

Comments

@stanstrup
Copy link
Collaborator

Evidently CS is sometimes not escaping properly special characters.
Here is a fix for the problem I found. I could not find a general way to
fix this. I guess if there was one parsers would do it.

Example here: https://www.chemspider.com/Chemical-Structure.8012.html

The line: "MP (exp database): <-80 deg C" makes the parser terminate the tag so the rest is lost and things are not as cs_prop expects.

I added to the PR here: #136
I hope that is OK.

@eduardszoecs eduardszoecs added the bug Unexpected problem or unintended behavior label Nov 25, 2017
eduardszoecs added a commit that referenced this issue Nov 25, 2017
@eduardszoecs
Copy link
Member

I cherry-picked (and droped the dependency on curl).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Unexpected problem or unintended behavior
Projects
None yet
Development

No branches or pull requests

2 participants