Исправление ошибки "Отсутствует обязательный параметр "MrchLogin"" предложенное Anton L. Safin (http://drupal.ru/username/anton-l-safin) в теме http://drupal.ru/node/47866 . Патч ниже, исправляет эту ошибку, плюс некоторые исправления кода.

CommentFileSizeAuthor
#3 roboxchange.patch4.21 KBneochief
#2 roboxchange.patch2.4 KBneochief
uc_roboxchange.100812.patch3.84 KB-sibero-

Comments

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

neochief’s picture

neochief commented
Status: Closed (fixed) » Needs review
StatusFileSize
new roboxchange.patch2.4 KB

I'm not sure why it's closed. The issue is still actual. Here's a patch in unified diff format.

neochief’s picture

neochief commented
Title: Отсутствует обязательный параметр "MrchLogin" » Update to new Robokassa API
StatusFileSize
new roboxchange.patch4.21 KB

New patch solves following issues:
1. All request parameters now match specification.
2. Everything works both with POST and GET (if you use ip2locale, switch to get to avoid troubles, as I did)
3. Solved problem with wrong balance counting, which didn't take into account the currency rate.

So far, so good.

to.stepan.kuzmin@gmail.com’s picture

to.stepan.kuzmin@gmail.com commented

Thanks for patch! I will test it in production.
Is there any other issues for this module?

neochief’s picture

neochief commented

Except a lot of code standard violations, I didn't find anything buggy.

kudesa’s picture

kudesa commented

У нас тут проблема возникла с робокассой в том, что после оплаты не производился переброс обратно на интернет-магазин. Используем 6ю версию. Допускаю, что у нас что-то не так настроено было. Но в итоге пришлось немного поправить модуль вот в этой функции: function uc_roboxchange_done_payment()

В самом низу функции изменили строчки:

return "OK". $order_id;

на

print "OK". $order_id;
exit;

и

return $res;

на

print $res;
exit;

После этого возвраты заработали.