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

Problém se zpracováním datumů. #25

Closed
thomasdv opened this issue Nov 4, 2021 · 5 comments
Closed

Problém se zpracováním datumů. #25

thomasdv opened this issue Nov 4, 2021 · 5 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@thomasdv
Copy link

thomasdv commented Nov 4, 2021

Dobrý den, pokud udělám jednoduchou operaci:

$inv = new FakturaVydana($invoiceId);
$inv->takeData(["stavMailK" => "stavMail.odeslano"]);
$inv->sync();

Dostávám chybovou hlášku:

'message' => 'Zadaný text 'null' musí být datum'
'for' => 'eetDatCasTrzby'
'path' => 'faktura-vydana[id=539].eetDatCasTrzby'
'value' => 'null'
'code' => 'PROP'

Přitom se s tím údajem vůbec nepracuje. Asi by bylo dobré v entitě faktury kontrolovat textový "null" u datumu a případně opravdu nastavit na null?

@thomasdv
Copy link
Author

thomasdv commented Nov 4, 2021

Zároveň s tím souvisí i následující:

Error: 
Method AbraFlexi\Date::__toString() must return a string value

@Vitexus Vitexus self-assigned this Nov 4, 2021
@Vitexus Vitexus added the bug Something isn't working label Nov 4, 2021
@Vitexus Vitexus added this to the AbraFlexi milestone Nov 4, 2021
@Vitexus
Copy link
Collaborator

Vitexus commented Nov 4, 2021

Tam je hodně zabojováno, ale asi jste našel chybyičku. Děkuji za bugreport. Pořeším to to obratem.

@Vitexus
Copy link
Collaborator

Vitexus commented Nov 5, 2021

@thomasdv Vámi daný příklad bych zapsal úsporněji:

$inv = new FakturaVydana($invoiceId,['autoload'=>false]);
$inv->setDataValue("stavMailK" , "stavMail.odeslano");
$inv->saveToAbraFlexi();

Ale to samozřejmě neznamená že by vámi vrácený příkald měl končit chybou.

Vitexus added a commit that referenced this issue Nov 6, 2021
@Vitexus
Copy link
Collaborator

Vitexus commented Nov 6, 2021

@thomasdv zkuste to prosím s poslední aktualizací v mainu. Pokud vám bude šplapat všechno dobře, tak bych uďál release.

@thomasdv
Copy link
Author

thomasdv commented Nov 8, 2021

@Vitexus, díky main je nyní bez chyby.

@Vitexus Vitexus closed this as completed Nov 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants