search

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

Update README.md #15

Closed rwkyyy closed 8 months ago

rwkyyy commented 8 months ago

added notification for setNumber field that tho it is mandatory, it is ignored on the AWB and only the setStreet field is shown!

shusaura85 commented 8 months ago

Campul nu este ignorat. Si chiar daca ar fi fost cazul, astfel de comentarii cum ca un camp ar fi obligatoriu dar ignorat in anumite cazuri nu isi au locul in readme

rwkyyy commented 8 months ago

fără câmp nu poți genera awb, iar ce este în el (ca și cifră) este ignorat (dpdv ce se afișează pe awb), eu m-am blocat când am făcut implementarea pentru că nu înțelegeam de ce (și de când) are nevoie de câmp separat de cifră, ulterior am văzut în exemple că orice ai pune el merge.

nb. în testele inițiale nu am folosit număr (setNumber) la adresa de livrare, fiind mai mult o interpretare eronată din partea mea unde am crezut că este numărul de colete din awb (ex. 1, 2, 3 colete) și am considerat că este opțional. Desigur am ajuns în această situație după ce am văzut în niște exemple de la fan în care ei trimiteau totul doar pe câmpul de adresă (dar este alt endpoint față de cel oficial).

shusaura85 commented 8 months ago

Campul este text, a fost pus de mine incorect ca numeric. Asta e motivul pentru care serverul fan vrea campul dar poate fi "ignorat".
Exemplele au ramas cu valoare numerica, dar intern se face conversia la string inainte sa fie trimis la fan.
Campul este trimis intotdeauna in request dar serverul respinge daca e string gol (valoarea default) sau chiar si " " (spatiu) dar accepta daca e "-" (daca incerci cu exemplul inclus si pui "-" ca numar o sa vezi ca asa apare si pe awb).

Cum am de gand sa fac upgrade la cod sa fie definite tipurile de parametri (dupa ce sunt sigur ca nu mai sunt sanse de buguri in cod), o sa fie definite exact variabilele si ce tip de informatie e acceptata (in cazul asta, setNumber va accepta int|string cu conversie interna de la int la string)