The first commit fixes a trivial bug: in the zotero-setup directive, the style attribute was ignored, and the default value was always used.
The second commit does the following: whenever zotxt returns an HTTP error, the HTTPError exception is only caught at the highest level by Sphinx, with the following message:
Please also report this if it was a user error, so that a better error message can be provided next time. A bug report can be filed in the tracker at https://bitbucket.org/birkenfeld/sphinx/issues/. Thanks!
This is misleading.
With the patch, those errors are repported on the output but do not stop the processing. The bibliography directive will render to an empty text, and the :xcite: texts will render to "[Broken citation]".
Note that in https://bitbucket.org/egh/zotxt/issue/16 I have suggested that zotxt could be more tolerant and return a partial solution rather than failing completely if one citation is broken. If this was done, then zot4rst would require some more changes to handle such partial solution (and still report the problem).
The first commit fixes a trivial bug: in the zotero-setup directive, the style attribute was ignored, and the default value was always used.
The second commit does the following: whenever zotxt returns an HTTP error, the HTTPError exception is only caught at the highest level by Sphinx, with the following message:
This is misleading.
With the patch, those errors are repported on the output but do not stop the processing. The bibliography directive will render to an empty text, and the :xcite: texts will render to "[Broken citation]".
Note that in https://bitbucket.org/egh/zotxt/issue/16 I have suggested that zotxt could be more tolerant and return a partial solution rather than failing completely if one citation is broken. If this was done, then zot4rst would require some more changes to handle such partial solution (and still report the problem).