-
Notifications
You must be signed in to change notification settings - Fork 10
STDEV-20296 #57
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
STDEV-20296 #57
Conversation
Добавлены поля для передачи состояния оплаты и id банковской сессии.
# Conflicts: # app/src/main/java/ru/evotor/devices/drivers/paysystem/PayResult.java
подлит девелоп. Резолв конфликтов.
Вернул readExpandData вместо readExpand
npe при paymentState.name()
temp commit (test)
temp commit (test №2)
Вернул версионирование
# Conflicts: # app/src/main/java/ru/evotor/devices/drivers/paysystem/PayResult.java
merge develop
This reverts commit 4fca07e.
Добавлен LoyaltyCardId
Расширен конструктор PaymentRequest
Актуализация состава PayResult
Конструктор для PayResult
Конструктор для PayResult #2
This reverts commit 223b501.
Сумма бонусов в PaymentRequest
| @@ -0,0 +1,28 @@ | |||
| package ru.evotor.devices.drivers.paysystem; | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
кажется, это уже было в симпсонах =/
я видел этот код, и кажется он уже где-то есть
| if (VERSION >= 2) { | ||
| parcel1.writeString(paymentSessionId); | ||
| parcel1.writeString(loyaltyCardId); | ||
| parcel1.writeString(earnedBonus.toPlainString()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NPE ?
| if (currentVersion >= 2) { | ||
| paymentSessionId = parcel1.readString(); | ||
| loyaltyCardId = parcel1.readString(); | ||
| earnedBonus = new BigDecimal(parcel1.readString()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BigDecimal(null)?
| * Может быть null | ||
| */ | ||
| private final String additionalDescription; | ||
| private String additionalDescription; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
final пропали
| if (currentVersion >= 7) { | ||
| String paymentStateName = parcel1.readString(); | ||
| if (paymentStateName != null) { | ||
| paymentState = Constants.PaymentState.valueOf(paymentStateName); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
тут может быть рандомная строка, нужна валидация, хотя бы через try/catch
Поддержка работы драйверов, допускающих работу в режиме прерывания (с поддержкой программ лояльности).
PaymentRequest:
PayResult