Skip to content
This repository has been archived by the owner on Nov 6, 2023. It is now read-only.

Bug in Tantalus #23

Open
TartarusTim opened this issue Dec 15, 2021 · 2 comments
Open

Bug in Tantalus #23

TartarusTim opened this issue Dec 15, 2021 · 2 comments

Comments

@TartarusTim
Copy link

hoi hoi,

Ik ben de zoveelste gebruiker van tantalus, laatst heb ik een bug ontdekt waarbij als ik een bepaalde transactie uit conscribo verwijder en deze transactie in tantalus aanpas deze niet meer wil syncen met conscribo.

dit is de feedback die tantalus terug geeft als ik het probeer te syncen:

Error voor thijs: Traceback (most recent call last): File "/app/ConscriboPyAPI/conscribo_sync.py", line 26, in sync_transactions conscribo.add_change_transaction(xml) File "/app/ConscriboPyAPI/conscribo_api.py", line 66, in add_change_transaction res = TransactionPutResult(tod.content, transaction) File "/app/ConscriboPyAPI/conscribo_mapper.py", line 199, in init super(TransactionPutResult, self).init(data) File "/app/ConscriboPyAPI/conscribo_mapper.py", line 17, in init tree = etree.fromstring(bytes(xml)) File "src/lxml/etree.pyx", line 3237, in lxml.etree.fromstring File "src/lxml/parser.pxi", line 1896, in lxml.etree._parseMemoryDocument File "src/lxml/parser.pxi", line 1784, in lxml.etree._parseDoc File "src/lxml/parser.pxi", line 1141, in lxml.etree._BaseParser._parseDoc File "src/lxml/parser.pxi", line 615, in lxml.etree._ParserContext._handleParseResultDoc File "src/lxml/parser.pxi", line 725, in lxml.etree._handleParseResult File "src/lxml/parser.pxi", line 654, in lxml.etree._raiseParseError File "", line 6 lxml.etree.XMLSyntaxError: StartTag: invalid element name, line 6, column 2

@thijsmie
Copy link
Owner

Hoi Tim!

Hmm beetje jammer dat Conscribo niet een fatsoenlijke error teruggeeft (geen error code, geen messages, maar gewoon invalid data, tsk tsk). Eigenlijk denk ik dat de fout aan de Conscribo API kant ligt, maar dat kan ik niet zo makkelijk bewijzen. Zou je in conscribo een dummy boeking kunnen maken met de juiste referentie en kijken of het dan wel werkt?

@TartarusTim
Copy link
Author

TartarusTim commented Dec 16, 2021 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants