nav-gov-hu / Online-Invoice

Public repository of the Online Invoice System
Other
138 stars 52 forks source link

[Q&A] QueryTaxpayerResponse hiányos XML struktúra #151

Closed cycle20 closed 4 years ago

cycle20 commented 4 years ago

Sziasztok!

QueryTaxpayerResponse-al kapcsolatban kérdezném az alábbiakat:

Üdv, Csongor

NTCA-developer commented 4 years ago

Szia @csgerdan !

Ez számunkra is ismert hiba, már folyamatban van a javítása. A probléma azért jelentkezik mert vannak olyan címadatok, amelyek nem tartalmaznak kis szemcsés, bontott címadatot, hanem csak konkatenált, bontatlan címet. A javítás a 2.8.3-as verzióban fog élesedni (a teszten most a 2.8.2 van) és úgy fog működni, hogy:

A lekérdezésekkor a fenti logikára számíthatsz.

cycle20 commented 4 years ago

Köszönöm szépen a gyors választ. Már csak egy extra kérdés kíváncsiságból:

Milyen technikai/fejlesztési akadálya van annak, hogy a taxpayerAddress AddressType típusú legyen, így ha nincs elég adat DetailedAddressType-hoz, akkor SimpleAddressType-ot adna vissza a lekérdezés?

Egyelőre nem látom át az egész API-t teljes egészében, az is lehet, hogy történeti/jogszabálybeli háttere van annak, hogy itt mindenképpen DetailedAddress-nek kell visszajönnie.

NTCA-developer commented 4 years ago

Nincs akadálya, sőt pont ez volna a leguniverzálisabb. Hogy most nem cseréljük le a típust a sémában annak az az oka, hogy túl sokan függenek a válasz formájától (Online Számlázó, mobilapp) és jelenleg nem tudjuk a meglévő feladatok mellett úgy ütemezni, hogy mindenkinek jó legyen. A másik ok valóban egy üzleti döntés, hogy amennyiben lehet akkor mindig adjunk detailedAddress-t. A frontend is ennek megfelelően működik, csak ugye ott van ember, aki szétbontja a címet.

A harmadik oka (és ez már csak metodika) hogy jót tenne a sémának, ha a choice-ok kikopnának belőle, vagy legalább a számuk csökkenne, mert segíti az átláthatóságot és könnyebb is erre programozni.

Majd kérlek zárd le ha nincs több kérdésed.

cycle20 commented 4 years ago

Köszönöm szépen. Részemről lezárva.