Popraviti bug kojim preuzeta slika uplatnice ponekad nema ekstenziju

Ovaj problem je prijavljen u komentaru moje objave u Facebook grupi WooCommerce Hrvatska.
Nakon neuspje?nog reproduciranja tog istog problema na 3 razli?ita sustava (WAMP s PHP 7.3, LAMP s PHP 5.6 te LAMP s PHP 7.2), bacila sam se u pregled izvornog koda te tra?enje ?to bi moglo biti uzrok prijavljenom problemati?nom pona?anju.

Ubrzo sam shvatila da sljede?a linija JavaScript k?da koja se nalazi unutar znakovnog niza radi vi?e nego ?to bi trebala:
link.download = /\.\w+/.test(name) ? name :`\${name}.\${type}`;
Kako sam logiku za preuzimanje slike uplatnice sa stranice odradila oslanjav?i se na ovaj odgovor sa StackOverflowa (u ?ijem k?du je prisutan taj bug), tako se taj isti bug javlja i u ovom pluginu. U kojim okolnostima? S obzirom na uvjet unutar ternarnog operatora, problemati?no pona?anje se manifestira kada naziv datoteke sa slikom sadr?i to?ku u sebi. A kako to da bi naziv datoteke sa slikom (ukoliko u naziv ne uklju?ujemo ekstenziju datoteke koja bi se tek trebala dodati) mogao sadr?avati to?ku? Odgovor na to pitanje jest ukoliko naziv web-stranie (koja koristi ovaj plugin) sadr?i to?ku u sebi, s obzirom da naziv datoteke slike ovisi o nazivu web-stranice. Npr. ukoliko je naziv stranice, tj. vrijednost polja “Naziv web-stranice” (unutar izbornika Postavke > Op?enito kod WordPress admin dashboarda), jednaka “F.R.I.E.N.D.S”, tada bi naziv datoteke slike sadr?avao to?ku u sebi s obzirom da se naziv datoteke slike odre?uje sljede?om programskom linijom:
$file_name = sprintf( '%s-%s-%s', __('payment-slip', $this->domain), $webapp_name_for_filename, $order_id);
Tj. kako bi se u tom slu?aju u gore-navedenom ternarnom operatoru provjeravalo ima li znakovni niz ‘uplatnica-F.R.I.E.N.D.S-1234’ (koji bi predstavljao naziv datoteke slike) to?ku u sebi, odgovor bi bio istinit te se ne bi na taj naziv nadodala ‘.png’ ekstenzija.

Patch za ovaj problem ?u objaviti za par minuta.

Ujedno, ako tko od ?itatelja ove poruke ima tamo reputaciju ve?u od 50, mo?e slobodno ostaviti komentar na toj objavi kako se jo? netko ne bi opekao na istoj stvari.

• This topic was modified 5 years, 1 month ago by Marinela Levak.
• This topic was modified 5 years, 1 month ago by Marinela Levak. Reason: escaping backtick characters