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

Chyba filtrování s gt #35

Open
vojtasvoboda opened this issue Dec 13, 2023 · 4 comments
Open

Chyba filtrování s gt #35

vojtasvoboda opened this issue Dec 13, 2023 · 4 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@vojtasvoboda
Copy link
Contributor

vojtasvoboda commented Dec 13, 2023

Od 5.12. mi přestalo fungovat filtrování:

$response = $invoices->getColumnsFromFlexiBee($columns, [
        'id' => "gt $lastInvoiceId",
        'storno' => false,
        'limit' => 100,
]);

které pak generuje URL ve tvaru:

/faktura-vydana/?(id%20gt%20%2789587%27and%20storno%20eq%20false)&limit=100

Dle podpory je problém v konstrukci id gt '89587' a správně to má být id gt 89587, což dává smysl a doopravdy při manuální změně v URL to funguje. Nicméně v rámci https://www.flexibee.eu/k/prehled-verzi-flexibee/prehled-verzi-abra-flexi-2023/ jsem zmínku o této změně nedohledal, asi to nebrali jako breaking change :-)

Jako hot fix jsem použil:

$response = $invoices->getColumnsFromFlexiBee($columns, [
        0 => "id gt $lastInvoiceId",
        'storno' => false,
        'limit' => 100,
]);

Ale mělo by to být ošetřeno na úrovni knihovny nejspíš.

@Vitexus Vitexus self-assigned this Dec 13, 2023
@Vitexus Vitexus added the bug Something isn't working label Dec 13, 2023
@Vitexus Vitexus added this to the AbraFlexi milestone Dec 13, 2023
@Vitexus
Copy link
Collaborator

Vitexus commented Dec 13, 2023

@vojtasvoboda čuji podraz tady:

$parts[$column] = $column .= ' ' . $condParts[0] . " '" . $condParts[1] . "'";

@Vitexus
Copy link
Collaborator

Vitexus commented Dec 13, 2023

Jářku, @vojtasvoboda prosím Vás už to bude nějakej ten rok co jsem to přejmenoval z

⚠️ getColumnsFromFlexiBee na getColumnsFromAbraFlexi ⚠️

PHP Fatal error:  Uncaught Error: Call to undefined method AbraFlexi\FakturaVydana::getColumnsFromFlexiBee() in /home/vitex/Projects/EASE/php-spojenet-abraflexi/Examples/Filtering.php:26

#JenomUpozorňuju

@vojtasvoboda
Copy link
Contributor Author

Já bohužel jedu ještě na spoje.net/flexipeehp:1.6.10, ale právě se chystám na postupný update :-)

@Vitexus
Copy link
Collaborator

Vitexus commented Dec 14, 2023

Já bohužel jedu ještě na spoje.net/flexipeehp:1.6.10, ale právě se chystám na postupný update :-)

Chápu: když něco funguje taxe f tom nehrab ...

Tak to jsem zvědav jak to pujde. Za ty +-3roky je tam navíc pár tisíc commitů.

Poslední dobou hodně používám phpstan, takže v samotnté flexibee knihovně se snížil počet breberek z několika set na teď asi cca 30. + se snažím alespoň trochu řešit unit testy a ty taky sem tam nejaky ten problem ukazou.

Aktuální verze PHP ve které probíhá vývoj je 8.2

Podporované platformy jsou debian 11-12 a ubuntu 20-22
Pod windows by to mělo taky šlapat, ale nezkouším to bez extra požadavku.

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

When branches are created from issues, their pull requests are automatically linked.

2 participants