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.

Sign up for GitHub

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

Různé chybky #16

Open

Vitexus opened this issue May 20, 2021 · 1 comment

Collaborator

Vitexus commented May 20, 2021

Chybějící return (je tam, ale vrací jen pokud je splněna podmínka)
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/FakturaVydana.php#L198

Chybějící return volá rodiče, ale nic nevrací (případně odebrat z annotace)
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/RO.php#L2804

Předává parametr, ale metoda žádný nechce
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/Stitky.php#L50
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/Stitky.php#L25

Vyžaduje parametry 2 dostává 3
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/RO.php#L2766

Zbytečný switch, vrací true uplně vždy
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/RO.php#L731

Překlad textu výjimky podrtzítkovou funkcí?

navíc kromě výjimky nedělá nic, a jestě se v annotaci píše, že vrací null
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/Company.php#L243

Stačí jen $options, toto co tam je nyní, je dostupné až od PHP 7.2
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/RW.php#L76

Parent vrací void!
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/RW.php#L76
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/Report.php#L53

Prázdný IF
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/Adresar.php#L103

Chybějící 'break;'
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/RO.php#L631
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/RO.php#L1137
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/RO.php#L1149
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/RO.php#L1712
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/RO.php#L1714

Nahradit NULL za "", neboť metoda vyžaduje string a není to typově kompatibilní
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/RO.php#L1625
https://github.com/Spoje-NET/php-abraflexi/blob/2.0/src/AbraFlexi/RO.php#L2106

Collaborator Author

Vitexus commented May 21, 2021

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment