shusaura85 / fancourier-api

PHP library to work with the new Fan Courier API v2.0 (selfawb.ro / api.fancourier.ro)
MIT License
21 stars 9 forks source link

date() to gmdate()? #16

Closed rwkyyy closed 4 months ago

rwkyyy commented 6 months ago

Salut,

Rulam un tool de verificare și am primit următorul notice: date() is affected by runtime timezone changes which can cause date/time to be incorrectly displayed. Use gmdate() instead.

În realitate nu aș vedea asta ca o problemă, iar pe proiectul unde folosesc libs pare ok, dar mă gândesc că pot apărea discrepanțe când se face trecerea de la ora de vară (și vice-versa)

nb. super work cu acest api 💯!

shusaura85 commented 6 months ago

Salut,

Nu cred ca isi are rost schimbarea in gmdate.

Sunt 2 obiecte care folosesc date() in care returneaza si ora/minut: AwbTracker si CourierOrderTracker. In ambele cazuri, functia e folosita cand nu exista status sa fie returnat deci e mai ok sa returneze cu timpul local in aceste cazuri.

Sunt 3 request-uri care folosesc date() dar nu folosesc si ora/minut: GetBankTransfers, GetCourierOrders si GetShippingSlip. In toate cazurile functia e folosita doar pt initializarea datei (ziua curenta).

Deci momentan nu cred ca e cazul sa fie facuta schimbarea. Daca o sa fie necesara in viitor schimbarea, o s-o fac atunci.