romul / spree-russian-adaptation

Adapts Spree to the Russian reality.
http://spreecommerce.ru/
Other
28 stars 5 forks source link

robokassa + tweaks = russian_adaptation? #3

Open konzeptual opened 14 years ago

konzeptual commented 14 years ago

Доброго времени. Спасибо за то, что делитесь своей работой!

Предложение у меня такое - может быть стоит разбить функционал расширения? К примеру, было бы неплохо иметь функционал Робокассы в отдельном расширении. Это удобно, если хочешь пользоваться Робокассой, но при этом иметь альтернативный процесс оформления заказа.

Если у вас нет времени, то я бы это сам сделал, мне все равно придется выделить робокассу в отдельное расширение. Работы должно быть не так много...

romul commented 14 years ago

Это есть в планах. Только там не столько выделять, сколько переписывать надо. Теперь в Spree есть BillingIntegration как раз для подобных целей.

konzeptual commented 14 years ago

Посмотрел на новый механизм BillingIntegrtaion. С ним действительно выделение робокассы в отдельное расширение совсем не обязательно.

romul commented 14 years ago

Да, я потихоньку начал обновление. План пока такой: 1) Перевести РобоКассу на гейтвэй ActiveMerchant, т.к. он уже есть готовый, спасибо parallel588. (сделано) 2) Изменить контроллер PaymentsController, надо дать ему более специфичное имя, чтобы было понятно, что это контроллер для RoboKassa и генерить ошибку 404 или 500, если текущий гейтвэй не является РобоКассой. 3) Посмотреть есть ли смысл переделывать на BillingIntegration или достаточно оставить в качестве Gateway. Основная разница между ними, что гейтвэи работают через ActiveMerchant, а BillingIntegration может быть абсолютно независимым.

konzeptual commented 14 years ago

Руки наконец дошли до робокассы. Сделал следующим образом: 1) Оставил робокассу гейтом 2) Процесс заказа такой: Адрес => вид доставки => вид оплаты => заказ формируется, имеет статус "новый", теперь покупатель может нажать кнопку "оплатить" и иметь дело с робокассой (если он перед этим выбрал робокассу в качестве вида оплаты).

http://github.com/krick/spree-russian-adaptation/tree/robokassa