No transaction is created and does not change the order status [#2497615]

Не создается транзакция и не меняется статус заказа.

Связка commerce_robokassa и kickstart 2

Устанавливается и настраивается модуль без проблем.
Проверяется работа на тестовом сервере Robokassa.

Во время оформления заказа переходим к пункту с оплатой.
Выбираем Robokassa.
Происходит перенаправление на сайт Robokassa.
Нажимаем успешную операцию.
Возвращаемся на сайт.
Показывается сообщение, что платеж прошел.

Проверяем:
Переходим в Заказы.
Смотрим статус заказа. Он замерз на статусе Оформление заказа: Оплата.
Заходим в раздел Оплаты заказа. Видис, что баланс не 0.00.
Транзакция не создана.

Должно работать так:
....... Возвращаемся на сайт.
Показано сообщение об успешной оплате.
Автомтически создается транзакция.
Происходит смена статуса с Оплаты на Завершен.

No transaction is created and does not change the status of your order.

Ligament commerce_robokassa and kickstart 2

Installed and configured the module without problems.
Checks the operation on the test server Robokassa.

During checkout, go to step with payment.
Choose Robokassa.
Redirected to the website of Robokassa.
Push successful operation.
Return to the site.
Shows a message that the payment has gone through.

Check:
Turn in Orders.
Watch the status of your order. He froze on the status of my order: Payment.
Go to the Payment section of the order. See that the balance is not 0.00.
The transaction is not created.

Should work like this:
....... Return to the site.
See the message on successful payment.
Avtomticheski transaction is created.
There is a change of status with Payment Completed.

Comment	File	Size	Author
#39	commerce_robokassa-with_through_transactions-2497615-39.patch	14.25 KB	lukasss
#32	syayus_robox.png	18.86 KB	lukasss
#29	rules_change_order_status_after_payment.png	38.9 KB	UksusoFF
#17	commerce_robokassa-with_through_transactions-2497615-17.patch	10.03 KB	lukasss
#16	commerce_robokassa-with_through_transactions-2497615-16.patch	10.01 KB	lukasss
#13	commerce_robokassa-with_through_transactions-2497615-13.patch	10.1 KB	lukasss
#12	commerce_robokassa-with_through_transactions-2497615-12.patch	10.15 KB	lukasss
#10	patch_with_transaction.patch	8 KB	lukasss
#9	patch_with_transaction.patch	8.17 KB	lukasss
#5	robox_l.patch	6.67 KB	lukasss

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

andypost

he/him

Russian

CreditAttribution: andypost at Skilld commented 3 July 2015 at 14:03

Похоже придется переделать часть логики, чтобы в пакет обмена данными передавался не заказ, а транзакция, поэтому нужно делать 7.x-2.x ветку по аналогии с https://www.drupal.org/project/commerce_interkassa

Comment #2

UksusoFF CreditAttribution: UksusoFF commented 10 August 2015 at 06:40

Есть решение для этой проблемы?

Comment #3

shevgeny

English

CreditAttribution: shevgeny commented 16 August 2015 at 20:01

Тоже интересует. Интеркасса — не катит.

Comment #4

andypost

he/him

Russian

CreditAttribution: andypost at Skilld commented 17 August 2015 at 10:25

решение есть - переделать на передачу транзакции, патчи приветствуются

Comment #5

lukasss CreditAttribution: lukasss commented 17 August 2015 at 11:08

File	Size
robox_l.patch	6.67 KB

Тоже недавно ковырялся с вашим модулем, на самом деле там всё основное реализовано и исправить этот недостаток не сложно. Я не брал модуль с интерфейсом, а просто решил переписать, чтобы самому вникнуть в код. Сейчас сделал патч с изменениями уже в самом модуле, патч не тестировал вообще, так что на ваш страх и риск. Там есть некоторые вещи которые неплохо было бы в интерфейс перенести, например, выбор статуса заказа после проведения и т.д., но это мелочи. Если что-то не заработает - пишите. В патче изменил функции:

commerce_robokassa_result
commerce_robokassa_create_transaction
commerce_robokassa_success
commerce_robokassa_fail

Comment #6

andypost

he/him

Russian

CreditAttribution: andypost at Skilld commented 17 August 2015 at 22:52

Status:

Active

» Needs work

Исправлено, я так понимаю, только обновление статуса, но теперь транзакции плодятся при каждом ответе шлюза, а всё потому, что передавать шлюзу нужно идентификатор транзакции, а не заказа.

Обязательно нужно учесть, что пользователь может настроить более одного метода оплаты робокассой, с разными параметрами, и тогда транзакция не будет найдена.

Также очень большие нарекания на форматирование кода, соблюдайне плиз https://www.drupal.org/coding-standards

+++ b/commerce_robokassa.module
@@ -176,29 +176,21 @@ function commerce_robokassa_get_settings($settings) {
 function commerce_robokassa_success() {
...
+     drupal_goto('checkout/'.$_POST['InvId'].'/complete');
+  } else {
+     drupal_goto('robokassa/fail'); ¶

это как success перенаправляет на fail?

+++ b/commerce_robokassa.module
@@ -176,29 +176,21 @@ function commerce_robokassa_get_settings($settings) {
 function commerce_robokassa_fail() {
...
-  if ($message = commerce_robokassa_get_settings('fail_message')) {
-    $build['message_fail']['#markup'] = $message;
-  }
+  $build = array();  ¶
+  $build['#markup'] = t('Payment unsuccessful!');

те настроенное сообщение не выведется

Comment #7

lukasss CreditAttribution: lukasss commented 26 August 2015 at 07:26

Ещё раз всё протестировал:

Нажимаем успешную операцию.
Возвращаемся на сайт.
Показывается сообщение, что платеж прошел.

Проверяем:
Переходим в Заказы.
Смотрим статус заказа. Он замерз на статусе Оформление заказа: Оплата.
Заходим в раздел Оплаты заказа. Видис, что баланс не 0.00.
Транзакция не создана.

Пробовали на тестовом сервере?
Если так, то скорее так и должно быть: вы, наверное, переходили по Success Url. Если так, то да, транзакция не создаётся и выводится простое сообщение об оплате, но по Result Url транзакция создается! и по моему тут всё корректно, единственное, что AMOUNT = 0.

Исправлено, я так понимаю, только обновление статуса, но теперь транзакции плодятся при каждом ответе шлюза, а всё потому, что передавать шлюзу нужно идентификатор транзакции, а не заказа.

Транзакции плодятся и без патча, но передавать, наверное нужно действительно их.

И вторая проблема - это то что когда происходит отказ от платежа, вот тогда он и зависает в корзине.

Сейчас пробую сделать нормальный патч

Comment #8

andypost

he/him

Russian

CreditAttribution: andypost at Skilld commented 26 August 2015 at 09:02

спасибо за тестирование, я настоятельно рекомендую посмотреть комиты http://cgit.drupalcode.org/commerce_interkassa/log/

там боролись с абсолютно той же проблемой, в итоге единственный выход - сделать версию 2.х (один из важнейших комитов)

то есть - нужно переходить к отправке именно кода транзакции, также крайне важно оперировать правильно инстансом платежной системы, те пользователь запросто может настроить более одного шлюза робокассы (например с разными реквизитами)... в текущей версии это работать не будет, но в интеркассе это исправлено

Comment #9

lukasss CreditAttribution: lukasss commented 26 August 2015 at 12:13

File	Size
patch_with_transaction.patch	8.17 KB

Коммиты не читал, но всё же посмотрите пожалуйста ещё один патч

Comment #10

lukasss CreditAttribution: lukasss commented 26 August 2015 at 12:19

File	Size
patch_with_transaction.patch	8 KB

1 file was hidden/shown/deleted

File	Size
patch_with_transaction.patch	8.17 KB

немного исправил

Comment #11

andypost

he/him

Russian

CreditAttribution: andypost at Skilld commented 28 August 2015 at 07:48

Assigned:

g0nz01

» Unassigned

1 file was hidden/shown/deleted

File	Size
robox_l.patch	6.67 KB

В целом теперь правильно передается транзакция, так что осталлось дочистить код
очень рекомендую выкладывая новый патч прилагать interdiff изменений
тогда реально легче делать ревью и следить за ходом изменений/мыслей

также нужно подумать, как разделить транзакции сохраненные до патча и после

+++ b/commerce_robokassa.module
@@ -40,8 +40,7 @@ function commerce_robokassa_commerce_payment_method_info() {
-    // @todo Maybe better make it auto?
-    //'offsite_autoredirect' => TRUE,
+    'offsite_autoredirect' => commerce_robokassa_get_settings('offsite_autoredirect'),

+++ b/includes/commerce_robokassa.admin.inc
@@ -13,6 +13,7 @@ function commerce_robokassa_settings_form($settings = NULL) {
+    'offsite_autoredirect' => 'true',

@@ -45,6 +46,16 @@ function commerce_robokassa_settings_form($settings = NULL) {
+    '#options' => array(
+      'true' => ('True'),
+      'false' => ('False'),

строка или булево значение?
полагаю, что комерсу нужно булево

+++ b/commerce_robokassa.module
@@ -102,10 +101,12 @@ function commerce_robokassa_build_redirect_form($form, &$form_state, $order, $se
+  ¶
...
+  ¶

@@ -208,51 +209,69 @@ function commerce_robokassa_fail() {
+   ¶
...
+    $transaction_id = $_POST['InvId'];  ¶
+     ¶
...
+    ¶
...
+      dsm($transaction->status);      ¶
+      // Get robokassa other variables.  ¶
...
+      $signature_in = $_POST['SignatureValue'];    ¶
...
+          //Success          ¶
...
+          ¶
...
+          $transaction->message = 'Payment has been received';          ¶
...
+    } ¶
...
+      watchdog('commerce_robokassa', 'There is no transaction or she was paid.', array(), WATCHDOG_ERROR);    ¶
...
+  }  ¶

@@ -282,14 +301,18 @@ function commerce_robokassa_create_transaction($order_id, $status) {
+  $transaction->status = $status; ¶
...
+  ¶

+++ b/includes/commerce_robokassa.admin.inc
@@ -45,6 +46,16 @@ function commerce_robokassa_settings_form($settings = NULL) {
+  ¶

лишние пробелы в конце строки, www.dreditor.org показывает всё

+++ b/commerce_robokassa.module
@@ -208,51 +209,69 @@ function commerce_robokassa_fail() {
+          echo 'OK' . $_POST['InvId'];
...
+          echo 'bad'.$amount_in;
...
+        echo 'bad'.$signature_in;
...
+      echo 'bad'.$transaction->transaction_id;

уверен, что это отладочный код. ибо выводить что-либо не нужно.

Comment #12

lukasss CreditAttribution: lukasss commented 23 September 2015 at 15:01

File	Size
commerce_robokassa-with_through_transactions-2497615-12.patch	10.15 KB

1 file was hidden/shown/deleted

File	Size
patch_with_transaction.patch	8 KB

Вроде всё почистил, исправил, просмотрел coder.

Проверял только на тестовом сервере!!!

Comment #13

lukasss CreditAttribution: lukasss commented 23 September 2015 at 15:11

File	Size
commerce_robokassa-with_through_transactions-2497615-13.patch	10.1 KB

1 file was hidden/shown/deleted

File	Size
commerce_robokassa-with_through_transactions-2497615-12.patch	10.15 KB

исправил кое, что

также нужно подумать, как разделить транзакции сохраненные до патча и после

Для чего это может потребоваться?

Comment #14

andypost

he/him

Russian

CreditAttribution: andypost at Skilld commented 23 September 2015 at 15:14

бегло просмотрел патч, в целом - супер!
довольно много нареканий на пробелы в пустых строках... просьба поправить

#13 чтобы люди после установки новой версии модуля могли обслужить оплаты созданные до обновления
полагаю, что можно написать hook_update_N() чтобы обновить данные в таблицах

Comment #15

andypost

he/him

Russian

CreditAttribution: andypost as a volunteer and at Skilld commented 23 September 2015 at 15:16

Status:

Needs work

» Needs review

также приветствуются ревью от остальных заинтересованых! :)
нужно просто проверить работу модуля с патчем... у меня сейчас нет активных акаутов, так что это затруднительно

Comment #16

lukasss CreditAttribution: lukasss commented 23 September 2015 at 15:51

File	Size
commerce_robokassa-with_through_transactions-2497615-16.patch	10.01 KB

1 file was hidden/shown/deleted

File	Size
commerce_robokassa-with_through_transactions-2497615-13.patch	10.1 KB

вроде все пробелы убрал...

Comment #17

lukasss CreditAttribution: lukasss commented 23 September 2015 at 15:52

File	Size
commerce_robokassa-with_through_transactions-2497615-17.patch	10.03 KB

1 file was hidden/shown/deleted

File	Size
commerce_robokassa-with_through_transactions-2497615-16.patch	10.01 KB

нет, не все 1 упустил )

Comment #18

lukasss CreditAttribution: lukasss commented 24 September 2015 at 10:30

Протестировал сам на рабочем аккаунте, Вроде все работает...

Comment #19

andypost

he/him

Russian

CreditAttribution: andypost as a volunteer and at Skilld commented 24 September 2015 at 10:45

@lukasss большое спасибо за новые патчи и тестирование
главная проблема для принятия - это отсутствие обновления с предыдущей версии, мы не можем потерять платежки, которые Уже созданы на заказы...
ну и было бы хорошо еще пару глаз привлеч для ревью и тестирования

ну и интересный момент с настройками остался:

+++ b/commerce_robokassa.module
@@ -40,8 +40,7 @@ function commerce_robokassa_commerce_payment_method_info() {
-    // @todo Maybe better make it auto?
-    //'offsite_autoredirect' => TRUE,
+    'offsite_autoredirect' => (boolean) commerce_robokassa_get_settings('offsite_autoredirect'),

как-то странно, настроек еще нет но умолчания берутся из них?! по идее должно быть TRUE

+++ b/commerce_robokassa.module
@@ -208,49 +209,74 @@ function commerce_robokassa_fail() {
+      if (commerce_robokassa_get_settings('server') == 'test') {

а если 2 ака робокассы добавлено? с разными настройками...

Comment #20

lukasss CreditAttribution: lukasss commented 24 September 2015 at 11:00

а если 2 ака робокассы добавлено? с разными настройками...

Вообще не думал об этом...Вы имеете в виду создание ещё одного способа оплаты на основе уже имеющегося со статусом Custom?

Comment #21

lukasss CreditAttribution: lukasss commented 24 September 2015 at 11:06

как-то странно, настроек еще нет но умолчания берутся из них?! по идее должно быть TRUE

Ну они же берутся из умолчания, которое в форме прописывается

Comment #22

andypost

he/him

Russian

CreditAttribution: andypost as a volunteer and at Skilld commented 24 September 2015 at 11:33

#20 да можно включить более одного идентичного плагина оплаты
#21 это как раз умолчание, которое потом должно в форму попасть, насколько я понимаю

Comment #23

lukasss CreditAttribution: lukasss commented 24 September 2015 at 11:45

Ещё потестирую...

Comment #24

sozonov CreditAttribution: sozonov commented 25 September 2015 at 12:20

Хотел уточнить, можно ли как-нибудь передать номер заказа в робокассу, чтобы там можно было посмотреть номер заказа для текущей транзакции?!
(я использовал #10 патч, может сейчас что-то изменилось уже).
спасибо!

Comment #25

andypost

he/him

Russian

CreditAttribution: andypost as a volunteer and at Skilld commented 25 September 2015 at 14:29

хорошее замечание! сейчас как раз именно номер заказа и передается, но так как патч делает правильно - передает номер транзакции, то стоит передавать и дополнительную информацию
насколько я помню, можно передавать пару дополнительных параметров, но можно и просто расширить информацию в передаваемом описание

Comment #26

UksusoFF CreditAttribution: UksusoFF commented 27 September 2015 at 09:53

Когда при использовании тестового сервера должен изменятся статус заказа?
Пробовал отправлять по result url и переходить по success url.
На сайте отображается "Спасибо, оплата прошла успешно!", но статус заказа все равно в ожидании.
Патч применял из #17 поста.

Comment #27

lukasss CreditAttribution: lukasss commented 27 September 2015 at 10:30

Вы должны в первую очередь смотреть не на статус заказа, а на статус оплаты.
А нужный статус выставляется в настройках рулеса

Comment #28

lukasss CreditAttribution: lukasss commented 27 September 2015 at 10:31

А нужный статус

статус заказа я имел в виду

Comment #29

UksusoFF CreditAttribution: UksusoFF commented 27 September 2015 at 11:32

File	Size
rules_change_order_status_after_payment.png	38.9 KB

Так?

Comment #30

lukasss CreditAttribution: lukasss commented 27 September 2015 at 14:19

меняется здесь: admin/commerce/config/payment-methods/manage/commerce_payment_commerce_robokassa/edit/3

Comment #31

UksusoFF CreditAttribution: UksusoFF commented 27 September 2015 at 14:29

Можно скрин или экспорт рулса?

Comment #32

lukasss CreditAttribution: lukasss commented 28 September 2015 at 08:49

File	Size
syayus_robox.png	18.86 KB

Comment #33

UksusoFF CreditAttribution: UksusoFF commented 28 September 2015 at 08:57

Там как раз и стоит "Выполнено", только статусы в этом чекбоксе не совпадают с возможными статусами заказа.

Comment #34

lukasss CreditAttribution: lukasss commented 28 September 2015 at 09:03

в этом чекбоксе

имеете в виду селекте?

Comment #35

UksusoFF CreditAttribution: UksusoFF commented 28 September 2015 at 11:01

Да, в селекте.

Comment #36

lukasss CreditAttribution: lukasss commented 28 September 2015 at 11:09

Вы используете кастомные статусы?

Comment #37

UksusoFF CreditAttribution: UksusoFF commented 28 September 2015 at 11:58

Нет: http://screenshot.net/lwv43ao

Comment #38

lukasss CreditAttribution: lukasss commented 28 September 2015 at 12:14

да, вы правы, сейчас не работает, я работаю над патчем

Comment #39

lukasss CreditAttribution: lukasss commented 12 October 2015 at 08:30

File	Size
commerce_robokassa-with_through_transactions-2497615-39.patch	14.25 KB

Только прошу сразу камнями не закидывать )))
Много чего поисправлял...

Самое трудное оказалось решить вопрос:

а если 2 ака робокассы добавлено? с разными настройками...

Точнее ту часть с настройками авторедиректа. Ничего лучшего не придумал, как аттачить файл js прямо в альтере формы.

Пришлось полностью переделать логику получению настроек. Теперь они берутся прямо из payment_method, а не из переменной как раньше.

Протестировал с двумя акаунтами: в рабочим и с тестовым.
Вроде всё работает...
У меня теперь новый вопрос: Что делать с рулесом оплаты после удаления модуля? Он остается...

Жду отзывов )

P.S.: Код на корректность и лишние пробелы не проверял пока.

Comment #40

andypost

he/him

Russian

CreditAttribution: andypost as a volunteer and at Skilld commented 12 October 2015 at 10:19

3 files were hidden/shown/deleted

File	Size
commerce_robokassa-with_through_transactions-2497615-17.patch	10.03 KB
rules_change_order_status_after_payment.png	38.9 KB
syayus_robox.png	18.86 KB

О! это уже больше похоже на то, что я пытался показать в 1-м коментарии
просто в интеркассе тоже было переписывание на вторую версию точно по тем же причинам и там как раз учтены все нюансы
Может имеет смысл пока перенести код на github и заанонсить где-нить, чтобы люди потестировали

Comment #41

lukasss CreditAttribution: lukasss commented 12 October 2015 at 10:48

Может имеет смысл пока перенести код на github

Мне можно это сделать?

заанонсить где-нить

Если только на drupal.ru )))

Comment #42

lukasss CreditAttribution: lukasss commented 12 October 2015 at 10:58

Может быть Вы обновите свой пост: http://www.drupal.ru/node/70117 ?

Comment #43

shevgeny

English

CreditAttribution: shevgeny commented 14 October 2015 at 05:07

А вы не смотрели подобный модуль от PayAnyWay (https://www.payanyway.ru/info/w/ru/public/w/partnership/developers/instr...)?

Comment #44

lukasss CreditAttribution: lukasss commented 14 October 2015 at 07:55

А вы не смотрели подобный модуль от PayAnyWay (https://www.payanyway.ru/info/w/ru/public/w/partnership/developers/instr...)?

Бегло просмотрел. Не увидел реализации авторедиректа.

Comment #45

keereel

Russian

CreditAttribution: keereel commented 16 February 2016 at 15:24

Скачал, пропатчил #39, установил. Проверил... Статус заказа меняется на тот, который в rules для этого способа поставил.. Но! Мне необходимо, что бы срабатывало ещё и другое правило, в котором Event - "When an order is first paid in full" (по этому событию активируются проданные лицензии на файлы), и оно не срабатывает, судя по всему это событие не активируется. Уже всю голову сломал, подскажите, плс, как быть?

Comment #46

lukasss CreditAttribution: lukasss commented 18 February 2016 at 14:29

Недавно работал c продажей файлов, честно говоря не замечал проблем.
Всё как обычно было, при смене статуса заказа активировалась лицензия.
Смогу протестировать ещё раз чуть позже

Comment #47

dillix CreditAttribution: dillix commented 21 February 2016 at 14:16

#39 пашет отлично! Энди сделай приз коммит)

Comment #48

dillix CreditAttribution: dillix commented 21 February 2016 at 14:16

Status:

Needs review

» Reviewed & tested by the community

Comment #49

fortis CreditAttribution: fortis as a volunteer commented 21 February 2016 at 14:50

я правильно понимаю что commerce_robokassa_create_transaction вызывается в конструкторе формы?
кажется в патче транзакция создается при показе формы, имхо это не верно, но нужно меня проверить)

Comment #50

andypost

he/him

Russian

CreditAttribution: andypost as a volunteer and at Skilld commented 21 February 2016 at 20:01

Полагаю, что изменение довольно сильно изменяет модуль, и его нужно выносить в отдельную ветку 2.x
Также необходимо продумать upgrade path, чтобыпользователи 1.x могли обновиться связав данные предыдущих транзакций.
Именно для миграции данных таск всё еще требует доработок.
Как только будет готов патч, я сразу сделаю 2ю ветку

Comment #51

niko- CreditAttribution: niko- commented 21 February 2016 at 20:09

На досуге сделаю тут рефакторинг в стиле интеркассы

Comment #52

fortis CreditAttribution: fortis as a volunteer commented 24 February 2016 at 08:45

в интеркассе транзакция создается в колбеке который вызывает шлюз, так правильнее, создавать ее до оплаты при показе формы не стоит, если есть необходимость отслеживать какие платежи бросили то это видно по статусу заказа да и можно прикрутить воронку/цели от аналитики или метрики

Comment #53

fortis CreditAttribution: fortis as a volunteer commented 24 February 2016 at 08:48

и сейчас в робокассе настраивается метод хеширования, лучше сделать чтобы можно было выбирать в админке md5 или другое
> Поддерживаются следующие алгоритмы расчёта хэша: MD5, RIPEMD160, SHA1, SHA256, SHA384, SHA512

Comment #54

dillix CreditAttribution: dillix commented 24 February 2016 at 12:03

fortis, если я буду юзать патченную версию у меня сайт сломается?

Comment #55

fortis CreditAttribution: fortis as a volunteer commented 24 February 2016 at 12:23

@dillix, нет, но я указал на недостаток патча, теоретически тут можно бесконтрольно генерировать сколько угодно транзакций для этого заказа, в итоге будет оплачена одна транзакция, но возможен флуд pending-транзакциями.

ну и так как транзакция создается заранее и ее id открыт (в форме он как hidden), то если на сайте будет возможно провести csrf атаку - получим возможность направлять пользователей на оплату чужих заказов и транзакций

Comment #56

andypost

he/him

Russian

CreditAttribution: andypost as a volunteer and at Skilld commented 24 February 2016 at 12:47

Status:

Reviewed & tested by the community

» Needs work

правильно подмечено, по идее транзакция должна создаваться только когда запускается оплата
стоит проверить вариант с редиректом, чтобы не делать hidden поля, по хорошему там должен быть хеш

+++ b/commerce_robokassa.module
@@ -40,8 +40,6 @@ function commerce_robokassa_commerce_payment_method_info() {
-    //'offsite_autoredirect' => TRUE,

полагаю, что нужно оставить и добавить комент, почему именно FALSE/TRUE

+++ b/commerce_robokassa.module
@@ -94,8 +92,7 @@ function commerce_robokassa_redirect_form($form, &$form_state, $order, $payment_
+  $form['#action'] = commerce_robokassa_server_url($settings['server']);  ¶

@@ -103,25 +100,29 @@ function commerce_robokassa_build_redirect_form($form, &$form_state, $order, $se
+    ¶

@@ -130,44 +131,36 @@ function commerce_robokassa_build_redirect_form($form, &$form_state, $order, $se
+  unset($form["form_build_id"]);  ¶
+  $payment_method = commerce_payment_method_instance_load($form_state['order']->data['payment_method']);  ¶
...
+    $form['#attached']['js'][] = drupal_get_path('module', 'commerce_payment') . '/commerce_payment.js';    ¶
...
+  $payment_method = commerce_payment_method_instance_load($order->data['payment_method']);  ¶
+  if ($payment_method) {    ¶
...
+    }    ¶

@@ -208,49 +201,74 @@ function commerce_robokassa_fail() {
+          ¶
...
+      // Calculate the hash.             ¶
...
+      if ($md5 == $signature_in) {        ¶

trailing whitespace

Comment #57

niko- CreditAttribution: niko- commented 26 February 2016 at 00:29

Как и обещал сделал версию робокассы с учетом наработок интеркассы тут

Фичи
1. Использованы все наработки интеркассы с учетом того, что протокол робокассы обрезан по сравнению с интеркассой (к примеру нет статустов wait process и отложеных транзакций)
2. модуль умеет грузить и работать с не завершенніми транзациями версии 1.x Для определения что транзакция родная используется параметр shp_order_id если он присутствует это версия 2 если нет 1. соотвественно по разному грузится тразакция.
3. Добавленна поддержка нескольких рулов
4. Версия 2.х не совместима с версией 1.х по настройкам - прийдется перстроить платежный метод и кабинет в робокассе (урлы).
5. Простановка статусов транзакции и ордера происходит drupal commerce way
6. Убраны кастомные страницы success/fail вместо них используется стандартный процесинг редиректа ордера на следующий\прошлый степ (опять же drupal commerce way ) Что позволяет после платежа добавлять шаги как это обычно делается у комерса а не строить костыли
7. Платежная транзакция создается когда ей положено при редиректе на платежку (по аналогии с интеркассой)

Ложка дегтя
1. Так как я не резидент РФ то я не смог зарегестрировать хост и протестировать в связке с робокассой.
2. Провел проверку на включение и "офлайн" тестирование на сайте. Ничего не упало

Онлайн тесты, фидбек, мерж реквесты приветствуются.

Comment #58

niko- CreditAttribution: niko- commented 26 February 2016 at 00:33

И да поддерживаются все доступные в робокассе алгоритмы хеширования

Comment #59

niko- CreditAttribution: niko- commented 26 February 2016 at 18:52

Тестинг\фиксы прошли успешно вроде все ровно.
Спасибо @dillix за помощь с тестированием

Comment #60

dillix CreditAttribution: dillix commented 26 February 2016 at 18:56

Подтверждаю, протестировали в тест моде все работает отлично!

Comment #61

niko- CreditAttribution: niko- commented 27 February 2016 at 14:40

ну и так как транзакция создается заранее и ее id открыт (в форме он как hidden), то если на сайте будет возможно провести csrf атаку - получим возможность направлять пользователей на оплату чужих заказов и транзакций

Вернул отправку order ID в InvId
shp_order_id заменен на shp_trxid
В shp_trxid отправляю GUID транзакции и по нему же происходит загрузка необходимой транзакции на result/fail GUID хранится в remote_id так как Робокасса не использует его.
Добавлена проверка суммы