Генерация форма для оплыты

[dapi]

Генерируем объект типа Form, с помощью FormBuilder из объекта Payment, для дальнейшего использования для генерации формы на странице

[artemeff]

@dapi вот как-то так

[dapi]

Слушай, а я думаю, а зачем в payment выдумывать название для полей, если можно использовать сразу именно из walletone?

То есть вместо payment_no -> PAYMENT_NO и тп.

Тут можно смело
attribute :merchant_id, Integer, required: true

walletone/lib/walletone/payment.rb

Line 6 in 8beb375

attribute :merchant_id, Integer

Прислай реквестом. Так обсуждать проще.

[dapi]

Слушай,я FormBuilder осилит не могу. Не понимаю что он делает. Он в принципе должен просто из Payment сделать Form.

Payment по-большому счету и есть Form, только туда надо добавить сигнатуру и все.

Какие-то extra,options, attributes смущает ) чем они отличаются?

Смущает еще и то, что я не могу проверить работает ли он так-же и как прошлый и что в нем изменилось.

Не стесняйся в Payment перечислить ВСЕ поля что есть в документации walletone, потому что именно в Payment будут заглядывать чтобы узнать какие поля он принимает. Можно даже комментарии на против каждого поля и ссылку на доку сверху файла.

[dapi]

В общем начало нормуль.

[dapi]

Тут:

walletone/spec/walletone/payment_spec.rb

Lines 4 to 5 in 8beb375

	payment = described_class.new(merchant_id: 123)
	expect(payment.merchant_id).to eq(123)

123 выноси в let, чтобы дать понять что это один ии тоже значение

let(:merchant_id) { 123 }

а еще лучше

let(:merchant_id) { generate :merchant_id } - https://github.com/thoughtbot/factory_girl/blob/master/GETTING_STARTED.md#sequences

[dapi]

Что за опции тут предполагаются?

walletone/lib/walletone/payment.rb

Line 26 in 8beb375

def build_form(options = {})

[dapi]

walletone/lib/walletone/payment.rb

Line 3 in 8beb375

include Virtus.model

Будь с ней по-строже )

include Virtus.model strict: true

[artemeff]

Так комментируй коммиты :)

Да, я сначала сделал required, но не увидел в доке этой опции, сделаю.

Слушай,я FormBuilder осилит не могу. Не понимаю что он делает. Он в принципе должен просто из Payment сделать Form.
Payment по-большому счету и есть Form, только туда надо добавить сигнатуру и все.
Какие-то extra,options, attributes смущает ) чем они отличаются?

Ну он просто эти поля приводит к нужному формату, extra – в walletone можно отправлять кастомные поля, которые потом придут вместе с POST запросом. options – это я так спрятал success_url и fail_url, чтобы их к пейменту не прицеплять. А attributes – это все поля от payment. Вообще да, надо этот билдер сделать не билдером, а чтобы оно подписывало поля и проверяло подписи.

Вообще, было бы очень круто вешать что-то типа коллбеков на поля, чтобы он при добавлении форматировались. Или мы форматирование оставим на пользователя гемом?

То есть вместо payment_no -> PAYMENT_NO и тп.

Меня капс сильно смущает :)

Смущает еще и то, что я не могу проверить работает ли он так-же и как прошлый и что в нем изменилось.

А что именно в работе смущает? Я тестами покрыл, подписывает, вроде, правильно – сам проверял.

Не стесняйся в Payment перечислить ВСЕ поля что есть в документации walletone, потому что именно в Payment будут заглядывать чтобы узнать какие поля он принимает. Можно даже комментарии на против каждого поля и ссылку на доку сверху файла.

Да я все перечислил, кроме delivery и ссылок, доставку же отложили :)