Skip to content
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

Неверное имя параметра для описания покупки #2

Closed
beautyfree opened this issue Nov 3, 2013 · 4 comments

Comments

@beautyfree
Copy link

Документация написана так что заблуждает в том какое правильное имя у параметра покупки.
http://robokassa.ru/ru/DocTest/Ru/Interface.aspx#222

Параметры запроса (метод POST либо GET): 
MrchLogin=sMerchantLogin&
OutSum=nOutSum&
InvId=nInvId&
Desc=sInvDesc&
SignatureValue=sSignatureValue
IncCurrLabel=sIncCurrLabel&
Culture=sCulture
[&shpa=yyy&shpb=xxx...-пользовательские_параметры_начинающиеся_с_SHP_в_сумме_до_2048_знаков] 

Проводя реверс оплаты на brainstorage вижу что у них используется параметр Desc для описания. А у нас https://github.com/ZeroOneStudio/rubykassa/blob/master/lib/rubykassa/payment_interface.rb#L15

К сожалению в тестовом режиме нет возможности проверить проходит ли параметр InvDesc, но если верить документации правильнее отправлять Desc и если судить по опыту друго сервиса при оплате которого успешно отображается описание.

Если проверяли работу в боевом режиме и описание отображалось - извиняюсь за тикет и порчу кармы)

@beautyfree beautyfree reopened this Nov 3, 2013
@SergeyKishenin
Copy link
Member

Нет, в продакшне пока не доводилось проверить. Так что в итоге, кто прав оказался?

@SergeyKishenin
Copy link
Member

Да, кажется, это мой факап. Пофиксил в 2fbc2a5. Если все ок, то релизим версию с этим патчем

@beautyfree
Copy link
Author

У меня так же нет возможности проверить в продакшене.
Но brainstrorage отправляет описание вида "Платеж №__" через параметр Desc который отображается, поэтому думаю надо релизить.

@SergeyKishenin
Copy link
Member

Bumped to 0.2.2 with 71723b3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants