Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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 opened this issue Jul 10, 2020 · 4 comments

Comments

@GaBbB
Copy link

GaBbB commented Jul 10, 2020

if (empty($responseXml->taxpayerValidity) or $responseXml->taxpayerValidity === "false") {

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
Copy link
Owner

pzs commented Jul 10, 2020

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
Copy link
Author

GaBbB commented Jul 10, 2020

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
Copy link
Author

GaBbB commented Jul 10, 2020

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
Copy link
Owner

pzs commented Jul 10, 2020

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

@pzs pzs closed this as completed Jul 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants