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

Balikobot::trackPackage chybně detekuje 404 české pošty jako Nedoručený balík #15

Closed
josefbenjac opened this issue Jul 20, 2021 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@josefbenjac
Copy link

josefbenjac commented Jul 20, 2021

Tracking api Balíkobotu vrací v určitých případech české pošty zvláštní odpověď, kde je namísto pole eventů, pouze pole se stringem 404 .

{
	"packages": [
		{
			"carrier_id": "NBxxxxxxxxxxxxxx",
			"status": 200,
			"states": ["404"]
		}
	]
}	

Z Balíkobotu máme vyjádření, že pouze přeposílají data z České Pošty a tudíž se tomu nebudou věnovat.

Na základě této chyby, knihovna špatně detekuje stav 4 - Nedoručený balík.

image

Toto se dá jednoduše hacknout na straně použití knihovny přibližně takto:

$statuses = $this->balikobot->trackPackage($orderedPackage);

// Hack / fix 404 Czech Post in Inspirum library
if (
	$package->getShipper()->isCeskaPosta()
	&& count($statuses) === 1
	&& $statuses[0]->getId() === 4.0
	&& $statuses[0]->getName() === '4'
	&& $statuses[0]->getDescription() === '4'
	&& $statuses[0]->getDate() === null
) {
	$statuses = [];
}

Nicméně bylo by moc fajn, kdyby toto uměla řešit přímo knihovna.

Díky za váš čas, pokud se tomu budete věnovat.

@tomas-novotny tomas-novotny self-assigned this Jul 21, 2021
@tomas-novotny tomas-novotny added the bug Something isn't working label Jul 21, 2021
@tomas-novotny
Copy link
Member

Zdravím,

Pro PHP 8.0+ knihovna "správně" spadla, jelikož již není možné přistupovat k textu jako k poli (Error: Cannot access offset of type string on string)

Opraveno 386c988 – nyní bude tyto stavy knihovna ignorovat.

Nový release včetně tohoto bugfixu bude po zpracovaní nových změn v dokumentaci Balikobot API.

@tomas-novotny
Copy link
Member

It have been solved in v5.5.0

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