ISBN ref support - Githubissues

lucasgautheron / CosmologyWebsite

History of modern cosmology

https://cosmology.education/

4 stars 1 forks source link

ISBN ref support #48

Closed lucasgautheron closed 8 years ago

lucasgautheron commented 8 years ago

Closes #47

lucasgautheron commented 8 years ago

@ScalaWilliam

I don't really like having to deal with something other than JSON
I don't like that I have to process ISBN et DOI references differently

Any idea on a better implementation than this one ?

PS : ISBN databases are usually poorly maintained - they contain errors, or lack entries.

lucasgautheron commented 8 years ago

Container-based travis-ci doesn't have basex whitelisted under, travis with sudo: required, it seems to be an old version, with no 'fetch' support

Saving properties in "/home/travis/.basex"...
Stopped at line 1, column 105:
[XPST0081] Unknown prefix: "fetch".
Stopped at line 1, column 108:
[XPST0081] Unknown prefix: "fetch".

In the end these are only "warnings" and tests succeed but that's not great.

lucasgautheron commented 8 years ago

Debian Jessie official repositories provide BaseX 7.9 which doesn't have json-to-xml implemented.

root/.basex: writing new configuration file.
Stopped at ., 1/96:
[XPST0017] Function 'json-to-xml' is unknown.
Stopped at ., 1/99:
[XPST0017] Function 'json-to-xml' is unknown.

EDIT: now using basex from the testing channel on my server, which is 8.5+.