Adószám ellenőrzés a nav válasz xml-ben ha false, akkor sem tér vissza false-al, hanem érvényesnek mutatja.

pzs / nav-online-invoice

PHP interface for Online Invoice Data Reporting System of Hungarian Tax Office (NAV)

MIT License

146 stars 52 forks source link

Adószám ellenőrzés a nav válasz xml-ben ha false, akkor sem tér vissza false-al, hanem érvényesnek mutatja. #24

Closed GaBbB closed 4 years ago

GaBbB commented 4 years ago

https://github.com/pzs/nav-online-invoice/blob/e555baefa51785957bf975c0ee58382b4e4442fa/src/NavOnlineInvoice/Reporter.php#L216

php 7.0, és 7.4-es verzió alatt az ebben a sorban található feltétel nem fogja meg, hogy érvényes-e az adószám, a visszatérési érték pedig csak az adózó adatait tartalmazza. Ezzel az adószámmal tudjátok tesztelni: 68833109

pzs commented 4 years ago

Szia, nálam a kérdéses adószámra azt hozza, hogy érvényes PHP 7.2 és 7.4 alatt (nav test rendszer). Érvénytelen lenne? Vagy mi pontosan a hiba?

GaBbB commented 4 years ago

Igen, ez egy érvénytelen adószám. Az éles nav rendszere a taxpayerValidity-t false-al adja vissza, viszont a kód nem fogja meg és visszaadja az adózó adatait, pedig false-al kéne visszatérnie.

GaBbB commented 4 years ago

Emiatt most így péntek délután beleírtam egy tákolást (most ennyire tellett hirtelen), hogy átalakítom tömbbé az xml választ és úgy ellenőrzöm, mert éles rendszeren becsúszott 2 ilyen számla. Csak gondoltam jelzem, hogy másnál se fordulhasson elő.

pzs commented 4 years ago

Köszi a jelentést! Valóban, ha az adózó létezett valaha, de időközben érvénytelenné vált az adószám, akkor is visszatért az adózó adataival. Ezt most javítottam, szóval ez esetben false a visszatérési érték, ahogy az a README-ben is szerepel. Javított verzió: https://github.com/pzs/nav-online-invoice/releases/tag/v2.0.3