Kdyby / CsobPaymentGateway

[DEPRECATED] Client library for ČSOB Payment Gateway eAPI 1.5+
Other
13 stars 3 forks source link

CartItem price #5

Closed soukicz closed 7 years ago

soukicz commented 8 years ago

U metody addCartItem je atribut $price a popis "In hundredth of currency units". Z toho bych chápal, že je to cena za kus. V dokumentaci ČSOB je ale uvedeno, že je to celková cena. Nenašel jsem, že by se to někde násobilo kusy (a asi ani nenásobí, protože pak by se muselo řešit zaokrouhlení).

jiripudil commented 8 years ago

Je to celková cena. Currency unit (měnová jednotka) je terminus technicus, vůbec to nesouvisí s počtem kusů zboží :)

soukicz commented 8 years ago

Třetí argument addCartItem je $quantity

jiripudil commented 8 years ago

To se ale imo předává jen kvůli přehledu, který je pak vidět přímo na bráně při platbě. Ta cena je každopádně celková. Jestli ti někde nějaká formulace připadá nejasná, pošli PR :)

soukicz commented 8 years ago

Šlo mi jen o to, že posílám cenu a množství - očekával bych tedy, že to je cena za kus.

fprochazka commented 8 years ago

@soukiii máš pravdu, to api té třídy je nelogické. Dal bych tam asi raději to násobení, co myslíš?

soukicz commented 8 years ago

U násobení ale zase narazíš na zaokrouhlování. Bez násobení dostaneš celkovou částku po zaokrouhlení, což není to samé jako zaokrouhlené částky vynásobené.

fprochazka commented 8 years ago

Násobení celých čísel je celkem safe, ne?

soukicz commented 8 years ago

To jo. Ale já to musím zaokrouhlit, abych ti to celé číslo mohl dát.

fprochazka commented 8 years ago

Hmm.. takže lepší asi bude zůstat u debilního api, ale přidat k tomu nějakou dokumentaci.

soukicz commented 8 years ago

Ono nejspíš bude stačit jen přejmenovat ten atribut. Když to bude $totalPriceInCents, tak už to při psaní v IDE bude rovnou vidět

fprochazka commented 8 years ago

@soukiii $priceInCents už jsem tam jednou málem dal, ale pak jsem se zasekl na tom, jestli se cents dá použít jako univerzální slovo i pro koruny a eura :)

soukicz commented 8 years ago

Ono je to víceméně univerzální - https://en.wikipedia.org/wiki/Centi-

jiripudil commented 8 years ago

Univerzální pojmenování je "fractional unit", ale to už je zase takové docela neohrabané

soukicz commented 8 years ago

Každopádně ale v dokumentaci (i u Spořky nebo GoPay) je to popisované jako "cents"