Bug: Fatal error

[8915b]

Hezký dobrý den, plugin byl skvělý moc za něj děkujeme. Aktuálně evidujeme chybu s děsivým názvem Fatal error :( Máme WordPress 5.5.3, woocommerce Verze 4.7.1. Zákazník to reportoval takto:

Vypnuli jsme všechny pluginy kromě woocommerce a Zásilkovna Shipping Method (Verze 0.1.1). A po kliknutí na výběr zásilkovna kde očekáváme rozbalovací menu se všechno zasekne a nic se neděje.

Můžete mi prosím poradit, co s tím? Moc děkuji.

[morvy]

aktualizujte plugin, nedavno bola vydana verzia 0.1.1, ktora tento problem riesi

[8915b]

aktualizujte plugin, nedavno bola vydana verzia 0.1.1, ktora tento problem riesi Zdravím, ve výpisu vidíte (nad druhým obrázkem), že máme Zásilkovna Shipping Method (Verze 0.1.1) tohle to opravdu neřeší. Děkuji za snahu.

[vyskoczilova]

@8915b Dobrý den, můžete se podívat, jestli máte na řádku 225 opravdu toto? Mělo by tam být empty místo size_of. Nevypadá to, že byste měl stejnou verzi (alespoň v kódu):

[8915b]

@vyskoczilova Dobrý den, přikládám. Přijde mi to stejné. Přikládám originál soubor z webu (přípona změněna z php na txt). class-zasilkovna.txt. Velmi děkuji.

[vyskoczilova]

@8915b To vypadá v pořádku, jenže si u vás web stěžuje na parametr size_of, který v kódu na daném místě není. Nemáte tam nějakou cache?

[8915b]

@vyskoczilova všechno je vypnuté je tam jen jádro wordpressu, woocommerce a zásilkovna. Zkusil jsem to v chrome v anonymním okně a teď to zas píše tohle. Jsem z toho úplnej jelen.

[morvy]

To vyzera lepsie, hodilo to teda chybu, ktoru to ma hodit ak je json prazdny, cize teraz treba hladat chybu v odpovedi zo zasielkovne, napr. API kluc nespravny a pod.

[maleksandrov]

Hi guys,

Thank you very much for creating and supporting this component for free. :) Really appreciated. Unfortunately, I have a similar issue as the user 8915b.

Fatal error: Uncaught Exception: WC_Zasilkovna_Shipping_Method::fetch_pickup_points: JSON data empty. in /home/kotito/adapted/wp-content/plugins/woocommerce-zasilkovna-master/class-zasilkovna.php:226 Stack trace: #0 /home/kotito/adapted/wp-content/plugins/woocommerce-zasilkovna-master/class-zasilkovna.php(196): WC_Zasilkovna_Shipping_Method->fetch_pickup_points('7e6831f41cfa71f...') #1 /home/kotito/adapted/wp-content/plugins/woocommerce-zasilkovna-master/class-zasilkovna.php(180): WC_Zasilkovna_Shipping_Method->load_pickup_points() #2 /home/kotito/adapted/wp-content/plugins/woocommerce-zasilkovna-master/zasilkovna.php(73): WC_Zasilkovna_Shipping_Method->pickup_points('CZ') #3 /home/kotito/adapted/wp-includes/class-wp-hook.php(287): wc_zasilkovna_review_order_after_shipping('') #4 /home/kotito/adapted/wp-includes/class-wp-hook.php(311): WP_Hook->apply_filters('', Array) #5 /home/kotito/adapted/wp-includes/plugin.php(478): WP_Hook->do_action(Array) #6 /home/kotito/adapted/wp-content/plugins/woocommerce/templates/checkout/re in /home/kotito/adapted/wp-content/plugins/woocommerce-zasilkovna-master/class-zasilkovna.php on line 226

On line 226, I see this:

That's from Zasilkovna website, doesn't seems empty:

I already test the API key, and it's working fine. WordPress v.5.5.3 WooCommerce v.4.7.1

Thank you!

Best, Milen

[ostryweb-cz]

fix your problems by correcting #225 to basically: if ( empty( $json->data ) ) {

(also it doesnt seem to be optimal downloading the whole 13MB list every time anyone enters the checkout...)

[8915b]

@ostryweb-cz Dobrý den, zkusili jsme změnu z: if ( empty( $json->data ) <= 0 ) { throw new Exception( METHOD . ": JSON data empty." ); } Na uvedené: if ( empty( $json->data ) ) { throw new Exception( METHOD . ": JSON data empty." ); } A bohužel to nezafungovalo. Teď to zašedne a zamrzne. Moc děkuji za snahu pomoci. Určitě na to vy magici přijdete. Moc rád bych pomohl, ale proti vám jsem lama :)

[ostryweb-cz]

@ostryweb-cz Dobrý den, zkusili jsme změnu z: if ( empty( $json->data ) <= 0 ) { throw new Exception( METHOD . ": JSON data empty." ); } Na uvedené: if ( empty( $json->data ) ) { throw new Exception( METHOD . ": JSON data empty." ); } A bohužel to nezafungovalo. Teď to zašedne a zamrzne. Moc děkuji za snahu pomoci. Určitě na to vy magici přijdete. Moc rád bych pomohl, ale proti vám jsem lama :)

a mate to zaktualizovano? tzn od toho uvodniho screenshotu jste stahli aktualni verzi, kde je chyba u toho empty() ? jinak z obrazku se tezko soudi, ale jak pisu anglicky vyse - ten plugin pokazde stahuje cely 13MB seznam odbernych mist, coz muze jednak chvili trvat a druhak ja jsem nejmin minuly tyden zaregistroval vysoke zatizeni serveru zasilkovny, obecne bych to zkusil znovu. mel byste mit posledni verzi pluginu a v ni opravenu tu vec, co jsem psal vyse.

