search

enricobacis / wos

📚 Web of Science python client
https://wos.readthedocs.io/
MIT License
210 stars 45 forks source link

Adding support for multiple records to lite #8

Closed ianabc closed 4 years ago

ianabc commented 8 years ago

This isn't a finished pull request, it's just for discussion.

At the moment wos.utils._get_records returns a single record (the first) when there might be multiple results in the XML. the pull request just goes one level up in the XML to display all of the records returned by the search, once at line 13 and again at line 36.

I'm not really familiar with the schema so I'm not sure about how to handle this case better.

enricobacis commented 8 years ago

It's so strange that they made the lite and premium services to act so differently. I would expect only the output data to be different, but not the format. By the way you are correct, I will merge it and improve a bit, and let inform you when to test it again.

Thanks

zuphilip commented 6 years ago

Is this superseeded by the wos-lite branch?

enricobacis commented 6 years ago

Yes I eventually plan to merge that branch in master. I just need testers

enricobacis commented 4 years ago

Closing this since the wos-lite branch was merged.