zeko868 / croatian-payment-slip-generator-for-woocommerce Goto Github PK

From marlevak on October 17, 2019 21:04 CEST

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.

Original issue: Popraviti bug kojim preuzeta slika uplatnice ponekad nema ekstenziju

Problem koji se desavao je sledeci,

Ukoliko korisnik obavi jednu narudzbinu njemu stigne email sa ispravnom uplatnicom medjutim ukoliko odmah nakon toga odluci da uradi novu narudzbinu za drugi proizvod u mailu mu stigne uplatnica sa podacima iz prethodne narudzbine sto je nama pravilo veliki problem.

Kopajuci malo po kodu video sam da se slika uplatnice cuva u sesiji.

Wooplatnica.php fajl 83. linija. Nakon te izmene sve je pocelo da radi kako treba.

Tako da mozda mozete ovo ukljuciti kao update ukoliko ne remeti neke druge stvari u pluginu.

Hi, I am using your plugin and it is great!
Recently I have updated Woocommerce to 4.8 version, and planning to update it to 4.9.
Now, using 4.8 Woocommerce version I have noticed issue with no Processing Order Mail sent to customer.
After I disabled Croatian-payment-slip-generator-for-WooCommerce plugin mail was sent.
Can you help me regarding this issue.

Hi,

there's no payment-slip in email.

From zeko868 on October 16, 2019 02:50 CEST

Currently this plugin has bug that manifests in inconsistency of callout number of payment slip through time.

Let’s take the following scenario:
suppose that %order%-%date% is being set as callout number pattern. If user ordered something yesterday (on 15th October 2019) by selecting paying using payment slip, but decides to perform that payment today (on 16th October 2019), then when loading previously generated payment slip from the order history the wrong callout number would be displayed. In other words, callout number would be based on current date (i.e. XXXX-2019-10-16) instead on the date when the order was placed/created (i.e. XXXX-2019-10-15).
The idea should be to use order creation date for setting the callout number value since that way the callout number would be the same, regardless of when the order page page with the payment slip image would be loaded.

Therefore, this pull request contains changes for replacing date placeholders in callout number with date components of order creation date instead of current date.

Original issue: Fix value interpolation of date placeholders in callout number

From marlevak on October 17, 2019 23:27 CEST

Ovaj prijedlog je naveden u komentaru komentara na moju objavu u Facebook grupi WooCommerce Hrvatska.

U postavkama plugina (tj. načina plaćanja ili payment gatewaya) se nalazi novi atribut ‘Glavni font’ (engl. Main font), a čija se vrijednost može odabrati kroz padajući izbornik. Za sada su ponuđene sljedeće vrijednosti:
– “proporcionalni”/”proportional” (njegovim odabirom se koristi font Times New Roman)
– “neproporcionalni”/”monospaced” (njegovim odabirom se koristi font Roboto Mono)
Ukoliko se ne vrši nikakvo dodatno podešavanje plugina, defaultno se koristi proporcionalni font, tj. Times New Roman.
Odabrani spomenuti font se koristi za tekst koji se ne prikazuje unutar ćelija, tj. čiji znakovi se ne moraju nalaziti u točno određenim kućicama jednake duljine.

Ovu funkcionalnost sam već razradila te će biti objavljena u repozitoriju na GitHubu kroz par minuta.

Original issue: Omogućiti biranje fonta za tekst koji se ne prikazuje unutar ćelija

From mvrban on October 15, 2019 21:16 CEST

Jeli bi bilo moguće napraviti da je %order% može ubaciti u Opis plaćanaja, pa onda i u pozivu na broj imamo broj narudžbe u opisu koji je vidljiviji na bankovnim izvodima.

Original issue: Ubacivanje %order% u Opis plaćanja

If one is using custom order numbers, they will have no clue where to look for specific order, so they need order number, not ID. If this number is the same, there will be no changes.

I changed it already, but it could be implemented to this plugin.

In wooplatnica.php

$order->get_id(),

could be replaced with

$order->get_order_number(),

I'm new here so I don't really know what I am doing.

Hope I've been helpful.