納品書の送料が２つ表示される問題を修正

[kazumiiiiiiiiiii]

概要(Overview・Refs Issue)

納品書出力した際に「送料」が２つ表示される。

方針(Policy)

• 受注管理に影響が無いよう、OrderPdfServiceのrenderOrderDetailDataで対応。
• 複数配送の納品書では送料が必要なので、複数配送ではない場合に限定。

実装に関する補足(Appendix)

単価1,000円、税込1,000円という表記もおかしい気がするので、詳細の方から除きました。

テスト（Test)

手動で動作確認。

相談（Discussion）

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更
• フックポイントの呼び出しタイミングの変更
• フックポイントのパラメータの削除・データ型の変更
• twigファイルに渡しているパラメータの削除・データ型の変更
• Serviceクラスの公開関数の、引数の削除・データ型の変更
• 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[okazy]

@kazumiiiiiiiiiii

ありがとうございます！
表示わかりやすくなったと思います！
動作も確認して問題ありませんでした。
細かいですが、 isProduct() の方が文脈としてわかりやすいかと思いましたのでsuggestionしてみました。

[okazy]

変更前

変更後

[kazumiiiiiiiiiii]

@okazy そうですね！isProductでいいですね。
ありがとうございます！

[okazy]

ご確認ありがとうございます！

[okazy]

@kazumiiiiiiiiiii ありがとうございます！マージしました 🎉