リンクタイプ決済の「配送区分」と注文情報の詳細の設定要否を引数で指定できるようにする #116
Conversation
yuriko1211
changed the title
yuriko1211
changed the title
…formation' に true を指定することを追記した
yuriko1211
changed the title
| @@ -307,6 +307,9 @@ end | |||
| EpsilosLinkPaymentGateway is available in all link payments. | |||
| For example, GMO Payment After Delivery. | |||
|
|
|||
| If you don't need send paramaters of delivery information details(e.g. consignee_postal, consignee_name, orderer_postal, and orderer_name), you set `delivery_info_required` to `false`. | |||
| When you don't set `delivery_info_required`, you must set purchase_detail to delivery information details. | |||
|
|
|||
There was a problem hiding this comment.
デフォルトはtrueだよ(default value is true.) 的なことをどこかに明記したい気がしました。
あと、英語的な部分でなんとなく気になった点…
don't need send => don't need to send ?
you must set purchase_detail to delivery information details
=>
you must set delivery information details to purchase_detail
（「配送情報詳細を purchase_detail にセットしてね」と言わんとしているのだとしたらこちらな気がしました）
There was a problem hiding this comment.
英文を意味が通るように修正しました。
Default value of `delivery_info_required` is `true`, therefore you must set delivery information details to purchase_detail When you don't set `delivery_info_required`.
「delivery_info_required のデフォルト値は true なので、 delivery_info_requiredをセットしない場合は、必ず配達情報の詳細をpurchase_detailに設定してください」の意味になるよう書いてみました。
| @@ -15,6 +15,15 @@ def test_epsilon_link_type_purchase_successfull | |||
| end | |||
| end | |||
|
|
|||
| def test_epsilon_link_type_not_sending_delivery_information_purchase_successfull | |||
| VCR.use_cassette(:epsilon_link_type_not_sending_delivery_information_purchase_successfull) do | |||
There was a problem hiding this comment.
successfull => successful ?
There was a problem hiding this comment.
successfulが正しいです。
テストの中でミススペルしている箇所を直しました。
|
@itakurara
また、仕様書p51に下記のような記載があったため、このPRでの対応内容で問題ないと思います。 |
|
レビューありがとうございました。 |
実現したいこと
これまで、イプシロンリンクタイプ決済のオーダー情報送信を行う
purchaseメソッドでは 、delivery_code: 99を無条件で指定していた。そのため注文情報の詳細（注文主・配送先の氏名住所等）をパラメータにセットすることが必須となっていた。この修正で、新たに追加した引数
delivery_info_requiredの値でdelivery_codeを設定するかを判断するようにした。（falseの場合は注文情報の詳細の送信が不要）なお、
delivery_info_requiredを指定しない場合（既存通りamountとdetailのみ引数で渡す場合）は、delivery_info_requiredをtrueとみなし、これまで通り注文情報の詳細を必須とする。やったこと
purchaseメソッドに引数でdelivery_info_requiredを渡せるようにしたdelivery_info_requiredの値でdelivery_codeと注文情報の詳細をセットするかを判断できるようにしたdelivery_info_requiredがfalseの場合のテストを追加参考：
イプシロン決済システム仕様書 Ver.1.0.42 p46