pzs / nav-online-invoice

PHP interface for Online Invoice Data Reporting System of Hungarian Tax Office (NAV)
MIT License
146 stars 53 forks source link

Egyedi RequestId generálás, naplózás bővítése #17

Closed scr34m closed 4 years ago

scr34m commented 4 years ago

A RequestID generálást most egy fixen beállított függvény végzi ezen szeretnék javítani ezért egy interface-t vezettem be. Így akár saját adatbázisban lehetőség van a request id egyediségét biztosítani a config létrehozása során $config->setRequestIdGenerator(new RequestIdGeneratorSql());

Valamint a naplózást segítendő ezt az azonosítót beraktam a getLastRequestData() hívás eredményébe, amennyiben ki tudja nyerni.

pzs commented 4 years ago

Köszi a PR-t! Jó ötlet mind a két fejlesztés! Egyetlen kérdésem lenne merge előtt: Config osztályban szándékosan lett getRequestId-nak elnevezve a metódus generateRequestId helyett? Miért? Ha nem, esetleg átírnád? (csak a Config osztályra gondolok, BaseRequestXml-ben értem)

scr34m commented 4 years ago

Az valóban nem beszédes helyette a getRequestIdGenerator lenne célravezető, mindjárt javítom.

pzs commented 4 years ago

Az a metódus már a generált értéket adja vissza, nem a genetátort.

scr34m commented 4 years ago

Igen, de módosítom, hogy a generatort adja vissza és az értéket már a BaseRequestXml kérje le, mivel különben akár milyen nevet adunk nem lesz egyértelmű a működés szerintem.

pzs commented 4 years ago

Rendben, így értem :)

scr34m commented 4 years ago

Módosítottam

scr34m commented 4 years ago

Köszönöm, a fejlesztés is!

pzs commented 4 years ago

Szükséged van esetleg RC2-es release most ezzel a fejlesztéssel, vagy elég lesz a végleges 2.0 release, amikor a NAV elérhetővé teszi a 2.0-ás interfészt élesben is?

scr34m commented 4 years ago

Nekem nincs szükség mivel nem olyan változást hozott szerintem.

pzs commented 4 years ago

Rendben, és nincs mit!