pokud to nefunguje, tak to je fakt nejaka specificka vec u vas, coz z obrazku nerozklicujem.

[8915b]

@ostryweb-cz ano prosím, staženo z gitu ->replace current with uploaded (verze je u staženého uvedena stejně, tak to píše 0.1.1.) A háže to tu stejnou chybu nebo mrzne.

Fatal error: Uncaught Exception: WC_Zasilkovna_Shipping_Method::fetch_pickup_points: JSON data empty. in /www/doc/www.ipmakeup.com/www/wp-content/plugins/woocommerce-zasilkovna-master/class-zasilkovna.php:226 Stack trace: #0 /www/doc/www.ipmakeup.com/www/wp-content/plugins/woocommerce-zasilkovna-master/class-zasilkovna.php(196): WC_Zasilkovna_Shipping_Method->fetch_pickup_points('c963d8422c9514e...') #1 /www/doc/www.ipmakeup.com/www/wp-content/plugins/woocommerce-zasilkovna-master/class-zasilkovna.php(180): WC_Zasilkovna_Shipping_Method->load_pickup_points() #2 /www/doc/www.ipmakeup.com/www/wp-content/plugins/woocommerce-zasilkovna-master/zasilkovna.php(73): WC_Zasilkovna_Shipping_Method->pickup_points('CZ') #3 /www/doc/www.ipmakeup.com/www/wp-includes/class-wp-hook.php(287): wc_zasilkovna_review_order_after_shipping('') #4 /www/doc/www.ipmakeup.com/www/wp-includes/class-wp-hook.php(311): WP_Hook->apply_filters('', Array) #5 /www/doc/www.ipmakeup.com/www/wp-includes/plugin.php(478): WP_Hook->do_action(Array) #6 /www/doc/w in /www/doc/www.ipmakeup.com/www/wp-content/plugins/woocommerce-zasilkovna-master/class-zasilkovna.php on line 226 Na webu došlo k závažné chybě.

• Něco specifického to být nemůže, když na webu je jen WordPress 5.5.3, woocommerce Verze 4.7.1. a Zásilkovna Shipping Method (Verze 0.1.1). když zásilkovnu vypnu není problém.
• rychlost downloadu 133.0 Mbit/s
• reload nepomůže. Mohu nějak pomoc s dodáním něčeho dalšího potřebnýho? Tuto chybu má i Milen (@maleksandrov) viz pár komentů výše. Testováno bylo i API a to je taky OK. Díky moc za pomoc.

[ostryweb-cz]

@ostryweb-cz

Fatal error: Uncaught Exception: WC_Zasilkovna_Shipping_Method::fetch_pickup_points: JSON data empty. in /www/doc/www.ipmakeup.com/www/wp-content/plugins/woocommerce-zasilkovna-master/class-zasilkovna.php:226

tuto chybu vam dela to, co jsem psal zde: https://github.com/oerdnj/woocommerce-zasilkovna/issues/20#issuecomment-737775604

[8915b]

@ostryweb-cz Pane máte pravdu!! if ( empty( $json->data ) ) { je řešení (asi to bylo nakešovaný a já to nevymazal). MOC DĚKUJI! JSTE MAGIK. Krásný den!

A ještě jednou děkujeme za SUPER PLUGIN.

[maleksandrov]

fix your problems by correcting #225 to basically: if ( empty( $json->data ) ) {

(also it doesnt seem to be optimal downloading the whole 13MB list every time anyone enters the checkout...)

Thank you, @ostryweb-cz, for the support. Unfortunately, it is not a good idea to download the full XML every time on checkout. My client decided to use a different component.

Please keep it going and bring more joy to the people who are using your code.

Happy holidays and stay healthy. Best, Milen Aleksandrov

[8915b]

Hello Milen, thank you for the answer and help. Mentioned fix worked for a few days. But now we found another one.

https://github.com/oerdnj/woocommerce-zasilkovna/issues/21

Is there any way to download XML eg. only for Czech republic? It could be smaller :) 

I am not so skilled like you, but I would like to help.

Regards,

Marcel

---------- Původní e-mail ---------- Od: Milen Aleksandrov notifications@github.com Komu: oerdnj/woocommerce-zasilkovna <woocommerce-zasilkovna@noreply.github. com> Datum: 14. 12. 2020 20:15:47 Předmět: Re: [oerdnj/woocommerce-zasilkovna] Bug: Fatal error (#20) "

" fix your problems by correcting #225 (https://github.com/oerdnj/woocommerce-zasilkovna/blob/master/class-zasilkovna.php#L225) to basically: if ( empty( $json->data ) ) {

(also it doesnt seem to be optimal downloading the whole 13MB list every time anyone enters the checkout...) " Thank you, @ostryweb-cz(https://github.com/ostryweb-cz), for the support. Unfortunately, it is not a good idea to download the full XML every time on checkout. My client decided to use a different component.

Please keep it going and bring more joy to the people who are using your code.

Happy holidays and stay healthy. Best, Milen Aleksandrov

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub (https://github.com/oerdnj/woocommerce-zasilkovna/issues/20#issuecomment-744651421) , or unsubscribe (https://github.com/notifications/unsubscribe-auth/AOQIZ4PB433ZUBK4CBB2GF3SUZP57ANCNFSM4UJSCDSA) . "